Project

General

Profile

Actions
Copy link

Bug #1355

closed

xxv 1.7.0 keine Menüdarstellung links unter Programm

Added by faup over 11 years ago. Updated almost 11 years ago.

Status:
Closed
Priority:
Normal
Assignee:
faup
Target version:
xxv-1.7.0
Start date:
05/02/2013
Due date:
% Done:

100%

Estimated time:

Description

Bedingungen :
xxv 1.7.0 git vom 2.5.2013
mysql 5.5.31-0ubuntu0.12.04.1 (Ubuntu)

Variablen programgroup für WebFXTreeItem werden nicht richtig aufgelöst.

Lösung : help.tmpl korrigiert :

--- ./pure_xxv/xxv/skins/html/help.tmpl    2013-05-02 14:09:23.291694474 +0200
+++ ./xxv/xxv/skins/html/help.tmpl    2013-05-02 14:26:18.032726310 +0200
@@ -50,7 +50,7 @@
 <?% IF allow('schema') %?>
     var program = new WebFXTreeItem("<?% gettext("Program guide") %?>","?cmd=schema",tree);
     <?% IF allow('topten') %?>program.add(new WebFXTreeItem("<?% gettext("Top ten entry") %?>",    "?cmd=t10"));<?% END %?>
-    <?% FOREACH ch = param.CHANNELS %?>
+    <?% FOREACH ch IN param.CHANNELS %?>
         <?% IF ch.2 != lastgroup;lastgroup = ch.2 %?>
         <?% CNT = CNT + 1 %?>
     var programgroup<?% CNT %?> = new WebFXTreeItem("<?% escape(chop(lastgroup,10)) %?>",  "?cmd=program&amp;data=<?% ch.1 %?>",program);

Files

xxv.1.7.0._help.tmpl_.precise_speedup_2013-05-02-alexrichter.patch (719 Bytes) xxv.1.7.0._help.tmpl_.precise_speedup_2013-05-02-alexrichter.patch faup, 05/02/2013 02:40 PM
Actions
Copy link
 #1

Updated by anbr over 11 years ago

  • Status changed from New to Assigned
  • Assignee changed from faup to anbr
  • Target version set to xxv-1.7.0

Hmm,

dokumentiert sind beide Methoden, und bei mir funktionieren beide Methoden

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

Wenn wir konsequent wären, dürfe nur eine Schreibweise in den Templatedateien verwendet werden.

Actions
Copy link
 #2

Updated by anbr over 11 years ago

So ich teste gerade die globale Änderung aller Skin-Template Dateien mit dem folgende Änderung Stream-Editor-Befehl

cd skins
find -type f -name '*.tmpl' -exec sed -i.tmpl~ -e 's/FOREACH \([a-z]*\) =/FOREACH \1 IN/g' {} \;
Actions
Copy link
 #3

Updated by anbr over 11 years ago

  • Status changed from Assigned to Feedback
  • Assignee changed from anbr to faup
  • % Done changed from 100 to 90

Bitte mal den aktuellen Stand des Git (fd3f221e) testen.

Actions
Copy link
 #4

Updated by anbr almost 11 years ago

  • Status changed from Feedback to Closed
  • % Done changed from 90 to 100
Actions
Copy link

Also available in: Atom PDF