From 1cf0564ef8ae86ac5124b05c783133a63a776fde Mon Sep 17 00:00:00 2001 From: Leila Date: Fri, 30 Jan 2026 16:07:00 +0100 Subject: [PATCH] Improved: Lookup don't work on area just updated (OFBIZ-13332) fix to avoid bindObserver events duplication --- .../webapp/common-theme/js/util/OfbizUtil.js | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/themes/common-theme/webapp/common-theme/js/util/OfbizUtil.js b/themes/common-theme/webapp/common-theme/js/util/OfbizUtil.js index ece42807f3..2824ec8565 100644 --- a/themes/common-theme/webapp/common-theme/js/util/OfbizUtil.js +++ b/themes/common-theme/webapp/common-theme/js/util/OfbizUtil.js @@ -666,15 +666,11 @@ function ajaxUpdateArea(areaId, target, targetParams) { function updateArea(areaId, data) { // If the area is indicate as embedded why replace the area instead inject into - var bindObserversArea = "#" + areaId if (/^embedded/.test(areaId)) { jQuery("#" + areaId).replaceWith(data); - const newContentId = $(data).filter('.embeddedScreen').attr('id'); - bindObserversArea = "#" + newContentId; } else { jQuery("#" + areaId).html(data); } - bindObservers(bindObserversArea); } /** Update multiple areas (HTML container elements). @@ -847,6 +843,12 @@ function ajaxSubmitFormUpdateAreas(formName, areaCsvString, close) { areaId = $form[0].target } updateArea(areaId, data) + var bindObserversArea = "#" + areaId + if (/^embedded/.test(areaId)) { + const newContentId = $(data).filter('.embeddedScreen').attr('id'); + bindObserversArea = "#" + newContentId; + } + bindObservers(bindObserversArea); } else { if (containsErrorMessages(data)) { displayErrorMessages(data)