https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32748

--- Comment #60 from Thibaud Guillot (thibaud_g) <[email protected]> 
---
(In reply to Jonathan Druart from comment #56)
> (In reply to Thibaud Guillot (thibaud_g) from comment #19)
> > Hello everyone,
> > 
> > I followed your advices and removed the display of the branchcode where the
> > values come from (and therefore the GROUP_CONCAT). I hope it can be tested
> > by as many people as possible :-)
> > 
> > Jonathan : What do you mean by adding the branchcode into the cache key? The
> > branchcode from which the VAs originate is not accessible until the cached
> > string is defined. Why should I get random results? Thanks for the
> > clarification
> 
> When I wrote that no-limit was part of the cache key
> (https://bugs.koha-community.org/bugzilla3/attachment.
> cgi?id=151989&action=edit)
> In the last version of the patch it's no longer there. Is there a reason for
> that?
> 
> Also there are modification to 2 different sub/methods but no tests.

Hi Jonathan,

You were right, and my earlier reply misunderstood the question — sorry for the
confusion. I fixed this and add some tests

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list -- [email protected]
To unsubscribe send an email to [email protected]
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to