[JCR-1537] ClassDescriptor.hasIdField() fails if id is declared in upper class

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.5
• Fix Version/s: 1.5
• Component/s: jackrabbit-ocm
• Environment: Mac OS X, JVM 1.5
• Assignee: Christophe Lombart
• Reporter: Stephane Landelle
• Created: 14/Apr/08 16:30
• Updated: 08/Dec/08 11:08
• Resolved: 21/May/08 20:51

Description

org.apache.jackrabbit.ocm.mapper.model.ClassDescriptor.hasIdField() looks up only current class and not the whole hierarchy, so it fails when the id field is declared in a upper class.

hasIdField should use getIdFieldDescriptor and not access idFieldDescriptor field directly, as follows :

public boolean hasIdField()

{ return (this.getIdFieldDescriptor() != null && this .getIdFieldDescriptor().isId()); }

Please find patch enclosed.

Sincerely yours,

Stéphane Landelle