Bug 38298 - EDIFACT breadcrumbs need to be permissions based
Summary: EDIFACT breadcrumbs need to be permissions based
Status: Needs Signoff
Alias: None
Product: Koha
Classification: Unclassified
Component: Acquisitions (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-10-30 12:10 UTC by Matt Blenkinsop
Modified: 2024-10-30 12:16 UTC (History)
0 users

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

Attachments
Bug 38298: Add permission checks to EDI breadcrumbs (1.61 KB, patch)
2024-10-30 12:14 UTC, Matt Blenkinsop 		Details | Diff | Splinter Review
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Matt Blenkinsop 2024-10-30 12:10:08 UTC 
The breadcrumbs in ed_accounts.pl point to Administration but if the user doesn't have permissions for this then they can't access the page. It should be permissions based and point to acquisitions if they don't have permission to access administrations
Comment 1 Matt Blenkinsop 2024-10-30 12:14:32 UTC 
Created attachment 173714 [details] [review]
Bug 38298: Add permission checks to EDI breadcrumbs

Test plan:
1) Choose a staff patron and assign them the following permissions
    - catalogue
    - edi_manage
2) Enable the edifact syspref
3) Log in as the chosen staff patron and navigate to Acquisitions > EDI accounts from the left hand menu
4) The breadcrumb will contain Administration
5) Clicking this will result in a permissions error
6) Apply patch
7) Refresh the page and the breadcrumb will now read Acquisitions and will work