Hi,

combing through the JBoss SF Bug database I found the following in the
patches section:

http://sourceforge.net/tracker/index.php?func=detail&aid=437043&group_id=22866&atid=376687

Whoever submitted this is actually correct; the EJB Spec (1.1 and 2.0)
state that Finder methods should be prefixed by "find" (i.e.
find<METHOD>). The spec doesn't require "findBy" except for the
findByPrimaryKey method of course.

The suggested "patch" would fix the problem but this is only half of the
story (it will only deal with custom finders declared in the home
interface): all of JBoss's auto-generated finder methods for CMP EB's
are also prefixed by "findBy" which doesn't seem valid according to the
specs; i.e. if you have a CMP field "String name;" the generated finder
method should read

        Collection findName(String parameter);

but JBoss generates

        Collection findByName(String parameter);

I haven't seen anybody complaining about this (the "findBy" prefix is
more intuitive anyways IMO) but this should be fixed. The fix seems easy
enough and I would be willing to take it but this could break a lot of
apps out there that rely on the method being prefixed with "findBy" ...

Any opinions on this? We could just generate the spec-compliant methods
AND the findBy stuff but this seems a bit awkward ...

Best regards,
        Christian


_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development

Reply via email to