For the approach 1) Create the accessor methods for the generic HashMap(roleTypeId, partyId) 2) Mark all of the specific accessor methods as deprecated or soon to be deprecated or whatever the customary approach is 3) Change those accessor methods to use the generic HashMap instead of the string variables 3a) Add log message about deprecation in the specific accessor methods?? 4) remove the string variables 5) clean up project code to use generic accessors 6) Not that this in itself warrants the maintaining of a separate document somewhere, but eventually we're going to need to maintain an upgrading pitfall document (be it an xml document in the project or simply a page on the docs site, if one doesn't already exist), especially if we're going to want to really take advantage of generics in java at some point.
If this looks good, I'll submit a patch. ----- Original Message ---- From: Jacopo Cappellato <[EMAIL PROTECTED]> To: dev@ofbiz.apache.org Sent: Friday, December 14, 2007 10:39:45 PM Subject: Re: Generic Roles in Shopping Cart Now it is clearer to me too: I actually misunderstood too, like David (or you didn't explain it very well). I'd say that is doable, but it seems like a lot of work, especially if you want to remove the 20+ accessor methods... and I can see backward compatibility issues. But I'm not against this, especially if we end up with a good approach. Jacopo