[gatein-issues] [JBoss JIRA] (GTNPORTAL-2751) <priority> in navigation.xml files ignored for merge importMode

[Peter Palaga (JIRA)]

Peter Palaga created GTNPORTAL-2751:
---------------------------------------

             Summary: <priority> in navigation.xml files ignored for merge importMode 
                 Key: GTNPORTAL-2751
                 URL: https://issues.jboss.org/browse/GTNPORTAL-2751
             Project: GateIn Portal
          Issue Type: Task
      Security Level: Public (Everyone can see)
            Reporter: Peter Palaga


Steps to reproduce:

Clone the Portal Extension Quickstart and deploy it changing the {{<priority>}} value in {{war/src/main/webapp/WEB-INF/conf/sample-ext/portal/group/platform/administrators/navigation.xml}}. 
NOT OK: The changes have no effect, because priority is ignored in {{org.exoplatform.portal.mop.importer.NavigationImporter.perform()}} with {{importMode}} {{merge}}.

h3. Solution Proposal A
* Make {{<priority>}} optional in {{gatein_objects}} XSD and  {{NavigationMarshaller.unmarshalNavigation(StaxNavigator<Element>)}}
* Explicitly state in the documentation [1] that {{<priority>}} is considered only if not already set (first wins).

h3. Solution Proposal B
* Make {{<priority>}} optional in {{gatein_objects}} XSD and  {{NavigationMarshaller.unmarshalNavigation(StaxNavigator<Element>)}}
* Change the behavior of {{NavigationMarshaller.unmarshalNavigation(StaxNavigator<Element>)}} to the effect that {{<priority>}} is handled in the same way as navigation nodes: it is used or ignored according to the chosen {{importMode}}. Esp. for {{importMode}} {{merge}} the last read {{<priority>}} should win.
* Document it in [1]

For both A and B the most or all occutences of {{<priority>}} should be removed from Portal Extension Quickstart.

[1] [https://docs.jboss.org/author/display/GTNPORTAL35/Portal+Navigation+Configuration]

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira