Jacopo, what shall i say? Having two additional fields in an already unique key, sorry i do not see the benefit, also because it is not following the general pattern used in invoice, quote and order etc. where the there is a header and items.
You seem to have strong feelings keeping this, so be my guest. It makes the application unnecessary complicated. Usable? After adding a review one still has to go back to he find option, find the review again and then finally items can add be added. Regards, Hans On Sat, 2011-01-15 at 15:17 +0100, Jacopo Cappellato wrote: > On Jan 15, 2011, at 6:45 AM, Jacopo Cappellato wrote: > > > 1) we revert the commit (or part of it) in which you altered the pk of the > > PerfReviewItem entity; if possible and easy to do, we can fix the > > screens/services that were not working before your commit > > Hans, thanks for reverting that code; in rev. 1059340 I have committed some > fixes to at least make the screens usable, please have a look at them when > you have a chance. > > Kind regards, > > Jacopo > -- Ofbiz on twitter: http://twitter.com/apache_ofbiz Myself on twitter: http://twitter.com/hansbak Antwebsystems.com: Quality services for competitive rates.
