[GitHub] struts pull request #115: WW-4737 Properly handle arrays containing nulls

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] struts pull request #115: WW-4737 Properly handle arrays containing nulls

yasserzamani
GitHub user ThrawnCA opened a pull request:

    https://github.com/apache/struts/pull/115

    WW-4737 Properly handle arrays containing nulls

    Don't convert null to "null"

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ThrawnCA/struts WW-4737

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/struts/pull/115.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #115
   
----
commit 58667e64970c528461cf7c5835f9a253c969f737
Author: antuarc <[hidden email]>
Date:   2017-02-01T02:26:31Z

    [WW-4737] preserve nulls instead of converting them to the string 'null'
   
    - Also simplify if-else logic to be more readable and avoid double-negatives

commit 26bc9cd64bd9e69f7bf2e1076150806581c355f4
Author: antuarc <[hidden email]>
Date:   2017-02-01T02:27:02Z

    [WW-4737] add unit testing for parameters being converted to string arrays

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] struts pull request #115: WW-4737 Properly handle arrays containing nulls

yasserzamani
Github user asfgit closed the pull request at:

    https://github.com/apache/struts/pull/115


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]