Shouldn't be CASCADE - I understand that you want link tables to be auto-deleted if the person record gets deleted, but in principle, we don't want person records to get deleted.

You can though override this in the s3db.pr_person_id() call - just make it s3db.pr_person_id(ondelete="CASCADE") there.