Dear Enrico,<br><br>Thanks for your quick reply. I&#39;ll test your patches and update you.<br><br>Thanks &amp; Regards,<br>Minnie.<br><br><br><br><div><span class="gmail_quote">On 10/26/07, <b class="gmail_sendername">Enrico Weigelt
</b> &lt;<a href="mailto:weigelt@metux.de">weigelt@metux.de</a>&gt; wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">* Minnie A &lt;
<a href="mailto:minnie@vtiger.com">minnie@vtiger.com</a>&gt; wrote:<br><br>Hi,<br>(CC&#39;in to vtiger-dev)<br><br><br>&gt; In these two patches, same method is defined with different argument list.<br><br>Ups ! Seems I&#39;ve mixed up some things.
<br>Best would be merging both together: make the second parameter optional<br>and use it as index in the result record if defined, otherwise take<br>the first column.<br><br>&gt; 3. remove noise in invoice module(02_nekrad-
remove-noise.diff) : while<br>&gt; creating a invoice if the same product is selected for multiple time, we<br>&gt; give the confirmation message for user saying that &quot;Same product selected<br>&gt; for multiple time , do you wan to continue&quot;. It is expected behaviour only.
<br>&gt; so we didn&#39;t integrate it with vtiger CRM.<br><br>You&#39;re assuming one product only appears once (with some factor).<br>Actually that&#39;s an bad assumpting. Think about services with<br>several tasks: we&#39;ve got an working hour as product and several
<br>items whith amounts of hours spent for certain tasks.<br><br>In such situations this warning is extremly annoying.<br><br><br>cu<br>--<br>----------------------------------------------------------------------<br> Enrico Weigelt, metux IT service -- 
<a href="http://www.metux.de/">http://www.metux.de/</a><br><br> cellphone: +49 174 7066481&nbsp;&nbsp; email: <a href="mailto:info@metux.de">info@metux.de</a>&nbsp;&nbsp; skype: nekrad666<br>----------------------------------------------------------------------
<br> Embedded-Linux / Portierung / Opensource-QM / Verteilte Systeme<br>----------------------------------------------------------------------<br></blockquote></div><br>