You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@isis.apache.org by "Daniel Keir Haywood (Jira)" <ji...@apache.org> on 2023/01/19 13:02:00 UTC

[jira] [Created] (ISIS-3331) Add a MM validation for number of parameters in supporting methods, eg choices5Act()

Daniel Keir Haywood created ISIS-3331:
-----------------------------------------

             Summary: Add a MM validation for number of parameters in supporting methods, eg choices5Act()
                 Key: ISIS-3331
                 URL: https://issues.apache.org/jira/browse/ISIS-3331
             Project: Isis
          Issue Type: Improvement
    Affects Versions: 2.0.0-M9
            Reporter: Daniel Keir Haywood
             Fix For: 2.0.0-RC1


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)