Problem ze skryptem aktualizującym Opcje

[Vertisan]

Witam!

Mam sobie taki kod HTML (a konkretniej link):

[HTML] pobierz, plaintext 
<a href="?rank='.$row['user_id'].'" class="btn btn-info btn-sm text-right" name="rank"> Zmień rangę </a>
[HTML] pobierz, plaintext 

oraz PHP:

[PHP] pobierz, plaintext 
 
 if( isset($_GET['rank']) )
            {
                 $rank = $_GET['rank'];
              $a_rank=mysql_query('SELECT `user_rank` FROM users');
              $res_a_rank = mysql_fetch_array($a_rank);
 
                if ($res_a_rank['user_rank'] == 1) {
 
                    $sql= "UPDATE users SET `user_rank` = '0' WHERE user_id = '$rank' ";
                    $res= mysql_query($sql) or die("Wystąpił błąd podczas zmiany uprawnien!".mysql_error());
                    echo "<meta http-equiv='refresh' content='0;url=listUsers.php'>";
                } elseif ($res_a_rank['user_rank'] == 0) {
 
                    $sql= "UPDATE users SET `user_rank` = '1' WHERE user_id = '$rank' ";
                    $res= mysql_query($sql) or die("Wystąpił błąd podczas zmiany uprawnien!".mysql_error());
                    echo "<meta http-equiv='refresh' content='0;url=listUsers.php'>";
                }
 
            }
[PHP] pobierz, plaintext 

Problem polega na tym, że po kliknięciu w link zmienia się wartość 1 -> 0, ale 0 -> 1 już nie.
Ma ktoś pomysł?

[Wazniak96]

Zamień

[PHP] pobierz, plaintext 
$a_rank=mysql_query('SELECT `user_rank` FROM users');
[PHP] pobierz, plaintext 

Na

[PHP] pobierz, plaintext 
 $a_rank=mysql_query('SELECT `user_rank` FROM users WHERE user_id='.$rank); 
[PHP] pobierz, plaintext 

Ps: Zdajesz sobie sprawę, że twój skrypt jest dziurawy jak sito? (IMG:style_emoticons/default/wink.gif)

[Vertisan]

Nie bardzo działa ...
Dopiero się ucze (IMG:style_emoticons/default/smile.gif)

[Pyton_000]

Jeżeli tylko zmieniasz zamiennie to możesz tak:

[PHP] pobierz, plaintext 
if( isset($_GET['rank']) )
            {
                 $rank = $_GET['rank'];
$sql = "UPDATE users SET `user_rank` = NOT `user_rank` WHERE user_id = '$rank' ";
$res= mysql_query($sql) or die("Wystąpił błąd podczas zmiany uprawnien!".mysql_error());
                    echo "<meta http-equiv='refresh' content='0;url=listUsers.php'>";
}
[PHP] pobierz, plaintext 

[Vertisan]

Jeżeli tylko zmieniasz zamiennie to możesz tak:

[PHP] pobierz, plaintext 
if( isset($_GET['rank']) )
            {
                 $rank = $_GET['rank'];
$sql = "UPDATE users SET `user_rank` = NOT `user_rank` WHERE user_id = '$rank' ";
$res= mysql_query($sql) or die("Wystąpił błąd podczas zmiany uprawnien!".mysql_error());
                    echo "<meta http-equiv='refresh' content='0;url=listUsers.php'>";
}
[PHP] pobierz, plaintext 

Nic nie dało ...

[Pyton_000]

Co nic nie dało?

[Vertisan]

To samo co na początku ...
Jeżeli jest ustawione 1 to zmienia na 0, a jeżeli jest 0 to nic nie robi.

[Pyton_000]

Zamieniłeś całość co miałeś tym co podałem?

[Forti]

Pobierasz cala tabele zamiast pojedynczy rekord user_id. W pierwszym zapytaniu dodaj WHERE. Wowczas powinno dzialac. Pokaz kod jaki masz po obecnych zmianach.

I tak, ten kod jest jak sitko z samymi uchwytami bez sitka (IMG:style_emoticons/default/smile.gif)


Pisze z telefonu wiec kodu ci nie podam.

[Vertisan]

Zamieniłeś całość co miałeś tym co podałem?

Tak

Pobierasz cala tabele zamiast pojedynczy rekord user_id. W pierwszym zapytaniu dodaj WHERE. Wowczas powinno dzialac. Pokaz kod jaki masz po obecnych zmianach.

I tak, ten kod jest jak sitko z samymi uchwytami bez sitka (IMG:style_emoticons/default/smile.gif)


Pisze z telefonu wiec kodu ci nie podam.

