[ 
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)

Reply via email to