It appears that there used to be a OrderItemShipGroup.productStoreShipMethId field, but there is no longer one, and when an order is saved the productStoreShipMethId in the ShoppingCart object is no longer saved. If anyone knows anything about this... Is that correct? If so, why was this change made? If no one knows anything about this then I think some investigation is needed... :( There is still some code that tries to use it that flares up when trying to edit an order item (orderdetail page in the Order Manager, Edit Items link, change quantity on any item): 2009-01-10 15:35:14,759 (http-0.0.0.0-8443-2) [ ServiceDispatcher.java:500:ERROR] ---- exception report ---------------------------------------------------------- Service [recalcShippingTotal] threw an unexpected exception/error Exception: org.ofbiz.service.GenericServiceException Message: Service [recalcShippingTotal] target threw an unexpected exception ([GenericEntity.get] "productStoreShipMethId" is not a field of OrderItemShipGroup) ---- cause --------------------------------------------------------------------- Exception: java.lang.IllegalArgumentException Message: [GenericEntity.get] "productStoreShipMethId" is not a field of OrderItemShipGroup ---- stack trace --------------------------------------------------------------- java.lang.IllegalArgumentException: [GenericEntity.get] "productStoreShipMethId" is not a field of OrderItemShipGroup org.ofbiz.entity.GenericEntity.get(GenericEntity.java:308) org.ofbiz.entity.GenericEntity.getString(GenericEntity.java:585) org .ofbiz .order .shoppingcart .shipping.ShippingEvents.getShipEstimate(ShippingEvents.java:121) org .ofbiz .order.order.OrderServices.recalcOrderShipping(OrderServices.java:1602) -David |
Administrator
|
I checked using FishEye and Nabble, it seems that OrderItemShipGroup.productStoreShipMethId field never existed. There are such
fields in some Product entities String productStoreShipMethId = cart.getProductStoreShipMethId(); was introduced in in r727381 (OFBIZ-2082) I guess we should better use cart.getProductStoreShipMethId(); But this means to pass a cart someway. I did not look further This prevent to add some types of Product to an existing order and should be fixed ASAP Jacques From: "David E Jones" <[hidden email]> > > It appears that there used to be a OrderItemShipGroup.productStoreShipMethId field, but there is no longer one, and when an > order is saved the productStoreShipMethId in the ShoppingCart object is no longer saved. > > If anyone knows anything about this... Is that correct? If so, why was this change made? > > If no one knows anything about this then I think some investigation is needed... :( > > There is still some code that tries to use it that flares up when trying to edit an order item (orderdetail page in the Order > Manager, Edit Items link, change quantity on any item): > > 2009-01-10 15:35:14,759 (http-0.0.0.0-8443-2) [ ServiceDispatcher.java:500:ERROR] > ---- exception report ---------------------------------------------------------- > Service [recalcShippingTotal] threw an unexpected exception/error > Exception: org.ofbiz.service.GenericServiceException > Message: Service [recalcShippingTotal] target threw an unexpected exception ([GenericEntity.get] "productStoreShipMethId" is not > a field of OrderItemShipGroup) > ---- cause --------------------------------------------------------------------- > Exception: java.lang.IllegalArgumentException > Message: [GenericEntity.get] "productStoreShipMethId" is not a field of OrderItemShipGroup > ---- stack trace --------------------------------------------------------------- > java.lang.IllegalArgumentException: [GenericEntity.get] "productStoreShipMethId" is not a field of OrderItemShipGroup > org.ofbiz.entity.GenericEntity.get(GenericEntity.java:308) > org.ofbiz.entity.GenericEntity.getString(GenericEntity.java:585) > org .ofbiz .order .shoppingcart .shipping.ShippingEvents.getShipEstimate(ShippingEvents.java:121) > org .ofbiz .order.order.OrderServices.recalcOrderShipping(OrderServices.java:1602) > > -David > |
Administrator
|
In reply to this post by David E Jones-3
I checked using FishEye and Nabble, it seems that OrderItemShipGroup.productStoreShipMethId field never existed. Note that there are
such fields in some Product entities String productStoreShipMethId = cart.getProductStoreShipMethId(); was introduced in in r727381 (OFBIZ-2082) I guess we should better use cart.getProductStoreShipMethId(); But this means to pass a cart somehow. I did not look further This prevent to add some types of Product to an existing order and should be fixed ASAP Thanks Jacques From: "David E Jones" <[hidden email]> > > It appears that there used to be a OrderItemShipGroup.productStoreShipMethId field, but there is no longer one, and when an > order is saved the productStoreShipMethId in the ShoppingCart object is no longer saved. > > If anyone knows anything about this... Is that correct? If so, why was this change made? > > If no one knows anything about this then I think some investigation is needed... :( > > There is still some code that tries to use it that flares up when trying to edit an order item (orderdetail page in the Order > Manager, Edit Items link, change quantity on any item): > > 2009-01-10 15:35:14,759 (http-0.0.0.0-8443-2) [ ServiceDispatcher.java:500:ERROR] > ---- exception report ---------------------------------------------------------- > Service [recalcShippingTotal] threw an unexpected exception/error > Exception: org.ofbiz.service.GenericServiceException > Message: Service [recalcShippingTotal] target threw an unexpected exception ([GenericEntity.get] "productStoreShipMethId" is not > a field of OrderItemShipGroup) > ---- cause --------------------------------------------------------------------- > Exception: java.lang.IllegalArgumentException > Message: [GenericEntity.get] "productStoreShipMethId" is not a field of OrderItemShipGroup > ---- stack trace --------------------------------------------------------------- > java.lang.IllegalArgumentException: [GenericEntity.get] "productStoreShipMethId" is not a field of OrderItemShipGroup > org.ofbiz.entity.GenericEntity.get(GenericEntity.java:308) > org.ofbiz.entity.GenericEntity.getString(GenericEntity.java:585) > org .ofbiz .order .shoppingcart .shipping.ShippingEvents.getShipEstimate(ShippingEvents.java:121) > org .ofbiz .order.order.OrderServices.recalcOrderShipping(OrderServices.java:1602) > > -David > |
Administrator
|
In reply to this post by Jacques Le Roux
Bump ?
Jacques From: "Jacques Le Roux" <[hidden email]> >I checked using FishEye and Nabble, it seems that OrderItemShipGroup.productStoreShipMethId field never existed. There are such >fields in some Product entities > > String productStoreShipMethId = cart.getProductStoreShipMethId(); > was introduced in in r727381 (OFBIZ-2082) > > I guess we should better use > cart.getProductStoreShipMethId(); > But this means to pass a cart someway. I did not look further > > This prevent to add some types of Product to an existing order and should be fixed ASAP > > Jacques > > From: "David E Jones" <[hidden email]> >> >> It appears that there used to be a OrderItemShipGroup.productStoreShipMethId field, but there is no longer one, and when an >> order is saved the productStoreShipMethId in the ShoppingCart object is no longer saved. >> >> If anyone knows anything about this... Is that correct? If so, why was this change made? >> >> If no one knows anything about this then I think some investigation is needed... :( >> >> There is still some code that tries to use it that flares up when trying to edit an order item (orderdetail page in the Order >> Manager, Edit Items link, change quantity on any item): >> >> 2009-01-10 15:35:14,759 (http-0.0.0.0-8443-2) [ ServiceDispatcher.java:500:ERROR] >> ---- exception report ---------------------------------------------------------- >> Service [recalcShippingTotal] threw an unexpected exception/error >> Exception: org.ofbiz.service.GenericServiceException >> Message: Service [recalcShippingTotal] target threw an unexpected exception ([GenericEntity.get] "productStoreShipMethId" is not >> a field of OrderItemShipGroup) >> ---- cause --------------------------------------------------------------------- >> Exception: java.lang.IllegalArgumentException >> Message: [GenericEntity.get] "productStoreShipMethId" is not a field of OrderItemShipGroup >> ---- stack trace --------------------------------------------------------------- >> java.lang.IllegalArgumentException: [GenericEntity.get] "productStoreShipMethId" is not a field of OrderItemShipGroup >> org.ofbiz.entity.GenericEntity.get(GenericEntity.java:308) >> org.ofbiz.entity.GenericEntity.getString(GenericEntity.java:585) >> org .ofbiz .order .shoppingcart .shipping.ShippingEvents.getShipEstimate(ShippingEvents.java:121) >> org .ofbiz .order.order.OrderServices.recalcOrderShipping(OrderServices.java:1602) >> >> -David >> > |
In reply to this post by Jacques Le Roux
We've had another complaint about this on the mailing list now... Here is the commit that Jacques is referring to: http://www.nabble.com/svn-commit:-r727381---in--ofbiz-trunk-applications:-ecommerce-script-org-ofbiz-ecommerce-customer--ecommerce-webapp-ecommerce-images--order-src-org-ofbiz-order-shoppingcart--order-src-org-ofbiz-order-shoppingcart-shipping--product-entitydef--product-se...-td21053476.html It looks like it was committed by Ashish, and there are credits for two of his coworkers: Arun Patidar & Jyotsna Rathore. Could one of you please look into this and comment on what this was meant for and if we can remove it and use something else, or should we build it out so it does not cause this error? Thanks, -David On Jan 18, 2009, at 12:25 PM, Jacques Le Roux wrote: > I checked using FishEye and Nabble, it seems that > OrderItemShipGroup.productStoreShipMethId field never existed. Note > that there are > such fields in some Product entities > > String productStoreShipMethId = cart.getProductStoreShipMethId(); > was introduced in in r727381 (OFBIZ-2082) > > I guess we should better use > cart.getProductStoreShipMethId(); > But this means to pass a cart somehow. I did not look further > > This prevent to add some types of Product to an existing order and > should be fixed ASAP > > Thanks > > Jacques > > From: "David E Jones" <[hidden email]> >> >> It appears that there used to be a >> OrderItemShipGroup.productStoreShipMethId field, but there is no >> longer one, and when an >> order is saved the productStoreShipMethId in the ShoppingCart >> object is no longer saved. >> >> If anyone knows anything about this... Is that correct? If so, why >> was this change made? >> >> If no one knows anything about this then I think some investigation >> is needed... :( >> >> There is still some code that tries to use it that flares up when >> trying to edit an order item (orderdetail page in the Order >> Manager, Edit Items link, change quantity on any item): >> >> 2009-01-10 15:35:14,759 (http-0.0.0.0-8443-2) >> [ ServiceDispatcher.java:500:ERROR] >> ---- exception report >> ---------------------------------------------------------- >> Service [recalcShippingTotal] threw an unexpected exception/error >> Exception: org.ofbiz.service.GenericServiceException >> Message: Service [recalcShippingTotal] target threw an unexpected >> exception ([GenericEntity.get] "productStoreShipMethId" is not >> a field of OrderItemShipGroup) >> ---- cause >> --------------------------------------------------------------------- >> Exception: java.lang.IllegalArgumentException >> Message: [GenericEntity.get] "productStoreShipMethId" is not a >> field of OrderItemShipGroup >> ---- stack trace >> --------------------------------------------------------------- >> java.lang.IllegalArgumentException: [GenericEntity.get] >> "productStoreShipMethId" is not a field of OrderItemShipGroup >> org.ofbiz.entity.GenericEntity.get(GenericEntity.java:308) >> org.ofbiz.entity.GenericEntity.getString(GenericEntity.java:585) >> org >> .ofbiz >> .order >> .shoppingcart >> .shipping.ShippingEvents.getShipEstimate(ShippingEvents.java:121) >> org >> .ofbiz >> .order.order.OrderServices.recalcOrderShipping(OrderServices.java: >> 1602) >> >> -David >> > |
I think this is related to something I suggested: modifying the
primary key of the shipping estimates entity to enhance flexibility. Probably the code that is broken now should be updated to reflect that change. Just my 2 cents... I will try to look at this as soon as I have some free time, if Ashish and co. will not reply before. Cheers, Jacopo On Feb 9, 2009, at 11:35 AM, David E Jones wrote: > > We've had another complaint about this on the mailing list now... > > Here is the commit that Jacques is referring to: > > http://www.nabble.com/svn-commit:-r727381---in--ofbiz-trunk-applications:-ecommerce-script-org-ofbiz-ecommerce-customer--ecommerce-webapp-ecommerce-images--order-src-org-ofbiz-order-shoppingcart--order-src-org-ofbiz-order-shoppingcart-shipping--product-entitydef--product-se...-td21053476.html > > It looks like it was committed by Ashish, and there are credits for > two of his coworkers: Arun Patidar & Jyotsna Rathore. Could one of > you please look into this and comment on what this was meant for and > if we can remove it and use something else, or should we build it > out so it does not cause this error? > > Thanks, > -David > > > On Jan 18, 2009, at 12:25 PM, Jacques Le Roux wrote: > >> I checked using FishEye and Nabble, it seems that >> OrderItemShipGroup.productStoreShipMethId field never existed. Note >> that there are >> such fields in some Product entities >> >> String productStoreShipMethId = cart.getProductStoreShipMethId(); >> was introduced in in r727381 (OFBIZ-2082) >> >> I guess we should better use >> cart.getProductStoreShipMethId(); >> But this means to pass a cart somehow. I did not look further >> >> This prevent to add some types of Product to an existing order and >> should be fixed ASAP >> >> Thanks >> >> Jacques >> >> From: "David E Jones" <[hidden email]> >>> >>> It appears that there used to be a >>> OrderItemShipGroup.productStoreShipMethId field, but there is no >>> longer one, and when an >>> order is saved the productStoreShipMethId in the ShoppingCart >>> object is no longer saved. >>> >>> If anyone knows anything about this... Is that correct? If so, why >>> was this change made? >>> >>> If no one knows anything about this then I think some >>> investigation is needed... :( >>> >>> There is still some code that tries to use it that flares up when >>> trying to edit an order item (orderdetail page in the Order >>> Manager, Edit Items link, change quantity on any item): >>> >>> 2009-01-10 15:35:14,759 (http-0.0.0.0-8443-2) >>> [ ServiceDispatcher.java:500:ERROR] >>> ---- exception report >>> ---------------------------------------------------------- >>> Service [recalcShippingTotal] threw an unexpected exception/error >>> Exception: org.ofbiz.service.GenericServiceException >>> Message: Service [recalcShippingTotal] target threw an unexpected >>> exception ([GenericEntity.get] "productStoreShipMethId" is not >>> a field of OrderItemShipGroup) >>> ---- cause >>> --------------------------------------------------------------------- >>> Exception: java.lang.IllegalArgumentException >>> Message: [GenericEntity.get] "productStoreShipMethId" is not a >>> field of OrderItemShipGroup >>> ---- stack trace >>> --------------------------------------------------------------- >>> java.lang.IllegalArgumentException: [GenericEntity.get] >>> "productStoreShipMethId" is not a field of OrderItemShipGroup >>> org.ofbiz.entity.GenericEntity.get(GenericEntity.java:308) >>> org.ofbiz.entity.GenericEntity.getString(GenericEntity.java:585) >>> org >>> .ofbiz >>> .order >>> .shoppingcart >>> .shipping.ShippingEvents.getShipEstimate(ShippingEvents.java:121) >>> org >>> .ofbiz >>> .order.order.OrderServices.recalcOrderShipping(OrderServices.java: >>> 1602) >>> >>> -David >>> >> > smime.p7s (3K) Download Attachment |
Hello David,
I am reviewing this now and will do the needful soon. -- Ashish On Mon, Feb 9, 2009 at 4:30 PM, Jacopo Cappellato <[hidden email]> wrote: > I think this is related to something I suggested: modifying the primary key > of the shipping estimates entity to enhance flexibility. > Probably the code that is broken now should be updated to reflect that > change. > > Just my 2 cents... I will try to look at this as soon as I have some free > time, if Ashish and co. will not reply before. > > Cheers, > > Jacopo > > On Feb 9, 2009, at 11:35 AM, David E Jones wrote: > >> >> We've had another complaint about this on the mailing list now... >> >> Here is the commit that Jacques is referring to: >> >> >> http://www.nabble.com/svn-commit:-r727381---in--ofbiz-trunk-applications:-ecommerce-script-org-ofbiz-ecommerce-customer--ecommerce-webapp-ecommerce-images--order-src-org-ofbiz-order-shoppingcart--order-src-org-ofbiz-order-shoppingcart-shipping--product-entitydef--product-se...-td21053476.html >> >> It looks like it was committed by Ashish, and there are credits for two of >> his coworkers: Arun Patidar & Jyotsna Rathore. Could one of you please look >> into this and comment on what this was meant for and if we can remove it and >> use something else, or should we build it out so it does not cause this >> error? >> >> Thanks, >> -David >> >> >> On Jan 18, 2009, at 12:25 PM, Jacques Le Roux wrote: >> >>> I checked using FishEye and Nabble, it seems that >>> OrderItemShipGroup.productStoreShipMethId field never existed. Note that >>> there are >>> such fields in some Product entities >>> >>> String productStoreShipMethId = cart.getProductStoreShipMethId(); >>> was introduced in in r727381 (OFBIZ-2082) >>> >>> I guess we should better use >>> cart.getProductStoreShipMethId(); >>> But this means to pass a cart somehow. I did not look further >>> >>> This prevent to add some types of Product to an existing order and should >>> be fixed ASAP >>> >>> Thanks >>> >>> Jacques >>> >>> From: "David E Jones" <[hidden email]> >>>> >>>> It appears that there used to be a >>>> OrderItemShipGroup.productStoreShipMethId field, but there is no longer >>>> one, and when an >>>> order is saved the productStoreShipMethId in the ShoppingCart object is >>>> no longer saved. >>>> >>>> If anyone knows anything about this... Is that correct? If so, why was >>>> this change made? >>>> >>>> If no one knows anything about this then I think some investigation is >>>> needed... :( >>>> >>>> There is still some code that tries to use it that flares up when >>>> trying to edit an order item (orderdetail page in the Order >>>> Manager, Edit Items link, change quantity on any item): >>>> >>>> 2009-01-10 15:35:14,759 (http-0.0.0.0-8443-2) [ >>>> ServiceDispatcher.java:500:ERROR] >>>> ---- exception report >>>> ---------------------------------------------------------- >>>> Service [recalcShippingTotal] threw an unexpected exception/error >>>> Exception: org.ofbiz.service.GenericServiceException >>>> Message: Service [recalcShippingTotal] target threw an unexpected >>>> exception ([GenericEntity.get] "productStoreShipMethId" is not >>>> a field of OrderItemShipGroup) >>>> ---- cause >>>> --------------------------------------------------------------------- >>>> Exception: java.lang.IllegalArgumentException >>>> Message: [GenericEntity.get] "productStoreShipMethId" is not a field of >>>> OrderItemShipGroup >>>> ---- stack trace >>>> --------------------------------------------------------------- >>>> java.lang.IllegalArgumentException: [GenericEntity.get] >>>> "productStoreShipMethId" is not a field of OrderItemShipGroup >>>> org.ofbiz.entity.GenericEntity.get(GenericEntity.java:308) >>>> org.ofbiz.entity.GenericEntity.getString(GenericEntity.java:585) >>>> >>>> org .ofbiz .order .shoppingcart .shipping.ShippingEvents.getShipEstimate(ShippingEvents.java:121) >>>> >>>> org .ofbiz .order.order.OrderServices.recalcOrderShipping(OrderServices.java:1602) >>>> >>>> -David >>>> >>> >> > > |
Hello All,
It was our mistake, we apologize for this. We Created a jira issue and also attached patch with proper solution. Please let me now if there is any more issue regarding this. Thanks and Regards --- Jyotsna Rathore & Arun Patidar Ashish Vijaywargiya wrote: > Hello David, > > I am reviewing this now and will do the needful soon. > > -- > Ashish > > On Mon, Feb 9, 2009 at 4:30 PM, Jacopo Cappellato > <[hidden email]> wrote: > >> I think this is related to something I suggested: modifying the primary key >> of the shipping estimates entity to enhance flexibility. >> Probably the code that is broken now should be updated to reflect that >> change. >> >> Just my 2 cents... I will try to look at this as soon as I have some free >> time, if Ashish and co. will not reply before. >> >> Cheers, >> >> Jacopo >> >> On Feb 9, 2009, at 11:35 AM, David E Jones wrote: >> >> >>> We've had another complaint about this on the mailing list now... >>> >>> Here is the commit that Jacques is referring to: >>> >>> >>> http://www.nabble.com/svn-commit:-r727381---in--ofbiz-trunk-applications:-ecommerce-script-org-ofbiz-ecommerce-customer--ecommerce-webapp-ecommerce-images--order-src-org-ofbiz-order-shoppingcart--order-src-org-ofbiz-order-shoppingcart-shipping--product-entitydef--product-se...-td21053476.html >>> >>> It looks like it was committed by Ashish, and there are credits for two of >>> his coworkers: Arun Patidar & Jyotsna Rathore. Could one of you please look >>> into this and comment on what this was meant for and if we can remove it and >>> use something else, or should we build it out so it does not cause this >>> error? >>> >>> Thanks, >>> -David >>> >>> >>> On Jan 18, 2009, at 12:25 PM, Jacques Le Roux wrote: >>> >>> >>>> I checked using FishEye and Nabble, it seems that >>>> OrderItemShipGroup.productStoreShipMethId field never existed. Note that >>>> there are >>>> such fields in some Product entities >>>> >>>> String productStoreShipMethId = cart.getProductStoreShipMethId(); >>>> was introduced in in r727381 (OFBIZ-2082) >>>> >>>> I guess we should better use >>>> cart.getProductStoreShipMethId(); >>>> But this means to pass a cart somehow. I did not look further >>>> >>>> This prevent to add some types of Product to an existing order and should >>>> be fixed ASAP >>>> >>>> Thanks >>>> >>>> Jacques >>>> >>>> From: "David E Jones" <[hidden email]> >>>> >>>>> It appears that there used to be a >>>>> OrderItemShipGroup.productStoreShipMethId field, but there is no longer >>>>> one, and when an >>>>> order is saved the productStoreShipMethId in the ShoppingCart object is >>>>> no longer saved. >>>>> >>>>> If anyone knows anything about this... Is that correct? If so, why was >>>>> this change made? >>>>> >>>>> If no one knows anything about this then I think some investigation is >>>>> needed... :( >>>>> >>>>> There is still some code that tries to use it that flares up when >>>>> trying to edit an order item (orderdetail page in the Order >>>>> Manager, Edit Items link, change quantity on any item): >>>>> >>>>> 2009-01-10 15:35:14,759 (http-0.0.0.0-8443-2) [ >>>>> ServiceDispatcher.java:500:ERROR] >>>>> ---- exception report >>>>> ---------------------------------------------------------- >>>>> Service [recalcShippingTotal] threw an unexpected exception/error >>>>> Exception: org.ofbiz.service.GenericServiceException >>>>> Message: Service [recalcShippingTotal] target threw an unexpected >>>>> exception ([GenericEntity.get] "productStoreShipMethId" is not >>>>> a field of OrderItemShipGroup) >>>>> ---- cause >>>>> --------------------------------------------------------------------- >>>>> Exception: java.lang.IllegalArgumentException >>>>> Message: [GenericEntity.get] "productStoreShipMethId" is not a field of >>>>> OrderItemShipGroup >>>>> ---- stack trace >>>>> --------------------------------------------------------------- >>>>> java.lang.IllegalArgumentException: [GenericEntity.get] >>>>> "productStoreShipMethId" is not a field of OrderItemShipGroup >>>>> org.ofbiz.entity.GenericEntity.get(GenericEntity.java:308) >>>>> org.ofbiz.entity.GenericEntity.getString(GenericEntity.java:585) >>>>> >>>>> org .ofbiz .order .shoppingcart .shipping.ShippingEvents.getShipEstimate(ShippingEvents.java:121) >>>>> >>>>> org .ofbiz .order.order.OrderServices.recalcOrderShipping(OrderServices.java:1602) >>>>> >>>>> -David >>>>> >>>>> >> |
Free forum by Nabble | Edit this page |