[PHP] pobierz, plaintext 
            if( isset($_GET['rank']) )
            {
                 $rank = $_GET['rank'];
              $a_rank=mysql_query('SELECT `user_rank` FROM users WHERE user_id ='.$rank.' ');
              $res_a_rank = mysql_fetch_array($a_rank);
 
                if ($res_a_rank['user_rank'] == 1) {
 
                    $sql= "UPDATE users SET `user_rank` = '0' WHERE user_id = '$rank' ";
                    $res= mysql_query($sql) or die("Wystąpił błąd podczas zmiany uprawnien!".mysql_error());
                    echo "<meta http-equiv='refresh' content='0;url=listUsers.php'>";
                } else if ($res_a_rank['user_rank'] == 0) {
 
                    $sql= "UPDATE users SET `user_rank` = '1' WHERE user_id = '$rank' ";
                    $res= mysql_query($sql) or die("Wystąpił błąd podczas zmiany uprawnien!".mysql_error());
                    echo "<meta http-equiv='refresh' content='0;url=listUsers.php'>";
                }
 
            }  
[PHP] pobierz, plaintext 

Coś takiego skleciłem ale nadal nic (w sensie nie zmienia rangi)

[Forti]

Ty chcesz zmienic range jednego usera czy wszystkich na raz? Jak wszystkich na raz? Skasuj caly twoj kod i wstaw to co python ci napisal. Jak nie dziala to blad masz gdzie indziej..

To co wstawiles where co prpsilem: szukasz usera z user_id odpowiadajacego wartosci zmiennej rank.. (IMG:style_emoticons/default/smile.gif)

[Vertisan]

Ty chcesz zmienic range jednego usera czy wszystkich na raz? Jak wszystkich na raz? Skasuj caly twoj kod i wstaw to co python ci napisal. Jak nie dziala to blad masz gdzie indziej..

To co wstawiles where co prpsilem: szukasz usera z user_id odpowiadajacego wartosci zmiennej rank.. (IMG:style_emoticons/default/smile.gif)

Chce zmienić tylko jednego użytkownika, tego przy którym kliknę w link

[Forti]

To musisz zrobić tak:

masz lista użytkowników. Klikasz w link i odsyła Cie np. do strony ze skryptem. w linku jako _GET pobieraj jego ID (w bazie unikalne ID dla user oczywiście). Następnie coś takiego:

[PHP] pobierz, plaintext 
<?php
            if( isset($_GET['id']) )  //jeżeli wybraliśmy user ID
            {
                 $user_id = $_GET['id'];  // przypisujemy _GET do zmiennej user_id
              $a_rank=mysql_query('SELECT `user_rank` FROM users WHERE user_id ='.$user_id.' ');  // pobieramy user_rank usera wg. jego ID
              $res_a_rank = mysql_fetch_array($a_rank);
 
                if ($res_a_rank['user_rank'] == 1) {  // jeżeli wartość user_rank 1 to:
 
                    $sql= "UPDATE users SET `user_rank` = '0' WHERE user_id = '$user_id' ";
                    $res= mysql_query($sql) or die("Wystąpił błąd podczas zmiany uprawnien!".mysql_error());
                    echo "<meta http-equiv='refresh' content='0;url=listUsers.php'>";
                } 
                 elseif ($res_a_rank['user_rank'] == 0) {  // jeżeli wartość user_rank 0 to:
 
                    $sql= "UPDATE users SET `user_rank` = '1' WHERE user_id = '$user_id' ";
                    $res= mysql_query($sql) or die("Wystąpił błąd podczas zmiany uprawnien!".mysql_error());
                    echo "<meta http-equiv='refresh' content='0;url=listUsers.php'>";
                }
 
			echo $res_a_rank['user_rank'];	 // wyświetl sobie wartość user_rank
 
            }  
?>
[PHP] pobierz, plaintext 

Jest sobota i ja nie myślę, więc mogę się mylić ale elseif piszę się razem ;]
oczywiście na stronie z listą uzytkowników musisz pobierać z bazy ich nazwy oraz id by tego przekazać.

Rozumiesz? (IMG:style_emoticons/default/wink.gif)


edit:

zastanów się nad PDO - ułatwia życie, zabezpiecza przed wieloma błędami i atakami. Troche możesz mieć problemów ze zrozumieniem i nauczeniem się, ale na prawdę warto od tego zaczynać. mysql_query przeszło do "lamusa" wieki temu i kolejne wersje php za pewne wogóle nie będą go obsługiwać.

Ten post edytował Forti 9.08.2014, 12:14:45

[Pyton_000]

Forti to co ja podałem realizuje dokładnie to samo tyle że dużo szybciej i prościej.

