Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Document references endpoints are not respecting role permissions #651

Closed
teosarca opened this issue Oct 30, 2017 · 1 comment
Closed

Document references endpoints are not respecting role permissions #651

teosarca opened this issue Oct 30, 2017 · 1 comment

Comments

@teosarca
Copy link
Member

Is this a bug or feature request?

Bug

Which are the steps to reproduce?

e.g.

  • open a completed sales order
  • check document references. You shall see something like MRP Datensätze, along to other references
  • go to role, disable the access to that window
  • go back to sales order and check the references again
    ** the "MRP Datensätze" is still there => NOK
    ** when clicking on it a new window is opened with "No Access" error

What is the expected or desired behavior?

If user has no access to that window, don't show it in references.

@metas-lc
Copy link

metas-lc commented Nov 1, 2017

IT

  1. open a completed sales order
  2. check document references.
    => there is MRP Datensätze, along to other references. Open it => window opens, no errors OK
  3. go to role window, select the one you logged in with, go to window subtab and disable MRP Datensätze
  4. go back to sales order, check document references
    => MRP Datensätze is not shown anymore OK
  5. try to open MRP Datensätze with direct link => no access error OK
  6. go to role and put MRP Datensätze back
  7. check sales order => MRP Datensätze is shown OK

@metas-lc metas-lc closed this as completed Nov 1, 2017
metas-ts added a commit to metasfresh/metasfresh that referenced this issue Nov 5, 2017
[#2881](#2881) Add a
process that removes pauses and can be ran from Contract window
[#633](metasfresh/metasfresh-webui-api-legacy#633)
Materialdispo Target Warehouse checks Warehouse Routing
[#2895](#2895)
AD_Ref_Table needs lower-case WHERE and FROM
[#2273](#2273) Payment
Selection as real Document
[#2862](#2862) Set
MasterStartDate for a contract when creating from order
[#2880](#2880) Add
greeting to partner quick creation from order
[#656](metasfresh/metasfresh-webui-api-legacy#656)
Address editor fields are not translated
[#654](metasfresh/metasfresh-webui-api-legacy#654)
Address country lookup not working correctly
[#1295](metasfresh/metasfresh-webui-frontend-legacy#1295)
processes: provide current selected tab and rows
[#1295](metasfresh/metasfresh-webui-api-legacy#653)
metasfresh.webui.debug.showColumnNamesForCaption shall be false by
default
[#651](metasfresh/metasfresh-webui-api-legacy#651)
Document references endpoints are not respecting role permissions
[#2878](#2878) Support
for 6 digits AccountNo length in ReferenceNumber
[#1312](metasfresh/metasfresh-webui-frontend-legacy#1312)
Wrong process call
[#2876](#2876) Fix
Recreate printing queue for webui
[#2874](#2874) Create
special filters in printing queue
[metasfresh-dist-orgs#82](metasfresh/metasfresh-dist-orgs#82)
Invoice Medium Address Layout Changes
[#2866](#2866) Make sure
that 'Print All' is working in webui
[#2803](#2803) Set
hostkey for printing module when login in webui
[#2113](#2113) Move
verify BOM process to BOM window

me-45
metas-ts added a commit to metasfresh/metasfresh that referenced this issue Nov 5, 2017
[#2881](#2881) Add a
process that removes pauses and can be ran from Contract window
[#633](metasfresh/metasfresh-webui-api-legacy#633)
Materialdispo Target Warehouse checks Warehouse Routing
[#2895](#2895)
AD_Ref_Table needs lower-case WHERE and FROM
[#2273](#2273) Payment
Selection as real Document
[#2862](#2862) Set
MasterStartDate for a contract when creating from order
[#2880](#2880) Add
greeting to partner quick creation from order
[#656](metasfresh/metasfresh-webui-api-legacy#656)
Address editor fields are not translated
[#654](metasfresh/metasfresh-webui-api-legacy#654)
Address country lookup not working correctly
[#1295](metasfresh/metasfresh-webui-frontend-legacy#1295)
processes: provide current selected tab and rows
[#1295](metasfresh/metasfresh-webui-api-legacy#653)
metasfresh.webui.debug.showColumnNamesForCaption shall be false by
default
[#651](metasfresh/metasfresh-webui-api-legacy#651)
Document references endpoints are not respecting role permissions
[#2878](#2878) Support
for 6 digits AccountNo length in ReferenceNumber
[#1312](metasfresh/metasfresh-webui-frontend-legacy#1312)
Wrong process call
[#2876](#2876) Fix
Recreate printing queue for webui
[#2874](#2874) Create
special filters in printing queue
[metasfresh-dist-orgs#82](metasfresh/metasfresh-dist-orgs#82)
Invoice Medium Address Layout Changes
[#2866](#2866) Make sure
that 'Print All' is working in webui
[#2803](#2803) Set
hostkey for printing module when login in webui
[#2113](#2113) Move
verify BOM process to BOM window

me-45

(cherry picked from commit b4208ff)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants