From ded7232146600db5b3efaff5b17c6af46fa2b87b Mon Sep 17 00:00:00 2001
From: Jan Dittberner <jan@dittberner.info>
Date: Sat, 11 Jul 2009 15:13:40 +0000
Subject: [PATCH] move more theme related functionality into the theme class
 (addresses #46)

---
 includes/theme.class.php            | 22 +++++++++++++++++++++-
 index.php                           | 29 ++++++-----------------------
 themes/default_horizontal/theme.php | 16 ++++++++++++++++
 3 files changed, 43 insertions(+), 24 deletions(-)

diff --git a/includes/theme.class.php b/includes/theme.class.php
index b8ccdfe..67b2be7 100644
--- a/includes/theme.class.php
+++ b/includes/theme.class.php
@@ -28,6 +28,8 @@
  * <http://www.gnu.org/licenses/>.
  */
 
+require('includes/template.class.php');
+
 /**
  * Theme class.
  */
@@ -52,6 +54,9 @@ class Theme {
    */
   var $thumbsize;
 
+  var $name;
+  var $template;
+
   /**
    * Constructor for themes. Expects a theme name and initializes the
    * internal state of the instance from the 'theme.ini' file in the
@@ -61,10 +66,25 @@ class Theme {
   function __construct($name) {
     $themeconfig = parse_ini_file(
       realpath(implode(DIRECTORY_SEPARATOR,
-                       array('themes', $name, 'theme.ini'))));
+        array('themes', $name, 'theme.ini'))));
+    $this->name = $name;
     $this->themetype = $themeconfig['themetype'];
     $this->previewsize = intval($themeconfig['previewsize']);
     $this->thumbsize = intval($themeconfig['thumbsize']);
   }
+
+  function getTemplate() {
+    if (!$this->template) {
+      $this->template = new Template();
+      $this->template->assign('themepath',
+        implode(DIRECTORY_SEPARATOR, array('themes', $this->name)));
+    }
+    return $this->template;
+  }
+
+  function display() {
+    $this->template->display(implode(DIRECTORY_SEPARATOR,
+      array('themes', $this->name, 'theme.php')));
+  }
 }
 ?>
\ No newline at end of file
diff --git a/index.php b/index.php
index 4fe31eb..138d682 100644
--- a/index.php
+++ b/index.php
@@ -47,35 +47,19 @@ $gallery   = getCurrentGallery();
  */
 $thumbinfo = getThumbNailInfo($gallery);
 
-require('includes/template.class.php');
-
-$template = new Template();
-
 $scripts = array('js/jquery.js',
                  'js/jquery.colorBlend.js',
                  'js/jquery.lightbox.js',
                  'scripts/ourhandlers.js');
 $styles  = array('css/jquery.lightbox.css');
 
-$inlinestyles = sprintf("\n#scrollable { width:%dpx; }\n", $thumbinfo[0]);
+$template = $theme->getTemplate();
 
-$inlinestyles .=  <<<EOD
-a {
-	font-size: 9px;
+if ($theme->themetype == 'horizontal') {
+  $inlinestyles = sprintf("\n#scrollable { width:%dpx; }\n", $thumbinfo[0]);
+} else {
+  $inlinestyles = sprintf("\n#scrollable { height:%dpx; }\n", $thumbinfo[0]);
 }
-a:link {
-	text-decoration: none;
-}
-a:visited {
-	text-decoration: none;
-}
-a:hover {
-	text-decoration: none;
-}
-a:active {
-	text-decoration: none;
-}
-EOD;
 
 $template->assign('scripts', $scripts);
 $template->assign('styles', $styles);
@@ -86,9 +70,8 @@ $template->assign('thumbnails', getAllThumbnails($thumbinfo));
 $template->assign('firstpreview', getFirstPreview($thumbinfo));
 $template->assign('firstdescription', getFirstDescription($thumbinfo));
 $template->assign('lang', 'de');
-$template->assign('themepath', 'themes/default_horizontal');
 $template->assign('gallery', $gallery);
 
-$template->display('themes/default_horizontal/theme.php');
+$theme->display();
 
 ?>
\ No newline at end of file
diff --git a/themes/default_horizontal/theme.php b/themes/default_horizontal/theme.php
index 322d155..e1f5763 100644
--- a/themes/default_horizontal/theme.php
+++ b/themes/default_horizontal/theme.php
@@ -6,6 +6,22 @@
 <title><?php echo $content['title']; ?></title>
 <style type="text/css">
 <?php print($content['inlinestyle']); ?>
+
+a {
+	font-size: 9px;
+}
+a:link {
+	text-decoration: none;
+}
+a:visited {
+	text-decoration: none;
+}
+a:hover {
+	text-decoration: none;
+}
+a:active {
+	text-decoration: none;
+}
 </style>
 <?php foreach ($content['styles'] as $style) { ?>
   <link rel="stylesheet" type="text/css" href="<?php echo $style; ?>" />