Ticket #160: another_updated_cruft_free_pagination.patch

File another_updated_cruft_free_pagination.patch, 17.5 KB (added by sidtheduck, 4 months ago)

  • plog-admin/plog-feedback.php

     
    109109#$url = "&entries_per_page=$_SESSION[entries_per_page]&level=$_REQUEST[level]&id=$_REQUEST[id]"; 
    110110$url = "?entries_per_page=$_SESSION[entries_per_page]"; 
    111111 
    112 $first_item = ($_REQUEST['plog_page'] - 1) * $_SESSION['entries_per_page']; 
     112$plog_page = isset($_REQUEST["plog_page"]) ? $_REQUEST["plog_page"] : 1; 
     113 
     114$first_item = ($plog_page - 1) * $_SESSION['entries_per_page']; 
    113115$limit = "LIMIT $first_item, $_SESSION[entries_per_page]"; 
    114116 
    115117// lets generate the pagination menu as well 
    … …  
    121123$mod_result = run_query($query); 
    122124$num_comments_im = mysql_result($mod_result, "in_moderation"); 
    123125 
    124 $page = isset($_GET["plog_page"]) ? $_GET["plog_page"] : 1; 
    125  
    126126// filter based on whether were looking at approved comments or unmoderated comments 
    127 $approved = isset($_GET["moderate"]) ? 0 : 1; 
     127$approved = isset($_REQUEST['moderate']) ? 0 : 1; 
    128128 
    129 if ($approved) 
    130         $pagination_menu = generate_pagination('plog-feedback.php'.$url,$page,$num_comments,$_SESSION['entries_per_page']); 
    131 else 
    132         $pagination_menu = generate_pagination('plog-feedback.php'.$url,$page,$num_comments_im,$_SESSION['entries_per_page'],"&moderate=1"); 
     129if ($approved) { 
     130        $pagination_menu = generate_pagination("admin", "feedback", $plog_page, $num_comments, $_SESSION['entries_per_page']); 
     131} else { 
     132        $pagination_menu = generate_pagination("admin", "feedback", $plog_page, $num_comments_im, $_SESSION['entries_per_page'], array("moderate" => 1)); 
     133} 
    133134 
    134135// generate javascript init function for ajax editing 
    135136$query = "SELECT *, UNIX_TIMESTAMP(`date`) AS `unix_date` from ".TABLE_PREFIX."comments WHERE `approved` = $approved ORDER BY `id` DESC $limit"; 

  • plog-admin/plog-manage.php

     
    300300 
    301301        // handle pagination 
    302302        // lets determine the limit filter based on current page and number of results per page 
    303         if (!isset($_REQUEST["page"])) $_REQUEST["page"] = "1"; // we're on the first page 
     303        if (isset($_REQUEST['entries_per_page'])) { 
     304                $_SESSION['entries_per_page'] = $_REQUEST['entries_per_page']; 
     305        } else { 
     306                $_SESSION['entries_per_page'] = 20; 
     307        } 
    304308 
    305         if (isset($_REQUEST['entries_per_page'])) 
    306         $_SESSION['entries_per_page'] = $_REQUEST['entries_per_page']; 
    307         else 
    308         $_SESSION['entries_per_page'] = 20; 
    309  
    310309        $cond = ""; 
    311310 
    312311        // determine the filtering conditional based on the level and id number 
    … …  
    317316                $cond = "WHERE `parent_album` = '$id'"; 
    318317        } 
    319318 
    320         $url = "?entries_per_page=$_SESSION[entries_per_page]&level=$_REQUEST[level]&id=$id"; 
    321  
    322         $plog_page = isset($_REQUEST['plog_page']) ? $_REQUEST['plog_page'] : 0; 
     319        $plog_page = isset($_REQUEST['plog_page']) ? $_REQUEST['plog_page'] : 1; // we're on the first page 
    323320        $first_item = ($plog_page - 1) * $_SESSION['entries_per_page']; 
    324321        if ($first_item < 0) { 
    325322                $first_item = 0; 
    326         }; 
    327         $limit = "LIMIT $first_item, $_SESSION[entries_per_page]"; 
     323        } 
     324        $limit = "LIMIT ".$first_item.", ".$_SESSION['entries_per_page']; 
    328325 
    329326        // lets generate the pagination menu as well 
    330327        $recordCount = "SELECT COUNT(*) AS num_items FROM ".TABLE_PREFIX."$level $cond"; 
    331328        $totalRowsResult = mysql_query($recordCount); 
    332329        $totalRows = mysql_result($totalRowsResult,'num_items'); 
    333330 
    334         $page = isset($_GET["plog_page"]) ? $_GET["plog_page"] : 1; 
    335         $pagination_menu = "\n\t\t" . '<div class="pagination">'.generate_pagination('plog-manage.php'.$url,$page,$totalRows,$_SESSION['entries_per_page']).'</div>'; 
     331        $pagination_menu = "\n\t\t" . '<div class="pagination">'.generate_pagination("admin", "manage", $plog_page, $totalRows, $_SESSION['entries_per_page'], array("level" => $level, "id" => $id, "entries_per_page" => $_SESSION['entries_per_page'])).'</div>'; 
    336332 
    337333        $output .= "\n\t\t" . '<form id="contentList" action="'.$_SERVER["PHP_SELF"].'" method="get">'; 
    338334 
    339335        $level = $_REQUEST['level']; 
    340336 
    341         if (empty($level)) { 
     337        if (empty($level) || $level == "collections") { 
    342338                $output .= generate_breadcrumb_admin("").$pagination_menu; 
    343339                $output .= plog_collection_manager($first_item,$_SESSION['entries_per_page']); 
    344         }; 
     340        } 
    345341 
    346342        if ($level == "albums") { 
    347343                $output .= generate_breadcrumb_admin("albums", $id).$pagination_menu; 
    … …  
    365361                <input type="hidden" name="id" value="'.$id.'" /> 
    366362                <input type="hidden" name="action" value="1" /> 
    367363                <input class="submit" type="submit" name="delete_checked" onclick="return confirm(\'' . plog_tr('Are you sure you want to delete selected items?') . '\');" value="' . plog_tr('Delete Checked') . '" />'; 
    368         if (!empty($level) && $level != "comments"){ 
     364        if (!empty($level) && $level != "collections" && $level != "comments"){ 
    369365                $output .= generate_move_menu($level); 
    370366        }; 
    371367        $output .= "\n\t\t</div>\n\t\t</form>\n"; 

  • plog-includes/plog-functions.php

     
    10241024                } 
    10251025        } 
    10261026 
    1027         if (!empty($names["collection"])) { 
     1027        if (!empty($names['collection'])) { 
    10281028                $sql = "SELECT * 
    10291029                FROM `".TABLE_PREFIX."collections` 
    1030                 WHERE `path`='".$names["collection"]."'"; 
     1030                WHERE `path`='".$names['collection']."'"; 
    10311031                $result = run_query($sql); 
    10321032 
    1033                 // No such collection, fall back to main page 
     1033                // no such collection 
    10341034                if (mysql_num_rows($result) == 0){ 
    1035                         return $rv; 
     1035                        //check if it's collections pagination 
     1036                        if (is_numeric($names['collection']) && intval($names['collection']) == $names['collection']) { 
     1037                                return array("level" => "collections", "id" => 0, "plog_page" => intval($names['collection'])); 
     1038                        } 
    10361039                } 
    10371040 
    10381041                $collection = mysql_fetch_assoc($result); 
    10391042 
    10401043                // what if there are multiple collections with same names? I hope there aren't .. this would 
    10411044                // suck. But here is an idea, we shouldn't allow the user to enter similar names 
    1042                 $rv = array("level" => "collection","id" => $collection["id"]); 
     1045                $rv = array("level" => "collection","id" => $collection['id']); 
    10431046        } 
    10441047 
    10451048        if (!empty($names['album'])) { 
    10461049                $sql = "SELECT * 
    10471050                FROM `".TABLE_PREFIX."albums` 
    1048                 WHERE `path`='".$names["album"]."' 
    1049                 AND `parent_id`=".intval($collection["id"]); 
     1051                WHERE `path`='".$names['album']."' 
     1052                AND `parent_id`=".intval($collection['id']); 
    10501053                $result = run_query($sql); 
    10511054 
    1052                 // no such album, fall back to collection 
     1055                // no such album 
    10531056                if (mysql_num_rows($result) == 0){ 
    1054                         return $rv; 
     1057                        //check if it's collection pagination 
     1058                        if (is_numeric($names['album']) && intval($names['album']) == $names['album']) { 
     1059                                return array("level" => "collection", "id" => $collection['id'], "plog_page" => intval($names['album'])); 
     1060                        } 
    10551061                } 
    10561062 
    10571063                $album = mysql_fetch_assoc($result); 
    … …  
    10771083                $rv = array('level' => 'album','id' => $album['id']); 
    10781084        } 
    10791085 
    1080         if (!empty($names["picture"])) { 
     1086        if (!empty($names['picture'])) { 
    10811087                $sql = "SELECT * 
    10821088                FROM `".TABLE_PREFIX."pictures` 
    1083                 WHERE `caption`='".$names["picture"]."' 
    1084                 AND `parent_album`=".intval($album["id"]); 
     1089                WHERE `caption`='".$names['picture']."' 
     1090                AND `parent_album`=".intval($album['id']); 
    10851091                $result = run_query($sql); 
    10861092 
    10871093                $picture = mysql_fetch_assoc($result); 
    … …  
    11001106                        $picture = mysql_fetch_assoc($result); 
    11011107                } 
    11021108 
    1103                 // no such picture, fall back to album 
     1109                // no such picture 
    11041110                if (!$picture) { 
    1105                         return $rv; 
     1111                        //check if it's album pagination 
     1112                        if (is_numeric($names['picture']) && intval($names['picture']) == $names['picture']) { 
     1113                                return array("level" => "album", "id" => $album['id'], "plog_page" => intval($names['picture'])); 
     1114                        } 
    11061115                } 
    11071116 
    1108                 $rv = array("level" => "picture", "id" => $picture["id"]); 
     1117                $rv = array("level" => "picture", "id" => $picture['id']); 
    11091118        } 
    11101119 
    11111120        return $rv; 
    11121121} 
    11131122 
    1114 function generate_pagination($url, $current_page, $items_total, $items_on_page, $extra_params = ''){ 
     1123function generate_pagination($level, $id, $current_page, $items_total, $items_on_page, $args = array()){ 
    11151124        $output = ''; 
    11161125 
    1117         if (!isset($GLOBALS["total_pictures"])) $GLOBALS["total_pictures"] = 0; 
     1126        if (!isset($GLOBALS['total_pictures'])) $GLOBALS['total_pictures'] = 0; 
    11181127 
    1119         if (($items_total == 0) && ($GLOBALS["total_pictures"] > 0)) { 
    1120                 $items_total = $GLOBALS["total_pictures"]; 
     1128        if (($items_total == 0) && ($GLOBALS['total_pictures'] > 0)) { 
     1129                $items_total = $GLOBALS['total_pictures']; 
    11211130        } 
    11221131 
    11231132        $num_pages = ceil($items_total / $items_on_page); 
    11241133 
    1125         // if adding arguments to mod_rewritten urls, then I need ? (question mark) before the arguments 
    1126         // otherwise I want &amp; 
    1127         //$last = substr($url,-1); 
    1128  
    1129         if (!strpos($url,"?")) { 
    1130                 $separator = "?"; 
    1131         } else { 
    1132                 $separator = "&amp;"; 
    1133         } 
    1134  
    11351134        if ($num_pages > 1){ 
    11361135                if ($current_page > 1){ 
    1137                         $output .= ' <a accesskey="," class="pagPrev" href="'.$url.$separator.'plog_page='.($current_page - 1).$extra_params.'"><span>&laquo;</span></a> '; 
     1136                        $args['plog_page'] = $current_page - 1; 
     1137                        $output .= ' <a accesskey="," class="pagPrev" href="'.generate_url($level, $id, $args).'"><span>&laquo;</span></a> '; 
    11381138                } 
    11391139 
    11401140                for ($i = 1; $i <= $num_pages; $i++){ 
    11411141                        if ($i == $current_page){ 
    11421142                                $output .= '<span class="page_link"> ['.$i.'] </span>'; 
    11431143                        } else{ 
    1144                                 $output .= '<a href="'.$url.$separator.'plog_page='.$i.$extra_params.'" class="page_link">'.$i.'</a> '; 
     1144                                $args['plog_page'] =  $i; 
     1145                                $output .= '<a href="'.generate_url($level, $id, $args).'" class="page_link">'.$i.'</a> '; 
    11451146                        } 
    11461147                } 
    11471148 
    11481149                if ($current_page != $num_pages){ 
    1149                         $output .= ' <a accesskey="." class="pagNext" href="'.$url.$separator.'plog_page='.($current_page + 1).$extra_params.'"><span>&raquo;</span></a> '; 
     1150                        $args['plog_page'] =  $current_page + 1; 
     1151                        $output .= ' <a accesskey="." class="pagNext" href="'.generate_url($level, $id, $args).'"><span>&raquo;</span></a> '; 
    11501152                } 
    11511153        } 
    11521154 
    … …  
    11661168        $rv = ''; 
    11671169 
    11681170        if ($config['use_mod_rewrite']){ 
     1171                $args = ''; 
     1172                // I need to give additional arguments to the url-s 
     1173                if (sizeof($arg) > 0) { 
     1174                        foreach($arg as $aval) { 
     1175                                $args .= $aval."/"; 
     1176                        } 
     1177                } 
     1178 
    11691179                switch($level){ 
    11701180                        case "collection": 
    11711181                                $query = "SELECT `path` FROM `".TABLE_PREFIX."collections` WHERE `id`=".intval($id); 
    11721182                                $result = run_query($query); 
    11731183                                $row = mysql_fetch_assoc($result); 
    1174                                 $rv = $config['baseurl'].rawurlencode(SmartStripSlashes($row['path']))."/"; 
     1184                                $rv = $config['baseurl'].rawurlencode(SmartStripSlashes($row['path']))."/".$args; 
    11751185                                break; 
    11761186                        case "album": 
    11771187                                $query = "SELECT 
    … …  
    11821192                                WHERE `a`.`id`=".intval($id); 
    11831193                                $result = run_query($query); 
    11841194                                $row = mysql_fetch_assoc($result); 
    1185  
    1186                                 $rv = $config['baseurl'].rawurlencode(SmartStripSlashes($row['collection_path'])) . '/' . rawurlencode(SmartStripSlashes($row['album_path']))."/"; 
    1187  
    1188                                 // I need to give additional arguments to the url-s 
    1189                                 if (sizeof($arg) > 0) { 
    1190                                         foreach($arg as $aval) { 
    1191                                                 $rv .= $aval."/"; 
    1192                                         } 
    1193                                 } 
     1195                                $rv = $config['baseurl'].rawurlencode(SmartStripSlashes($row['collection_path'])) . '/' . rawurlencode(SmartStripSlashes($row['album_path']))."/".$args; 
    11941196                                break; 
    11951197                        case "picture": 
    11961198                                $pic = get_picture_by_id($id); 
    … …  
    12031205                                if (sizeof($arg) > 0) { 
    12041206                                        foreach($arg as $akey => $aval) { 
    12051207                                                // mod_rewrite url-s need /sorted in them, the old style ones do not. 
    1206                                                 // this temporary workaround removes the 'sorted' string 
    1207                                                 if ($aval != 'sorted') { 
     1208                                                // this temporary workaround removes the 'sorted' and 'plog_page' strings 
     1209                                                if ($aval != "sorted") { 
    12081210                                                        $rv .= "&amp;".$akey."=".$aval; 
    12091211                                                } 
    12101212                                        } 
    12111213                                } 
    12121214                                break; 
    12131215                        case "collections": 
    1214                                 default: 
    1215                                 $rv = $config['baseurl']; 
     1216                        default: 
     1217                                $rv = $config['baseurl'].$args; 
    12161218                                break; 
    12171219                } 
    12181220        } else { 
    12191221                // if there are non-Plogger query items, get them here to prepend to the URL query string 
    1220                 $query = (isset($config['query_args'])) ? "?".$config['query_args']."&amp;" : $query = "?"; 
     1222                // non-Plogger query items only work with old style URLs (not with mod_rewrite URLs) 
     1223                $query = (isset($config['query_args'])) ? "?".$config['query_args']."&amp;" : "?"; 
     1224                 
     1225                $args = ''; 
     1226                // add on any additional arguments from the $arg array 
     1227                if (sizeof($arg) > 0) { 
     1228                        foreach($arg as $akey => $aval) { 
     1229                                // mod_rewrite url-s need /sorted and /plog_page in them, the old style ones do not. 
     1230                                // this temporary workaround removes the 'sorted' and 'plog_page' strings 
     1231                                if ($aval != "sorted") { 
     1232                                        $args .= "&amp;".$akey."=".$aval; 
     1233                                } 
     1234                        } 
     1235                } 
    12211236 
    12221237                switch($level){ 
     1238                        // admin section for generate_url 
     1239                        case "admin": 
     1240                                $rv = $config['baseurl']."plog-admin/plog-".$id.".php?".substr($args, 5); 
     1241                                break; 
     1242                        // front end section for generate_url 
    12231243                        case "collection": 
    1224                                 return $config['baseurl'].$query.'level=collection&amp;id='.$id; 
     1244                                $rv = $config['baseurl'].$query.'level=collection&amp;id='.$id.$args; 
    12251245                                break; 
    12261246                        case "album": 
    1227                                 $rv = $config['baseurl'].$query.'level=album&amp;id='.$id; 
    1228                                 if (sizeof($arg) > 0) { 
    1229                                         foreach($arg as $akey => $aval) { 
    1230                                                 // mod_rewrite url-s need /sorted in them, the old style ones do not. 
    1231                                                 // this temporary workaround removes the 'sorted' string 
    1232                                                 if ($aval != 'sorted') { 
    1233                                                         $rv .= "&amp;".$akey."=".$aval; 
    1234                                                 } 
    1235                                         } 
    1236                                 } 
     1247                                $rv = $config['baseurl'].$query.'level=album&amp;id='.$id.$args; 
    12371248                                break; 
    12381249                        case "picture": 
    12391250                                $rv = $config['baseurl'].$query.'level=picture&amp;id='.$id; 
    12401251                                break; 
    12411252                        case "search": 
    1242                                 $rv = $config['baseurl'].$query."level=search"; 
    1243                                 // I need to give additional arguments to the url-s 
    1244                                 if (sizeof($arg) > 0) { 
    1245                                         foreach($arg as $akey => $aval) { 
    1246                                                 // mod_rewrite url-s need /sorted in them, the old style ones do not. 
    1247                                                 // this temporary workaround removes the 'sorted' string 
    1248                                                 if ($aval != 'sorted') { 
    1249                                                         $rv .= "&amp;".$akey."=".$aval; 
    1250                                                 } 
    1251                                         } 
    1252                                 } 
     1253                                $rv = $config['baseurl'].$query."level=search".$args; 
    12531254                                break; 
    12541255                        case "collections": 
    1255                                 default: 
    1256                                 $query = (isset($config['query_args'])) ? "?".$config['query_args'] : $query = ""; 
    1257                                 $rv = $config['baseurl'].$query; 
     1256                        default: 
     1257                                $rv = $config['baseurl']; 
     1258                                if ($query != "?" && empty($args)) { 
     1259                                        if ($query == "?") { $args = substr($args, 5); } 
     1260                                        $rv .= $query.$args; 
     1261                                } 
    12581262                                break; 
    12591263                } 
    12601264        } 
    12611265 
    1262         // replace &amp; with & if outputting to email 
     1266        // replace &amp; with & if formatting plaintext (i.e. outputting to email) 
    12631267        if ($plaintext !== false){ 
    12641268                $rv = str_replace("&amp;","&",$rv); 
    12651269        } 
    … …  
    19741978        global $config; 
    19751979 
    19761980        if ($GLOBALS['plogger_mode'] != 'slideshow') { 
    1977                 $page = isset($_GET["plog_page"]) ? intval($_GET["plog_page"]) : 1; 
    1978  
    1979                 if ($GLOBALS['plogger_level'] == "search") { 
    1980                         $p_url = generate_url("search", -1, array('searchterms'=>urlencode($_GET["searchterms"]))); 
    1981                 } 
    1982                 else { 
    1983                         if ($GLOBALS['plogger_level']) { 
    1984                                 $p_url = generate_url($GLOBALS['plogger_level'], $GLOBALS['plogger_id']); 
    1985                         } else { 
    1986                                 $p_url = generate_url("collections"); 
    1987                         } 
    1988                 } 
    1989  
    1990                 switch($GLOBALS['plogger_level']) { 
     1981                $page = isset($_GET['plog_page']) ? intval($_GET['plog_page']) : 1; 
     1982                $level = $GLOBALS['plogger_level']; 
     1983                $id = $GLOBALS['plogger_id']; 
     1984                switch($level) { 
    19911985                        case 'search': 
    1992                                 $num_items = $GLOBALS["total_pictures"]; 
    1993                         break; 
     1986                                $num_items = $GLOBALS['total_pictures']; 
     1987                                return generate_pagination("search", -1, $page, $num_items, $config['thumb_num'], array('searchterms'=>urlencode($_GET['searchterms']))); 
     1988                                break; 
    19941989 
    19951990                        case 'album': 
    19961991                                $num_items = plogger_album_picture_count(); 
    1997                         break; 
     1992                                break; 
    19981993 
    19991994                        case 'collection': 
    20001995                                $num_items = plogger_collection_album_count(); 
    2001                         break; 
     1996                                break; 
    20021997 
    20031998                        default: 
    2004                         $num_items = plogger_count_collections(); 
    2005                         break; 
     1999                                $level = "collections"; 
     2000                                $id = 0; 
     2001                                $num_items = plogger_count_collections(); 
     2002                                break; 
    20062003                } 
    20072004 
    2008                 return generate_pagination($p_url, $page, $num_items, $config["thumb_num"]); 
     2005                return generate_pagination($level, $id, $page, $num_items, $config['thumb_num']); 
    20092006        } 
    20102007 
    20112008} 

  • plog-load-config.php

     
    8585// remove plog-admin/ from the end, if present .. is there a better way to determine the full url? 
    8686// had to update this for new use of links without mod_rewrite turned on (only affects the View gallery greybox in Admin) 
    8787if (strpos($config['baseurl'], "plog-admin/")) { 
    88         $config['baseurl'] = substr($config['baseurl'],0,strpos($config['baseurl'], "plog-admin/")); 
     88        $config['use_mod_rewrite'] = 0; 
     89        $config['baseurl'] = substr($config['baseurl'], 0, strpos($config['baseurl'], "plog-admin/")); 
    8990} 
    9091 
    9192$config['theme_url'] = $config['gallery_url']."plog-content/themes/".basename($config['theme_dir'])."/"; 

  • plogger.php

     
    2121                if (isset($resolved_path['id'])) { 
    2222                        $_GET['id'] = $resolved_path['id']; 
    2323                } 
     24                if (isset($resolved_path['plog_page'])) { 
     25                        $_GET['plog_page'] = $resolved_path['plog_page']; 
     26                } 
    2427                if (isset($resolved_path['mode'])) { 
    2528                        $_GET['mode'] = $resolved_path['mode']; 
    2629                } 
    2730 
    28                 // get page number from url, if present 
     31                // get the path for RSS links (maybe should rework this) 
    2932                $parts = parse_url($_SERVER['REQUEST_URI']); 
    30                 if (isset($parts['query'])) { 
    31                         parse_str($parts['query'],$query_parts); 
    32                         if (!empty($query_parts['plog_page'])) { 
    33                                 $_GET['plog_page'] = $query_parts['plog_page']; 
    34                         } 
    35                 } 
    3633                $path = $parts['path']; 
    3734        } 
    3835} else {