Move addArticle and removeArticle methods into a service class

[npeterson]

Cross-pubbed a couple of genetics articles in compbiol

info:doi/10.1371/journal.pgen.1000390 info:doi/10.1371/journal.pgen.1000410

added them to issue.pcbi.v04.i12 and issue.pcbi.v04.i11 respectively.

each time I added one, all the other articles in teh issue were deleted, leaving the cross-pubbed article as the issues only member. I was able to add them back for v04.i12, and the crosspubbed article stayed in place. Didn't try to v04.i11, just left it as-is, with the cross-pubbed as the only member

[dragisak]

(In [7548]) Get methods in entities should not set fields.

Because of lazy loading, all values in the entity might not be initialized when get method is called. So simpleCollection might not have been initialized when getArticleList is called.

Addresses #1201, #1215 and #1228

[rcave]

Verified that cross published articles do not delete current articles when added to an issue.

[npeterson]

This still seems to happen sometimes, but it appears to have nothing to do with cross-pubbed articles -- happens for journal-native articles too. The first thing that I add to any issue (in a particular session?) reliably removes everything else. I can subsequently add other articles, or delete and re-add that same one without any problems.

[wtoconnor]

During an update the articleList gets a copy of the new information from simpleCollection. This is not the case for addArticle and removeArticle. If this is done before articleList is set with new information we loose the previous articles.

This is not the best way to solve this but it is the only thing we have time for.

[wtoconnor]

(In [7558]) Make sure articleList is set by Adds and Removes of articles.

Add and Removed worked before the Update fix. However in the process of fixing update we failed to take in to account problems with Add and Remove.

Addresses #1215

[rcave]

addArticle and removeArticle methods hold business logic and, as a part of layering the architecture, it would have been better if they were in a service class. Moving to 0.9.3

[josowski]

(In [7604]) Moving the addArticle and removeArticle methods into a service class and some minor updates to the javadocs

Addresses #1215

[rcave]

Confirmed that the addArticle and removeArticle methods were made private functions within the AdminService? class.