Improve framework code to support $(this) in data attributes

modules/backend/formwidgets/repeater/partials/_repeater_item.htm

@@ -11,7 +11,7 @@
             aria-label="Remove"
             data-request="<?= $this->getEventHandler('onRemoveItem') ?>"
             data-request-data="'index': '<?= $indexValue ?>'"
-            data-request-success="$el.closest('.field-repeater-item').remove()"
+            data-request-success="$(this).closest('.field-repeater-item').remove()"
             data-request-confirm="Are you sure?">
             <span aria-hidden="true">&times;</span>
         </button>

modules/backend/widgets/reportcontainer/partials/_new_widget_popup.htm

@@ -1,6 +1,6 @@
 <?= Form::open([
     'data-request' => $this->getEventHandler('onAddWidget'),
-    'data-request-success' => "\$el.trigger('close.oc.popup'); \$(window).trigger('oc.report-widget-added')",
+    'data-request-success' => "\$(this).trigger('close.oc.popup'); \$(window).trigger('oc.report-widget-added')",
     'data-popup-load-indicator' => 1
 ]) ?>
     <div class="modal-header">

modules/cms/controllers/index/_content_toolbar.htm

@@ -13,6 +13,6 @@
         class="btn btn-default empty oc-icon-trash-o <?php if (!$templatePath): ?>hide<?php endif ?>"
         data-request="onDelete"
         data-request-confirm="<?= e(trans('cms::lang.content.delete_confirm_single')) ?>"
-        data-request-success="$.oc.cmsPage.updateTemplateList('content'); $(element).trigger('close.oc.tab', [{force: true}])"
+        data-request-success="$.oc.cmsPage.updateTemplateList('content'); $(this).trigger('close.oc.tab', [{force: true}])"
         data-control="delete-button"></button>
 </div>

modules/cms/controllers/index/_layout_toolbar.htm

@@ -13,6 +13,6 @@
         class="btn btn-default empty oc-icon-trash-o <?php if (!$templatePath): ?>hide<?php endif ?>"
         data-request="onDelete"
         data-request-confirm="<?= e(trans('cms::lang.layout.delete_confirm_single')) ?>"
-        data-request-success="$.oc.cmsPage.updateTemplateList('layout'); $(element).trigger('close.oc.tab', [{force: true}])"
+        data-request-success="$.oc.cmsPage.updateTemplateList('layout'); $(this).trigger('close.oc.tab', [{force: true}])"
         data-control="delete-button"></button>
 </div>

modules/cms/controllers/index/_page_toolbar.htm

@@ -24,6 +24,6 @@
         class="btn btn-default empty oc-icon-trash-o <?php if (!$templatePath): ?>hide<?php endif ?>"
         data-request="onDelete"
         data-request-confirm="<?= e(trans('cms::lang.page.delete_confirm_single')) ?>"
-        data-request-success="$.oc.cmsPage.updateTemplateList('page'); $(element).trigger('close.oc.tab', [{force: true}])"
+        data-request-success="$.oc.cmsPage.updateTemplateList('page'); $(this).trigger('close.oc.tab', [{force: true}])"
         data-control="delete-button"></button>
 </div>

modules/cms/controllers/index/_partial_toolbar.htm

@@ -13,6 +13,6 @@
         class="btn btn-default empty oc-icon-trash-o <?php if (!$templatePath): ?>hide<?php endif ?>"
         data-request="onDelete"
         data-request-confirm="<?= e(trans('cms::lang.partial.delete_confirm_single')) ?>"
-        data-request-success="$.oc.cmsPage.updateTemplateList('partial'); $(element).trigger('close.oc.tab', [{force: true}])"
+        data-request-success="$.oc.cmsPage.updateTemplateList('partial'); $(this).trigger('close.oc.tab', [{force: true}])"
         data-control="delete-button"></button>
 </div>

modules/cms/widgets/assetlist/partials/_move_form.htm

@@ -1,6 +1,6 @@
 <?= Form::open([
     'data-request'=>$this->getEventHandler('onMove'),
-    'data-request-success'=>"\$el.trigger('close.oc.popup')",
+    'data-request-success'=>"\$(this).trigger('close.oc.popup')",
     'data-stripe-load-indicator'=>1,
     'id'=>'asset-move-popup-form'
 ]) ?>

