Glad I could help! A couple of points:
–It looks like your commStatus takes on a few limited number of values. In this case, it’s probably better to use a kind of value for it rather than a text variable. So you could say:
CommStatus is a kind of value. The commStatuses are Undecided, Agree, Disagree, and None. A person has a commStatus.
Also, it looks to me as though your code for adjusting the Table of Relationships doesn’t actually affect relationships between the two people. If you’re adjusting the relationship between Darwin and Lifeshare, first you’ll choose a table row with first friend of Lifeshare, and then you’ll choose a table row with first friend of Darwin, but you won’t have chosen a row with both Darwin and Lifeshare (except by coincidence).
You want to do a two-column table lookup. There’s no built-in way to do that but you can adapt something like this. (I kind of feel like I should be able to change that so it’s completely general, with argument places for the table name, table column, etc., but not today… anyway if you have only one table you need to do a two-column lookup in, it should be OK to hardcode all that.)