On Donnerstag 25 Dezember 2008, Duncan Webb wrote: > The number of chapters should be len(chapters) > [...] > - <if expression="chapters"> > - <text font="info value" expression="chapters"/> > + <if expression="len(chapters)"> > + <text font="info value"
<nitpick>Maybe the if-expression should have remained "chapters".
Rationale: chapters without len is more safe, since it works not only for
empty lists but e.g. also for None.</nitpick>
Anyhow, I assume you tested this and it works with the current code.
Just my 2 cents.
--
Ciao, / / .o.
/--/ ..o
/ / ANS ooo
signature.asc
Description: This is a digitally signed message part.
------------------------------------------------------------------------------
_______________________________________________ Freevo-devel mailing list Freevo-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/freevo-devel
