Re: Merge PR 621
Hi!
> Yes, that's probably what it was supposed to do, but as you can see
> from bug 66834 that's clearly not the case. I had earlier approached
> the problem purely from SPL standpoint (see linked PR) but Etienne
> said that it would be a better idea to tackle this issue further up
> the chain so to speak :)
I think we need to first find out why that code did not work, instead of
ripping out the code that already was supposed to do exactly what this
bug is saying and replacing it with other code. At least we need to know
the reason why that did not work. So please do not merge this patch
until we know what's going on there.
--
Stanislav Malyshev, Software Architect
SugarCRM: http://www.sugarcrm.com/
(408)454-6900 ext. 227
Thread (23 messages)