From ea36e81b91c8689ae44d413eca4697d1993ac30b Mon Sep 17 00:00:00 2001
From: Andreas Unterkircher <unki@netshadow.at>
Date: Sat, 23 May 2009 09:45:29 +0200
Subject: avoid notice messages about undefined variables, resolves #179

Signed-off-by: Andreas Unterkircher <unki@netshadow.at>
---
 themes/default/templates/calendar.tpl     |  2 +-
 themes/default/templates/header.tpl       |  4 ++--
 themes/default/templates/photo_index.tpl  | 32 +++++++++++++++----------------
 themes/default/templates/search.tpl       | 12 ++++++------
 themes/default/templates/single_photo.tpl | 32 ++++++++++++++++++-------------
 themes/default/templates/tags.tpl         |  2 +-
 6 files changed, 45 insertions(+), 39 deletions(-)

(limited to 'themes')

diff --git a/themes/default/templates/calendar.tpl b/themes/default/templates/calendar.tpl
index 1f52732..8c1d27e 100644
--- a/themes/default/templates/calendar.tpl
+++ b/themes/default/templates/calendar.tpl
@@ -18,7 +18,7 @@
  <!-- matrix -->
  {section name="row" loop=$rows step=1}
   {section name="col" loop=8 step=1}
-   {if $matrix[row][col] }
+   {if isset($matrix[row][col]) }
     {$matrix[row][col]}
    {/if}
   {/section}
diff --git a/themes/default/templates/header.tpl b/themes/default/templates/header.tpl
index c1d7a0c..d592ebf 100644
--- a/themes/default/templates/header.tpl
+++ b/themes/default/templates/header.tpl
@@ -8,13 +8,13 @@
   <script type="text/javascript" src="{$web_path}/phpfspot.js"></script>
   <link rel="shortcut icon" href="{$web_path}/resources/gpl_16.png" type="image/png" />
   <link rel="icon" href="{$web_path}/resources/gpl_16.png" type="image/png" />
-  { if $use_lightbox }
+  { if isset($use_lightbox) }
   <script type="text/javascript" src="{$web_path}/lightbox2/js/prototype.js"></script>
   <script type="text/javascript" src="{$web_path}/lightbox2/js/scriptaculous.js?load=effects,builder"></script>
   <script type="text/javascript" src="{$web_path}/lightbox2/js/lightbox.js"></script>
   <link rel="stylesheet" href="{$web_path}/lightbox2/css/lightbox.css" type="text/css" media="screen" />
   { /if }
-  { if $use_autocomplete }
+  { if isset($use_autocomplete) }
   <script type="text/javascript" src="{$web_path}/autosuggest/js/bsn.AutoSuggest_2.1.3_comp.js"></script>
   <link rel="stylesheet" href="{$web_path}/autosuggest/css/autosuggest_inquisitor.css" type="text/css" media="screen" charset="utf-8" />
   { /if }
diff --git a/themes/default/templates/photo_index.tpl b/themes/default/templates/photo_index.tpl
index fa63b31..ddc3229 100644
--- a/themes/default/templates/photo_index.tpl
+++ b/themes/default/templates/photo_index.tpl
@@ -5,13 +5,13 @@
 <div class="header">
  <div class="header_title">
   <b>Photo Index</b>
-  {if $searchfor }
+  {if isset($searchfor) }
    {if $count == 1}
     {$count} image is the result for your search about "{$searchfor}".
    {else}
     {$count} images are the result for your search about "{$searchfor}".
    {/if}
-  {elseif $tag_result}
+  {elseif isset($tag_result)}
    {if $count == 1}
     {$count} image has been found for the selected tags.
    {else}
@@ -24,22 +24,22 @@
     {$count} images have been found.
    {/if}
   {/if}
-  {if $from_date && $to_date }
+  {if isset($from_date) && isset($to_date) }
    <br />
    Results are limited to a date within {$from_date} to {$to_date}.
   {/if}
  </div>
  <div class="header_menu">
-  {if $slideshow_link }
+  {if isset($slideshow_link) }
    <a href="{$slideshow_link}" title="Slideshow" target="_blank"><img src="{$web_path}/resources/slideshow.png" />&nbsp;Slideshow</a>
   {/if}
-  {if $extern_link }
+  {if isset($extern_link) }
    <a href="{$extern_link}" title="Use this link to return to the current view"><img src="{$web_path}/resources/link.png" />&nbsp;External Link</a>
   {/if} 
