Folks, as a rule of thumb always use JIRA to get patches. Applying a patch without a JIRA stating the license is granted is illegal as well.

Ruwan

On 10/12/10 8:58 PM, Senaka Fernando wrote:
Azeez,

This does not have an accompanying JIRA issue, since Sadeep fixed an issue (which occurs @ runtime) that was introduced by us - when we fixed issues reported by FindBugs, while he was working on improving the advanced search UI. I will ask Sadeep to create a JIRA issue for this fix and attach the patch to it.

Sadeep,

Can you create a JIRA issue for this fix and reply to this thread with the corresponding issue id?

Thanks,
Senaka.

On Tue, Oct 12, 2010 at 8:08 PM, Afkham Azeez <[email protected] <mailto:[email protected]>> wrote:

    Senaka,
    Please include the JIRA key which includes this patch.

    Azeez




    On Tue, Oct 12, 2010 at 12:01 AM, <[email protected]
    <mailto:[email protected]>> wrote:

        Author: senaka
        Date: Mon Oct 11 11:31:51 2010
        New Revision: 75426
        URL: http://wso2.org/svn/browse/wso2?view=rev&revision=75426
        <http://wso2.org/svn/browse/wso2?view=rev&revision=75426>

        Log:
        Fixing issue in Registry search feature. Applying patch from
        Sadeep.

        Modified:
trunk/carbon/components/registry/org.wso2.carbon.registry.search/src/main/java/org/wso2/carbon/registry/search/services/utils/AdvancedResourceQuery.java

        Modified:
        
trunk/carbon/components/registry/org.wso2.carbon.registry.search/src/main/java/org/wso2/carbon/registry/search/services/utils/AdvancedResourceQuery.java
        URL:
        
http://wso2.org/svn/browse/wso2/trunk/carbon/components/registry/org.wso2.carbon.registry.search/src/main/java/org/wso2/carbon/registry/search/services/utils/AdvancedResourceQuery.java?rev=75426&r1=75425&r2=75426&view=diff
        
<http://wso2.org/svn/browse/wso2/trunk/carbon/components/registry/org.wso2.carbon.registry.search/src/main/java/org/wso2/carbon/registry/search/services/utils/AdvancedResourceQuery.java?rev=75426&r1=75425&r2=75426&view=diff>
        
==============================================================================
        ---
        
trunk/carbon/components/registry/org.wso2.carbon.registry.search/src/main/java/org/wso2/carbon/registry/search/services/utils/AdvancedResourceQuery.java
           (original)
        +++
        
trunk/carbon/components/registry/org.wso2.carbon.registry.search/src/main/java/org/wso2/carbon/registry/search/services/utils/AdvancedResourceQuery.java
           Mon Oct 11 11:31:51 2010
        @@ -300,7 +300,7 @@
            }

            public Date getCreatedAfter() {
        -        return new Date(createdAfter);
        +        return (createdAfter == Long.MIN_VALUE) ? null : new
        Date(createdAfter);
            }

            public void setCreatedAfter(Date createdAfter) {
        @@ -308,15 +308,15 @@
            }

            public Date getCreatedBefore() {
        -        return new Date(createdBefore);
        +        return (createdBefore == Long.MIN_VALUE) ? null : new
        Date(createdBefore);
            }

            public void setCreatedBefore(Date createdBefore) {
        -        this.createdBefore = (createdBefore == null) ?
        Long.MAX_VALUE : createdBefore.getTime();
        +        this.createdBefore = (createdBefore == null) ?
        Long.MIN_VALUE : createdBefore.getTime();
            }

            public Date getUpdatedAfter() {
        -        return new Date(updatedAfter);
        +        return (updatedAfter == Long.MIN_VALUE) ? null : new
        Date(updatedAfter);
            }

            public void setUpdatedAfter(Date updatedAfter) {
        @@ -324,11 +324,11 @@
            }

            public Date getUpdatedBefore() {
        -        return new Date(updatedBefore);
        +        return (updatedBefore == Long.MIN_VALUE) ? null : new
        Date(updatedBefore);
            }

            public void setUpdatedBefore(Date updatedBefore) {
        -        this.updatedBefore = (updatedBefore == null) ?
        Long.MAX_VALUE : updatedBefore.getTime();
        +        this.updatedBefore = (updatedBefore == null) ?
        Long.MIN_VALUE : updatedBefore.getTime();
            }

            public void setTags(String tags) {

        _______________________________________________
        Carbon-commits mailing list
        [email protected] <mailto:[email protected]>
        https://wso2.org/cgi-bin/mailman/listinfo/carbon-commits




-- *Afkham Azeez*
    Senior Software Architect & Senior Manager; WSO2, Inc.;
    http://wso2.com,
    /
    /
    /Member; Apache Software Foundation; //http://www.apache.org///
    email: //[email protected]/ <mailto:[email protected]>/ cell: +94 77 3320919
    blog: //http://blog.afkham.org//
    twitter: //http://twitter.com/afkham_azeez//
    linked-in: //http://lk.linkedin.com/in/afkhamazeez/
    /
    /
    /Lean . Enterprise . Middleware/




--
Senaka Fernando
Associate Technical Lead
WSO2 Inc.
E-mail: senaka AT wso2.com <http://wso2.com>;  Mobile: +94 77 322 1818

http://www.wso2.com/ - "Lean . Enterprise . Middleware"


_______________________________________________
Carbon-dev mailing list
[email protected]
https://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev


--
Ruwan Linton
Software Architect&  Product Manager, WSO2 ESB; http://wso2.org/esb
WSO2 Inc.; http://wso2.com

Lean . Enterprise . Middleware

phone: +1 408 754 7388 ext 51789
email: [email protected]; cell: +94 77 341 3097
blog: http://blog.ruwan.org
linkedin: http://www.linkedin.com/in/ruwanlinton
tweet: http://twitter.com/ruwanlinton

_______________________________________________
Carbon-dev mailing list
[email protected]
https://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev

Reply via email to