Re: svn commit: r1840659 - in /ofbiz/ofbiz-framework/trunk/applications: order/template/order/OrderShippingInfo.ftl product/minilang/shipment/shipment/ShipmentServices.xml product/servicedef/services_
Le 18/09/2018 à 17:37, Jacques Le Roux a écrit : Le 18/09/2018 à 09:26, Jacques Le Roux a écrit : Hi, We had this discussion before, I made a proposition but w/o much feedback did not work on it https://markmail.org/message/wvlaie5lro4w7pp4 Propositions so far seem good to me, looking forward for action, oOps, actually I did https://markmail.org/message/qnckq5zxvdj563ny :D Please feel free to improve/augment if needed Jacques Hi Michael, It seems what I missed then to write is your comment at https://markmail.org/message/hhuaddrrj3wfa2xs Also Taher's last proposition (process), if everybody agree with it, is not yet documented. Jacques Jacques Le 17/09/2018 à 22:25, Gil Portenseigne a écrit : Hello, There is an old wiki page around entity deprecation (https://cwiki.apache.org/confluence/display/OFBIZ/General+Entity+Overview#GeneralEntityOverview-DeprecatedEntities) But I couldn't find any about service or others. I agree with the need of deprecation process for services/entity/methods. I met some deprecation examples with no date to remove them (see oldXXXSequenceEnumId in PartyAcctgPreference entity, that date from 2010). So +1 for Michael suggestion ! And I agree with Taher process example ! Should we file a jira to populate developer-manual.adoc with those guidelines and maintain another adoc for the deprecation changelog ? Gil Le mercredi 12 sept. 2018 à 15:42:23 (+0300), Taher Alkhateeb a écrit : One way to make the transition smooth, is to do the following: 1- create a new service definition (the renamed service) 2- rename the implementation service (where the source code lives) 3- point both the old and new service definitions to the same new and renamed implementation 4- Mark the old service definition as deprecated and specify _when_ you would intend to remove it. On Wed, Sep 12, 2018 at 3:39 PM Taher Alkhateeb wrote: +1 to Michael's suggestion. We are changing core services in core applications. On Wed, Sep 12, 2018 at 3:22 PM Michael Brohl wrote: Hi Jacques, all, I think these renames are problematic for exitsing users who might use these services in their productive environments. We should agree upon a "deprecation period" where these changes are announced but not implemented and implement them in the next major release. We also need a proper documentation/changelog for these changes between major releases to allow for easy migration. The changelog and/or documentation should be part of the commit. What do you and others think? Regards, Michael Am 12.09.18 um 13:09 schrieb jler...@apache.org: Author: jleroux Date: Wed Sep 12 11:09:50 2018 New Revision: 1840659 URL: http://svn.apache.org/viewvc?rev=1840659=rev Log: Improved: [Naming Convention] Change 'quickShipPurchaseOrder' to 'quickReceivePurchaseOrder' (OFBIZ-10558) We have the option of 'Quick Receive Purchase Order' from Order Overview screen. In the feature, the request and service name is 'quickShipPurchaseOrder which is confusing. Change the name to 'quickReceivePurchaseOrder'. Thanks: Deepak Nigam for the patch and Suraj Khurana for review Modified: ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml ofbiz/ofbiz-framework/trunk/applications/product/servicedef/services_shipment.xml ofbiz/ofbiz-framework/trunk/applications/product/webapp/facility/WEB-INF/controller.xml Modified: ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl?rev=1840659=1840658=1840659=diff == --- ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl (original) +++ ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl Wed Sep 12 11:09:50 2018 @@ -69,7 +69,7 @@ under the License. <#if ownedFacilities?has_content> <#if !allShipments?has_content> - + <#-- destination form (/facility/control/ReceiveInventory) wants purchaseOrderId instead of orderId, so we set it here as a workaround --> @@ -83,7 +83,7 @@ under the License. - action="/facility/control/quickShipPurchaseOrder?externalLoginKey=${externalLoginKey}" method="post"> + action="/facility/control/quickReceivePurchaseOrder?externalLoginKey=${externalLoginKey}" method="post"> Modified: ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml URL:
Re: svn commit: r1840659 - in /ofbiz/ofbiz-framework/trunk/applications: order/template/order/OrderShippingInfo.ftl product/minilang/shipment/shipment/ShipmentServices.xml product/servicedef/services_
Le 18/09/2018 à 09:26, Jacques Le Roux a écrit : Hi, We had this discussion before, I made a proposition but w/o much feedback did not work on it https://markmail.org/message/wvlaie5lro4w7pp4 Propositions so far seem good to me, looking forward for action, oOps, actually I did https://markmail.org/message/qnckq5zxvdj563ny :D Please feel free to improve/augment if needed Jacques Jacques Le 17/09/2018 à 22:25, Gil Portenseigne a écrit : Hello, There is an old wiki page around entity deprecation (https://cwiki.apache.org/confluence/display/OFBIZ/General+Entity+Overview#GeneralEntityOverview-DeprecatedEntities) But I couldn't find any about service or others. I agree with the need of deprecation process for services/entity/methods. I met some deprecation examples with no date to remove them (see oldXXXSequenceEnumId in PartyAcctgPreference entity, that date from 2010). So +1 for Michael suggestion ! And I agree with Taher process example ! Should we file a jira to populate developer-manual.adoc with those guidelines and maintain another adoc for the deprecation changelog ? Gil Le mercredi 12 sept. 2018 à 15:42:23 (+0300), Taher Alkhateeb a écrit : One way to make the transition smooth, is to do the following: 1- create a new service definition (the renamed service) 2- rename the implementation service (where the source code lives) 3- point both the old and new service definitions to the same new and renamed implementation 4- Mark the old service definition as deprecated and specify _when_ you would intend to remove it. On Wed, Sep 12, 2018 at 3:39 PM Taher Alkhateeb wrote: +1 to Michael's suggestion. We are changing core services in core applications. On Wed, Sep 12, 2018 at 3:22 PM Michael Brohl wrote: Hi Jacques, all, I think these renames are problematic for exitsing users who might use these services in their productive environments. We should agree upon a "deprecation period" where these changes are announced but not implemented and implement them in the next major release. We also need a proper documentation/changelog for these changes between major releases to allow for easy migration. The changelog and/or documentation should be part of the commit. What do you and others think? Regards, Michael Am 12.09.18 um 13:09 schrieb jler...@apache.org: Author: jleroux Date: Wed Sep 12 11:09:50 2018 New Revision: 1840659 URL: http://svn.apache.org/viewvc?rev=1840659=rev Log: Improved: [Naming Convention] Change 'quickShipPurchaseOrder' to 'quickReceivePurchaseOrder' (OFBIZ-10558) We have the option of 'Quick Receive Purchase Order' from Order Overview screen. In the feature, the request and service name is 'quickShipPurchaseOrder which is confusing. Change the name to 'quickReceivePurchaseOrder'. Thanks: Deepak Nigam for the patch and Suraj Khurana for review Modified: ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml ofbiz/ofbiz-framework/trunk/applications/product/servicedef/services_shipment.xml ofbiz/ofbiz-framework/trunk/applications/product/webapp/facility/WEB-INF/controller.xml Modified: ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl?rev=1840659=1840658=1840659=diff == --- ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl (original) +++ ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl Wed Sep 12 11:09:50 2018 @@ -69,7 +69,7 @@ under the License. <#if ownedFacilities?has_content> <#if !allShipments?has_content> - + <#-- destination form (/facility/control/ReceiveInventory) wants purchaseOrderId instead of orderId, so we set it here as a workaround --> @@ -83,7 +83,7 @@ under the License. - action="/facility/control/quickShipPurchaseOrder?externalLoginKey=${externalLoginKey}" method="post"> + action="/facility/control/quickReceivePurchaseOrder?externalLoginKey=${externalLoginKey}" method="post"> Modified: ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml?rev=1840659=1840658=1840659=diff == ---
Re: svn commit: r1840659 - in /ofbiz/ofbiz-framework/trunk/applications: order/template/order/OrderShippingInfo.ftl product/minilang/shipment/shipment/ShipmentServices.xml product/servicedef/services_
Hi, We had this discussion before, I made a proposition but w/o much feedback did not work on it https://markmail.org/message/wvlaie5lro4w7pp4 Propositions so far seem good to me, looking forward for action, Jacques Le 17/09/2018 à 22:25, Gil Portenseigne a écrit : Hello, There is an old wiki page around entity deprecation (https://cwiki.apache.org/confluence/display/OFBIZ/General+Entity+Overview#GeneralEntityOverview-DeprecatedEntities) But I couldn't find any about service or others. I agree with the need of deprecation process for services/entity/methods. I met some deprecation examples with no date to remove them (see oldXXXSequenceEnumId in PartyAcctgPreference entity, that date from 2010). So +1 for Michael suggestion ! And I agree with Taher process example ! Should we file a jira to populate developer-manual.adoc with those guidelines and maintain another adoc for the deprecation changelog ? Gil Le mercredi 12 sept. 2018 à 15:42:23 (+0300), Taher Alkhateeb a écrit : One way to make the transition smooth, is to do the following: 1- create a new service definition (the renamed service) 2- rename the implementation service (where the source code lives) 3- point both the old and new service definitions to the same new and renamed implementation 4- Mark the old service definition as deprecated and specify _when_ you would intend to remove it. On Wed, Sep 12, 2018 at 3:39 PM Taher Alkhateeb wrote: +1 to Michael's suggestion. We are changing core services in core applications. On Wed, Sep 12, 2018 at 3:22 PM Michael Brohl wrote: Hi Jacques, all, I think these renames are problematic for exitsing users who might use these services in their productive environments. We should agree upon a "deprecation period" where these changes are announced but not implemented and implement them in the next major release. We also need a proper documentation/changelog for these changes between major releases to allow for easy migration. The changelog and/or documentation should be part of the commit. What do you and others think? Regards, Michael Am 12.09.18 um 13:09 schrieb jler...@apache.org: Author: jleroux Date: Wed Sep 12 11:09:50 2018 New Revision: 1840659 URL: http://svn.apache.org/viewvc?rev=1840659=rev Log: Improved: [Naming Convention] Change 'quickShipPurchaseOrder' to 'quickReceivePurchaseOrder' (OFBIZ-10558) We have the option of 'Quick Receive Purchase Order' from Order Overview screen. In the feature, the request and service name is 'quickShipPurchaseOrder which is confusing. Change the name to 'quickReceivePurchaseOrder'. Thanks: Deepak Nigam for the patch and Suraj Khurana for review Modified: ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml ofbiz/ofbiz-framework/trunk/applications/product/servicedef/services_shipment.xml ofbiz/ofbiz-framework/trunk/applications/product/webapp/facility/WEB-INF/controller.xml Modified: ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl?rev=1840659=1840658=1840659=diff == --- ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl (original) +++ ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl Wed Sep 12 11:09:50 2018 @@ -69,7 +69,7 @@ under the License. <#if ownedFacilities?has_content> <#if !allShipments?has_content> - + <#-- destination form (/facility/control/ReceiveInventory) wants purchaseOrderId instead of orderId, so we set it here as a workaround --> @@ -83,7 +83,7 @@ under the License. - + Modified: ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml?rev=1840659=1840658=1840659=diff == --- ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml (original) +++ ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml Wed Sep 12 11:09:50 2018 @@ -1359,12 +1359,12 @@ under the License. - + - +
Re: svn commit: r1840659 - in /ofbiz/ofbiz-framework/trunk/applications: order/template/order/OrderShippingInfo.ftl product/minilang/shipment/shipment/ShipmentServices.xml product/servicedef/services_
Hello, There is an old wiki page around entity deprecation (https://cwiki.apache.org/confluence/display/OFBIZ/General+Entity+Overview#GeneralEntityOverview-DeprecatedEntities) But I couldn't find any about service or others. I agree with the need of deprecation process for services/entity/methods. I met some deprecation examples with no date to remove them (see oldXXXSequenceEnumId in PartyAcctgPreference entity, that date from 2010). So +1 for Michael suggestion ! And I agree with Taher process example ! Should we file a jira to populate developer-manual.adoc with those guidelines and maintain another adoc for the deprecation changelog ? Gil Le mercredi 12 sept. 2018 à 15:42:23 (+0300), Taher Alkhateeb a écrit : > One way to make the transition smooth, is to do the following: > 1- create a new service definition (the renamed service) > 2- rename the implementation service (where the source code lives) > 3- point both the old and new service definitions to the same new and > renamed implementation > 4- Mark the old service definition as deprecated and specify _when_ > you would intend to remove it. > On Wed, Sep 12, 2018 at 3:39 PM Taher Alkhateeb > wrote: > > > > +1 to Michael's suggestion. We are changing core services in core > > applications. > > On Wed, Sep 12, 2018 at 3:22 PM Michael Brohl > > wrote: > > > > > > Hi Jacques, all, > > > > > > I think these renames are problematic for exitsing users who might use > > > these services in their productive environments. > > > > > > We should agree upon a "deprecation period" where these changes are > > > announced but not implemented and implement them in the next major > > > release. > > > > > > We also need a proper documentation/changelog for these changes between > > > major releases to allow for easy migration. The changelog and/or > > > documentation should be part of the commit. > > > > > > What do you and others think? > > > > > > Regards, > > > > > > Michael > > > > > > Am 12.09.18 um 13:09 schrieb jler...@apache.org: > > > > Author: jleroux > > > > Date: Wed Sep 12 11:09:50 2018 > > > > New Revision: 1840659 > > > > > > > > URL: http://svn.apache.org/viewvc?rev=1840659=rev > > > > Log: > > > > Improved: [Naming Convention] Change 'quickShipPurchaseOrder' to > > > > 'quickReceivePurchaseOrder' > > > > (OFBIZ-10558) > > > > > > > > We have the option of 'Quick Receive Purchase Order' from Order > > > > Overview screen. > > > > In the feature, the request and service name is 'quickShipPurchaseOrder > > > > which > > > > is confusing. Change the name to 'quickReceivePurchaseOrder'. > > > > > > > > Thanks: Deepak Nigam for the patch and Suraj Khurana for review > > > > > > > > Modified: > > > > > > > > ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl > > > > > > > > ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml > > > > > > > > ofbiz/ofbiz-framework/trunk/applications/product/servicedef/services_shipment.xml > > > > > > > > ofbiz/ofbiz-framework/trunk/applications/product/webapp/facility/WEB-INF/controller.xml > > > > > > > > Modified: > > > > ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl > > > > URL: > > > > http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl?rev=1840659=1840658=1840659=diff > > > > == > > > > --- > > > > ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl > > > > (original) > > > > +++ > > > > ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl > > > > Wed Sep 12 11:09:50 2018 > > > > @@ -69,7 +69,7 @@ under the License. > > > > <#if ownedFacilities?has_content> > > > > <#if !allShipments?has_content> > > > > > > > > - > > > action="/facility/control/quickShipPurchaseOrder?externalLoginKey=${externalLoginKey}" > > > > method="post"> > > > > + > > > action="/facility/control/quickReceivePurchaseOrder?externalLoginKey=${externalLoginKey}" > > > > method="post"> > > > > > > > value="Y"/> > > > > > > > value="${orderId}"/> > > > > <#-- destination form > > > > (/facility/control/ReceiveInventory) wants purchaseOrderId instead of > > > > orderId, so we set it here as a workaround --> > > > > @@ -83,7 +83,7 @@ under the License. > > > > > > > > > > > > > > > > - > > > action="/facility/control/quickShipPurchaseOrder?externalLoginKey=${externalLoginKey}" > > > > method="post"> > > > > + > > > action="/facility/control/quickReceivePurchaseOrder?externalLoginKey=${externalLoginKey}" > > > >
Re: svn commit: r1840659 - in /ofbiz/ofbiz-framework/trunk/applications: order/template/order/OrderShippingInfo.ftl product/minilang/shipment/shipment/ShipmentServices.xml product/servicedef/services_
One way to make the transition smooth, is to do the following: 1- create a new service definition (the renamed service) 2- rename the implementation service (where the source code lives) 3- point both the old and new service definitions to the same new and renamed implementation 4- Mark the old service definition as deprecated and specify _when_ you would intend to remove it. On Wed, Sep 12, 2018 at 3:39 PM Taher Alkhateeb wrote: > > +1 to Michael's suggestion. We are changing core services in core > applications. > On Wed, Sep 12, 2018 at 3:22 PM Michael Brohl > wrote: > > > > Hi Jacques, all, > > > > I think these renames are problematic for exitsing users who might use > > these services in their productive environments. > > > > We should agree upon a "deprecation period" where these changes are > > announced but not implemented and implement them in the next major release. > > > > We also need a proper documentation/changelog for these changes between > > major releases to allow for easy migration. The changelog and/or > > documentation should be part of the commit. > > > > What do you and others think? > > > > Regards, > > > > Michael > > > > Am 12.09.18 um 13:09 schrieb jler...@apache.org: > > > Author: jleroux > > > Date: Wed Sep 12 11:09:50 2018 > > > New Revision: 1840659 > > > > > > URL: http://svn.apache.org/viewvc?rev=1840659=rev > > > Log: > > > Improved: [Naming Convention] Change 'quickShipPurchaseOrder' to > > > 'quickReceivePurchaseOrder' > > > (OFBIZ-10558) > > > > > > We have the option of 'Quick Receive Purchase Order' from Order Overview > > > screen. > > > In the feature, the request and service name is 'quickShipPurchaseOrder > > > which > > > is confusing. Change the name to 'quickReceivePurchaseOrder'. > > > > > > Thanks: Deepak Nigam for the patch and Suraj Khurana for review > > > > > > Modified: > > > > > > ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl > > > > > > ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml > > > > > > ofbiz/ofbiz-framework/trunk/applications/product/servicedef/services_shipment.xml > > > > > > ofbiz/ofbiz-framework/trunk/applications/product/webapp/facility/WEB-INF/controller.xml > > > > > > Modified: > > > ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl > > > URL: > > > http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl?rev=1840659=1840658=1840659=diff > > > == > > > --- > > > ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl > > > (original) > > > +++ > > > ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl > > > Wed Sep 12 11:09:50 2018 > > > @@ -69,7 +69,7 @@ under the License. > > > <#if ownedFacilities?has_content> > > > <#if !allShipments?has_content> > > > > > > - > > action="/facility/control/quickShipPurchaseOrder?externalLoginKey=${externalLoginKey}" > > > method="post"> > > > + > > action="/facility/control/quickReceivePurchaseOrder?externalLoginKey=${externalLoginKey}" > > > method="post"> > > > > > value="Y"/> > > > > > value="${orderId}"/> > > > <#-- destination form > > > (/facility/control/ReceiveInventory) wants purchaseOrderId instead of > > > orderId, so we set it here as a workaround --> > > > @@ -83,7 +83,7 @@ under the License. > > > > > > > > > > > > - > > action="/facility/control/quickShipPurchaseOrder?externalLoginKey=${externalLoginKey}" > > > method="post"> > > > + > > action="/facility/control/quickReceivePurchaseOrder?externalLoginKey=${externalLoginKey}" > > > method="post"> > > > > > value="Y"/> > > > > > value="${orderId}"/> > > > > > value="${orderId}"/> > > > > > > Modified: > > > ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml > > > URL: > > > http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml?rev=1840659=1840658=1840659=diff > > > == > > > --- > > > ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml > > > (original) > > > +++ > > > ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml > > > Wed Sep 12 11:09:50 2018 > > > @@ -1359,12 +1359,12 @@ under the License. > > > > > > > > > > > > - > >
Re: svn commit: r1840659 - in /ofbiz/ofbiz-framework/trunk/applications: order/template/order/OrderShippingInfo.ftl product/minilang/shipment/shipment/ShipmentServices.xml product/servicedef/services_
+1 to Michael's suggestion. We are changing core services in core applications. On Wed, Sep 12, 2018 at 3:22 PM Michael Brohl wrote: > > Hi Jacques, all, > > I think these renames are problematic for exitsing users who might use > these services in their productive environments. > > We should agree upon a "deprecation period" where these changes are > announced but not implemented and implement them in the next major release. > > We also need a proper documentation/changelog for these changes between > major releases to allow for easy migration. The changelog and/or > documentation should be part of the commit. > > What do you and others think? > > Regards, > > Michael > > Am 12.09.18 um 13:09 schrieb jler...@apache.org: > > Author: jleroux > > Date: Wed Sep 12 11:09:50 2018 > > New Revision: 1840659 > > > > URL: http://svn.apache.org/viewvc?rev=1840659=rev > > Log: > > Improved: [Naming Convention] Change 'quickShipPurchaseOrder' to > > 'quickReceivePurchaseOrder' > > (OFBIZ-10558) > > > > We have the option of 'Quick Receive Purchase Order' from Order Overview > > screen. > > In the feature, the request and service name is 'quickShipPurchaseOrder > > which > > is confusing. Change the name to 'quickReceivePurchaseOrder'. > > > > Thanks: Deepak Nigam for the patch and Suraj Khurana for review > > > > Modified: > > > > ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl > > > > ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml > > > > ofbiz/ofbiz-framework/trunk/applications/product/servicedef/services_shipment.xml > > > > ofbiz/ofbiz-framework/trunk/applications/product/webapp/facility/WEB-INF/controller.xml > > > > Modified: > > ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl > > URL: > > http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl?rev=1840659=1840658=1840659=diff > > == > > --- > > ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl > > (original) > > +++ > > ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl > > Wed Sep 12 11:09:50 2018 > > @@ -69,7 +69,7 @@ under the License. > > <#if ownedFacilities?has_content> > > <#if !allShipments?has_content> > > > > - > action="/facility/control/quickShipPurchaseOrder?externalLoginKey=${externalLoginKey}" > > method="post"> > > + > action="/facility/control/quickReceivePurchaseOrder?externalLoginKey=${externalLoginKey}" > > method="post"> > > > value="Y"/> > > > value="${orderId}"/> > > <#-- destination form > > (/facility/control/ReceiveInventory) wants purchaseOrderId instead of > > orderId, so we set it here as a workaround --> > > @@ -83,7 +83,7 @@ under the License. > > > > > > > > - > action="/facility/control/quickShipPurchaseOrder?externalLoginKey=${externalLoginKey}" > > method="post"> > > + > action="/facility/control/quickReceivePurchaseOrder?externalLoginKey=${externalLoginKey}" > > method="post"> > > > value="Y"/> > > > value="${orderId}"/> > > > value="${orderId}"/> > > > > Modified: > > ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml > > URL: > > http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml?rev=1840659=1840658=1840659=diff > > == > > --- > > ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml > > (original) > > +++ > > ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml > > Wed Sep 12 11:09:50 2018 > > @@ -1359,12 +1359,12 @@ under the License. > > > > > > > > - > short-description="Quick ships an entire purchase order to a facility"> > > + > short-description="Quick receives an entire purchase order in a facility"> > > > > > > > >> method-name="createShipmentForFacilityAndShipGroup"/> > > - > > + > > > > > > > > > > Modified: > > ofbiz/ofbiz-framework/trunk/applications/product/servicedef/services_shipment.xml > > URL: > > http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/product/servicedef/services_shipment.xml?rev=1840659=1840658=1840659=diff > >
Re: svn commit: r1840659 - in /ofbiz/ofbiz-framework/trunk/applications: order/template/order/OrderShippingInfo.ftl product/minilang/shipment/shipment/ShipmentServices.xml product/servicedef/services_
Hi Jacques, all, I think these renames are problematic for exitsing users who might use these services in their productive environments. We should agree upon a "deprecation period" where these changes are announced but not implemented and implement them in the next major release. We also need a proper documentation/changelog for these changes between major releases to allow for easy migration. The changelog and/or documentation should be part of the commit. What do you and others think? Regards, Michael Am 12.09.18 um 13:09 schrieb jler...@apache.org: Author: jleroux Date: Wed Sep 12 11:09:50 2018 New Revision: 1840659 URL: http://svn.apache.org/viewvc?rev=1840659=rev Log: Improved: [Naming Convention] Change 'quickShipPurchaseOrder' to 'quickReceivePurchaseOrder' (OFBIZ-10558) We have the option of 'Quick Receive Purchase Order' from Order Overview screen. In the feature, the request and service name is 'quickShipPurchaseOrder which is confusing. Change the name to 'quickReceivePurchaseOrder'. Thanks: Deepak Nigam for the patch and Suraj Khurana for review Modified: ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml ofbiz/ofbiz-framework/trunk/applications/product/servicedef/services_shipment.xml ofbiz/ofbiz-framework/trunk/applications/product/webapp/facility/WEB-INF/controller.xml Modified: ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl?rev=1840659=1840658=1840659=diff == --- ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl (original) +++ ofbiz/ofbiz-framework/trunk/applications/order/template/order/OrderShippingInfo.ftl Wed Sep 12 11:09:50 2018 @@ -69,7 +69,7 @@ under the License. <#if ownedFacilities?has_content> <#if !allShipments?has_content> - + <#-- destination form (/facility/control/ReceiveInventory) wants purchaseOrderId instead of orderId, so we set it here as a workaround --> @@ -83,7 +83,7 @@ under the License. - + Modified: ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml?rev=1840659=1840658=1840659=diff == --- ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml (original) +++ ofbiz/ofbiz-framework/trunk/applications/product/minilang/shipment/shipment/ShipmentServices.xml Wed Sep 12 11:09:50 2018 @@ -1359,12 +1359,12 @@ under the License. - + - + Modified: ofbiz/ofbiz-framework/trunk/applications/product/servicedef/services_shipment.xml URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/product/servicedef/services_shipment.xml?rev=1840659=1840658=1840659=diff == --- ofbiz/ofbiz-framework/trunk/applications/product/servicedef/services_shipment.xml (original) +++ ofbiz/ofbiz-framework/trunk/applications/product/servicedef/services_shipment.xml Wed Sep 12 11:09:50 2018 @@ -85,8 +85,8 @@ under the License. - + The mirror of quickShipEntireOrder, this service automatically creates shipments for an entire purchase order. All order items on each ship group is created as a Shipment. All items on a Shipment are automatically issued to a Package. The shipment's status is first set to CREATED and then set as SHIPPED. The facilityId is used to set the destinationFacilityId Modified: ofbiz/ofbiz-framework/trunk/applications/product/webapp/facility/WEB-INF/controller.xml URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/product/webapp/facility/WEB-INF/controller.xml?rev=1840659=1840658=1840659=diff == --- ofbiz/ofbiz-framework/trunk/applications/product/webapp/facility/WEB-INF/controller.xml (original) +++ ofbiz/ofbiz-framework/trunk/applications/product/webapp/facility/WEB-INF/controller.xml Wed Sep 12 11:09:50 2018 @@