Changeset 1084 for trunk/include/functions_html.inc.php

Timestamp:

Mar 16, 2006, 11:34:45 PM (18 years ago)

Author:

plg

Message:

bug fixed: create_navigation_bar take into account clean URL if requested.

deletion: argument link_class (HTML class of links) in function
create_navigation_bar was removed, useless since branch 1.5.

bug fixed: rate_items are now a configuration parameter (set in config file)

modification: new functions library functions_rate.inc.php to reduce
picture.php length.

bug fixed: categories were never expanded in the menu since clean URLs.

bug fixed: changing pictures sorting order in main page was always
rederecting to root category.

File:

• trunk/include/functions_html.inc.php (modified) (7 diffs)

trunk/include/functions_html.inc.php

@@ -76 +76 @@
 }
 
-function create_navigation_bar($url, $nb_element, $start,
-                               $nb_element_page, $link_class)
+function create_navigation_bar(
+  $url, $nb_element, $start, $nb_element_page, $clean_url = false
+  )
 {
   global $lang, $conf;
 
   $pages_around = $conf['paginate_pages_around'];
+  $start_str = $clean_url ? '/start-' : '&start=';
   
   $navbar = '';
@@ -103 +105 @@
     if ($cur_page != 1)
     {
-      $navbar.= '<a href="';
-      $navbar.= $url;
-      $navbar.= '" class="'.$link_class.'" rel="start">'.$lang['first_page'];
-      $navbar.= '</a>';
+      $navbar.=
+        '<a href="'.$url.'" rel="start">'
+        .$lang['first_page']
+        .'</a>';
     }
     else
@@ -114 +116 @@
     $navbar.= ' | ';
     // link on previous page ?
-    if ( $start != 0 )
+    if ($start != 0)
     {
       $previous = $start - $nb_element_page;
-      $navbar.= '<a href="';
-      $navbar.= $url;
-      if ($previous>0)
-      {
-        $navbar.= '&amp;start='.$previous;
-      }
-      $navbar.= '" class="'.$link_class.'" rel="prev">'.$lang['previous_page'];
-      $navbar.= '</a>';
+      
+      $navbar.=
+        '<a href="'
+        .$url.($previous > 0 ? $start_str.$previous : '')
+        .'" rel="prev">'
+        .$lang['previous_page']
+        .'</a>';
     }
     else
@@ -134 +135 @@
     if ($cur_page > $pages_around + 1)
     {
-      $navbar.= '&nbsp;<a href="';
-      $navbar.= $url;
-      $navbar.= '" class="'.$link_class.'">1</a>';
+      $navbar.= '&nbsp;<a href="'.$url.'">1</a>';
+      
       if ($cur_page > $pages_around + 2)
       {
@@ -155 +155 @@
       {
         $temp_start = ($i - 1) * $nb_element_page;
-        $navbar.= '&nbsp;<a href="';
-        $navbar.= $url;
-        if ($temp_start>0)
-        {
-          $navbar.= '&amp;start='.$temp_start;
-        }
-        $navbar.= '" class="'.$link_class.'"';
-        $navbar.='>'.$i.'</a>';
+        
+        $navbar.=
+          '&nbsp;'
+          .'<a href="'.$url
+          .($temp_start > 0 ? $start_str.$temp_start : '')
+          .'">'
+          .$i
+          .'</a>';
       }
       else
       {
-        $navbar.= '&nbsp;<span class="pageNumberSelected">';
-        $navbar.= $i.'</span>';
+        $navbar.=
+          '&nbsp;'
+          .'<span class="pageNumberSelected">'
+          .$i
+          .'</span>';
       }
     }
@@ -174 +177 @@
     {
       $temp_start = ($maximum - 1) * $nb_element_page;
+      
       if ($cur_page < ($maximum - $pages_around - 1))
       {
         $navbar.= ' ...';
       }
-      $navbar.= ' <a href="';
-      $navbar.= $url.'&amp;start='.$temp_start;
-      $navbar.= '" class="'.$link_class.'">'.$maximum.'</a>';
+      
+      $navbar.= ' <a href="'.$url.$start_str.$temp_start.'">'.$maximum.'</a>';
     }
     
     $navbar.= ' | ';
     // link on next page ?
-    if ( $nb_element > $nb_element_page
-         && $start + $nb_element_page < $nb_element )
+    if ($nb_element > $nb_element_page
+        and $start + $nb_element_page < $nb_element)
     {
       $next = $start + $nb_element_page;
-      $navbar.= '<a href="';
-      $navbar.= $url.'&amp;start='.$next;
-      $navbar.= '" class="'.$link_class.'" rel="next">'.$lang['next_page'].'</a>';
+      
+      $navbar.=
+        '<a href="'.$url.$start_str.$next.'" rel="next">'
+        .$lang['next_page']
+        .'</a>';
     }
     else
@@ -203 +208 @@
     {
       $temp_start = ($maximum - 1) * $nb_element_page;
-      $navbar.= '<a href="';
-      $navbar.= $url.'&amp;start='.$temp_start;
-      $navbar.= '" class="'.$link_class.'" rel="last">'.$lang['last_page'];
-      $navbar.= '</a>';
+      
+      $navbar.=
+        '<a href="'.$url.$start_str.$temp_start.'" rel="last">'
+        .$lang['last_page']
+        .'</a>';
     }
     else