Merge branch 'master' of git://github.com/ckieffer/gallery3

5 files changed, 102 insertions, 31 deletions

diff --git a/modules/info/helpers/info_block.php b/modules/info/helpers/info_block.php
index 3cbbb12b..9596bc41 100644
--- a/modules/info/helpers/info_block.php
+++ b/modules/info/helpers/info_block.php
@@ -31,6 +31,47 @@ class info_block_Core {
         $block->css_id = "g-metadata";
         $block->title = $theme->item()->is_album() ? t("Album info") : t("Photo info");
         $block->content = new View("info_block.html");
+        if ($theme->item->title && module::get_var("info", "show_title")) {
+          $info["title"] = array(
+            "label" => t("Title:"),
+            "value" => html::purify($theme->item->title)
+          );
+        }
+        if ($theme->item->description && module::get_var("info", "show_description")) {
+          $info["description"] = array(
+            "label" => t("Description:"),
+            "value" => nl2br(html::purify($theme->item->description))
+          );
+        }
+        if (!$theme->item->is_album() && module::get_var("info", "show_name")) {
+          $info["file_name"] = array(
+            "label" => t("File name:"),
+            "value" => html::clean($theme->item->name)
+          );
+        }
+        if ($theme->item->captured && module::get_var("info", "show_captured")) {
+          $info["captured"] = array(
+            "label" => t("Captured:"),
+            "value" => gallery::date_time($theme->item->captured)
+          );
+        }
+        if ($theme->item->owner && module::get_var("info", "show_owner")) {
+          $display_name = $theme->item->owner->display_name();
+          if ($theme->item->owner->url) {
+            $info["owner"] = array(
+              "label" => t("Owner:"),
+              "value" => "<a href=\"{$theme->item->owner->url}\">" . html::clean($display_name) . "</a>"
+            );
+          } else {
+            $info["owner"] = array(
+              "label" => t("Owner:"),
+              "value" => html::clean($display_name)
+            );
+          }
+        }
+        $block->content->metadata = $info;
+
+        module::event("info_block_get_metadata", $block, $theme->item);
       }
       break;
     }
diff --git a/modules/info/helpers/info_installer.php b/modules/info/helpers/info_installer.php
new file mode 100644
index 00000000..51962173
--- /dev/null
+++ b/modules/info/helpers/info_installer.php
@@ -0,0 +1,41 @@
+<?php defined("SYSPATH") or die("No direct script access.");
+/**
+ * Gallery - a web based photo album viewer and editor
+ * Copyright (C) 2000-2010 Bharat Mediratta
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or (at
+ * your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street - Fifth Floor, Boston, MA  02110-1301, USA.
+ */
+class info_installer {
+
+  static function install() {
+    module::set_var("info", "show_title", 1);
+    module::set_var("info", "show_description", 1);
+    module::set_var("info", "show_owner", 1);
+    module::set_var("info", "show_name", 1);
+    module::set_var("info", "show_captured", 1);
+    module::set_version("info", 2);
+  }
+
+  static function upgrade($version) {
+    if ($version == 1) {
+      module::set_var("info", "show_title", 1);
+      module::set_var("info", "show_description", 1);
+      module::set_var("info", "show_owner", 1);
+      module::set_var("info", "show_name", 1);
+      module::set_var("info", "show_captured", 1);
+      module::set_version("info", $version = 2);
+    }
+  }
+}
diff --git a/modules/info/module.info b/modules/info/module.info
index e352213c..5f84cbb9 100644
--- a/modules/info/module.info
+++ b/modules/info/module.info
@@ -1,3 +1,3 @@
 name = "Info"
 description = "Display extra information about photos and albums"
-version = 1
+version = 2
diff --git a/modules/info/views/info_block.html.php b/modules/info/views/info_block.html.php
index ebe9bd28..b296fa1d 100644
--- a/modules/info/views/info_block.html.php
+++ b/modules/info/views/info_block.html.php
@@ -1,35 +1,8 @@
 <?php defined("SYSPATH") or die("No direct script access.") ?>
 <ul class="g-metadata">
+  <? foreach($metadata as $info): ?>
   <li>
-    <strong class="caption"><?= t("Title:") ?></strong>
-    <?= html::purify($item->title) ?>
+    <strong class="caption"><?= $info["label"] ?></strong> <?= $info["value"] ?>
   </li>
-  <? if ($item->description): ?>
-  <li>
-    <strong class="caption"><?= t("Description:") ?></strong>
-     <?= nl2br(html::purify($item->description)) ?>
-  </li>
-  <? endif ?>
-  <? if (!$item->is_album()): ?>
-  <li>
-    <strong class="caption"><?= t("File name:") ?></strong>
-    <?= html::clean($item->name) ?>
-  </li>
-  <? endif ?>
-  <? if ($item->captured): ?>
-  <li>
-    <strong class="caption"><?= t("Captured:") ?></strong>
-    <?= gallery::date_time($item->captured)?>
-  </li>
-  <? endif ?>
-  <? if ($item->owner): ?>
-  <li>
-    <strong class="caption"><?= t("Owner:") ?></strong>
-    <? if ($item->owner->url): ?>
-    <a href="<?= $item->owner->url ?>"><?= html::clean($item->owner->display_name()) ?></a>
-    <? else: ?>
-    <?= html::clean($item->owner->display_name()) ?>
-    <? endif ?>
-  </li>
-  <? endif ?>
+  <?  endforeach; ?>
 </ul>
diff --git a/modules/tag/helpers/tag_event.php b/modules/tag/helpers/tag_event.php
index 829089c4..6b8de723 100644
--- a/modules/tag/helpers/tag_event.php
+++ b/modules/tag/helpers/tag_event.php
@@ -147,4 +147,20 @@ class tag_event_Core {
       }
     }
   }
+
+  static function info_block_get_metadata($block, $item) {
+    $tags = array();
+    foreach (tag::item_tags($item) as $tag) {
+      $tags[] = "<a href=\"" . url::site("tag/{$tag->name}") . "\">{$tag->name}</a>";
+    }
+    if ($tags) {
+      $info = $block->content->metadata;
+      $info["tags"] = array(
+         "label" => t("Tags:"),
+         "value" => implode(", ", $tags)
+      );
+      $block->content->metadata = $info;
+    }
+  }
+  
 }