[CAUSEWAY-2152] FactoryService (API): remove 'm' shortcut, add 'viewModel'

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.0.0-M3
• Assignee: Andi Huber
• Reporter: Andi Huber
• Created: 11/Jul/19 10:31
• Updated: 11/Jul/19 11:05
• Resolved: 11/Jul/19 11:05

Description

Opinionated: remove this shortcut for 'mixin', don't encourage bad code readablity, especially when the verbose variant 'mixin' is already short enough:

 

<T> T m(Class<T> mixinClass, Object mixedIn);

Add

 

 

/**
 * 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);

However, I'm not entirely sure whether its a good idea to expose the mementoStr with the API that way.