Witam pisze sobie modul nawigaji i mam jeden problem chce zrobic przesuwanie linkow w gore/dol w danej kategorii na stronie myslalem ze bedzie to pestka ale trzeba tam duzo warunkow i mam problem bo mi sie juz to wszystko placze mam taka "podstawowa" funkcje:

[PHP] pobierz, plaintext 
<?php
function move_link($id, $site, $wh) {
 
$sql_old = mysql_query('select * from navigation where id = '.(int)$id.' limit 1');
$old = mysql_fetch_assoc($sql_old);
 
if($wh = 'up') $sql_move = mysql_query('select * from navigation where id < '.(int)$id.' and site = "'.mysql_real_escape_string($site).'" order by id asc limit 1'); else $sql_move = mysql_query('select * from navigation where id > '.(int)$id.' and site = "'.mysql_real_escape_string($site).'" order by id desc limit 1');
 
if(mysql_num_rows($sql_move) > 0) $new = mysql_fetch_assoc($sql_move);
 
if(!empty($new['id'])) {
 
 if(mysql_query('update navigation set link = "'.$old['link'].'", name_href = "'.$old['name_href'].'", site = "'.$old['site'].'" where id = '.$new['id'].'') && mysql_query('update navigation set link = "'.$new['link'].'", name_href = "'.$new['name_href'].'", site = "'.$new['site'].'" where id = '.$old['id'].'')) $message = get_error('#0B0', '#eee', '#0B0', 'Link zostal przesuniety'); else $message = get_error('#FF0000', '#eee', '#FF0000', 'Blad: '.mysql_error());
} 
else $message .= 'Brak nowego id<Br>';
$data = $message;
return $data;
}
 
//wywoluje
if(isset($_GET['view']) && $_GET['view'] == 'navigation' && isset($_GET['select']) && !empty($_GET['select']) && isset($_GET['action']) && ($_GET['action'] == 'up' || $_GET['action'] == 'down') && isset($_GET['id']) && !empty($_GET['id'])) $zawartosc = move_link($_GET['id'], $_GET['select'], $_GET['action']);
?>
[PHP] pobierz, plaintext 

No i tu zaczynaja sie schodki bo linki czasami sie nie zmieniaja a czasami sie zmieniaja tak jak ja nie chce.

Mialo by to tak dzialac mam np takie linki na stronie index.php:
Strona glowna
Rejestracja
Administracja

Teraz po kliknieciu na Przesun do goru na link Strona Glowna albo link ma sie nie przesuwac albo ma sie zamienic miejscem z ostatnim linkiem skoro jest on pierwszy ale tak sie nie dzieje

Gdy klikne na Przesun w dol na link Administracja zamienia sie on z pierwszy czyli to jest ok

Gdy klikne na Przesun w dol na link Strona Glowna mam blad ze nie ma nowego ID wiec warunek z zapytaniu sql sie nie spelnia.

Gdy klikne na Przesun w gore na link Rejestracja link sie zamienia miejscem z linkiem Strona glowna czyli jest ok

Gdy klikne przesun w dol na link Rejestracja to Rejestracja staje sie pierwszym linkiem a Administracja ostatnim

Gdy klikne przesun w gore na link Administracja dostaje:
Administracja
Rejestracja
Strona glowna

Jak widac czasami dobrze dziala czasami zle musze uzupelnic tu kilka warynkow tylko nie wiiem na jakiej zasadzie je tam wepchnac i kiedy.

Oczywiscie jesli zmienie ten order by id desc na asc i viceversa i tak czy siak nie dziala w 100%

Czy ktos jest w stanie pomoc?

A nie łatwiej byłoby zroibć kolumne "kolejność" w bazie danych i robić to jakoś tak:

