You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@struts.apache.org by "Lukasz Lenart (Jira)" <ji...@apache.org> on 2020/04/04 06:42:00 UTC

[jira] [Updated] (WW-5063) Use null check of passed in invocation in all the results

     [ https://issues.apache.org/jira/browse/WW-5063?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Lukasz Lenart updated WW-5063:
------------------------------
    Description: 
Right now in most cases passed in {{invocation}} isn't validated if it isn't {{null}}, the below code should be applied to all {{Result}}s

{code}
if (invocation == null) {
    throw new IllegalArgumentException("invocation cannot be null!");
}
{code}

This was pointed out during a PR review
https://github.com/apache/struts/pull/397/files#r396134222

  was:
Right now in most cases passed in {{invocation}} isn't validated if it isn't {{null}}, the below code should be applied to all {{Result}}s

{code}
if (invocation == null) {
    throw new IllegalArgumentException("invocation cannot be null!");
}
{code}


> Use null check of passed in invocation in all the results
> ---------------------------------------------------------
>
>                 Key: WW-5063
>                 URL: https://issues.apache.org/jira/browse/WW-5063
>             Project: Struts 2
>          Issue Type: Improvement
>          Components: Core Results
>            Reporter: Lukasz Lenart
>            Priority: Minor
>             Fix For: 2.6
>
>
> Right now in most cases passed in {{invocation}} isn't validated if it isn't {{null}}, the below code should be applied to all {{Result}}s
> {code}
> if (invocation == null) {
>     throw new IllegalArgumentException("invocation cannot be null!");
> }
> {code}
> This was pointed out during a PR review
> https://github.com/apache/struts/pull/397/files#r396134222



--
This message was sent by Atlassian Jira
(v8.3.4#803005)