Re: svn commit: r588002 - /openjpa/branches/1.0.x/openjpa-kernel/src/main/resources/org/apache/openjpa/enhance/localizer.properties

[Craig L Russell]

Hi Mike,

On Oct 24, 2007, at 5:25 PM, Michael Dick wrote:

The first time I read it I thought it was referring to property "{0}", not to the method, ie because the property / field was marked private, you need

a getter / setter.


adds much to the message besides confusing us.


and \

\

private \
+       Please create a setter method for this property.

I think I'd prefer to omit the last bit altogether, I don't think it really property-no-setter: No setter method found for persistent property "{0}", bytecode analysis of the getter did not reveal a clear backing field. - Please create a setter method for this property. which may be a

+1.

Craig

-Mike

On 10/24/07, Albert Lee <[EMAIL PROTECTED]> wrote:

we should change "private property" to "private method".

Albert Lee.

On 10/24/07, Craig L Russell <[EMAIL PROTECTED]> wrote:

I agree with Craig. The "private" is referring to the setter "method", so

Hi Mike,

1. I don't think the text is an improvement. What is a private
property? Should it not be:


private \
+       method.

or,

private).

Craig

On Oct 24, 2007, at 1:22 PM, [EMAIL PROTECTED] wrote:

+ Please create a setter method for this property which may be a - Please create a setter method for this property (which may be

Author: mikedd
Date: Wed Oct 24 13:22:40 2007
New Revision: 588002

URL: http://svn.apache.org/viewvc?rev=588002&view=rev
Log:
OPENJPA-413 committing for Teresa

Modified:
    openjpa/branches/1.0.x/openjpa-kernel/src/main/resources/org/
apache/openjpa/enhance/localizer.properties

Modified: openjpa/branches/1.0.x/openjpa-kernel/src/main/resources/
org/apache/openjpa/enhance/localizer.properties
URL: http://svn.apache.org/viewvc/openjpa/branches/1.0.x/openjpa-
kernel/src/main/resources/org/apache/openjpa/enhance/
localizer.properties?rev=588002&r1=588001&r2=588002&view=diff

========
--- openjpa/branches/1.0.x/openjpa-kernel/src/main/resources/org/
apache/openjpa/enhance/localizer.properties (original)
+++ openjpa/branches/1.0.x/openjpa-kernel/src/main/resources/org/

@@ -128,7 +128,8 @@
      persistent properties must have getter methods.
 property-no-setter: No setter method found for persistent property
"{0}", and \

field. \
-     Please create a setter method for this property (it may be

=================================================================== === apache/openjpa/enhance/localizer.properties Wed Oct 24 13:22:40 2007 bytecode analysis of the getter did not reveal a clear backing

private).

+ Please create a setter method for this property which may be a

private \
+     property.
 property-setter-getter-mismatch: Bytecode analysis of the setter
and getter \
      methods for persistent property "{0}" indicates that the two
methods \

assigns to \

might be acting on different backing fields. The setter method

Craig Russell

Architect, Sun Java Enterprise System http://java.sun.com/ products/jdo

408 276-5638 mailto:[EMAIL PROTECTED]
P.S. A good JDO? O, Gasp!

Craig Russell
Architect, Sun Java Enterprise System http://java.sun.com/products/jdo
408 276-5638 mailto:[EMAIL PROTECTED]
P.S. A good JDO? O, Gasp!

smime.p7s
Description: S/MIME cryptographic signature