You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@causeway.apache.org by "Daniel Keir Haywood (Jira)" <ji...@apache.org> on 2023/03/13 06:29:00 UTC
[jira] [Updated] (CAUSEWAY-3331) Add a MM validation for number of parameters in supporting methods, eg choices5Act()
[ https://issues.apache.org/jira/browse/CAUSEWAY-3331?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Daniel Keir Haywood updated CAUSEWAY-3331:
------------------------------------------
Fix Version/s: 2.0.0-RC2
(was: 2.0.0-RC1)
> Add a MM validation for number of parameters in supporting methods, eg choices5Act()
> ------------------------------------------------------------------------------------
>
> Key: CAUSEWAY-3331
> URL: https://issues.apache.org/jira/browse/CAUSEWAY-3331
> Project: Causeway
> Issue Type: Improvement
> Affects Versions: 2.0.0-M9
> Reporter: Daniel Keir Haywood
> Priority: Major
> Fix For: 2.0.0-RC2
>
>
> In converting v1 to v2, we came across a supporting "choices" method that was incorrectly accepted in v1, but rejected in v2 as being orphaned.
> ```
> public Foo myAction(X x, Y y, Z z, W w) \{ ... }
> public List<Z> choices2(X x, Y y, Z z, W w) \{ ... }
> ```
> The error was that the choices method incorrectly included parameters AFTER the 'Nth' parameter, in other words it should have been:
> ```
> public List<Z> choices2(X x, Y y) \{ ... }
> ```
> It would be preferable if a MM validation (with higher priority than the one that triggered) could have given a message something like:
> "there should only be 2 parameters in "choices2(...)", instead found 4."
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)