Skip to content

Improved: VIEW permissions - invoice notes (OFBIZ-12431)#383

Merged
JacquesLeRoux merged 1 commit intoapache:trunkfrom
PierreSmits:OFBIZ-12431-VIEW-permissions-invoice-notes
Dec 4, 2021
Merged

Improved: VIEW permissions - invoice notes (OFBIZ-12431)#383
JacquesLeRoux merged 1 commit intoapache:trunkfrom
PierreSmits:OFBIZ-12431-VIEW-permissions-invoice-notes

Conversation

@PierreSmits
Copy link
Member

Though there is an entity InvoiceNote, this cannot be accessed by user with VIEW permissions when accessing an invoice.
See:
for entity data:https://demo-trunk.ofbiz.apache.org/webtools/control/entity/find/InvoiceNote
test invoice: https://demo-trunk.ofbiz.apache.org/accounting/control/invoiceOverview?invoiceId=demo11001

Modified:
AccountingMenus.xml

  • added menu-item for invoice notes to InvoiceTabBar menu, reordered menu-items alphabetically
    controller.xml
  • added request-map and view-map for request invoiceNotes
    InvoiceScreens.xml
  • added screen for invoice notes
    InvoiceForms.xml
  • added grid to display InvoiceNote records
    AccountingUiLabels.xml
  • added label for page title invoice notes.

Though there is an entity InvoiceNote, this cannot be accessed by user with VIEW permissions when accessing an invoice.
See:
for entity data:https://demo-trunk.ofbiz.apache.org/webtools/control/entity/find/InvoiceNote
test invoice: https://demo-trunk.ofbiz.apache.org/accounting/control/invoiceOverview?invoiceId=demo11001

Modified:
AccountingMenus.xml
- added menu-item for invoice notes to InvoiceTabBar menu, reordered menu-items alphabetically
controller.xml
- added request-map and view-map for request invoiceNotes
InvoiceScreens.xml
- added screen for invoice notes
InvoiceForms.xml
- added grid to display InvoiceNote records
AccountingUiLabels.xml
- added label for page title invoice notes.
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 3, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@JacquesLeRoux
Copy link
Contributor

Hi Pierre,

It works but you forgot to (at least) mention the (right) changes in AccountingMenus::editInvoiceApplications and EditInvoiceTimeEntries. It's OK but please rather open specific issues for these kind of changes. It's much work for reviewers/committers to understand what is really done, TIA

@JacquesLeRoux JacquesLeRoux merged commit 32c4b6b into apache:trunk Dec 4, 2021
@PierreSmits PierreSmits deleted the OFBIZ-12431-VIEW-permissions-invoice-notes branch December 4, 2021 09:03
@PierreSmits
Copy link
Member Author

Good morning Jacques,

Are you referring to the alphabetically reordering of certain menu-items in the InvoiceTabBar menu? If you are, I suggest you pay attention to the first 2 lines beneath 'modified in both commit and PR message.

@JacquesLeRoux
Copy link
Contributor

I meant the "add" of


But it's indeed actually the reordering that mislead me (removed on one side added to other, I used the horizontal GH review).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants