[Koha-devel] TT loop count VS size

[dcook at prosentient.com.au]

I was going to say… I’d never even heard of the “count” call in TT. Looking at http://template-toolkit.org/docs/manual/VMethods.html, it’s not a method for a data structure. Looking at http://template-toolkit.org/docs/manual/Directives.html#section_FOREACH, it’s only a method of the “loop” variable. 

 

grep -R "loop\.count" *   | wc -l

86

 

What’s the actual problem being observed?

 

David Cook

Software Engineer

Prosentient Systems

Suite 7.03

6a Glen St

Milsons Point NSW 2061

Australia

 

Office: 02 9212 0899

Online: 02 8005 0595

 

From: Koha-devel <koha-devel-bounces at lists.koha-community.org> On Behalf Of Jonathan Druart
Sent: Friday, 7 May 2021 5:58 PM
To: Fridolin SOMERS <fridolin.somers at biblibre.com>
Cc: koha-devel <koha-devel at lists.koha-community.org>
Subject: Re: [Koha-devel] TT loop count VS size

 

You should grep "loop.count", not ".count"

Most of the count you have with your regex is coming from koha_objects->count

 

Le ven. 7 mai 2021 à 09:17, Fridolin SOMERS <fridolin.somers at biblibre.com <mailto:fridolin.somers at biblibre.com> > a écrit :

Hi community,

I've been on a mindblowing search on a template error :D

I was thinking that "loop.count" was wrong for testing if loop is not 
empty, the method is "loop.size".

See doc 
http://template-toolkit.org/docs/manual/Directives.html#section_FOREACH

Looks like we use both actually :
  > git grep 'IF.*\.count ' origin/master | wc -l
602
  > git grep 'IF.*\.size ' origin/master | wc -l
1636

So using count seems to work because it starts with 1.
But in my opinion it is a wrong call.

We can see for example :
  <a href="#images">Images ([% localimages.count || 0 | html %])</a>
This is wrong no ?

Should be create a coding guideline ?

-- 
Fridolin SOMERS <fridolin.somers at biblibre.com <mailto:fridolin.somers at biblibre.com> >
Software and system maintainer 🦄
BibLibre, France
_______________________________________________
Koha-devel mailing list
Koha-devel at lists.koha-community.org <mailto:Koha-devel at lists.koha-community.org> 
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel
website : https://www.koha-community.org/
git : https://git.koha-community.org/
bugs : https://bugs.koha-community.org/

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.koha-community.org/pipermail/koha-devel/attachments/20210510/23a000da/attachment.htm>