Re: svn commit: r502824 -/ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml

2007-02-07 Thread Jacopo Cappellato
Jacques, that's a great comment! You've made me smile, thanks. Jacopo Jacques Le Roux wrote: Just one word : Please "-*DON ' T*_ use absolute pathes in your diff/patches but relative to root" Jacques

Re: svn commit: r502824 -/ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml

2007-02-07 Thread Jacques Le Roux
Just one word : Please "-*DON ' T*_ use absolute pathes in your diff/patches but relative to root" Jacques - Original Message - From: "Andrew Sykes" <[EMAIL PROTECTED]> To: Sent: Thursday, February 08, 2007 12:17 AM Subject: Re: svn commit: r502824 -/o

Re: svn commit: r502824 - /ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml

2007-02-07 Thread Andrew Sykes
Please please guys, just drop this, I'm sure none of us who are having to delete all this nonsense really care. - Andrew On Thu, 2007-02-08 at 06:56 +0800, Jonathon -- Improov wrote: > Tim, Jacopo, > > Well, since you put this in the open, I guess I might as well re-examine my > previous pos

Re: svn commit: r502824 - /ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml

2007-02-07 Thread Jonathon -- Improov
Tim, Jacopo, Well, since you put this in the open, I guess I might as well re-examine my previous posts. In Jacopo's case, it all did start out with me appreciating his help. In one of my first posts, I made the mistake of recalling that it was Jacopo who corrected me on the "please use absolu

Re: svn commit: r502824 - /ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml

2007-02-07 Thread Tim Ruppert
Jonathon as an innocent bystander, all I can say is that with every word you type you are rubbing people the wrong way. Please just take a moment to read thru your posts before hitting send - and think about the fact that there are tons of people here from different backgrounds who are pus

Re: svn commit: r502824 - /ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml

2007-02-07 Thread Jonathon -- Improov
Jacopo, No, I'm not trolling. Are you? If you meant the last concise post to you, it's because I know you're really really busy. Just a sincere effort to cut down on "extras" in my writing to you. If you meant my comment on David's and Si Chen's discussion, like I said, you better re-read for

Re: svn commit: r502824 - /ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml

2007-02-07 Thread Jacopo Cappellato
Jonathon, are you trolling? Jacopo Jonathon -- Improov wrote: Er, Jacopo. Maybe I moved your cheese some time, I don't know. If so, sorry! Since you like conciseness, please re-read this thread between David and Si Chen. As for incorrect information, re-read my previous posts. Jonathon

Re: svn commit: r502824 - /ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml

2007-02-06 Thread Jonathon -- Improov
Er, Jacopo. Maybe I moved your cheese some time, I don't know. If so, sorry! Since you like conciseness, please re-read this thread between David and Si Chen. As for incorrect information, re-read my previous posts. Jonathon Jacopo Cappellato wrote: Jonathon -- Improov wrote: Si Chen, I wa

Re: svn commit: r502824 - /ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml

2007-02-06 Thread Jacopo Cappellato
Jonathon -- Improov wrote: Si Chen, I wasn't "saying" anything. I think that this is the problem with many of your (and with those of some new guys that recently are around in these lists) posts, Jonathon: you say nothing with too many words, and this is confusing, especially for new users

Re: svn commit: r502824 - /ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml

2007-02-06 Thread Jonathon -- Improov
Si Chen, > You're welcome to have and express whatever opinion you wish, but what > you are saying is not true. I wasn't "saying" anything. I was observing the fact that it seemed there was gonna be a change to something core. I've not known "+/-" to be a standard SQL notation, and quickly assu

Re: svn commit: r502824 - /ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml

2007-02-06 Thread Si Chen
Jonathon -- Improov wrote: David, Si Chen, Just to let you know my impression as an observer of this thread. I was shocked to read "the +/- notation does not work well with PostgreSQL". Was bracing myself for a tidal wave of a change in the OFBiz framework. Jonathon David E. Jones wrote:

Re: svn commit: r502824 - /ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml

2007-02-05 Thread Jonathon -- Improov
David, Si Chen, Just to let you know my impression as an observer of this thread. I was shocked to read "the +/- notation does not work well with PostgreSQL". Was bracing myself for a tidal wave of a change in the OFBiz framework. Jonathon David E. Jones wrote: On Feb 5, 2007, at 10:16 AM,

Re: svn commit: r502824 - /ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml

2007-02-05 Thread Si Chen
David E. Jones wrote: On Feb 5, 2007, at 10:16 AM, Si Chen wrote: David E. Jones wrote: Hold on a minute there did you actually test and find this to be a problem? The +/- notation is an entity engine ONLY thing and should never make it to the database. This patch should be reverted

Re: svn commit: r502824 - /ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml

2007-02-05 Thread David E. Jones
On Feb 5, 2007, at 10:16 AM, Si Chen wrote: David E. Jones wrote: Hold on a minute there did you actually test and find this to be a problem? The +/- notation is an entity engine ONLY thing and should never make it to the database. This patch should be reverted and if +/- are making

Re: svn commit: r502824 - /ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml

2007-02-05 Thread Si Chen
David E. Jones wrote: Hold on a minute there did you actually test and find this to be a problem? The +/- notation is an entity engine ONLY thing and should never make it to the database. This patch should be reverted and if +/- are making it to the database instead of being replaced wi

Re: svn commit: r502824 - /ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml

2007-02-03 Thread David E. Jones
Why change it then? Why put the effort into it, and then require corresponding review effort? Why comment "the +/- notation does not work well with PostgreSQL"? Either way I don't care a lot, as long as it's clear what is being fixed and what is just being changed because "I like my way be

Re: svn commit: r502824 - /ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml

2007-02-03 Thread Leon Torres
Hi David, I was involved in fixing this issue. There was a genuine bug where the inventory reservation with FIFO method was ordered the wrong way, causing it to behave like LIFO. This commit fixes the issue by reversing the order for the FIFO case so it works correctly. So unless you want

Re: svn commit: r502824 - /ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml

2007-02-02 Thread David E. Jones
Hold on a minute there did you actually test and find this to be a problem? The +/- notation is an entity engine ONLY thing and should never make it to the database. This patch should be reverted and if +/- are making it to the database instead of being replaced with an ASC/DESC by th