On Sun, 17 Apr 2016 23:16:27 +0200
Holger Wansing wrote:

tested on a virtualbox vm on Windows10 64bit;
copyright view stats checked using "Edge" web browser:

wrong translations: original is missing;
no translators info: original is there but no translators' ones
broken: entire pdf is broken or out of space; original is not completed
OK: both of original and translators' exists; translators' info may be wrong

base, Makefile patch applied (same results):
  wrong translations:cs,he,zh_CN
  no translators 
info:ar,da,el,es,eu,gl,hi,hu,it,ja,lt,ml,ms,pl,pt_BR,sk,sv,tr,zh_TW
  broken:bg,ca,de,fi,fr,nb,nl,pt,ro,ru,uk,vi
  OK: [none]

fo.xsl patch applied, fo.xsl and Makefile patches applied (same results):
  wrong translations:cs,he,zh_CN
  no translators info:ar,da,el,es,hu,ml,
  broken:bg,ca,de,fi,fr,nb,nl,pt,ro,ru,uk,vi
  OK:eu,gl,hi,it,ja,lt,ms,pl,pt_BR,sk,sv,tr,zh_TW

so ..., at least on my env,
the Makefile patch didn't make significant improvement from this POV

about the patch:
fo.xsl is used *after* preproc.xsl,
 so here no need to exclude the dummy 3rd item in en,
 just need to wrap each of them in a block to separate

-- 
victory
no need to CC me :-)
--
Index: fo.xsl
===================================================================
--- fo.xsl      (R2)
+++ fo.xsl      (working copy)
@@ -79,14 +79,13 @@
       <fo:block text-align="justify">
        <xsl:apply-templates select="/article/articleinfo/legalnotice"
                             mode="titlepage.mode"/>
-       <xsl:apply-templates select="/article/articleinfo/copyright[1]"
-                            mode="titlepage.mode"/>
-       <xsl:if test="/article/articleinfo/copyright[2]">
-         <fo:block>
-           <xsl:apply-templates select="/article/articleinfo/copyright[2]"
-                                mode="titlepage.mode"/>
-         </fo:block>
-       </xsl:if>
+       <xsl:for-each select="/article/articleinfo/copyright"> 
+         <xsl:if test="/article/articleinfo/copyright[position()]">
+           <fo:block>
+               <xsl:apply-templates select="." mode="titlepage.mode"/>
+           </fo:block>
+         </xsl:if>
+       </xsl:for-each> 
       </fo:block>
     </fo:block>
   </xsl:template>

Reply via email to