[ https://issues.apache.org/jira/browse/ISIS-2152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16882844#comment-16882844 ]
ASF subversion and git services commented on ISIS-2152: ------------------------------------------------------- Commit 9ad77a45a24ea4b886d79fc62e029a1d8dd5ceb2 in isis's branch refs/heads/v2 from Andi Huber [ https://gitbox.apache.org/repos/asf?p=isis.git;h=9ad77a4 ] ISIS-2152 adds ViewModel creation to the FactoryService (API) - also removes 'm' (=mixin) shortcut from FactoryService interface > FactoryService (API): remove 'm' shortcut, add 'viewModel' > ---------------------------------------------------------- > > Key: ISIS-2152 > URL: https://issues.apache.org/jira/browse/ISIS-2152 > Project: Isis > Issue Type: Improvement > Reporter: Andi Huber > Assignee: Andi Huber > Priority: Major > Fix For: 2.0.0 > > > Opinionated: remove this shortcut for 'mixin', don't encourage bad code > readablity, especially when the verbose variant 'mixin' is already short > enough: > > {code:java} > <T> T m(Class<T> mixinClass, Object mixedIn); > {code} > Add > > > {code:java} > /** > * Creates a new ViewModel instance, and initializes according to the given > {@code mementoStr} > * @param viewModelClass > * @param mementoStr > * @since 2.0 > */ > <T> T viewModel(Class<T> viewModelClass, @Nullable String mementoStr); > {code} > However, I'm not entirely sure whether its a good idea to expose the > mementoStr with the API that way. > > -- This message was sent by Atlassian JIRA (v7.6.14#76016)