modules/cms/widgets/assetlist/partials/_new_dir_form.htm

@@ -1,6 +1,6 @@
 <?= Form::open([
     'data-request'=>$this->getEventHandler('onNewDirectory'),
-    'data-request-success'=>"\$el.trigger('close.oc.popup')",
+    'data-request-success'=>"\$(this).trigger('close.oc.popup')",
     'data-stripe-load-indicator'=>1,
     'id'=>'asset-new-dir-popup-form'
 ]) ?>

modules/system/assets/js/framework.js

@@ -95,7 +95,7 @@ if (window.jQuery === undefined)
                  * Halt here if beforeUpdate() or data-request-before-update returns false
                  */
                 if (this.options.beforeUpdate.apply(this, [data, textStatus, jqXHR]) === false) return
-                if (options.evalBeforeUpdate && eval('(function($el, context, data, textStatus, jqXHR) {'+options.evalBeforeUpdate+'}($el, context, data, textStatus, jqXHR))') === false) return
+                if (options.evalBeforeUpdate && eval('(function($el, context, data, textStatus, jqXHR) {'+options.evalBeforeUpdate+'}.call($el.get(0), $el, context, data, textStatus, jqXHR))') === false) return
 
                 /*
                  * Trigger 'ajaxBeforeUpdate' on the form, halt if event.preventDefault() is called
@@ -111,7 +111,7 @@ if (window.jQuery === undefined)
 
                 updatePromise.done(function(){
                     form.trigger('ajaxSuccess', [context, data, textStatus, jqXHR])
-                    options.evalSuccess && eval('(function($el, context, data, textStatus, jqXHR) {'+options.evalSuccess+'}($el, context, data, textStatus, jqXHR))')
+                    options.evalSuccess && eval('(function($el, context, data, textStatus, jqXHR) {'+options.evalSuccess+'}.call($el.get(0), $el, context, data, textStatus, jqXHR))')
                 })
 
                 return updatePromise
@@ -158,7 +158,7 @@ if (window.jQuery === undefined)
                     /*
                      * Halt here if the data-request-error attribute returns false
                      */
-                    if (options.evalError && eval('(function($el, context, textStatus, jqXHR) {'+options.evalError+'}($el, context, textStatus, jqXHR))') === false)
+                    if (options.evalError && eval('(function($el, context, textStatus, jqXHR) {'+options.evalError+'}.call($el.get(0), $el, context, textStatus, jqXHR))') === false)
                         return
 
                     requestOptions.handleErrorMessage(errorMsg)
@@ -167,10 +167,8 @@ if (window.jQuery === undefined)
                 return updatePromise
             },
             complete: function(data, textStatus, jqXHR) {
-
                 form.trigger('ajaxComplete', [context, data, textStatus, jqXHR])
-                options.evalComplete && eval('(function($el, context, data, textStatus, jqXHR) {'+options.evalComplete+'}($el, context, data, textStatus, jqXHR))')
+                options.evalComplete && eval('(function($el, context, data, textStatus, jqXHR) {'+options.evalComplete+'}.call($el.get(0), $el, context, data, textStatus, jqXHR))')
-
             },
 
             /*

modules/system/controllers/eventlogs/_list_toolbar.htm

@@ -17,7 +17,7 @@
         data-trigger-action="enable"
         data-trigger=".control-list input[type=checkbox]"
         data-trigger-condition="checked"
-        data-request-success="$el.prop('disabled', false)"
+        data-request-success="$(this).prop('disabled', false)"
         data-stripe-load-indicator>
         <?= e(trans('backend::lang.list.delete_selected')) ?>
     </button>

modules/system/controllers/requestlogs/_list_toolbar.htm

@@ -17,7 +17,7 @@
         data-trigger-action="enable"
         data-trigger=".control-list input[type=checkbox]"
         data-trigger-condition="checked"
-        data-request-success="$el.prop('disabled', false)"
+        data-request-success="$(this).prop('disabled', false)"
         data-stripe-load-indicator>
         <?= e(trans('backend::lang.list.delete_selected')) ?>
     </button>