Ewidentnie błąd jest nie tu tylko w innym miejscu.

Vertisan wklej jeszcze raz mój kod do postu i po $sql = ...
daj
echo $sql; Zobacz jakie zapytania Ci się generują.

[Forti]

Tak, twój kod wykona się szybciej, jednak napisałem mu tak aby zrozumiał gdzie ma błąd. Bardziej w celach edukacyjnych.

[Vertisan]

Pogubiłem się już .. (IMG:style_emoticons/default/sad.gif)
To może od początku (może coś bardziej rozjaśni).

To jest moja struktura tabeli

(IMG:http://screenshu.com/static/uploads/temporary/er/22/dz/qz1vln.jpg)


To jest kod w całości:

[PHP] pobierz, plaintext 
<html>
<head>
  <title>Zmiana rangi</title>
  <meta charset="utf-8">
 
  <style>
    table {
      border: 1px solid black;
    }
    td {
      padding: 5px;
    }
  </style>
</head>
<body>
 
 
<?php 
 
define('DBHOST', 'localhost');
define('DBUSER', 'root');
define('DBPASS', '');
define('DBNAME', 'service');
 
function db_connect() {
    mysql_connect(DBHOST, DBUSER, DBPASS) or die('Server MySQL nie odpowiada!');
 
    mysql_select_db(DBNAME) or die('Nie można wybrać bazy!');
}
 
function db_close() {
    mysql_close();
}
 
// -------------------------------------------------------------------------------------
db_connect();
 
            if($result = mysql_query('SELECT * FROM users ORDER BY user_id')) 
              {
 
echo ' <table>
          <thead>
            <tr>
              <th>#</th>
              <th>Nazwa</th>
              <th>Email</th>
              <th>Ranga</th>
              <th>Data rejestracji</th>
              <th>&nbsp</th>
            </tr>
          </thead> 
      ';
 
 
          while($row = mysql_fetch_array($result)){
 
          echo '<tbody>';
          echo '<tr>';
          echo '<td>' . $row['user_id'] . '</td>';
          echo '<td>' . $row['user_name'] . '</td>';
          echo '<td>' . $row['user_email'] . '</td>';
          echo '<td>' . $row['user_rank'] . '</td>';
          echo '<td>' . $row['user_regdate'] . '</td>';
          echo '<td> <a href="?rank='.$row['user_id'].'" name="rank"> Zmień rangę </a>';
 
              // ---------------------------------------------------------------------
 
            if( isset($_GET['rank']) )
            {
                 $rank = $_GET['rank'];
              $a_rank=mysql_query('SELECT `user_rank` FROM users');
              $res_a_rank = mysql_fetch_array($a_rank);
 
                if ($res_a_rank['user_rank'] == 1) {
 
                    $sql= "UPDATE users SET `user_rank` = '0' WHERE user_id = '$rank' ";
                    $res= mysql_query($sql) or die("Wystąpił błąd podczas zmiany uprawnien!".mysql_error());
                } elseif ($res_a_rank['user_rank'] == 0) {
 
                    $sql= "UPDATE users SET `user_rank` = '1' WHERE user_id = '$rank' ";
                    $res= mysql_query($sql) or die("Wystąpił błąd podczas zmiany uprawnien!".mysql_error());
                }
 
            } 
 
            echo '</tr>';
              echo '</tbody>';
 
        }
      }
 
          echo '</table>';
 
 
db_close();
 ?>
 
</body>
</html>
[PHP] pobierz, plaintext 

I teraz.
Gdy jest ustawiona wartość na 1 to zmienia na 0, natomiast jeżeli jest 0 to nic się nie dzieje (a ma zmieniać na 1)

[Pyton_000]

Wywal cały ten if z rank
i wstaw to co Ci dałem zaraz po db_connect();

[Forti]

Wkleiłeś to ci ci napisałem ja lub python? W linku przekazujesz $_GET['rank']. Co to jest rank? 0 lub 1 odpowiadające randze usera czy jego ID?


już wiem gdzie masz kod, zrób to co mówi python, to musi dzialać.

Co ten kod Tobie wyświetla? Bo jak dla mnie to nie powinna Ci sie wogole strona otworzyć ;] masz pobrać jednego użytkownika czy wszystkich?

Ja to cały ten kod bym skasował i napisał od nowa.


edit

Ten post edytował Forti 9.08.2014, 13:10:52

[Vertisan]

Nareszcie ...
Widzę że jeszcze sporo muszę się nauczyć (IMG:style_emoticons/default/smile.gif)
Dzięki wielkie chłopaki za pomoc (IMG:style_emoticons/default/wink.gif)