Fixes page deletion for newer platform

Fixes #477
studiobosco · Apr 18, 2021 · 2427d56 · 2427d56
1 parent 1ca6f68
commit 2427d56
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 4 deletions.
diff --git a/assets/js/pages-page.js b/assets/js/pages-page.js
@@ -253,6 +253,18 @@
  this.updateContentEditorMode($tabPane, false)
  }
 
+ PagesPage.prototype.onDeletePageSingle = function(el) {
+ var $el = $(el);
+
+ $el.request('onDelete', {
+ success: function(data) {
+ $.oc.pagesPage.closeTabs(data, 'page');
+ $.oc.pagesPage.updateObjectList('page');
+ $(this).trigger('close.oc.tab', [{force: true}]);
+ }
+ });
+ }
+
  /*
  * Updates the browser title when an object is saved.
  */
@@ -275,7 +287,7 @@
  }
  }).always(function(){
  $.oc.stripeLoadIndicator.hide()
- })
+ });
  }
 
  /*

diff --git a/controllers/index/_page_toolbar.htm b/controllers/index/_page_toolbar.htm
@@ -26,11 +26,10 @@
  <button
  type="button"
  class="btn btn-default empty oc-icon-trash-o <?php if (!$objectPath): ?>hide<?php endif ?>"
- data-request="onDelete"
+ onclick="$.oc.pagesPage.onDeletePageSingle(this)"
  data-request-confirm="<?= e(trans('rainlab.pages::lang.page.delete_confirm_single')) ?>"
- data-request-success="$.oc.pagesPage.closeTabs(data, 'page'); $.oc.pagesPage.updateObjectList('page'); $(this).trigger('close.oc.tab', [{force: true}])"
  data-control="delete-button">
  </button>
 
  <?= $this->makePartial('~/modules/cms/controllers/index/_button_lastmodified.htm'); ?>
-</div>
+</div>
diff --git a/updates/version.yaml b/updates/version.yaml
@@ -56,3 +56,4 @@
 1.3.7: Adds MenuPicker formwidget. Adds future support for v2.0 of October CMS.
 1.4.0: Fixes bug when adding menu items in October CMS v2.0.
 1.4.1: Fixes support for configuration values.
+1.4.3: Fixes page deletion is newer platform builds.