Changeset 7456

Timestamp:

03/17/09 18:24:09 (17 months ago)

Author:

ronald

Message:

Display authors in search results in the order they appear in the article.

DublinCore?.creators is currently unordered; while it would be possible to
change it to be ordered, that would not come for free in terms of query
performance. Using the already ordered author list from the citation is a
bit slower, but not hugely so: it slows down retrieval of the article
info for a search hit by 10% (i.e. a page full of results now takes 10s
instead of 9s).

This closes #1124.

Location:

head/ambra/webapp/src/main/java/org/topazproject/ambra/search/service

Files:

• Results.java (modified) (1 diff)
• SearchHit.java (modified) (2 diffs)

head/ambra/webapp/src/main/java/org/topazproject/ambra/search/service/Results.java

@@ -175 +175 @@
       return new SearchHit(hit.getHitScore(), hit.getUri(), article.getDublinCore().getTitle(),
                            createHighlight("body", findTextRepresentation(article).getBodyAsText()),
-                           article.getDublinCore().getCreators(),
+                           article.getDublinCore().getBibliographicCitation().getAuthorsRealNames(),
                            article.getDublinCore().getDate());
     } catch (IOException ioe) {

head/ambra/webapp/src/main/java/org/topazproject/ambra/search/service/SearchHit.java

@@ -19 +19 @@
 package org.topazproject.ambra.search.service;
 
+import java.util.Collection;
 import java.util.Date;
-import java.util.Set;
 
 import org.apache.commons.lang.StringUtils;
@@ -42 +42 @@
    */
   public SearchHit(double hitScore, String uri, String title, String highlight,
-                   Set<String> creators, Date date) {
+                   Collection<String> creators, Date date) {
     this.hitScore  = hitScore;
     this.uri       = uri;