[Vtigercrm-developers] Bug in Current/Previous Week options in Reports?

Alan Lord alanslists at gmail.com
Fri Oct 16 12:57:32 GMT 2015


In 6.4.0 you seems to have addressed the specific issue I reported here: 
http://trac.vtiger.com/cgi-bin/trac.cgi/ticket/8547

Yet you appear to have completely ignored the suggested patch (below) 
from Błażej which could have removed (as a guess) a couple of thousand 
lines of duplicated code? Why?


Al

On 13/08/15 17:18, Alan Lord (News) wrote:
> On 13/08/15 16:49, Błażej Pabiszczak wrote:
>> Hi
>>
>> check this:
>> https://github.com/YetiForceCompany/YetiForceCRM/commit/eb8827ef15e9860a3d5b5ddb74fdad4db7d15822
>>
>
>
> That's a mother of a patch but looks good! Thanks:
>
>
> "384 additions and 1,239 deletions."
>
> vtiger devs, please take a look. This is major bit of code cleaning :-)
>
> Al
>




More information about the vtigercrm-developers mailing list