-  {if $export_link }
+  {if isset($export_link) }
    <a href="{$export_link}" target="_blank" title="this will open a new browser window where you can export this page in several formats"><img src="{$web_path}/resources/export.png" />&nbsp;Export</a>
   {/if}
-  {if $rss_link }
+  {if isset($rss_link) }
    <a href="{$rss_link}" target="_blank" title="RSS feed"><img src="{$web_path}/resources/rss.png" />&nbsp;RSS-Feed</a>
   {/if}
  </div>
@@ -48,7 +48,7 @@
 <!-- if result of a tag-search is displayed, show the selected tags
      with some small pictures in a bar. -->
 
-{ if $tag_result }
+{ if isset($tag_result) }
 <!-- seperator -->
  <div class="tagresult"></div>
  <!-- /seperator -->
@@ -66,10 +66,10 @@
 
  {section name="thumb" loop=$thumbs step=1}
 
-  {if $images[thumb] }
+  {if isset($images[thumb]) }
 
    <div class="thumb" onmouseover="setBackGrdColor(this, 'mouseover');" onmouseout="setBackGrdColor(this, 'mouseout');" style="width: { $thumb_container_width }px; height: { $thumb_container_height }px;">
-    { if $user_friendly_url }
+    { if isset($user_friendly_url) }
     <a href="{$web_path}/photoview/{$images[thumb]}" onclick="showPhoto({$images[thumb]}, 'scrollup'); return false;" id="thumblink{$images[thumb]}" name="image{$img_id[thumb]}" class="thumblink" onfocus="click(this);" title="{$img_title[thumb]}">
     { else }
     <a href="{$web_path}/index.php?mode=showp&amp;id={$images[thumb]}" onclick="showPhoto({$images[thumb]}, 'scrollup'); return false;" id="thumblink{$images[thumb]}" name="image{$img_id[thumb]}" class="thumblink" onfocus="click(this);" title="{$img_title[thumb]}">
@@ -79,11 +79,11 @@
     {$img_name[thumb]}
     </a>
     <!-- show lightbox eyes, if enabled -->
-    { if $use_lightbox }
+    { if isset($use_lightbox) }
     <a href="{$web_path}/phpfspot_img.php?idx={$images[thumb]}&amp;width={$preview_width}" alt="thumb_{$images[thumb]}" rel="lightbox[photoidx]" title="Preview of {$img_fullname[thumb]}"><img src="{$web_path}/resources/eyes.png" /></a>
     { /if }
     <!-- show F-Spot photo rating value, if available -->
-    { if $img_rating[thumb] }
+    { if isset($img_rating[thumb]) }
      <br />
      {section name="rating" loop=$img_rating[thumb] step=1}
       <img src="{$web_path}/resources/star.png" />
@@ -101,7 +101,7 @@
 
  <!-- the right nav arrow -->
  <div class="indexnavigationright">
- { if $next_url == "" }
+ { if !isset($next_url) || $next_url == "" }
   { if $count != 0 }
    <img src="{$web_path}/resources/arrow_right_gray.png" alt="last page reached" />
   { /if }
@@ -114,7 +114,7 @@
 
  <!-- the left nav arrow -->
  <div class="indexnavigationleft">
- { if $previous_url == "" }
+ { if !isset($previous_url) || $previous_url == "" }
   { if $count != 0 }
    <img src="{$web_path}/resources/arrow_left_gray.png" alt="first page reached" />
   { /if }
@@ -127,7 +127,7 @@
 
  <!-- the middle page selector -->
  <div class="indexnavigationcenter">
- { if $page_selector != "" }
+ { if isset($page_selector) && $page_selector != "" }
   {$page_selector}
  { /if}
  </div>
@@ -149,7 +149,7 @@
    var last_thumb;
 
    {section name="thumb" loop=$thumbs step=1}
-      {if $images[thumb] }
+      {if isset($images[thumb]) }
          image_urls[{counter}] = '{$web_path}/phpfspot_img.php?idx={$images[thumb]}&width={$width}';
          last_thumb = {$images[thumb]};
       {/if}
diff --git a/themes/default/templates/search.tpl b/themes/default/templates/search.tpl
index 672bb35..69de0a1 100644
--- a/themes/default/templates/search.tpl
+++ b/themes/default/templates/search.tpl
@@ -7,7 +7,7 @@
  <tr>
   <td>Tag:</td>
   <td>
