[PAGES-1533] Green edit bars allow to edit components regardless of availability Created: 27/Nov/23  Updated: 10/Jan/24

Status: Open
Project: Magnolia pages module
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Bug Priority: Neutral
Reporter: Roberto Gaona Assignee: Unassigned
Resolution: Unresolved Votes: 4
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
causality
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
Date of First Response:
Epic Link: AuthorX Support
Team: AuthorX

 Description   

Steps to reproduce

Add on pages app the following rule to editComponent:

            justSomethingThatDisablesTheAction:
              $type: jcrIsDeletedRule 

After doing so, try to edit a component on any page from the pages app.

Expected result:

Since the action is disabled, user shouldn't be able to edit components at any point

Actual results

Using the greenbars, the user is able to open edit dialogs.

Workaround

Development notes



 Comments   
Comment by Roman Kovařík [ 27/Nov/23 ]

Page editor (Pages app specific component) is not aware of the actionbar availability (generic concept) but vice versa.

Component definition has separate concept of properties to define availability https://docs.magnolia-cms.com/product-docs/6.2/Developing/Templating/Template-definition/Component-definition.html#_component_properties.

The action bar then reads this config https://git.magnolia-cms.com/projects/MODULES/repos/pages/browse/magnolia-pages-app/src/main/resources/pages-app/apps/pages-app.yaml#361

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