Quantcast

[GitHub] struts pull request #128: Multidimensional validation

classic Classic list List threaded Threaded
8 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[GitHub] struts pull request #128: Multidimensional validation

lukaszlenart-2
GitHub user lukaszlenart opened a pull request:

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

    Multidimensional validation

    This PR adds support for validation of Collections in field validators
   
    Resolves WW-4578

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

    $ git pull https://github.com/lukaszlenart/struts multidimensional-validation

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

    https://github.com/apache/struts/pull/128.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 #128
   
----
commit caca3d057d56f43699c1ef5d37e7800eb7b0e5a5
Author: Lukasz Lenart <[hidden email]>
Date:   2017-04-06T06:59:11Z

    WW-4578 Makes RegexFieldValidator to support collections

commit 158eda97834645a542e81d7414b67f8285042b8b
Author: Lukasz Lenart <[hidden email]>
Date:   2017-04-10T06:37:22Z

    WW-4578 Changes range validators to support collections

commit e66fd538b519b1a156d76e4bbe4e994a35e3c464
Author: Lukasz Lenart <[hidden email]>
Date:   2017-04-10T07:13:14Z

    WW-4578 Makes required validator to support collections

----


---
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
|  
Report Content as Inappropriate

[GitHub] struts issue #128: WW-4578: Multidimensional validation

lukaszlenart-2
Github user cnenning commented on the issue:

    https://github.com/apache/struts/pull/128
 
    IMO these 4 could be considered, too:
   
    - `DoubleRangeFieldValidator`
    - `URLValidator`
    - `RequiredStringValidator`
    - `StringLengthFieldValidator`



---
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
|  
Report Content as Inappropriate

[GitHub] struts issue #128: WW-4578: Multidimensional validation

lukaszlenart-2
In reply to this post by lukaszlenart-2
Github user lukaszlenart commented on the issue:

    https://github.com/apache/struts/pull/128
 
    - ~`DoubleRangeFieldValidator`~
    - `URLValidator`
    - `RequiredStringValidator`
    - `StringLengthFieldValidator`


---
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
|  
Report Content as Inappropriate

[GitHub] struts issue #128: WW-4578: Multidimensional validation

lukaszlenart-2
In reply to this post by lukaszlenart-2
Github user lukaszlenart commented on the issue:

    https://github.com/apache/struts/pull/128
 
    - ~`DoubleRangeFieldValidator`~
    - `URLValidator`
    - ~`RequiredStringValidator`~
    - `StringLengthFieldValidator`


---
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
|  
Report Content as Inappropriate

[GitHub] struts issue #128: WW-4578: Multidimensional validation

lukaszlenart-2
In reply to this post by lukaszlenart-2
Github user lukaszlenart commented on the issue:

    https://github.com/apache/struts/pull/128
 
    - ~`DoubleRangeFieldValidator`~
    - `URLValidator`
    - ~`RequiredStringValidator`~
    - ~`StringLengthFieldValidator`~


---
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
|  
Report Content as Inappropriate

[GitHub] struts issue #128: WW-4578: Multidimensional validation

lukaszlenart-2
In reply to this post by lukaszlenart-2
Github user lukaszlenart commented on the issue:

    https://github.com/apache/struts/pull/128
 
    - ~`DoubleRangeFieldValidator`~
    - ~`URLValidator`~
    - ~`RequiredStringValidator`~
    - ~`StringLengthFieldValidator`~


---
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
|  
Report Content as Inappropriate

[GitHub] struts issue #128: WW-4578: Multidimensional validation

lukaszlenart-2
In reply to this post by lukaszlenart-2
Github user wolpi commented on the issue:

    https://github.com/apache/struts/pull/128
 
    +1 for merging 😆


---
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
|  
Report Content as Inappropriate

[GitHub] struts pull request #128: WW-4578: Multidimensional validation

lukaszlenart-2
In reply to this post by lukaszlenart-2
Github user asfgit closed the pull request at:

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


---
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]

Loading...