-   { if $use_autocomplete }
+   { if isset($use_autocomplete) }
    <input type="text" name="searchfor_tag" id="searchfor_tag" value="{$searchfor_tag}" size="15" acdropdown="true" autocomplete_list="url:rpc.php?action=getxmltaglist&search=[S]&length=10" />
    { literal }
    <script type="text/javascript">
@@ -21,7 +21,7 @@
    </script>
    { /literal }
    { else }
-   <input type="text" name="searchfor_tag" value="{$searchfor_tag}" size="15" />
+   <input type="text" name="searchfor_tag" value="{ if isset($searchfor_tag) }{$searchfor_tag}{ /if }" size="15" />
    { /if }
   </td>
   <td>
@@ -31,7 +31,7 @@
  <tr>
   <td>Name:</td>
   <td>
-   <input type="text" name="searchfor_name" value="{$searchfor_name}" size="15" />
+   <input type="text" name="searchfor_name" value="{ if isset($searchfor_name) }{$searchfor_name}{ /if }" size="15" />
   </td>
   <td>
    <input type="image" class="submit" src="{$web_path}/resources/doit.png" alt="start search" title="start search" onclick="click(this);" />
@@ -40,7 +40,7 @@
  <tr>
   <td>Date:</td>
   <td class="nowarp">
-   { if ! $date_search_enabled }
+   { if ! isset($date_search_enabled) }
     <input type="checkbox" class="checkbox" name="consider_date" value="Y" onclick="datesearch();" />
    { else }
     <input type="checkbox" class="checkbox" name="consider_date" value="Y" onclick="datesearch();" checked="checked" />
@@ -60,11 +60,11 @@
    <input type="image" class="submit" src="{$web_path}/resources/doit.png" alt="start search" title="start search" onclick="click(this);" />
   </td>
  </tr>
- { if $has_rating }
+ { if isset($has_rating) }
  <tr>
   <td>Rating:</td>
   <td class="nowarp">
-   { if ! $rate_search_enabled }
+   { if ! isset($rate_search_enabled) }
     <input type="checkbox" class="checkbox" name="consider_rate" value="Y" />
    { else }
     <input type="checkbox" class="checkbox" name="consider_rate" value="Y" checked="checked" />
diff --git a/themes/default/templates/single_photo.tpl b/themes/default/templates/single_photo.tpl
index 572bef7..f74b34c 100644
--- a/themes/default/templates/single_photo.tpl
+++ b/themes/default/templates/single_photo.tpl
@@ -9,7 +9,7 @@
    {else}
     Photos
    {/if}
-   {if $description}
+   {if isset($description) }
     -&nbsp;{$description}<br />
    {/if}
   {/if}
@@ -19,7 +19,7 @@
   <a href="javascript:zoom(0);" title="zoom_100" onclick="click(this);"><img src="{$web_path}/resources/zoom-100.png" /></a>
   <a href="javascript:zoom(10);" title="zoom_in" onclick="click(this);"><img src="{$web_path}/resources/zoom-in.png" /></a>
   <a href="javascript:startAutoBrowse();" title="auto browsing"><img id="autobrowse_ico" src="{$web_path}/resources/16_play.png" />&nbsp;Auto-Browse</a>
-  {if $extern_link }
+  {if isset($extern_link) }
    <a href="{$extern_link}" title="Use this link to return to the current view"><img src="{$web_path}/resources/link.png" />&nbsp;External Link</a>
   {/if}
   <a href="{$image_url_full}" title="orig_image" target="_blank" onclick="click(this);"><img src="{$web_path}/resources/original.png" alt="original resolution" />&nbsp;Fullsize</a>
@@ -29,27 +29,27 @@
 <p class="cb">
 
  <div id="photo_details">
-   { if $ExifMadeWith }
+   { if isset($ExifMadeWith) }
    <u><img src="{$web_path}/resources/camera.png" alt="camera icon" />&nbsp;Image taken with:</u><br />
    {$ExifMadeWith}<br />
    { /if }
-   { if $ExifMadeOn }
+   { if isset($ExifMadeOn) }
    <u><img src="{$web_path}/resources/date.png" alt="calendar icon" />&nbsp;Image made on:</u><br />
    {$ExifMadeOn}<br />
    { /if }
