jackrabbit-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christophe Lombart (JIRA)" <j...@apache.org>
Subject [jira] Assigned: (JCR-1398) [PATCH] ClassDescriptor.hasIdField uses faulty logic
Date Sun, 02 Mar 2008 23:09:50 GMT

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

Christophe Lombart reassigned JCR-1398:
---------------------------------------

    Assignee: Christophe Lombart

> [PATCH] ClassDescriptor.hasIdField uses faulty logic
> ----------------------------------------------------
>
>                 Key: JCR-1398
>                 URL: https://issues.apache.org/jira/browse/JCR-1398
>             Project: Jackrabbit
>          Issue Type: Bug
>          Components: jackrabbit-ocm
>    Affects Versions: 1.4
>            Reporter: Dave Brosius
>            Assignee: Christophe Lombart
>            Priority: Minor
>             Fix For: 1.4.1
>
>         Attachments: classdescriptor_isid.patch
>
>
> hasIdField tries to compare a FieldDescriptor to an empty string, which doesn't make
sense, here:
>      public boolean hasIdField() {
>         return (this.idFieldDescriptor != null && ! this.idFieldDescriptor.equals(""));
>      }
> i'm assuming it should be
>        return (this.idFieldDescriptor != null && this.idFieldDescriptor.isId());
> patch does this

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message