You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by Ashish Vijaywargiya <vi...@gmail.com> on 2008/09/01 06:33:40 UTC
Re: Method deprecation WAS svn commit: r690573 - in /ofbiz/trunk/applications: accounting/src/org/ofbiz/accounting/payment/ party/src/org/ofbiz/party/contact/ party/src/org/ofbiz/party/party/ product/src/org/ofbiz/product/category/ product/src/org/of
+1 on David's comment.
--
Ashish Vijaywargiya
Indore (M.P), India
http://en.wikipedia.org/wiki/Indore
On Sun, Aug 31, 2008 at 6:52 AM, David E. Jones <jo...@hotwaxmedia.com>wrote:
>
> While deprecations is okay pretty much any time, removing methods or
> anything else is maybe better done just after a release branch. That was
> people can more easily use the release branch, and when upgrading in the
> future know that there may be some effort involved.
>
> The idea is the same though, ie that moving from one release to another
> chances are you'll have functionality changes that you may have to adjust
> your code for. The only difference is that if we're going to do it close to
> a release let's do it just after instead of just before so that people using
> release branches have more time to adjust.
>
> -David
>
>
>
> Scott Gray wrote:
>
>> Removing deprecated methods before a release would be good but who
>> knows when that would be? A search reveals 121 matches for
>> @deprecated and I think it would be good to set a timeframe for their
>> removal. How about 12 months after deprecation or right before a
>> release (whichever comes first)?
>>
>> Regards
>> Scott
>>
>> 2008/8/31 Jacques Le Roux <ja...@les7arts.com>:
>>
>>> Why not at the next release ? But sometimes it's better to keep it
>>> forever
>>> (working with API is hard)... So I think each case should
>>> be discussed...
>>>
>>> My 2 cents
>>>
>>> Jacques
>>>
>>> From: <le...@apache.org>
>>>
>>>> Author: lektran
>>>> Date: Sat Aug 30 13:05:40 2008
>>>> New Revision: 690573
>>>>
>>>> URL: http://svn.apache.org/viewvc?rev=690573&view=rev
>>>> Log:
>>>> Added deprecation flags to the old jsp based worker methods, any
>>>> thoughts
>>>> on how long a method should be deprecated for before we
>>>> can remove it?
>>>>
>>>> Modified:
>>>>
>>>>
>>>> ofbiz/trunk/applications/accounting/src/org/ofbiz/accounting/payment/PaymentWorker.java
>>>>
>>>>
>>>> ofbiz/trunk/applications/party/src/org/ofbiz/party/contact/ContactMechWorker.java
>>>>
>>>>
>>>> ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyWorker.java
>>>>
>>>>
>>>> ofbiz/trunk/applications/product/src/org/ofbiz/product/category/CategoryWorker.java
>>>>
>>>>
>>>> ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductWorker.java
>>>>
>>>>
>>>> ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/project/ProjectWorker.java
>>>>
>>>>
>>>> ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/workeffort/WorkEffortWorker.java
>>>>
>>>> Modified:
>>>>
>>>> ofbiz/trunk/applications/accounting/src/org/ofbiz/accounting/payment/PaymentWorker.java
>>>> URL:
>>>>
>>>>
>>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/src/org/ofbiz/accounting/payment/PaymentWorker.java?rev=690573&r1=690572&r2=690573&view=diff
>>>>
>>>>
>>>> ==============================================================================
>>>> ---
>>>>
>>>> ofbiz/trunk/applications/accounting/src/org/ofbiz/accounting/payment/PaymentWorker.java
>>>> (original)
>>>> +++
>>>>
>>>> ofbiz/trunk/applications/accounting/src/org/ofbiz/accounting/payment/PaymentWorker.java
>>>> Sat Aug 30 13:05:40 2008
>>>> @@ -49,6 +49,7 @@
>>>> private static int decimals =
>>>> UtilNumber.getBigDecimalScale("invoice.decimals");
>>>> private static int rounding =
>>>> UtilNumber.getBigDecimalRoundingMode("invoice.rounding");
>>>>
>>>> + /** @deprecated */
>>>> public static void getPartyPaymentMethodValueMaps(PageContext
>>>> pageContext, String partyId, boolean showOld, String
>>>> paymentMethodValueMapsAttr) {
>>>> GenericDelegator delegator = (GenericDelegator)
>>>> pageContext.getRequest().getAttribute("delegator");
>>>> List paymentMethodValueMaps =
>>>> getPartyPaymentMethodValueMaps(delegator, partyId, showOld);
>>>> @@ -92,6 +93,7 @@
>>>> }
>>>>
>>>> /** TODO: REMOVE (DEJ 20030301): This is the OLD style and should be
>>>> removed when the eCommerce and party mgr JSPs are */
>>>> + /** @deprecated */
>>>> public static void getPaymentMethodAndRelated(PageContext pageContext,
>>>> String partyId,
>>>> String paymentMethodAttr, String creditCardAttr, String
>>>> eftAccountAttr, String paymentMethodIdAttr, String
>>>> curContactMechIdAttr,
>>>> String donePageAttr, String tryEntityAttr) {
>>>>
>>>> Modified:
>>>>
>>>> ofbiz/trunk/applications/party/src/org/ofbiz/party/contact/ContactMechWorker.java
>>>> URL:
>>>>
>>>>
>>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/party/src/org/ofbiz/party/contact/ContactMechWorker.java?rev=690573&r1=690572&r2=690573&view=diff
>>>>
>>>>
>>>> ==============================================================================
>>>> ---
>>>>
>>>> ofbiz/trunk/applications/party/src/org/ofbiz/party/contact/ContactMechWorker.java
>>>> (original)
>>>> +++
>>>>
>>>> ofbiz/trunk/applications/party/src/org/ofbiz/party/contact/ContactMechWorker.java
>>>> Sat Aug 30 13:05:40 2008
>>>> @@ -20,9 +20,7 @@
>>>> package org.ofbiz.party.contact;
>>>>
>>>> import java.util.Collection;
>>>> -import java.util.HashMap;
>>>> import java.util.Iterator;
>>>> -import java.util.LinkedList;
>>>> import java.util.List;
>>>> import java.util.Map;
>>>>
>>>> @@ -40,7 +38,6 @@
>>>> import org.ofbiz.entity.GenericEntityException;
>>>> import org.ofbiz.entity.GenericValue;
>>>> import org.ofbiz.entity.condition.EntityCondition;
>>>> -import org.ofbiz.entity.condition.EntityExpr;
>>>> import org.ofbiz.entity.util.EntityUtil;
>>>>
>>>> /**
>>>> @@ -50,6 +47,7 @@
>>>>
>>>> public static final String module = ContactMechWorker.class.getName();
>>>>
>>>> + /** @deprecated */
>>>> public static void getPartyContactMechValueMaps(PageContext
>>>> pageContext, String partyId, boolean showOld, String
>>>> partyContactMechValueMapsAttr) {
>>>> GenericDelegator delegator = (GenericDelegator)
>>>> pageContext.getRequest().getAttribute("delegator");
>>>> List<Map<String, Object>> partyContactMechValueMaps =
>>>> getPartyContactMechValueMaps(delegator, partyId, showOld);
>>>> @@ -205,6 +203,7 @@
>>>> }
>>>>
>>>>
>>>> + /** @deprecated */
>>>> public static void getOrderContactMechValueMaps(PageContext
>>>> pageContext, String orderId, String orderContactMechValueMapsAttr)
>>>> {
>>>> GenericDelegator delegator = (GenericDelegator)
>>>> pageContext.getRequest().getAttribute("delegator");
>>>> List<Map<String, GenericValue>> maps =
>>>> getOrderContactMechValueMaps(delegator, orderId);
>>>>
>>>> Modified:
>>>>
>>>> ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyWorker.java
>>>> URL:
>>>>
>>>>
>>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyWorker.java?rev=690573&r1=690572&r2=690573&view=diff
>>>>
>>>>
>>>> ==============================================================================
>>>> ---
>>>>
>>>> ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyWorker.java
>>>> (original)
>>>> +++
>>>>
>>>> ofbiz/trunk/applications/party/src/org/ofbiz/party/party/PartyWorker.java
>>>> Sat Aug 30 13:05:40 2008
>>>> @@ -20,7 +20,6 @@
>>>> package org.ofbiz.party.party;
>>>>
>>>> import java.sql.Timestamp;
>>>> -import java.util.Iterator;
>>>> import java.util.List;
>>>> import java.util.Locale;
>>>> import java.util.Map;
>>>> @@ -84,6 +83,7 @@
>>>> return result;
>>>> }
>>>>
>>>> + /** @deprecated */
>>>> public static void getPartyOtherValues(PageContext pageContext, String
>>>> partyId, String partyAttr, String personAttr, String
>>>> partyGroupAttr) {
>>>> Map<String, GenericValue> partyMap =
>>>> getPartyOtherValues(pageContext.getRequest(), partyId, partyAttr,
>>>> personAttr,
>>>> partyGroupAttr);
>>>> for (Map.Entry<String, GenericValue> e: partyMap.entrySet()) {
>>>>
>>>> Modified:
>>>>
>>>> ofbiz/trunk/applications/product/src/org/ofbiz/product/category/CategoryWorker.java
>>>> URL:
>>>>
>>>>
>>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/src/org/ofbiz/product/category/CategoryWorker.java?rev=690573&r1=690572&r2=690573&view=diff
>>>>
>>>>
>>>> ==============================================================================
>>>> ---
>>>>
>>>> ofbiz/trunk/applications/product/src/org/ofbiz/product/category/CategoryWorker.java
>>>> (original)
>>>> +++
>>>>
>>>> ofbiz/trunk/applications/product/src/org/ofbiz/product/category/CategoryWorker.java
>>>> Sat Aug 30 13:05:40 2008
>>>> @@ -41,8 +41,6 @@
>>>> import org.ofbiz.entity.GenericEntityException;
>>>> import org.ofbiz.entity.GenericValue;
>>>> import org.ofbiz.entity.condition.EntityCondition;
>>>> -import org.ofbiz.entity.condition.EntityConditionList;
>>>> -import org.ofbiz.entity.condition.EntityExpr;
>>>> import org.ofbiz.entity.condition.EntityOperator;
>>>> import org.ofbiz.entity.util.EntityUtil;
>>>> import org.ofbiz.product.product.ProductWorker;
>>>> @@ -54,6 +52,7 @@
>>>>
>>>> public static final String module = CategoryWorker.class.getName();
>>>>
>>>> + /** @deprecated */
>>>> public static String getCatalogTopCategory(PageContext pageContext,
>>>> String defaultTopCategory) {
>>>> return getCatalogTopCategory(pageContext.getRequest(),
>>>> defaultTopCategory);
>>>> }
>>>> @@ -85,6 +84,7 @@
>>>> return topCatName;
>>>> }
>>>>
>>>> + /** @deprecated */
>>>> public static void getCategoriesWithNoParent(PageContext pageContext,
>>>> String attributeName) {
>>>> getCategoriesWithNoParent(pageContext.getRequest(),
>>>> attributeName);
>>>> }
>>>> @@ -113,6 +113,7 @@
>>>> request.setAttribute(attributeName, results);
>>>> }
>>>>
>>>> + /** @deprecated */
>>>> public static void getRelatedCategories(PageContext pageContext,
>>>> String
>>>> attributeName, boolean limitView) {
>>>> getRelatedCategories(pageContext.getRequest(), attributeName,
>>>> limitView);
>>>> }
>>>> @@ -130,6 +131,7 @@
>>>> getRelatedCategories(request, attributeName, requestId,
>>>> limitView);
>>>> }
>>>>
>>>> + /** @deprecated */
>>>> public static void getRelatedCategories(PageContext pageContext,
>>>> String
>>>> attributeName, String parentId, boolean limitView) {
>>>> getRelatedCategories(pageContext.getRequest(), attributeName,
>>>> parentId, limitView);
>>>> }
>>>> @@ -145,6 +147,7 @@
>>>> request.setAttribute(attributeName, categories);
>>>> }
>>>>
>>>> + /** @deprecated */
>>>> public static List getRelatedCategoriesRet(PageContext pageContext,
>>>> String attributeName, String parentId, boolean limitView)
>>>> {
>>>> return getRelatedCategoriesRet(pageContext.getRequest(),
>>>> attributeName, parentId, limitView);
>>>> }
>>>> @@ -270,6 +273,7 @@
>>>> return andCond;
>>>> }
>>>>
>>>> + /** @deprecated */
>>>> public static void setTrail(PageContext pageContext, String
>>>> currentCategory) {
>>>> setTrail(pageContext.getRequest(), currentCategory);
>>>> }
>>>> @@ -347,6 +351,7 @@
>>>> setTrail(request, crumb);
>>>> }
>>>>
>>>> + /** @deprecated */
>>>> public static List getTrail(PageContext pageContext) {
>>>> return getTrail(pageContext.getRequest());
>>>> }
>>>> @@ -357,6 +362,7 @@
>>>> return crumb;
>>>> }
>>>>
>>>> + /** @deprecated */
>>>> public static List setTrail(PageContext pageContext, List crumb) {
>>>> return setTrail(pageContext.getRequest(), crumb);
>>>> }
>>>> @@ -367,6 +373,7 @@
>>>> return crumb;
>>>> }
>>>>
>>>> + /** @deprecated */
>>>> public static boolean checkTrailItem(PageContext pageContext, String
>>>> category) {
>>>> return checkTrailItem(pageContext.getRequest(), category);
>>>> }
>>>> @@ -380,6 +387,7 @@
>>>> return false;
>>>> }
>>>>
>>>> + /** @deprecated */
>>>> public static String lastTrailItem(PageContext pageContext) {
>>>> return lastTrailItem(pageContext.getRequest());
>>>> }
>>>>
>>>> Modified:
>>>>
>>>> ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductWorker.java
>>>> URL:
>>>>
>>>>
>>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductWorker.java?rev=690573&r1=690572&r2=690573&view=diff
>>>>
>>>>
>>>> ==============================================================================
>>>> ---
>>>>
>>>> ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductWorker.java
>>>> (original)
>>>> +++
>>>>
>>>> ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductWorker.java
>>>> Sat Aug 30 13:05:40 2008
>>>> @@ -53,6 +53,7 @@
>>>> public static final String module = ProductWorker.class.getName();
>>>> public static final String resource = "ProductUiLabels";
>>>>
>>>> + /** @deprecated */
>>>> public static void getProduct(PageContext pageContext, String
>>>> attributeName) {
>>>> getProduct(pageContext, attributeName, null);
>>>> }
>>>> @@ -151,6 +152,7 @@
>>>> }
>>>> }
>>>>
>>>> + /** @deprecated */
>>>> public static void getProduct(PageContext pageContext, String
>>>> attributeName, String productId) {
>>>> GenericDelegator delegator = (GenericDelegator)
>>>> pageContext.getRequest().getAttribute("delegator");
>>>> ServletRequest request = pageContext.getRequest();
>>>> @@ -297,6 +299,7 @@
>>>> return available;
>>>> }
>>>>
>>>> + /** @deprecated */
>>>> public static void getAssociatedProducts(PageContext pageContext,
>>>> String productAttributeName, String assocPrefix) {
>>>> GenericValue product = (GenericValue)
>>>> pageContext.getAttribute(productAttributeName);
>>>>
>>>>
>>>> Modified:
>>>>
>>>> ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/project/ProjectWorker.java
>>>> URL:
>>>>
>>>>
>>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/project/ProjectWorker.java?rev=690573&r1=690572&r2=690573&view=diff
>>>>
>>>>
>>>> ==============================================================================
>>>> ---
>>>>
>>>> ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/project/ProjectWorker.java
>>>> (original)
>>>> +++
>>>>
>>>> ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/project/ProjectWorker.java
>>>> Sat Aug 30 13:05:40 2008
>>>> @@ -20,7 +20,6 @@
>>>> package org.ofbiz.workeffort.project;
>>>>
>>>> import java.util.Collection;
>>>> -import java.util.Iterator;
>>>>
>>>> import javax.servlet.jsp.PageContext;
>>>>
>>>> @@ -43,6 +42,7 @@
>>>>
>>>> public static final String module = ProjectWorker.class.getName();
>>>>
>>>> + /** @deprecated */
>>>> public static void getAssignedProjects(PageContext pageContext, String
>>>> projectsAttrName) {
>>>> GenericDelegator delegator = (GenericDelegator)
>>>> pageContext.getRequest().getAttribute("delegator");
>>>> GenericValue userLogin = (GenericValue)
>>>> pageContext.getSession().getAttribute("userLogin");
>>>> @@ -70,6 +70,7 @@
>>>> pageContext.setAttribute(projectsAttrName, validWorkEfforts);
>>>> }
>>>>
>>>> + /** @deprecated */
>>>> public static void getAllAssignedProjects(PageContext pageContext,
>>>> String projectsAttrName) {
>>>> GenericDelegator delegator = (GenericDelegator)
>>>> pageContext.getRequest().getAttribute("delegator");
>>>> GenericValue userLogin = (GenericValue)
>>>> pageContext.getSession().getAttribute("userLogin");
>>>> @@ -94,6 +95,7 @@
>>>> pageContext.setAttribute(projectsAttrName, validWorkEfforts);
>>>> }
>>>>
>>>> + /** @deprecated */
>>>> public static void getAllProjectPhases(PageContext pageContext, String
>>>> phasesAttrName) {
>>>> GenericDelegator delegator = (GenericDelegator)
>>>> pageContext.getRequest().getAttribute("delegator");
>>>> GenericValue userLogin = (GenericValue)
>>>> pageContext.getSession().getAttribute("userLogin");
>>>> @@ -141,6 +143,7 @@
>>>> pageContext.setAttribute(phasesAttrName, validWorkEfforts);
>>>> }
>>>>
>>>> + /** @deprecated */
>>>> public static void getAllPhaseTasks(PageContext pageContext, String
>>>> tasksAttrName) {
>>>> GenericDelegator delegator = (GenericDelegator)
>>>> pageContext.getRequest().getAttribute("delegator");
>>>> GenericValue userLogin = (GenericValue)
>>>> pageContext.getSession().getAttribute("userLogin");
>>>> @@ -187,6 +190,7 @@
>>>> pageContext.setAttribute(tasksAttrName, validWorkEfforts);
>>>> }
>>>>
>>>> + /** @deprecated */
>>>> public static void getTaskNotes(PageContext pageContext, String
>>>> notesAttrName) {
>>>> GenericDelegator delegator = (GenericDelegator)
>>>> pageContext.getRequest().getAttribute("delegator");
>>>> GenericValue userLogin = (GenericValue)
>>>> pageContext.getSession().getAttribute("userLogin");
>>>>
>>>> Modified:
>>>>
>>>> ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/workeffort/WorkEffortWorker.java
>>>> URL:
>>>>
>>>>
>>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/workeffort/WorkEffortWorker.java?rev=690573&r1=690572&r2=690573&view=diff
>>>>
>>>>
>>>> ==============================================================================
>>>> ---
>>>>
>>>> ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/workeffort/WorkEffortWorker.java
>>>> (original)
>>>> +++
>>>>
>>>> ofbiz/trunk/applications/workeffort/src/org/ofbiz/workeffort/workeffort/WorkEffortWorker.java
>>>> Sat Aug 30 13:05:40 2008
>>>> @@ -41,7 +41,7 @@
>>>>
>>>> public static final String module = WorkEffortWorker.class.getName();
>>>>
>>>> - // TODO: REMOVE this method when JSPs/etc are moved to FreeMarker;
>>>> this is replaced by a corresponding service
>>>> + /** @deprecated */
>>>> public static void getWorkEffort(PageContext pageContext, String
>>>> workEffortIdAttrName, String workEffortAttrName, String
>>>> partyAssignsAttrName,
>>>> String canViewAttrName, String tryEntityAttrName, String
>>>> currentStatusAttrName) {
>>>> GenericDelegator delegator = (GenericDelegator)
>>>> pageContext.getRequest().getAttribute("delegator");
>>>> @@ -125,12 +125,15 @@
>>>> pageContext.setAttribute(currentStatusAttrName,
>>>> currentStatus);
>>>> }
>>>>
>>>> + /** @deprecated */
>>>> public static void getMonthWorkEffortEvents(PageContext pageContext,
>>>> String attributeName) {}
>>>>
>>>> + /** @deprecated */
>>>> public static void getActivityContext(PageContext pageContext, String
>>>> workEffortId) {
>>>> getActivityContext(pageContext, workEffortId, "activityContext");
>>>> }
>>>>
>>>> + /** @deprecated */
>>>> public static void getActivityContext(PageContext pageContext, String
>>>> workEffortId, String attribute) {
>>>> LocalDispatcher dispatcher = (LocalDispatcher)
>>>> pageContext.getRequest().getAttribute("dispatcher");
>>>> GenericValue userLogin = (GenericValue)
>>>> pageContext.getSession().getAttribute("userLogin");
>>>>
>>>>
>>>>
>>>