[MGNLUI-4586] Vaadin integration swallow exception which made errors does not show up to user Created: 21/Aug/18  Updated: 08/Mar/21  Resolved: 08/Mar/21

Status: Closed
Project: Magnolia UI
Component/s: vaadin integration
Affects Version/s: 5.7.1
Fix Version/s: None

Type: Bug Priority: Neutral
Reporter: Viet Nguyen Assignee: Unassigned
Resolution: Incomplete Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Testcase included:
Yes
Date of First Response:

 Description   

AbstractJcrNodeAdapter updateProperty does NOT throw exception in case of exception.
This cause all errors could not be detected and shown up properly to end user.
Reference here for instance:
https://git.magnolia-cms.com/projects/PLATFORM/repos/ui.pub/browse/magnolia-ui-vaadin-integration/src/main/java/info/magnolia/ui/vaadin/integration/jcr/AbstractJcrNodeAdapter.java#313
Expected result:
Scan package source to detect this kind of common mistake and process exception properly. Error code or error message should be returned or throw accordingly.



 Comments   
Comment by Roman Kovařík [ 08/Mar/21 ]

Closing this as this is about the old framework and no support ticket is linked.

Generated at Mon Feb 12 09:18:06 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.