Merge branch...

Merge branch 'hotline#65216_pb_sur_mise_a_jour_des_articles_apres_modification_d_un_carrousel' into 'stable'

hotline #65216 article edit: when slideshow dialog is closed, validate button enabled

See merge request !2453

VERSIONS_HOTLINE/65216

+ - ticket #65216 : Articles: sur modification du carousel, le bouton "Valider" est accessible
\ No newline at end of file

public/opac/java/on_form_change_do/on_form_change_do.js

@@ -22,34 +22,29 @@
   $.fn.on_form_change_do = function(options) {
     var button = $(this);
 
-    var callback = options.callback;
+    var do_callback = function () { eval(options.callback); };
 
     var form = button.parent().nextAll('form');
 
-    if (!form.size())
+    if (!form.length)
       form = button.parent().prevAll('form');
 
-    if (!form.size())
+    if (!form.length)
       form = button.closest('form');
 
-    if(!form.size())
+    if(!form.length)
       return;
 
     form.on('change paste input',
             'input:not([id^="multiple-selection"]), select, textarea',
-            function()
-            {
-              eval(callback);
-            });
+            do_callback);
 
     if( 'undefined' == typeof(CKEDITOR))
       return;
 
-    for(var instance in CKEDITOR.instances)
-      CKEDITOR.instances[instance].on('change',
-                                      function()
-                                      {
-                                        eval(callback);
-                                      });
+    for(var instance in CKEDITOR.instances) {
+      CKEDITOR.instances[instance].on('change', do_callback);
+      CKEDITOR.instances[instance].on('dialogHide', do_callback);
+    }
   }
 } (jQuery));