[MGNLUI-6613] NotificationPropertySetFactory fails on unknown field Created: 11/Mar/21  Updated: 25/Oct/21  Resolved: 20/Oct/21

Status: Closed
Project: Magnolia UI
Component/s: None
Affects Version/s: 6.2.11
Fix Version/s: 6.2.13

Type: Bug Priority: Neutral
Reporter: Carlos Cantalapiedra Assignee: Roman Kovařík
Resolution: Fixed Votes: 0
Labels: maintenance, ui-team-support
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: PNG File notifications-app-fail.png    
Issue Links:
Problem/Incident
relation
Template:
Acceptance criteria:
Empty
Task DoD:
[X]* Doc/release notes changes? Comment present?
[X]* Downstream builds green?
[X]* Solution information and context easily available?
[X]* Tests
[X]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[X]* Steps to reproduce, expected, and actual results filled
[X]* Affected version filled
Release notes required:
Yes
Date of First Response:
Sprint: UI FW 39
Story Points: 3

 Description   

Description

After falling with an exception in Configuration App, if you click at the "Read more" button within the Warning Notification, it throws a Sub-app failed to start: detail error

Steps to reproduce

  1.  Go to Configurations app at demo
  2.  Try to delete any module
  3.  After the error that Root, Level 1 or level 2 nodes can't be unpublished, clic on the "Read more" button of the notification at the top of the screen
  4.  Check that it fails when opening the notification and a Sub app failed error is thrown instead

Expected results

After click on "Read more" the full Notification can be read at the Notification detail section of the Notifications app

Actual results

The app fails to start (check notifications-app-fail.png image)

java.lang.IllegalArgumentException: Could not resolve property name comment from info.magnolia.admincentral.apps.notifications.NotificationPropertySetFactory$1@1f072584 at

Workaround

N/A

Development notes

When accessing the notification detail from other app, like asset, it works. E.g, try to publish an asset that fails (public instance not accesible), then the notification appears and if you click the "Read more" button, it goes to the Notifications app and shows the full notification
http://localhost:8080/magnoliaAuthor/.magnolia/admincentral#app:configuration:browser;/modules/ui-admincentral/messageViews/longRunning/form/tabs/message/fields/comment:treeview:



 Comments   
Comment by Roman Kovařík [ 22/Oct/21 ]

For release notes: notification app detail doesn't crash anymore on unknown field name.

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