Changeset 1020 for trunk/picture.php

Timestamp:

Feb 1, 2006, 3:46:26 AM (18 years ago)

Author:

rvelices

Message:

feature 280: Allow visitors/users to choose image ordering inside a category

improvement 82: Viewing pictures from remote galleries does not check anymore
for the high pictures (existence flag added to create_listing_file and db)

correction: link element in picture is in the head instead of body (w3c spec)

correction: in profile.php the current template was not selected by default

File:

• trunk/picture.php (modified) (4 diffs)

trunk/picture.php

@@ -248 +248 @@
     // if we are working on the "current" element, we search if there is a
     // high quality picture
-    // FIXME : with remote pictures, this "remote fopen" takes long...
     if ($i == 'current')
     {
-      if (@fopen($cat_directory.'/pwg_high/'.$row['file'], 'r'))
+      $url_high=$cat_directory.'/pwg_high/'.$row['file'];
+      if (url_is_remote($cat_directory))
       {
-        $picture[$i]['high'] = $cat_directory.'/pwg_high/'.$row['file'];
+        if ($row['has_high'])
+        {
+          $picture[$i]['high'] = $url_high;
+        }
+      }
+      else
+      {
+        if (@fopen($url_high, 'r'))
+        {
+          $picture[$i]['high'] = $url_high;
+        }
       }
     }
@@ -527 +537 @@
 
 $page['body_id'] = 'thePicturePage';
+//-------------------------------------------------------- navigation management
+if ($has_prev)
+{
+  $template->assign_block_vars(
+    'previous',
+    array(
+      'TITLE_IMG' => $picture['prev']['name'],
+      'IMG' => $picture['prev']['thumbnail'],
+      'U_IMG' => $picture['prev']['url'],
+      'U_IMG_SRC' => $picture['prev']['src']
+      ));
+}
+
+if ($has_next)
+{
+  $template->assign_block_vars(
+    'next',
+    array(
+      'TITLE_IMG' => $picture['next']['name'],
+      'IMG' => $picture['next']['thumbnail'],
+      'U_IMG' => $picture['next']['url'],
+      'U_IMG_SRC' => $picture['next']['src'] // allow navigator to preload
+      ));
+}
+
 include(PHPWG_ROOT_PATH.'include/page_header.php');
 $template->set_filenames(array('picture'=>'picture.tpl'));
@@ -586 +621 @@
 if (isset($picture['current']['high']))
 {
-  $full_size = @getimagesize($picture['current']['high']);
-  $full_width = $full_size[0];
-  $full_height = $full_size[1];
   $uuid = uniqid(rand());
   $template->assign_block_vars('high', array(
     'U_HIGH' => $picture['current']['high'],
-        'UUID'=>$uuid,
-        'WIDTH_IMG'=>($full_width + 40),
-        'HEIGHT_IMG'=>($full_height + 40)
+        'UUID'=>$uuid
         ));
   $template->assign_block_vars(
@@ -672 +702 @@
 {
   $template->assign_block_vars('admin', array());
-}
-
-//-------------------------------------------------------- navigation management
-if ($has_prev)
-{
-  $template->assign_block_vars(
-    'previous',
-    array(
-      'TITLE_IMG' => $picture['prev']['name'],
-      'IMG' => $picture['prev']['thumbnail'],
-      'U_IMG' => $picture['prev']['url'],
-      'U_IMG_SRC' => $picture['prev']['src']
-      ));
-}
-
-if ($has_next)
-{
-  $template->assign_block_vars(
-    'next',
-    array(
-      'TITLE_IMG' => $picture['next']['name'],
-      'IMG' => $picture['next']['thumbnail'],
-      'U_IMG' => $picture['next']['url'],
-      'U_IMG_SRC' => $picture['next']['src'] // allow navigator to preload
-      ));
 }