[PHP] pobierz, plaintext 
<?php
$stara_koeljnosc = ... mysql_query("SELECT `kolejnosc` FROM `navigation` WHERE id=5");
mysql_query("UPDATE `navigation` SET koeljnosc=kolejnosc+1 WHERE koeljnosc=$stara_kolejnosc-1");
mysql_query("UPDATE `navigation` SET kolejnosc=kolejnosc-1 WHERE id=5);
?>
[PHP] pobierz, plaintext 

i analogicznie w 2 strone

Nom w sumie moglbym ale zmienialem juz kilka razy strukture tej tabeli i w sumie potem by bylo troche kombinowania z dawaniem kolejnosci danemu linku na danej stronie i z wartosciami ujemnymi etc... chcialbym to chyba jednak zrobic na takiej strukturze jaka jest no chyba ze jeszcze ktos sie wypowie.

Poprę słowa @lukaszgolder, to jest chyba najprostsza metoda na kolejność (dodac kolumnę w bazie odpowiadającą za kolejność). a wartości ujemnych ustrzeżesz się sprawdzając, czy dany item jest SELECT MIN(kolumna_kolejnosci) FROM ... lub SELECT MAX (kolumna_kolejnosci), nie dajesz wtedy guzika przesuwania odpowiednio w dół i w górę.
Za pomocą SELECT MAX(...) rozwiazesz tez problem kolejności nowego linka.
Pamiętac jedynie trzeba, ze przy usuwaniu dobrze jest przesunąć kolejne linki o jeden w dół, dzięki czemu unikasz dziur (ale tutaj są różne koncepcje - czy w ogóle usuwac, czy przesuwac itd)

OK mialem troche czasu i pobawilem sie prawie dziala ale nie do konca mam:

[PHP] pobierz, plaintext 
<?php
function move_link($id, $site, $wh) {
 
$sql_old = mysql_query('select * from navigation where id = '.(int)$id.' and site = "'.mysql_real_escape_string($site).'" limit 1');
$old = mysql_fetch_assoc($sql_old);
 
if($wh == 'up') {
 
$sql_move = mysql_query('select * from navigation where id < '.(int)$id.' and site = "'.mysql_real_escape_string($site).'" order by id desc'); 
 
if(mysql_num_rows($sql_move) > 0) $new = mysql_fetch_assoc($sql_move);
 
else {
 
$sql_move_one = mysql_query('select max(id) from navigation where site = "'.mysql_real_escape_string($site).'"');
 
if(mysql_num_rows($sql_move_one) > 0) {
$max_nav = mysql_fetch_row($sql_move_one);
$sql_max = mysql_query('select * from navigation where id = '.$max_nav[0].' limit 1');
$new = mysql_fetch_assoc($sql_max);
 
    }
  }
 
if(!empty($new['id'])) {
 
 if(mysql_query('update navigation set link = "'.$old['link'].'", name_href = "'.$old['name_href'].'", site = "'.$old['site'].'" where id = '.$new['id'].'') && mysql_query('update navigation set link = "'.$new['link'].'", name_href = "'.$new['name_href'].'", site = "'.$new['site'].'" where id = '.$old['id'].'')) $data = get_error('#0B0', '#eee', '#0B0', 'Link zostal przesuniety o jedna pozycje do gory'); else $data = get_error('#FF0000', '#eee', '#FF0000', 'Blad: '.mysql_error());
  } 
} 
 
else {
 
$sql_move = mysql_query('select * from navigation where id > '.(int)$id.' and site = "'.mysql_real_escape_string($site).'" order by id asc');
 
if(mysql_num_rows($sql_move_one) > 0) $new = mysql_fetch_assoc($sql_move_one);
 
else {
 
$sql_move_one = mysql_query('select min(id) from navigation where site = "'.mysql_real_escape_string($site).'"');
 
if(mysql_num_rows($sql_move_one) > 0) {
$min_nav = mysql_fetch_row($sql_move_one);
$sql_min = mysql_query('select * from navigation where id = '.$min_nav[0].' limit 1');
$new = mysql_fetch_assoc($sql_min);
 
        }
    }
if(!empty($new['id'])) {
 
 if(mysql_query('update navigation set link = "'.$old['link'].'", name_href = "'.$old['name_href'].'", site = "'.$old['site'].'" where id = '.$new['id'].'') && mysql_query('update navigation set link = "'.$new['link'].'", name_href = "'.$new['name_href'].'", site = "'.$new['site'].'" where id = '.$old['id'].'')) $data = get_error('#0B0', '#eee', '#0B0', 'Link zostal przesuniety o jedna pozycje w dol'); else $data = get_error('#FF0000', '#eee', '#FF0000', 'Blad: '.mysql_error());
  } 
} 
 
return $data;
}
?>
[PHP] pobierz, plaintext 

Oczywiscie potem funkcje poprawie zeby nie bylo tyle if'ow bedzie elastyczniejsza jednak nie dzialaja dobrze tylko te 2 rzeczy:


I teraz nie wiem zabardzo co jest przyczyna tego czy ktos moze pomoc?