[MGNLPN-574] NullPointer when directly deleting page Created: 13/Aug/21  Updated: 04/May/22  Resolved: 04/Oct/21

Status: Closed
Project: Magnolia Personalization
Component/s: None
Affects Version/s: 2.0.11
Fix Version/s: 2.1, 2.0.12

Type: Bug Priority: Neutral
Reporter: Carlos Cantalapiedra Assignee: Roman Kovařík
Resolution: Fixed Votes: 0
Labels: VN-Testing
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Problem/Incident
relation
is related to MGNLPN-560 Further Improve performance of Varian... Closed
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
Date of First Response:
Epic Link: Grid performance
Sprint: UI FW 38
Story Points: 8

 Description   

Steps to reproduce

  1.  Disable receivers https://demo.magnolia-cms.com/.magnolia/admincentral#app:configuration:browser;/modules/publishing-core/config/receivers/magnoliaPublic8080@enabled:treeview:
  2.  Delete a page https://demo.magnolia-cms.com/.magnolia/admincentral#app:pages-app:browser;/sportstation::

Expected results

Page is deleted and success message is shown.

Actual results

Page is deleted and exception is thrown.

Workaround

Use default variant aware column definition (if amount of pages is not huge and you are not affected with performance issue MGNLPN-560)

Development notes

Deletion trigger observation which rerenders the grid asynchronously (without a webcontext/request).
CachingVariantAwareTitleColumnDefinition fails to access request.
Checks if a simple check for null doesn't introduce a performance problem again for asynchronous requests.
Can we cache in ValueProvider and flush if mgnl:variant is added (via observation)?



 Comments   
Comment by Roman Kovařík [ 27/Aug/21 ]

Forwardport MGNLPN-560 to master as part of this ticket.

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