Attachment 118550 Details for Bug 23666 – Bug 23666: (QA follow-up) Fix wrong required permissions

[patch] Bug 23666: (QA follow-up) Fix wrong required permissions

Bug-23666-QA-follow-up-Fix-wrong-required-permissi.patch (text/plain), 2.06 KB, created by Tomás Cohen Arazi (tcohen) on 2021-03-19 15:05:42 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tomás Cohen Arazi (tcohen)

Created: 2021-03-19 15:05:42 UTC

Size: 2.06 KB

patch

obsolete

>From 7c673f23b2846c3b424ce85f9645e538f688fd3d Mon Sep 17 00:00:00 2001
>From: Tomas Cohen Arazi <tomascohen@theke.io>
>Date: Fri, 19 Mar 2021 12:05:24 -0300
>Subject: [PATCH] Bug 23666: (QA follow-up) Fix wrong required permissions
>
>Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
>---
> api/v1/swagger/paths/patrons_extended_attributes.json | 2 +-
> t/db_dependent/api/v1/patrons_extended_attributes.t   | 6 +++---
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
>diff --git a/api/v1/swagger/paths/patrons_extended_attributes.json b/api/v1/swagger/paths/patrons_extended_attributes.json
>index 048bae827b6..c721f409c35 100644
>--- a/api/v1/swagger/paths/patrons_extended_attributes.json
>+++ b/api/v1/swagger/paths/patrons_extended_attributes.json
>@@ -79,7 +79,7 @@
>       },
>       "x-koha-authorization": {
>         "permissions": {
>-          "catalogue": "1"
>+          "borrowers": "edit_borrowers"
>         }
>       }
>     },
>diff --git a/t/db_dependent/api/v1/patrons_extended_attributes.t b/t/db_dependent/api/v1/patrons_extended_attributes.t
>index 3ee17affa86..da80c2e23ea 100755
>--- a/t/db_dependent/api/v1/patrons_extended_attributes.t
>+++ b/t/db_dependent/api/v1/patrons_extended_attributes.t
>@@ -40,7 +40,7 @@ subtest 'list_patron_attributes() tests' => sub {
> 
>     my $patron = $builder->build_object({
>         class => 'Koha::Patrons',
>-        value => { flags => 2 ** 2 } # 'catalogue' flag == 2
>+        value => { flags => 2 ** 4 } # 'borrowers' flag == 4
>     });
>     my $password = 'thePassword123';
>     $patron->set_password({ password => $password, skip_validation => 1 });
>@@ -208,8 +208,8 @@ subtest 'overwrite() tests' => sub {
> 
>     my $patron = $builder->build_object({
>         class => 'Koha::Patrons',
>-        value => { flags => 2 ** 4 + 2 ** 2 } # 'borrowers' flag == 4
>-    });                                       # 'catalogue' flag == 2
>+        value => { flags => 2 ** 4 } # 'borrowers' flag == 4
>+    });
>     my $password = 'thePassword123';
>     $patron->set_password({ password => $password, skip_validation => 1 });
>     my $userid = $patron->userid;
>-- 
>2.31.0

Actions: View | Diff | Splinter Review

Attachments on bug 23666: 118508 | 118509 | 118510 | 118550 | 118699 | 118700 | 118701 | 118702 | 118818 | 118854 | 119042 | 119388 | 119389 | 119390 | 119391 | 119392 | 119393