Witam

Mam kod ktory dzieli mi wynik na kilka stron z piecioma rekordami. Wszystko dziala poprawnie do momentu kiedy chce zeby sortowanie bylo np. wg daty, poprostu nie chce oczytywac mi zmiennej $orderby przy pobieraniu rekordow z bazy danych.

Kod ktory mam na stronie glownej:

[PHP] pobierz, plaintext 
<?php
list($pagertop, $pagerbottom, $limit) = pager(5, $count, "mytorrents.php?");
	$sql = "SELECT A.*, IF(A.numratings < $minvotes, NULL, ROUND(A.ratingsum / A.numratings, 1)) AS rating, B.name AS cat_name, B.image AS cat_pic FROM torrent_torrents A 
L
FT JOIN torrent_categories B ON A.category = B.id ORDER BY $orderby $limit";
		$res = $db->sql_query($sql) or bterror($sql);
 
		print($pagertop);
?>
[PHP] pobierz, plaintext 

Kod z funkcja w oddzielnym pliku:

[PHP] pobierz, plaintext 
<?php
function pager($rpp, $count, $href, $opts = array()) {
		$pages = ceil($count / $rpp);
 
		if (!$opts["lastpagedefault"])
				$pagedefault = 0;
		else {
				$pagedefault = floor(($count - 1) / $rpp);
				if ($pagedefault < 0)
						$pagedefault = 0;
		}
 
		if (isset($_GET["page"])) {
				$page = 0 + $_GET["page"];
				if ($page < 0)
						$page = $pagedefault;
		}
		else
				$page = $pagedefault;
 
		$pager = "";
 
		$mp = $pages - 1;
		$as = "<b>&lt;&lt;&nbsp;Poprzednia</b>";
		if ($page >= 1) {
				$pager .= "<a href=\"modules.php?name=Bittorrent&page=" . ($page - 1) . "&orderby=$orderby\">";
				$pager .= $as;
				$pager .= "</a>";
		}
		else
				$pager .= $as;
		$pager .= "&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;";
		$as = "<b>Nastepna&nbsp;&gt;&gt;</b>";
		if ($page < $mp && $mp >= 0) {
				$pager .= "<a href=\"modules.php?name=Bittorrent&page=" . ($page + 1) . "&orderby=$orderby\">";
				$pager .= $as;
				$pager .= "</a>";
		}
		else
				$pager .= $as;
 
		if ($count) {
				$pagerarr = array();
				$dotted = 0;
				$dotspace = 3;
				$dotend = $pages - $dotspace;
				$curdotend = $page - $dotspace;
				$curdotstart = $page + $dotspace;
				for ($i = 0; $i < $pages; $i++) {
						if (($i >= $dotspace && $i <= $curdotend) || ($i >= $curdotstart && $i < $dotend)) {
								if (!$dotted)
										$pagerarr[] = "...";
								$dotted = 1;
								continue;
						}
						$dotted = 0;
						$start = $i * $rpp + 1;
						$end = $start + $rpp - 1;
						if ($end > $count)
								$end = $count;
						$text = "$start&nbsp;-&nbsp;$end";
						if ($i != $page)
								$pagerarr[] = "<a href=\"modules.php?name=Bittorrent&page=$i&orderby=$orderby\">$text</a>";
						else
								$pagerarr[] = "<b>$text</b>";
				}
				$pagerstr = join(" | ", $pagerarr);
				$pagertop = "<p align=\"center\">$pager<br />$pagerstr</p>\n";
				$pagerbottom = "<p align=\"center\">$pagerstr<br />$pager</p>\n";
		}
		else {
				$pagertop = "<p align=\"center\">$pager</p>\n";
				$pagerbottom = $pagertop;
		}
 
		$start = $page * $rpp;
 
		return array($pagertop, $pagerbottom, "LIMIT $start,$rpp");
}
?>
[PHP] pobierz, plaintext 

Bylbym wdzieczny za pomoc
Pozdrawiam

sprobuj dac do funkcji zmienna orderby:



//odpowiadam na szybko, bez glebszej analizy kodu

Tez nic nie daje, probowalem wczesniej. Na stronie wywala mi komunikat:

No to masz błąd w zapytaniu Po linijce $sql daj echo $sql i przenalizuj sobie tego sql'a dokładnie np. w phpmyadmin'ie.

[PHP] pobierz, plaintext 
<?php
// ...
 $sql = "SELECT A.*, IF(A.numratings < $minvotes, NULL, ROUND(A.ratingsum / A.numratings, 1)) AS rating, B.name AS cat_name, B.image AS cat_pic FROM torrent_torrents A 
L
FT JOIN torrent_categories B ON A.category = B.id ORDER BY $orderby $limit";
echo $sql; // wynik skopiuj i przeanalizuj :)
?>
[PHP] pobierz, plaintext 

No wlasnie chodzi o to ze mam ORDER BY LIMIT 0,5 czyli wpisana przezemnie zmienna $orderby nie jest czytana (mam ja wpisana w 'adresie strony' - ....&orderby=date ASC

Macie jeszcze jakies pomysly na rozwiazanie tego problemu?

Pozdrawiam

No to prawdopodobnie masz register_globals na off (i bardzo dobrze), aby rozwiązać ten problem zamiast $orderby staraj się używać $_GET['orderby'], czyli:

[PHP] pobierz, plaintext 
<?php
$sql = "SELECT A.*, IF(A.numratings < $minvotes, NULL, ROUND(A.ratingsum / A.numratings, 1)) AS rating, B.name AS cat_name, B.image AS cat_pic FROM torrent_torrents A 
L
FT JOIN torrent_categories B ON A.category = B.id ORDER BY " . $_GET['orderby'] . " $limit";
?>
[PHP] pobierz, plaintext 

I obowiązkowo jak tak będziesz robił to przeczytaj temat o SQL Injections.