You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Instead, OpinionReading can have opinion_author and opinion_type attributes corresponding to the author and type found on the Opinion model. That way the Decision and OpinionReading don't both need to have the Opinion as attributes.
Decision.posit could take author and type parameters to pass the Holdings that need to be posited down to the proper OpinionReading (but if there's only one OpinionReading, those params could be omitted).
The text was updated successfully, but these errors were encountered:
I implemented the approach above, but I still don't like it. It's too easy for the OpinionReading to get out of sync with the Opinion, without a direct link to the Opinion. There must be a more intuitive way to organize or consolidate these classes (and the DecisionReading).
mscarey
changed the title
OpinionReading should not have Opinion as attribute
Restructure Holding links to Opinions and Decisions
Oct 24, 2021
Instead, OpinionReading can have
opinion_author
andopinion_type
attributes corresponding to theauthor
andtype
found on the Opinion model. That way the Decision and OpinionReading don't both need to have the Opinion as attributes.Decision.posit could take
author
andtype
parameters to pass the Holdings that need to be posited down to the proper OpinionReading (but if there's only one OpinionReading, those params could be omitted).The text was updated successfully, but these errors were encountered: