[
https://issues.jboss.org/browse/GTNPORTAL-2751?page=com.atlassian.jira.pl...
]
Peter Palaga commented on GTNPORTAL-2751:
-----------------------------------------
Thanks for the comment, Nick.
I do not know how exactly the ordering using {{priority}} is implemented, but it indeed
makes sense only for the kinds of navigation for which there may exist more than one piece
of the same kind - i.e. for group and user navigations.
There is no point in stating {{<priority>}} for portal navigations because there is
always only one navigation per portal. I see this as a next reason for making the
{{<priority>}} optional.
<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+Confi...]
--
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