Bah, that was a short-lived experiment with returning from all functions.

git-svn-id: svn://svn.open-ils.org/ILS/trunk@13244 dcc99617-32d9-48b4-a31d-7c20da2025e4

diff --git a/Open-ILS/web/opac/skin/default/js/rdetail.js b/Open-ILS/web/opac/skin/default/js/rdetail.js
index 866771d..975a4c0 100644 (file)
--- a/Open-ILS/web/opac/skin/default/js/rdetail.js
+++ b/Open-ILS/web/opac/skin/default/js/rdetail.js
@@ -200,7 +200,6 @@ function OpenMarcEditWindow(pcrud, rec) {
                        },
                }
        };
-       return null;
 }
 
 function loadMarcEditor(recId) {
@@ -209,7 +208,6 @@ function loadMarcEditor(recId) {
        if (rec) {
                OpenMarcEditWindow(pcrud, rec);
        }
-       return null;
 }
 
 /*
@@ -221,7 +219,6 @@ function _holdingsDraw(h) {
        if (!holdings) { return null; }
 
        dojo.forEach(holdings, _holdingsDrawMFHD);
-       return null;
 }
 
 function _holdingsDrawMFHD(holdings, entryNum) {
@@ -261,13 +258,11 @@ function _holdingsDrawMFHD(holdings, entryNum) {
                dojo.require('openils.PermaCrud');
                dojo.place("<span> - </span><a class='classic_link' href='javascript:loadMarcEditor(" + holdings.id() + ")'> Edit</a>", "mfhdHoldingsCaption", "last");
        }
-       return null;
 }
 
 function _holdingsDrawMFHDEntry(entryNum, entryName, entry) {
        var flatEntry = entry.toString().replace(/,/g, ', ');
        dojo.place("<tr><td> </td><td nowrap='nowrap' class='rdetail_desc'>" + entryName + "</td><td class='rdetail_item'>" + flatEntry + "</td></tr>", "rdetail_holdings_tbody_" + entryNum, "last");
-       return null;
 }
 
 function _rdetailDraw(r) {