Hii Mohammad, I agree with your points. I think we should wait for other's reply too, then we can conclude the implementation.
Thanks Renuka Srishti On Fri, Oct 6, 2017 at 7:34 PM, Mohammad Kathawala < mohammad.kathaw...@hotwaxsystems.com> wrote: > Hi, > > IMO, if all the items of the order need to be canceled then the user > should/would directly cancel the order rather than canceling all the items > individually. This will automatically cancel all the items and the order at > header level. Despite this, still, if the user cancels all the items > individually then IMO the order should not get canceled automatically at > header level because after the order is canceled you cannot add new items > to it and it might be possible that user wants to add new items to the > order after canceling all the existing items. The order should be canceled > manually in this case. > > Thanks. > > Regards*,* > *Mohammad Kathawala* | Technical Consultant / Team Lead > *HotWax Commerce* by *HotWax Systems* > Plot no. 80, Scheme no. 78 Part 2, Near Brilliant Convention Center, > Indore, > M.P 452010 > Cell phone: 7772858789 > > HotWax Systems recently received 8 mentions in *The Gartner Digital > Commerce Vendor Guide, 2016 *by Gartner, Inc., the world's leading IT > research and advisory company. Learn more about our research here > <https://www.gartner.com/technology/media-products/ > newsletters/HotWax/1-2UVLP6M/index.html> > . > > On Fri, Oct 6, 2017 at 7:14 PM, Ratnesh Upadhyay < > upadhyay.ratn...@gmail.com > > wrote: > > > Hi Renuka, > > > > I would also prefer to remove this condition as it's not making sense to > > keep order in created status when all the order items get canceled. > > > > Thanks!! > > > > Regards, > > Ratnesh Upadhyay > > HotWax Systems | www.hotwaxsystems.com > > > > On Fri, Oct 6, 2017 at 12:52 PM, Deepak Dixit < > > deepak.di...@hotwaxsystems.com> wrote: > > > > > Hi Renuka, > > > > > > I see no reason to keen this condition. we can remove this. > > > As I know there is no business scenario around this. If all order item > > > cancelled than order should be cancel either its Sales or Purchase > order. > > > > > > > > > > > > Thanks & Regards > > > -- > > > Deepak Dixit > > > www.hotwaxsystems.com > > > www.hotwax.co > > > > > > On Thu, Oct 5, 2017 at 7:17 PM, Renuka Srishti < > > renuka.srisht...@gmail.com > > > > > > > wrote: > > > > > > > Hello Devs, > > > > > > > > This is reference to > > > > http://markmail.org/message/wggbvdumrndpkxrj > > > > > > > > In "applications/order/src/org/ofbiz/order/order/ > OrderServices.java", > > > > under > > > > "checkItemStatus" service, there is one check for all item cancelled: > > > > > > > > if (!"PURCHASE_ORDER".equals(orderTypeId)) { > > > > newStatus = "ORDER_CANCELLED"; > > > > } > > > > > > > > I was trying to find out the reason behind it but did not get any. > > > > > > > > When we cancel all the items of an order, Order should cancel at the > > > header > > > > level too. Then why we have added this special check for purchase > > order? > > > > > > > > Becuase of this check, after canceling of all the items, purchase > order > > > do > > > > not get canceled at the header level. Currently, we need to cancel > the > > > > purchase order manually. > > > > > > > > > > > > Thanks > > > > Renuka Srishti > > > > > > > > > >