page-templates-editing - Adding/Removing element to page template, should result in corresponding change to the page it is applied to.

[todor]

Adding/Removing element to page template, should result in corresponding change to the page it is applied to.

• When a frame is added to the page template, this should result in adding new frames in all pages that are using this template.
• Same thing must be valid for deleting frame from the template.

(The same must be applied to groups of elements or certain parts of the group (frame that is part of group) etc.)

[deni]

• The analysis is ok.
• Adding/removing elements as a result of changing the template will register significant AutoActions. This means that it can be undone.

[deni]

• Add an AutoProperty in ElementH that keeps track of the subelements of the element's template and fires an event when a new subelement is added or when an existing one is deleted.
  • Add an inner enum TemplateEventIds in ElementH that contains 2 events:
    • TEMPLATE_ELEMENT_ADDED fired when a new element is added to the template
    • TEMPLATE_ELEMENT_DELETED fired when an existing element is deleted from the template
    • Both events have a ResourceRefR4 parameter pointing to the resource that was added/removed.
  • The property keeps the previous subelements value in a private field and after computing the new subelements, compares the two lists to determine which elements have been added/removed.
  • The property should not fire any events when the template itself has changed. Otherwise when applying a page template, a new element will be added for each slot (in addition to the actions the user has chosen from the 'Apply page template' dialog).

• Make sure that while applying a page template in TemplateUtil.applyPageTemplate(...) when associating a group from the page with a group from the template, the KEY_TEMPLATE is appropriately set. This will ensure that when a frame is added to an existing group in the template of a page, a new frame will be added to the group on the page that is associated with the template group.

• Add a new logic class to handle events related to changes in the template of an element - CompositeTemplateLogic in org.sophie2.main.func.templates.composite package. It should contain the following 2 operations:
  • ON_TEMPLATE_SUBELEMENT_ADDED - handles adding a new element (sub-template) to the template of the source element. Creates a new sub-element by the sub-template.
    • Before adding the new element, it should check whether it has not already been added. This is possible when there are 2 ElementHs for the same resource at the same time or after undo-ing the action that removed a sub-element from the template.
      • Since a sub-template can be used only once, checking whether the sub-template is already used should be enough.
  • ON_TEMPLATE_SUBELEMENT_DELETED - handles deleting a sub-element (sub-template) from the template of the source element. Deletes the element that was using the sub-template.
    • There might be no such element or it might have alrealy been deleted.

• Make TemplateUtil.createTemplate(...) use a single AutoAction, so that the property updates after the template's creation is complete. Otherwise the order of the AutoActions was not correct.

[meddle]

• I think the idea is good, if it works with the server collaboration it is OK.
• Use ImmTreeList instead naive.

4p

[deni]

• Applied the same approach to change the resource kind of a templated frame when the main resource and the kind of its template change.
  • The property is in FrameH.

• Adding/removing elements as a result of adding/removing a subelement from the template registers significant autoactions. This means they can be undone. This might be inconvenient for the users, especially when there are many changes. However, I can't think of any solution...

[meddle]

Merged into the trunk at [9104].

• To many bugs.

[meddle]

Closing all the tickets before M Y1