[php][mysql] Kolejność bloków menu Opcje

[Lee]

Mam moduł bloków menu i gdy jedynie je przesuwam wszystko jest ok, ale gdy skasuje jakiś np. ze środka to już wszystko się rozwala :/

Tak przesuwam blok w górę (w doł pdobonie)

[PHP] pobierz, plaintext 
<?php
public function Up($id) {
	  global $tbl_menu;
		  $this -> query = mysql_query("SELECT * FROM $tbl_menu WHERE id='$id' LIMIT 1");
		  while ($this -> result = mysql_fetch_assoc($this -> query)) {
			  $up = $this -> result['order'];
			  $position = $this -> result['position'];
		  }
		  $order = $up - 1;
		  $this -> query = mysql_query("SELECT * FROM $tbl_menu WHERE `order`='$order' AND `position`='$position' LIMIT 1");
		  while ($this -> result = mysql_fetch_assoc($this -> query)) {
			  $down = $this -> result['order'];
			  $id_down = $this -> result['id'];
		  }
		  $this -> query = mysql_query("UPDATE $tbl_menu SET `order`='$down' WHERE id='$id' LIMIT 1");
		  $this -> query = mysql_query("UPDATE $tbl_menu SET `order`='$up' WHERE id='$id_down' LIMIT 1");
		  header ('Location: admin.php?mod=menu');
	  }
?>
[PHP] pobierz, plaintext 

a tak kasuję blok

[PHP] pobierz, plaintext 
<?php
public function Delete($id) {
	  global $tbl_menu;
		  echo 'Czy na pewno chcesz usunąć to menu ? (<a href="admin.php?mod=menu&act=delete&confirm=yes&id='.$id.'">Tak</a> | <a href="admin.php?mod=menu">Nie</a>)<br />';
		  if ($_GET['confirm'] == 'yes') {
			  @$this -> query = mysql_query("DELETE FROM $tbl_menu WHERE id='$id' LIMIT 1") or die (_MYSQL_ERROR_3 .''. mysql_error());
			  echo 'Menu zostało usunięte ! <a href="admin.php?mod=menu">Powrót</a>';
		  }
	  }
?>
[PHP] pobierz, plaintext 

co powinienem zmienić/dodać ? próbowałem po skasowaniu danej pozycji pozostałe zmniejszyć ale nie działało dobrze (może coś źle robiłem, ale kodu już niestety nie mam :/)

[wipo]

Nie działa poprawnie bo źle badasz poprzedni link
Ty badasz link mniejszy o 1 od poprzedniego, więc jak usuniesz jakiś to możesz go potem nie znależć.
Spróbuj where id<klikanego_id order bo id lub pozycja desc limit 0,1

[Lee]

A w którym momencie powinienem to wstawić ?

[wipo]

Kod

<?php
public function Up($id) {
      global $tbl_menu;
          $this -> query = mysql_query("SELECT * FROM $tbl_menu WHERE id='$id' ");
          while ($this -> result = mysql_fetch_assoc($this -> query)) {
              $up = $this -> result['order'];
              $position = $this -> result['position'];
          }
          $order = $up - 1;
          $this -> query = mysql_query("SELECT * FROM $tbl_menu WHERE position<$position ORDER BY position desc limit 0,1");
          while ($this -> result = mysql_fetch_assoc($this -> query)) {
              $down = $this -> result['order'];
              $id_down = $this -> result['id'];
          }
          $this -> query = mysql_query("UPDATE $tbl_menu SET `order`='$down' WHERE id='$id' LIMIT 1");
          $this -> query = mysql_query("UPDATE $tbl_menu SET `order`='$up' WHERE id='$id_down' LIMIT 1");
          header ('Location: admin.php?mod=menu');
      }
?>

Coś takiego powinno działać

[Lee]

No nie zabardzo, to co chciałem do góry podnieść kolumne "order" ma pustą

[wipo]

A pokaż jak masz zrobioną tabelę w bazie bo pewnie nie to zmieniam co trzeba

[Lee]

|id(int)|name(varchar)|content(text)|position(varchar:left/right)|order(int:1-n)|active(int0/1)|
|1|Główne|blabla|left|1|1
|2|Polecam|blabla|left|2|1

[wipo]

Kod

$this -> query = mysql_query("SELECT * FROM $tbl_menu WHERE order<$position ORDER BY order desc limit 0,1");

To teraz powinno działać

[Lee]

ok działa, nie zauważyłem, że rodzaj kolumny order to varchar btw. poprawny kod wygląda tak

[PHP] pobierz, plaintext 
<?php
$this -> query = mysql_query("SELECT * FROM $tbl_menu WHERE id='$id' LIMIT 1");
		  while ($this -> result = mysql_fetch_assoc($this -> query)) {
			  $up = $this -> result['order'];
			  $position = $this -> result['position'];
		  }
		  $this -> query = mysql_query("SELECT * FROM $tbl_menu WHERE `order`<$up AND `position`='$position' ORDER BY `order` DESC LIMIT 1");
		  while ($this -> result = mysql_fetch_assoc($this -> query)) {
			  $down = $this -> result['order'];
			  $id_down = $this -> result['id'];
		  }
		  $this -> query = mysql_query("UPDATE $tbl_menu SET `order`='$down' WHERE id='$id' LIMIT 1");
		  $this -> query = mysql_query("UPDATE $tbl_menu SET `order`='$up' WHERE id='$id_down' LIMIT 1");
		  header ('Location: admin.php?mod=menu');
?>
[PHP] pobierz, plaintext