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

[Prasad]

Alan,

We feel short of validation so had to push back on integration.

To give it right importance, we planned to focus our effort in our next
release.
We appreciate if such patch can be with reference to 6.4.0.

Not a problem if it can't be - a pointer on our trac.vtiger.com is
sufficient to reach.

Regards,
Prasad

--
FB <http://www.facebook.com/vtiger> I Twit <http://twitter.com/vtigercrm> I
LIn <https://www.linkedin.com/company/1270573?trk=tyah> I Blog
<https://blogs.vtiger.com> I Website <https://www.vtiger.com/>

On Fri, Oct 16, 2015 at 6:27 PM, Alan Lord <alanslists at gmail.com> wrote:

> 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
>>
>>
>
> _______________________________________________
> http://www.vtiger.com/
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.vtigercrm.com/pipermail/vtigercrm-developers/attachments/20151016/6bd6834e/attachment.html>