-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Relationship can be added between an entity and itself #1727
Comments
Is there ever an example where this is valid? I can't think of one. This is probably an edge case data entry mistake regardless. |
There are some. If you go down the rabbit hole of how we represent political campaign finance donations in LittleSis, you can find some valid cases. We create aggregate contribution relationships between the donor and the politician (i.e. donations than span multiple campaign entities). So this relationship https://littlesis.org/relationships/1157009 contains Trump's FEC donations to his own campaigns and should remain. But yes, position relationships don't make a lot of sense here and that one should probably be removed. |
These donations might actually be duplicates. They are made to "DONALD J. TRUMP FOR PRESIDENT, INC." I normally link PACs and a person, but keep any manually entered donations on the PAC. Probably at a minimum should not be allowed for:
Just an example, this is how a small campaign looks in Texas: Our local rules don't require a treasurer until they reach a certain amount. Although it could be argued that a campaign is always a separate entity. As an aside on that blob syntax, is that common enough that it should be supported? |
hmm, it was edited 7 years ago so you may be right about that one. but in general our fec matching tool does track 2 relationships and will update both the donor->candidate relationship and the donor->committee relationship https://github.com/public-accountability/littlesis-rails/blob/main/app/models/fec_match.rb We have automated importing tools for Federal and New York State data...every state has different rules and forms of course ! |
@mooninmyname What are your thoughts on whether entities should be allowed to have relationships with themselves? |
there are about 1k of them on the site Relationship.where("entity1_id = entity2_id").count
=> 1097 |
The text was updated successfully, but these errors were encountered: