[ 
https://issues.apache.org/jira/browse/ISIS-954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14217623#comment-14217623
 ] 

Martin Grigorov commented on ISIS-954:
--------------------------------------

After this change what will be the final state ?
Looking at the commit I think 'ToDoItemsFixturesService' will be ignored, no ?

Isn't it better to use Map<String, List<String>> instead of Set<String> ?
This way the menu items of both services will be collected and presented with 
dropdown headers (http://getbootstrap.com/components/#dropdowns-headers). 
Related to ISIS-854.

> Duplicate menu items in the application menu
> --------------------------------------------
>
>                 Key: ISIS-954
>                 URL: https://issues.apache.org/jira/browse/ISIS-954
>             Project: Isis
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: core-1.7.0
>            Reporter: Dan Haywood
>            Assignee: Dan Haywood
>            Priority: Minor
>             Fix For: core-1.8.0
>
>
> eg in Todo app, if change:
> @DomainService(menuOrder = "20")
> @Named("Analysis")
> public class ToDoItemAnalysis { ... }
> to
> @DomainService(menuOrder = "20")
> @Named("Prototyping")
> public class ToDoItemAnalysis { ... }
> then, because there is also:
> @DomainService(menuOrder = "40.1")
> @Named("Prototyping")
> public class ToDoItemsFixturesService extends FixtureScripts { ... }
> this will  result in two "Prototyping" menu items.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to