[Vtigercrm-developers] Reg : PATCH for PearDatabase

[Minnie A]

Dear Enrico,

Thanks for your quick reply. I'll test your patches and update you.

Thanks & Regards,
Minnie.



On 10/26/07, Enrico Weigelt <weigelt at metux.de> wrote:
>
> * Minnie A <minnie at vtiger.com> wrote:
>
> Hi,
> (CC'in to vtiger-dev)
>
>
> > In these two patches, same method is defined with different argument
> list.
>
> Ups ! Seems I've mixed up some things.
> Best would be merging both together: make the second parameter optional
> and use it as index in the result record if defined, otherwise take
> the first column.
>
> > 3. remove noise in invoice module(02_nekrad-remove-noise.diff) : while
> > creating a invoice if the same product is selected for multiple time, we
> > give the confirmation message for user saying that "Same product
> selected
> > for multiple time , do you wan to continue". It is expected behaviour
> only.
> > so we didn't integrate it with vtiger CRM.
>
> You're assuming one product only appears once (with some factor).
> Actually that's an bad assumpting. Think about services with
> several tasks: we've got an working hour as product and several
> items whith amounts of hours spent for certain tasks.
>
> In such situations this warning is extremly annoying.
>
>
> cu
> --
> ----------------------------------------------------------------------
> Enrico Weigelt, metux IT service -- http://www.metux.de/
>
> cellphone: +49 174 7066481   email: info at metux.de   skype: nekrad666
> ----------------------------------------------------------------------
> Embedded-Linux / Portierung / Opensource-QM / Verteilte Systeme
> ----------------------------------------------------------------------
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.vtigercrm.com/pipermail/vtigercrm-developers/attachments/20071026/d7186e18/attachment-0003.html