-   { if $ExifOrigResolution }
+   { if isset($ExifOrigResolution) }
    <u><img src="{$web_path}/resources/image.png" alt="resolution icon" />&nbsp;Original resolution:</u><br />
    {$ExifOrigResolution}px<br />
    { /if }
    <u>Size:</u>&nbsp;{$ExifFileSize}<br />
-   { if $image_rating }
+   { if isset($image_rating) && $image_rating > 0 }
     <u><img src="{$web_path}/resources/rating.png" alt="rating" />&nbsp;Rating:</u><br />
     {section name="rating" loop=$image_rating step=1}
      <img src="{$web_path}/resources/star.png" />
     {/section}
     <br />
    { /if }
-   { if $has_versions }
+   { if isset($has_versions) }
     <br />
     Versions:<br />
     <select onchange="update_photo_version(this, {$photo});">
@@ -57,7 +57,7 @@
     </select>
     <br />
    { /if }
-   { if $tags }
+   { if isset($tags) }
     <br />
     <u><img src="{$web_path}/resources/small_available_tags.png" alt="available tags" />&nbsp;Tagged with:</u><br />
     { foreach from=$tags key=id item=name }
@@ -65,13 +65,13 @@
     { /foreach }
    { /if }
    <hr>
-   { if $prev_img }
+   { if isset($prev_img) && isset($previous_url) }
     <u><img src="{$web_path}/resources/photo.png" alt="photo icon" />&nbsp;Previous:</u><br />
     <a href="{$previous_url}" onfocus="click(this);" id="prev_link">
      <img src="{$web_path}/phpfspot_img.php?idx={$prev_img}&amp;width={$mini_width}" /><br />
     </a>
    { /if}
-   { if $next_img }
+   { if isset($next_img) && isset($next_url) }
     <u><img src="{$web_path}/resources/photo.png" alt="photo icon" />&nbsp;Next:</u><br />
     <a href="{$next_url}" onfocus="click(this);" id="next_link">
      <img src="{$web_path}/phpfspot_img.php?idx={$next_img}&amp;width={$mini_width}" /><br />
@@ -82,7 +82,7 @@
  <!-- the photo itself -->
 
  <div class="photo">
- { if $next_url == "" }
+ { if !isset($next_url) || $next_url == "" }
   <a href="{$image_url}" onclick="showPhotoIndex(); return false;" onfocus="click(this);" name="photo">
  { else }
   <a href="{$image_url}" onclick="{$next_url}; return false;" onfocus="click(this);" title="click for the next photo" name="photo">
@@ -94,17 +94,23 @@
  <!-- navigation arrows -->
 
  <div class="photonavigation">
-  { if $previous_url == "" }
+
+  <!-- previous -->
+  { if !isset($previous_url) || $previous_url == "" }
    <img src="{$web_path}/resources/arrow_left_gray.png" alt="first photo reached" />
   { else }
    <a href="{$previous_url}" onfocus="click(this);" title="click for the previous photo (left cursor)">
     <img src="{$web_path}/resources/arrow_left.png" alt="previous photo" />
    </a>
   { /if }
+
+  <!-- back to index -->
    <a href="javascript:showPhotoIndex({$current_page}, {$current_img});" onfocus="click(this);" title="click to go back to the index">
     <img src="{$web_path}/resources/arrow_up.png" alt="photo index" />
    </a>
-  { if $next_url == "" }
+
+  <!-- next -->
+  { if !isset($next_url) || $next_url == "" }
    <img src="{$web_path}/resources/arrow_right_gray.png" alt="last photo reached" />
   { else }
    <a href="{$next_url}" onfocus="click(this);" title="click for the next photo (right cursor)">
diff --git a/themes/default/templates/tags.tpl b/themes/default/templates/tags.tpl
index 817cfbc..8a9521e 100644
--- a/themes/default/templates/tags.tpl
+++ b/themes/default/templates/tags.tpl
@@ -8,7 +8,7 @@
  &nbsp;
  <input type="radio" name="condition" value="or" onclick="Tags('condition', this);" {if $current_condition == "or" } checked="checked" { /if } title="OR condition" />||
  <input type="radio" name="condition" value="and" onclick="Tags('condition', this);" {if $current_condition == "and" } checked="checked" { /if } title="AND condition" />&amp;&amp;
-{ if $preset_selected_tags }
+{ if isset($preset_selected_tags) }
  <div id="selected_tags">{ $preset_selected_tags }</div>
 { else }
  <div id="selected_tags">no tags selected</div>
-- 
cgit v1.2.3-18-g5258