5912 2010-09-20 09:35:00 -0700 Monograph value object contains references to DAOs 2010-11-02 15:40:56 -0700 1 1 1 Unclassified OMP General 1.0 PC Linux RESOLVED DUPLICATE 5231 P3 normal --- 1 jerico.dev pkp-support mattcrider 21164 jerico.dev 2010-09-20 09:35:37 -0700 The signoff-related accessors in the Monograph object instantiate DAOs and access data in the database. We usually implement application objects as simple value objects and let them be populated by the DAO. So usually the DAOs populate the application objects while the application objects remain unaware of the DAO. (See the classical GoF DAO pattern). If these fields have an important performance impact when populated or cannot be populated in all circumstances then we usually design subclasses with these accessors to be used when use-cases require the extra fields while leaving the base Monograph class alone, see the many Submission derivatives for examples. What are the reasons for parting from the DAO pattern in Monograph? 21165 jerico.dev 2010-09-20 09:38:46 -0700 Same for getAuthors() and getPrimaryAuthor(). 21169 jerico.dev 2010-09-20 10:20:27 -0700 Please also see #5913 for the author accessors. 21555 mattcrider 2010-10-08 11:00:18 -0700 These are convenience functions, there mostly because authors used to be stored in the Monograph object as an array, but are now handled completely in the AuthorDAO (though there was really no reason to maintain backwards-compatibility there, since there is no backwards in OMP). I'm tempted to keep these methods in the Article and Paper classes to maintain backwards-compatibility though. Lets close this bug and continue any discussion in bug 5231. 21563 jerico.dev 2010-10-08 15:45:37 -0700 Thanks Matt, that makes sense to me. And yes, agreed, we can discuss that in 5231. 21897 jerico.dev 2010-11-02 15:40:56 -0700 *** This bug has been marked as a duplicate of bug 5231 ***