Bug 20861: Correct EDI permissions on EDI messsages

The permission for EDI is edi_manage, but 2 pages asked
for manage_edi, allowing users not to access those.

To test:
- Add edi_manage to your permissions
- Try to access the EDIFACT messages from the
  acq start page
- Verify it doesn't work
- Apply patch and try again
- You should be able to access the page now
- Try to access the other page directly (if you don't
  have EDI data):
  /cgi-bin/koha/acqui/edimsg.pl
- Verify you can access the page and don't get a
  permission error

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
(cherry picked from commit 7883b19de5)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
This commit is contained in:
Katrin Fischer 2018-06-03 10:09:23 +02:00 committed by Fridolin Somers
parent d65b20c673
commit cbeed6cf3d
2 changed files with 2 additions and 2 deletions

View File

@ -32,7 +32,7 @@ my ( $template, $loggedinuser, $cookie, $userflags ) = get_template_and_user(
query => $q,
type => 'intranet',
authnotrequired => 0,
flagsrequired => { acquisition => 'manage_edi' },
flagsrequired => { acquisition => 'edi_manage' },
debug => 1,
}
);

View File

@ -32,7 +32,7 @@ my ( $template, $loggedinuser, $cookie, $userflags ) = get_template_and_user(
query => $q,
type => 'intranet',
authnotrequired => 0,
flagsrequired => { acquisition => 'manage_edi' },
flagsrequired => { acquisition => 'edi_manage' },
debug => 1,
}
);