[Bug 38298] New: Edifact breadcrumbs need to be permissions based
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38298 Bug ID: 38298 Summary: Edifact breadcrumbs need to be permissions based Change sponsored?: --- Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Acquisitions Assignee: koha-bugs@lists.koha-community.org Reporter: matt.blenkinsop@ptfs-europe.com QA Contact: testopia@bugs.koha-community.org 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 -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38298 --- Comment #1 from Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> --- Created attachment 173714 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=173714&action=edit 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 -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38298 Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38298 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Edifact breadcrumbs need to |EDIFACT breadcrumbs need to |be permissions based |be permissions based -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38298 Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |matt.blenkinsop@ptfs-europe |ity.org |.com -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38298 Brendan Gallagher <brendan@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38298 Brendan Gallagher <brendan@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #173714|0 |1 is obsolete| | --- Comment #2 from Brendan Gallagher <brendan@bywatersolutions.com> --- Created attachment 180099 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=180099&action=edit 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 Signed-off-by: Brendan Gallagher <brendan@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38298 Baptiste Wojtkowski (bwoj) <baptiste.wojtkowski@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |baptiste.wojtkowski@biblibr |y.org |e.com CC| |baptiste.wojtkowski@biblibr | |e.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38298 Baptiste Wojtkowski (bwoj) <baptiste.wojtkowski@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38298 Baptiste Wojtkowski (bwoj) <baptiste.wojtkowski@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #180099|0 |1 is obsolete| | --- Comment #3 from Baptiste Wojtkowski (bwoj) <baptiste.wojtkowski@biblibre.com> --- Created attachment 182857 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=182857&action=edit 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 Signed-off-by: Brendan Gallagher <brendan@bywatersolutions.com> Signed-off-by: Baptiste Wojtkowski <baptiste.wojtkowski@biblibre.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38298 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |25.11.00 released in| | Status|Passed QA |Pushed to main -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38298 --- Comment #4 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Nice work everyone! Pushed to main for 25.11 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38298 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)|25.11.00 |25.11.00,25.05.01 released in| | Status|Pushed to main |Pushed to stable CC| |fridolin.somers@biblibre.co | |m --- Comment #5 from Fridolin Somers <fridolin.somers@biblibre.com> --- I see in 25.05.x -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38298 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to stable |Needs documenting --- Comment #6 from Fridolin Somers <fridolin.somers@biblibre.com> --- I prefer not backport to 24.11.x for stability -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38298 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This fixes the breadcrumbs release notes| |to EDI accounts, so that | |the 'Administration' | |breadcrumb link is not | |shown if the staff patron | |doesn't have permission to | |access administration | |pages, but does have | |permission to manage EDIT | |accounts (Acquisitions > | |Administration > EDI | |accounts). | | | |What happens after this | |change: | |- Administration | |permissions: sees | |Administration > EDI | |accounts (no change) | |- Only Manage EDIFACT | |transmissions (edi_manage) | |permissions: sees | |Acquisitions > EDI accounts | |(previously Administration | |> EDI accounts was shown, | |but they couldn't access | |the administration pages) CC| |david@davidnind.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38298 Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs documenting |RESOLVED Resolution|--- |FIXED --- Comment #7 from Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> --- Nothing to change in the manual. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org