[open-ils-commits] r19398 - trunk/Open-ILS/xul/staff_client/server/cat (dbs)

[svn at svn.open-ils.org]

Author: dbs
Date: 2011-02-07 22:43:08 -0500 (Mon, 07 Feb 2011)
New Revision: 19398

Modified:
   trunk/Open-ILS/xul/staff_client/server/cat/marcedit.js
Log:
Prevent authority context menu from being displayed on empty content (LP 712538)

It's confusing to have the authority context menu - particularly the
"Create authority" options - appear for empty content. Check to ensure
that the subfield being clicked on actually has some content before
displaying the authority context menu.


Modified: trunk/Open-ILS/xul/staff_client/server/cat/marcedit.js
===================================================================
--- trunk/Open-ILS/xul/staff_client/server/cat/marcedit.js	2011-02-04 20:29:03 UTC (rev 19397)
+++ trunk/Open-ILS/xul/staff_client/server/cat/marcedit.js	2011-02-08 03:43:08 UTC (rev 19398)
@@ -1928,7 +1928,9 @@
         return false;
     }
 
-    browseAuthority( sf_popup, menu_id, target, sf, 20, page);
+    if (sf.toString()) {
+        browseAuthority(sf_popup, menu_id, target, sf, 20, page);
+    }
 
     return true;
 }