Re: svn commit: r912248 - /ofbiz/trunk/framework/entity/src/org/ofbiz/entity/model/ModelEntity.java

[Adam Heath-2]

[hidden email] wrote:
> Author: lektran
> Date: Sat Feb 20 23:04:51 2010
> New Revision: 912248
>
> URL: http://svn.apache.org/viewvc?rev=912248&view=rev
> Log:
> Reverting r912247, not all methods removed had been deprecated prior to 9.04

When deprecation is added, in addition to adding @Deprecated
annotation and @deprecated javadoc, the javadoc should mention that it
was done 'after ${lastMajorRelease}', and say what you should do instead.

[Scott Gray-2]

On 20/02/2010, at 4:21 PM, Adam Heath wrote:

> [hidden email] wrote:
>> Author: lektran
>> Date: Sat Feb 20 23:04:51 2010
>> New Revision: 912248
>>
>> URL: http://svn.apache.org/viewvc?rev=912248&view=rev
>> Log:
>> Reverting r912247, not all methods removed had been deprecated prior to 9.04
>
> When deprecation is added, in addition to adding @Deprecated
> annotation and @deprecated javadoc, the javadoc should mention that it
> was done 'after ${lastMajorRelease}', and say what you should do instead.

I was thinking the same thing, it's a pain having to check back on each one.  I was planning to go through and hit them all in one go after I'm done (which won't be anytime soon)

Regards
Scott