Hmmm I am not sure if comments are the most appropriate form for deprecation. Usually deprecation is useful when it is programmatic because it goes beyond raw text to logging warnings and highlighting by tools. You want the system to constantly remind you (both the developer and author) to get rid of the deprecated code.
However if this entails a lot of code changes (not sure) then I think it might be useful to wait until we refactor the rest of the core components (entity engine, service engine, etc ...) On Aug 6, 2017 8:47 AM, "Deepak Dixit" <deepak.di...@hotwaxsystems.com> wrote: > Hi Nicolas, > > Idea is to mark service deprecated is looks good to me, > What I think instead of adding deprecated as engine we can set annotation > for deprecated service like we set in java, annotation can be simple > comment or xml annotation. > > And ideally we have to mark services deprecate instead of removing, with > expected release on which we will remove this deprecated code. > > > Thanks & Regards > -- > Deepak Dixit > www.hotwaxsystems.com > www.hotwax.co > > On Sat, Aug 5, 2017 at 1:34 AM, Nicolas Malin <nicolas.ma...@nereide.fr> > wrote: > > > Hello with the thread https://lists.apache.org/ > thread.html/Zoz5yfpkrfcxts1 > > and the voluntary to have a good coherence on crud service name, > > > > I would be have your suggest to manage old name and deprecated process. > > > > I review the issue OFBIZ-9550 [1] that contains this problematic : > > > > - <service name="addPromoToMarketingCampaign" ... > > + <service name="createMarketingCampaignPromo" ... > > > > Currently I follow this process : > > > > * duplicate the service definition > > * Rename the duplicate with the correct name > > * Set "DEPRECATED : use ${new service} instead" in the service > description > > * implement on old service definition : return error("use ${new service} > > instead") to help developer to correct their specific code. > > > > I image that we can create a generic code to return the error and change > > the old service definition like this : > > > > <service name="addPromoToMarketingCampaign" > > default-entity-name="MarketingCampaignPromo" engine="deprecated" > > invoke="create" auth="true"> > > <description>Deprecated please replace by > > createMarketingCampaignPromo</description> > > <auto-attributes include="pk" mode="IN" optional="false"/> > > <auto-attributes include="nonpk" mode="IN" optional="true"/> > > </service> > > > > And deprecated engine return always error with the service description. > > > > After a new ofbiz stable branche creation, we remove all deprecated > > service ? > > > > Any suggests, othet ideas, comments ? > > > > Cheers, > > > > Nicolas > > > > [1] https://issues.apache.org/jira/browse/OFBIZ-9550 > > > > -- > > logoNrd <https://nereide.fr/> > > Nicolas Malin > > The apache way <http://theapacheway.com/> : *Openness* Technical > > decisions are made publicly > > informat...@nereide.fr > > 8 rue des Déportés 37000 TOURS, 02 47 50 30 54 > > > > Apache OFBiz <http://ofbiz.apache.org/>|The Apache Way < > > http://theapacheway.com/>|ofbiz-fr <http://www.ofbiz-fr.org/>|réseau LE > < > > http://www.libre-entreprise.org/> > > >