[open-ils-commits] r1335 - conifer/branches/rel_2_0/web/opac/skin/lul/xml/rdetail (dbs)

svn at svn.open-ils.org svn at svn.open-ils.org
Tue Apr 12 17:10:19 EDT 2011


Author: dbs
Date: 2011-04-12 17:10:13 -0400 (Tue, 12 Apr 2011)
New Revision: 1335

Modified:
   conifer/branches/rel_2_0/web/opac/skin/lul/xml/rdetail/rdetail_summary.xml
Log:
IE really kicks it up a notch here; ugly.

It's still not clear why we seem to need to use raw DOM
accessors in subjectMe() in 2.0 but at least we have a working
solution for IE again.


Modified: conifer/branches/rel_2_0/web/opac/skin/lul/xml/rdetail/rdetail_summary.xml
===================================================================
--- conifer/branches/rel_2_0/web/opac/skin/lul/xml/rdetail/rdetail_summary.xml	2011-04-12 20:18:51 UTC (rev 1334)
+++ conifer/branches/rel_2_0/web/opac/skin/lul/xml/rdetail/rdetail_summary.xml	2011-04-12 21:10:13 UTC (rev 1335)
@@ -482,12 +482,16 @@
             var total = '';
             var output = [];
             dojo.query( 'subfield:not([code=2])', item ).forEach( function (onesub) {
-                var subtext;
+                var subtext = '';
                 try {   
+                    if (dojo.isIE) {
+                        subtext = onesub.firstChild.nodeValue;
+                    } else {
                         subtext = onesub.textContent;
-                        total +=  subtext + ' ';
+                    }
+                    total +=  subtext + ' ';
                 } catch (e) {
-                        return;
+                    return;
                 }
                 var current = '<a class="search_link" href="rresult.xml?rt=subject&tp=subject&t=' + total;
                 for (var p in other_params) {



More information about the open-ils-commits mailing list