Attachment 174132 Details for Bug 38010 – Bug 38010: Make the delete button conditional on whether a vendor has baskets or subscriptions

[patch] Bug 38010: Make the delete button conditional on whether a vendor has baskets or subscriptions

Bug-38010-Make-the-delete-button-conditional-on-wh.patch (text/plain), 2.04 KB, created by Matt Blenkinsop on 2024-11-07 11:48:55 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Matt Blenkinsop

Created: 2024-11-07 11:48:55 UTC

Size: 2.04 KB

patch

obsolete

>From dd52c786b9b5a6be1beab2bd4ad633a11b6b2f55 Mon Sep 17 00:00:00 2001
>From: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
>Date: Fri, 27 Sep 2024 14:34:42 +0000
>Subject: [PATCH] Bug 38010: Make the delete button conditional on whether a
> vendor has baskets or subscriptions
>
>---
> .../js/vue/components/Vendors/VendorList.vue   | 18 +++++++++++++++---
> 1 file changed, 15 insertions(+), 3 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/js/vue/components/Vendors/VendorList.vue b/koha-tmpl/intranet-tmpl/prog/js/vue/components/Vendors/VendorList.vue
>index 9f3fe26e686..25b1a94f0a7 100644
>--- a/koha-tmpl/intranet-tmpl/prog/js/vue/components/Vendors/VendorList.vue
>+++ b/koha-tmpl/intranet-tmpl/prog/js/vue/components/Vendors/VendorList.vue
>@@ -74,8 +74,20 @@ export default {
>                     }),
>                 },
>                 actions: {
>-                    0: ["show"],
>-                    "-1": ["edit", "delete"],
>+                    "-1": [
>+                        "edit",
>+                        {
>+                            delete: {
>+                                text: this.$__("Delete"),
>+                                icon: "fa fa-trash",
>+                                should_display: row =>
>+                                    (!row.baskets ||
>+                                        row.baskets.length === 0) &&
>+                                    (!row.subscriptions ||
>+                                        row.subscriptions.length === 0),
>+                            },
>+                        },
>+                    ],
>                 },
>             },
>             before_route_entered: false,
>@@ -121,7 +133,7 @@ export default {
>                     cancel_label: this.$__("No, do not delete"),
>                 },
>                 () => {
>-                    const client = APIClient.erm
>+                    const client = APIClient.acquisition
>                     client.vendors.delete(vendor.id).then(
>                         success => {
>                             this.setMessage(
>-- 
>2.39.3 (Apple Git-146)

Actions: View | Diff | Splinter Review

Attachments on bug 38010: 174119 | 174120 | 174121 | 174122 | 174123 | 174124 | 174125 | 174126 | 174128 | 174129 | 174130 | 174131 | 174132 | 174133 | 174134 | 174135 | 174136 | 174137 | 174138 | 174139 | 174140 | 174141 | 174142 | 174143 | 174144 | 174145 | 174146 | 174147 | 174148 | 174151 | 174152 | 174153 | 174154 | 174155 | 174156 | 174157 | 174158 | 174159 | 174160 | 174161 | 174162 | 174163 | 174164 | 174165 | 174166 | 174167 | 174168 | 174169 | 174170 | 174171 | 174172 | 174173 | 174174 | 174175 | 174176 | 174177 | 174178 | 174179 | 174194 | 174196 | 174197 | 174198 | 174199 | 174200 | 174201 | 174202 | 174203 | 174204 | 174205 | 174206 | 174207 | 174208 | 174209 | 174210 | 174211 | 174212 | 174213 | 174214 | 174215 | 174216 | 174217 | 174218 | 174220 | 174221 | 174222 | 174223 | 174224 | 174225 | 174226 | 174248 | 174266 | 174267 | 174269 | 174270 | 174273 | 174327 | 174328 | 174329 | 174330 | 174331 | 174332 | 174333 | 174334 | 174335 | 174336 | 174337 | 174338 | 174339 | 174340 | 174341 | 174342 | 174343 | 174344 | 174345 | 174346 | 174347 | 174348 | 174349 | 174350 | 174351 | 174352 | 174353 | 174354 | 174355 | 174356 | 174357 | 174358 | 174359 | 174360 | 174603