From 6c7591ab82d78135d968a82dbd14a4805763a857 Mon Sep 17 00:00:00 2001 From: Dan Wells Date: Mon, 14 Oct 2013 15:24:39 -0400 Subject: [PATCH] Fix integrated subscription editor record field When trying to edit a subscription using the new 'integrated' editor, it foolishly tries to populate an unfiltered bre dropdown selector. This works OK in test environments, but (as expected) fails miserably with production-size data sets. The end result is that the dropdown either never populates, or takes a really long time. If the user saves before the dropdown loads, the associated record ID is lost, and the subscription is no longer properly associated with the record. To fix this, simply turn the dropdown into a disabled text input instead, as the ability to edit this field would generally be more harmful than it is necessary. Signed-off-by: Dan Wells Signed-off-by: Lebbeous Fogle-Weekley --- Open-ILS/web/js/ui/default/serial/subscription.js | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Open-ILS/web/js/ui/default/serial/subscription.js b/Open-ILS/web/js/ui/default/serial/subscription.js index bedb5878b8..5860a56d56 100644 --- a/Open-ILS/web/js/ui/default/serial/subscription.js +++ b/Open-ILS/web/js/ui/default/serial/subscription.js @@ -334,6 +334,8 @@ openils.Util.addOnLoad( } ssub_grid.showCreateDialog(); + } else { + ssub_grid.overrideWidgetArgs.record_entry = {widgetClass : "dijit.form.TextBox", dijitArgs : {disabled : true}}; } ssub_grid.onPostUpdate = function(fmObject) { parent.document.getElementById(window.name).refresh_command(); -- 2.43.2