Changeset 7412

Timestamp:

03/12/09 16:17:43 (17 months ago)

Author:

josowski

Message:

Updates, changes and improvements to related articles linking logic.

References #1115

Files:

• head/ambra/webapp/src/main/webapp/javascript/init_related.js (modified) (6 diffs)

head/ambra/webapp/src/main/webapp/javascript/init_related.js

@@ -38 +38 @@
   dojo.fadeOut({ node:'relatedArticlesSpinner', duration: 1000 }).play();
   
-  if(response == null)
+  if (response == null)
   {
     setError('pubMedRelatedErr');
@@ -46 +46 @@
   var pubMedID = 0;
   
-  if(response.article.pub_med) {
+  if (response.article.pub_med) {
     pubMedID = response.article.pub_med;
     dojo.byId('pubMedRelatedURL').href="http://www.ncbi.nlm.nih.gov/sites/entrez?Db=pubmed&DbFrom=pubmed&Cmd=Link&LinkName=pubmed_pubmed&LinkReadableName=Related%20Articles&IdsFromResult=" + pubMedID + "&ordinalpos=1&itool=EntrezSystem2.PEntrez.Pubmed.Pubmed_ResultsPanel.Pubmed_RVCitation";
@@ -58 +58 @@
   dojo.fadeOut({ node:'relatedCitesSpinner', duration: 1000 }).play();
 
-  if(response == null)
+  if (response == null)
   {
     setError('relatedCites');
@@ -64 +64 @@
   }
 
-  if(response.article.sources.length > 0) {
+  var numCitesRendered = 0;
+  var doi = escape(dojo.byId('doi').value);
+
+  if (response.article.sources.length > 0) {
     var html = "<ul>";
 
     for(var a = 0; a < response.article.sources.length; a++)
     {
-      html = html + "<li><a href=\"\">" + response.article.sources[a].source + "(" + response.article.sources[a].count + ")</a></li>";
+      var url = "";
+
+      /**
+       * TODO: We really want to have the ALM server set these sources
+       * In the meantime we'll set them here.
+       *
+       * I figure we'll append a DOI to these urls with the right parameter name
+       * Or perhaps append an ID given to us from the ALM server?
+       *
+       * Nulling out any of the following values should stop the
+       * item from being displayed. 
+      */
+      switch (response.article.sources[a].source) {
+        case "CrossRef":
+            //Example of what it would like like to pass the doi to the website
+            url = "http://www.crossref.org/?paramName=" + doi; //+ response.article.sources[a].source.someID
+            //Example of what it would like if we got a response from the ALM server
+            //url = "http://www.crossref.org/?paramName=" + response.article.sources[a].source.someID;
+            //Example of null:
+            //url = null;
+          break;
+        case "PubMed":
+            url = "http://www.ncbi.nlm.nih.gov/pubmed/?paramName="; //+ doi (the article DOI)
+          break;
+        case "Scopus":
+            url = "http://www.scopus.com/scopus/home.url?paramName=";
+          break;
+        case "Citeulike":
+            url = "http://www.citeulike.org/?paramName=";
+          break;
+      }
+
+      //Only list links that HAVE DEFINED URLS
+      if (url) {
+        html = html + "<li><a href=\"" + url + "\">" + response.article.sources[a].source + "(" + response.article.sources[a].count + ")</a></li>";
+        numCitesRendered++;
+      }
     }
     
     html = html + "</ul>"
-  } else {
+  }
+
+  if (numCitesRendered == 0){
     html = "<ul><li>No related cites found</li></ul>";
   }
@@ -88 +129 @@
   dojo.fadeOut({ node:'relatedBlogSpinner', duration: 1000 }).play();
 
-  if(response == null)
+  if (response == null)
   {
     setError('relatedBlogPosts');
@@ -94 +135 @@
   }
 
-  if(response.article.sources.length > 0) {
+  var numBlogsRendered = 0;
+  var doi = escape(dojo.byId('doi').value);
+
+  if (response.article.sources.length > 0) {
     html = "<ul>";
 
     for(var a = 0; a < response.article.sources.length; a++)
     {
-      html = html + "<li><a href=\"\">" + response.article.sources[a].source + "(" + response.article.sources[a].count + ")</a></li>";
+      var url = "";
+
+      /**
+       * TODO: We really want to have the ALM server set these sources
+       * In the meantime we'll set them here.
+       *
+       * I figure we'll append a DOI to these urls with the right parameter name
+       * Or perhaps append an ID given to us from the ALM server?
+       *
+       * Nulling out any of the following values should stop the
+       * item from being displayed.
+      */
+      switch(response.article.sources[a].source) {
+        case "Bloglines":
+            url = "http://www.crossref.org/?paramName=" + doi;
+            //url = "http://www.crossref.org/?paramName=" + response.article.sources[a].source.someID;
+          break;
+        case "Nature":
+            url = "http://www.nature.com/?paramName=";
+          break;
+        case "Postgenomic":
+            url = "http://www.postgenomic.com/?paramName=";
+          break;
+      }
+
+      //Only list links that HAVE DEFINED URLS
+      if (url) {
+        html = html + "<li><a href=\"" + url + "\">" + response.article.sources[a].source + "(" + response.article.sources[a].count + ")</a></li>";
+        numBlogsRendered++;
+      }
     }
 
     html = html + "</ul>"
-  } else {
+  } else
+
+  if (numBlogsRendered == 0) {
     html = "<ul><li>No related blog enrties found</li></ul>"; 
   }