Changeset 8219

Timestamp:

02/11/10 13:57:36 (6 months ago)

Author:

dragisak

Message:

Fix missing collaborative authors in the feed.

Addresses JO-12697

Files:

• head/ambra/webapp/src/main/java/org/topazproject/ambra/struts2/AmbraFeedResult.java (modified) (3 diffs)

head/ambra/webapp/src/main/java/org/topazproject/ambra/struts2/AmbraFeedResult.java

@@ -2 +2 @@
  * $Id::                                                      $
  *
- * Copyright (c) 2006-2009 by Topaz, Inc. http://topazproject.org
+ * Copyright (c) 2006-2010 by Topaz, Inc. http://topazproject.org
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
@@ -28 +28 @@
 import com.sun.syndication.feed.atom.Person;
 import com.sun.syndication.io.WireFeedOutput;
+
+import java.util.HashSet;
 import java.util.List;
 import java.util.ArrayList;
@@ -699 +701 @@
   private List<Person> newContributorsList(Article article) {
     List<Person> newContributors = new ArrayList<Person>();
+
+    /*
+     * FIXME: contributors field exists in DublinCore but it's not being populated on ingest.
+     * article.getDublinCore().getContributors() is useless at the moment.
+     */
     Set<String> contributors = article.getDublinCore().getContributors();
+
+    Citation citation = article.getDublinCore().getBibliographicCitation();
+    if (citation != null) {
+      // Avoid duplicates
+      contributors = new HashSet<String>(contributors); 
+      contributors.addAll(citation.getCollaborativeAuthors());
+    }
 
     for (String contributor: contributors) {