fix javascript calls to delete routes which currently don't work

ChurchCRM · Dec 31, 2023 · 7884a00 · 7884a00
1 parent 42ac712
commit 7884a00
Show file tree

Hide file tree

Showing 6 changed files with 6 additions and 12 deletions.
diff --git a/react/components/Events/ExistingEvent.tsx b/react/components/Events/ExistingEvent.tsx
@@ -172,12 +172,11 @@ class ExistingEvent extends React.Component<EventFormProps, EventFormState> {
   delete() {
     fetch(CRMRoot + "/api/events/" + this.props.eventId, {
       credentials: "include",
-      method: "POST",
+      method: "DELETE",
       headers: {
         Accept: "application/json",
         "Content-Type": "application/json",
       },
-      body: JSON.stringify({ _METHOD: "DELETE" }),
     }).then(() => this.exit());
   }
 

diff --git a/src/skin/js/CRMJSOM.js b/src/skin/js/CRMJSOM.js
@@ -125,9 +125,9 @@ window.CRM.cart = {
     },
     removePerson: function (Persons, callback) {
         window.CRM.APIRequest({
-            method: "POST",
+            method: "DELETE",
             path: "cart/",
-            data: JSON.stringify({ _METHOD: "DELETE", Persons: Persons }),
+            data: JSON.stringify({ Persons: Persons }),
         }).done(function (data) {
             window.CRM.cart.refresh();
             if (callback) {

diff --git a/src/skin/js/FamilyView.js b/src/skin/js/FamilyView.js
@@ -298,17 +298,14 @@ $(document).ready(function () {
 
     $("#deletePhoto").click(function () {
         $.ajax({
-            type: "POST",
+            type: "DELETE",
             url:
                 window.CRM.root +
                 "/api/family/" +
                 window.CRM.currentFamily +
                 "/photo",
             encode: true,
             dataType: "json",
-            data: {
-                _METHOD: "DELETE",
-            },
         }).done(function (data) {
             location.reload();
         });

diff --git a/src/skin/js/GroupEditor.js b/src/skin/js/GroupEditor.js
@@ -109,11 +109,10 @@ $("document").ready(function () {
     $(document).on("click", ".deleteRole", function (e) {
         var roleID = e.currentTarget.id.split("-")[1];
         $.ajax({
-            method: "POST",
+            method: "DELETE",
             url:
                 window.CRM.root + "/api/groups/" + groupID + "/roles/" + roleID,
             encode: true,
-            data: { _METHOD: "DELETE" },
             dataType: "json",
         }).done(function (data) {
             dataT.clear();

diff --git a/src/skin/js/users.js b/src/skin/js/users.js
@@ -30,7 +30,6 @@ function deleteUser(userId, userName) {
                 window.CRM.APIRequest({
                     method: "DELETE",
                     path: "user/" + userId + "/",
-                    data: { _METHOD: "DELETE" },
                 }).done(function () {
                     window.location.href = window.CRM.root + "/UserList.php";
                 });

diff --git a/src/v2/templates/cart/cartlisting.php b/src/v2/templates/cart/cartlisting.php
@@ -33,7 +33,7 @@
             </td>
             <td><?= $Person->getAddress() ?></td>
             <td><?= $Person->getEmail() ?></td>
-            <td><a class="RemoveFromPeopleCart" data-personid="<?= $Person->getId() ?>"><?= gettext('Remove') ?></a>
+            <td><button class="RemoveFromPeopleCart" data-personid="<?= $Person->getId() ?>"><?= gettext('Remove') ?></button>
             </td>
             <td><?= $Person->getClassificationName() ?></td>
             <td><?= $Person->getFamilyRoleName() ?></td>