[PHP][MySQL]Dwa skrypty(Wysyłanie komunikatu oraz zmiana hasła) Opcje

[adrianozo]

Witam wszystkich. Od jakiegoś czasu nie mogę uporać się z dwoma funkcjami panelu administracyjnego. Pierwszym z nich jest skrypt komunikatu administratora.

Chodzi o to, że po zaznaczeniu pola checked nie zmienia się wartość w bazie danych na 1. Poza tym brakuje mi tutaj czegoś takiego, że jeśli wartość w bazie równa się 1 to checked jest zaznaczony.

Struktura bazy:

[SQL] pobierz, plaintext 
CREATE TABLE IF NOT EXISTS `komunikat` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `tresc` longtext collate utf8_polish_ci NOT NULL,
  `poziom` text collate utf8_polish_ci NOT NULL,
  `adres` text collate utf8_polish_ci NOT NULL,
  `numer` text collate utf8_polish_ci NOT NULL,
  PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 COLLATE=utf8_polish_ci AUTO_INCREMENT=2 ;
[SQL] pobierz, plaintext 

Skrypt:

http://wklej.org/id/388541/

Drugim skryptem, z którym mam problem to zmiana hasła administratora. Z tym skryptem według mnie będzie trudniej, bo o ile pierwsza część wykonuje się prawidłowo to, gdy nie występują błędy w pierwszej części formularz przechodzi do drugiego formularza, ale co teraz się dziej. Pokazują się oby dwa formularze. Zamiast tylko drugi to pokazuje się i pierwszy i drugi. A gdy tutaj złe dane się wpisze to powraca do pierwszego formularza, a chciałbym, żeby w tym drugim można było poprawić dane i kontynuować skrypt czyli zmienić hasło w bazie.

Struktura bazy:

[SQL] pobierz, plaintext 
CREATE TABLE IF NOT EXISTS `admin` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `loginek` text collate utf8_polish_ci NOT NULL,
  `email` text collate utf8_polish_ci NOT NULL,
  `haslo` text collate utf8_polish_ci NOT NULL,
  `accessik` int(1) NOT NULL DEFAULT '0',
  `klucz` varchar(40) collate utf8_polish_ci NOT NULL,
  PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 COLLATE=utf8_polish_ci AUTO_INCREMENT=2 ;
[SQL] pobierz, plaintext 

Skrypt:

http://wklej.org/id/388542/

Proszę o pomoc bo już nie wiem co jest nie tak.
Z góry dziękuje (IMG:style_emoticons/default/smile.gif)
Za pomoc oczywiście "POMÓGŁ" (IMG:style_emoticons/default/smile.gif)
Pozdrawiam.

[Armstrong]

Co do pierwszego tylko podpowiem:

porównaj linię 19 z 35 i zobacz czym się różni.

A co do zaznaczenia to linia:

49 zmień na:
$pobierztresc = mysql_query("SELECT tresc, poziom FROM komunikat WHERE id='1'");

i linię 56 z isset($poziom) na isset($tresc['poziom']) <-- nie wiem po co ale jest rano, więc może się to okazać konieczne.

oraz:

59 z $poziom na: $tresc['poziom');

co do drugiego nie mam już czasu... Może później.

[adrianozo]

To tak. Działać działa ale sypie błędem:

Notice: Undefined index: poziom in /a/b/c/d/e/komunikat.php on line 10

[PHP] pobierz, plaintext 
<?php
if(!$_SESSION['access'] || !$_SESSION['login'])
{
    header('Location: index.php');
}
include('include/db.php');
if(isset($_POST['tresc']))
{
    $tresc = strip_tags(htmlspecialchars(mysql_real_escape_string($_POST['tresc'])));
    $poziom = strip_tags(htmlspecialchars(mysql_real_escape_string($_POST['poziom'])));
}
if(isset($_GET['wpis']) && !empty($_GET['wpis']))
{
    if($_GET['wpis']==1)
    {
        if($_POST['poziom'] == 0) 
        { 
            $zapytanie = "INSERT INTO `komunikat` (`tresc`, `poziom`) VALUES ('".$tresc."', '1')";
            $idzapytania = mysql_query($zapytanie);
        } 
        else 
        { 
            $zapytanie = "INSERT INTO `komunikat` (`tresc`, `poziom`) VALUES ('".$tresc."', '0')";
            $idzapytania = mysql_query($zapytanie);
        }
    }
}
if(isset($_GET['wpis']) && !empty($_GET['wpis']))
{
    if($_GET['wpis']==2)
    {
        if($poziom == 1) 
        { 
            $zapytanie = "UPDATE `komunikat` SET `tresc`= '".$tresc."', `poziom`= '1' WHERE `id`='1'";
            $idzapytania = mysql_query($zapytanie);
        } 
        else 
        { 
            $zapytanie = "UPDATE `komunikat` SET `tresc`= '".$tresc."', `poziom`= '0' WHERE `id`='1'";
            $idzapytania = mysql_query($zapytanie);
        }
    }
}
if(mysql_num_rows(mysql_query("SELECT id FROM komunikat WHERE id='1'"))==1)
{
    $pobierztresc = mysql_query("SELECT tresc, poziom FROM komunikat WHERE id='1'");
    $tresc = mysql_fetch_assoc($pobierztresc);
    echo '<form action="index.php?page=komunikat&wpis=2" method="post">
    <input type="hidden" name="send" value="2"><br />Treść<br />
    <br /><textarea name="tresc" cols="50" rows="10" id="textarea">'.stripslashes($tresc['tresc']).'</textarea><br /><br />
    Nadaj komunikat:
    <input type="checkbox" name="poziom" value="1"';
    if(isset($tresc['poziom']))
    {
    ?> 
    <?=($tresc['poziom']==1)?' checked="checked" ':''?> 
    <?php
    }
    echo '<br /><br /><input name="Zapisz" type="submit" value="Zapisz" />
    <input name="wyczysc" type="reset" value="Wyczyść" />
    </form><br /><a style="text-decoration: none;" href="index.php?page=paneladmina">Wróć do Panelu Administratora</a><br /><br />';
}
else
{
    echo '<form action="index.php?page=komunikat&wpis=1" method="post">
    <input type="hidden" name="send" value="1"><br />Treść<br />
    <br /><textarea name="tresc" cols="50" rows="10" value="Nowa tresc" id="textarea">Tutaj będzie znajdował się komunikat administratora.</textarea>
    <br /><br />
    Nadaj komunikat:
    <input type="checkbox" name="poziom" value="1"'; 
    ?>
    <?=($poziom==1)?' checked="checked" ':''?> 
    <?php
    echo '<br /><br /><input name="Zapisz" type="submit" value="Zapisz" />
    <input name="wyczysc" type="reset" value="Wyczyść" />
    </form><br /><a style="text-decoration: none;" href="index.php?page=paneladmina">Wróć do Panelu Administratora</a><br /><br />';
}
if(isset($_POST['send']))
{
    if($_POST['send'])
    {
        if(isset($idzapytania))
        {
            if($idzapytania === TRUE)
            {
                echo '<font color="green"><strong>Komunikat został zapisany.</strong></font><br /><br />';
            }
            else
            {
                echo '<font color="red"><strong>Nie udało się zapisać komunikatu. Proszę spróbować za jakiś czas!</strong></font><br /><br />';
            }
        }
    }
}
mysql_close($connect);
?>
[PHP] pobierz, plaintext 

Ten post edytował adrianozo 14.09.2010, 13:53:20

[selecton]

Co do problemu pierwszego:
w lini 10 masz if(isset($poziom)) a nie widzę wcześniej deklaracji zmiennej $poziom. Ten kawałek kodu się nigdy nie wykona.
Dalej w lini 19 jest: if($_POST['poziom'] == 1) problem w tym, że gdy checkbox jest niezaznaczony to wogóle się nie wysyła w POST. sprawdzaj czy isset i czy równe jest równe 1.
Zmienna poziom w bazie: `poziom` text collate utf8_polish_ci NOT NULL,
jest typou text. Czy jest to konieczne jeśli przechowuje zmienną Bool? Pozatym jeśli to tekst to porównuj jak string == "1" a nie == 1 (taka drobna uwaga)

Co do problemu 2:
Nie wiem dokładnie o co Ci chodzi. Strzelam, że ma być tak:
Jeśli ktoś popełnij błąd w pierwszym formularzu to ten formularz ma się pokazać jeszcze raz a jeśli niema błędów to ma się pokazać formularz drugi. A teraz jest problem z tym, że gdy niema błędu to się pokazuje formularz pierwszy i drugi? Czy tak?
Jeśli tak to zwróć uwagę jak działa Twój algorytm:
1. jeśli NIE WYSŁANO pierwszego forma to go wyświetl (linia 8)
2. w przeciwnym razie WYŚWIETL pierwszy form i sprawdź czy jest on poprawny (linia 17)
3. sprawdź czy WYSŁANO pierwszy form i czy nie było błędów. Jesli tak to wyświetl form drugi. (linia 58)
Tak te warunki są ułożone, że gdy wyślesz pierwszy form to wyświetli się on jeszcze raz (linia 19) a potem sprawdzenie będzie czy są błędy i jeśli nie to wyświetli się drugi form.

Ten post edytował selecton 14.09.2010, 14:40:28

[adrianozo]

Pierwszy skrypt zrobiony.

Problem został w drugim skrypcie.

Teraz gdy nie ma błędu pokazuje się formularz drugi. Wypełniasz dane w drugim formularzu i jak wysyłasz i pokazują się błędy to pokazują się błędy oraz pierwszy formularz zamiast drugiego. To samo jest, gdy poprawnie dane wypełnisz. Pokazuje się pierwszy formularz, a nie pokazują się komunikaty, czy poprawnie dane wprowadzone, czy błędnie.

zmiana_hasla.php

[PHP] pobierz, plaintext 
<?php
if(!$_SESSION['access'] || !$_SESSION['login'])
{
    header('Location: index.php');
}
include('include/db.php');
$error = false;
if(!isset($_POST['dalej']))
{
    echo '<form action="index.php?page=zmien_haslo&wpis=1" method="post"><br />
    Podaj login:<br /><input type="text" name="loginek" /><br /><br />
    Podaj email:<br /><input type="text" name="email" /><br /><br />
    Podaj hasło:<br /><input type="password" name="haslo" /><br /><br />
    <input name="dalej" type="submit" value="Dalej"><br /><br />
    </form><a style="text-decoration: none;" href="index.php?page=paneladmina">Wróć do Panelu Administratora</a><br /><br />';
}
else
{
    echo '<form action="index.php?page=zmien_haslo&wpis=2" method="post"><br />
    Podaj login:<br /><input type="text" name="loginek" /><br /><br />
    Podaj email:<br /><input type="text" name="email" /><br /><br />
    Podaj hasło:<br /><input type="password" name="haslo" /><br /><br />
    <input name="dalej" type="submit" value="Dalej"><br /><br />
    </form><a style="text-decoration: none;" href="index.php?page=paneladmina">Wróć do Panelu Administratora</a><br /><br />';
    $login = strip_tags(htmlspecialchars(mysql_real_escape_string($_POST['loginek'])));
    $haslo = strip_tags(htmlspecialchars(mysql_real_escape_string(md5(sha1($_POST['haslo'])))));
    $email = strip_tags(htmlspecialchars(mysql_real_escape_string($_POST['email']))); 
    if(empty($login) || empty($haslo)  || empty($email)) 
    {
        echo 'Uzupełnij wszystkie pola.<br /><br />';
        $error = true;
    }
    else
    {
        $log = mysql_query("SELECT * FROM admin WHERE loginek='".$login."'");
        $log1 = mysql_num_rows($log);
        if($log1 === 0) 
        {
            echo 'Podany login jest nieprawidłowy.<br /><br />';
            $error = true;
        }
        $ema = mysql_query("SELECT * FROM admin WHERE email='".$email."'");
        $ema1 = mysql_num_rows($ema);
        if($ema1 === 0) 
        {
            echo 'Podany email jest nieprawidłowy.<br /><br />';
            $error = true;
        }
        $has = mysql_query("SELECT * FROM admin WHERE haslo='".$haslo."'");
        $has1 = mysql_num_rows($has);
        if($has1 === 0) 
        {
            echo 'Podane hasło jest nieprawidłowe.<br /><br />';
            $error = true;
        }
    }
}
if((isset($_POST['dalej']))&&($error==false))
{
    echo '<form action="index.php?page=zmien_haslo&wpis=3" method="post">
    <input type="hidden" name="login" value="'.$login.'"><br /> 
    Podaj stare hasło<br /><input type="password" name="haslos" /><br /><br />
    Podaj nowe hasło<br /><input type="password" name="haslo1" /><br /><br />
    Powtórz nowe hasło<br /><input type="password" name="haslo2" /><br /><br />
    <input name="zapisz" type="submit" value="Zapisz" /></form><br />
    <a style="text-decoration: none;" href="index.php?page=paneladmina">Wróć do Panelu Administratora</a><br /><br />';
}
if((isset($_POST['zapisz']))&&($error==false))
{
    $login = strip_tags(htmlspecialchars(mysql_real_escape_string($_POST['login'])));
    $haslos = strip_tags(htmlspecialchars(mysql_real_escape_string(md5(sha1($_POST['haslos'])))));
    $haslo1 = strip_tags(htmlspecialchars(mysql_real_escape_string(md5(sha1($_POST['haslo1'])))));
    $haslo2 = strip_tags(htmlspecialchars(mysql_real_escape_string(md5(sha1($_POST['haslo2']))))); 
    if(empty($haslos) || empty($haslo1)  || empty($haslo2)) 
    {
        echo 'Uzupełnij wszystkie pola.<br /><br />';
        $error = true;
    }
    elseif(!empty($haslos) || !empty($haslo1)  || !empty($haslo2)) 
    {
        $has = mysql_query("SELECT * FROM admin WHERE haslo='".$haslos."'");
        $has1 = mysql_num_rows($has);
        if($has1 === 0) 
        {
            echo 'Podane hasło jest nieprawidłowe.<br /><br />';
            $error = true;
        }
        if($haslo1 != $haslo2) 
        {
            echo 'Podane hasła nie są identyczne.<br /><br />';
            $error = true;
        }
    }
    else
    {
        $zapytanie1 = 'UPDATE `admin` SET `haslo`= "'.$haslo2.'" WHERE `loginek`="'.$login.'"';
        $idzapytania1 = mysql_query($zapytanie1) or die(mysql_error()); 
        if($idzapytania1 === TRUE)
        {
            echo '<font color="green"><strong>Hasło zostało zmienione.</strong></font><br /><br />';
            $error = true;
        }
        else
        {
            echo '<font color="red"><strong>Nie udało się zmienić hasła.<br />Proszę spróbować za jakiś czas</strong></font><br /><br />';
            $error = true;
        }
    }
}
?>
[PHP] pobierz, plaintext 

[selecton]

Troche niepotrzebnie to zagmatwałeś. Generalnie chcesz zrobić taki algorytm:

[PHP] pobierz, plaintext 
 
function isForm1Submitted() { return isset($_POST['zapisz'] ); }
function getForm1() { return '<form...'; }
function getForm1Errors() return null albo string z błędami }
 
function isForm2Submitted() { return isset($_POST['zapisz']); }
function getForm2Errors() { return null albo string z błędami }
function getForm2() {return '<form...'; }
function saveForm2() { update table... }
 
if( isForm2Submitted() ) {
    $validation = getForm2Errors() 
    if( $errors === null) {
        saveForm2();
        echo 'Hasło zostało zmienione';
    } else {
        echo getForm2();
        echo $errors;
    }
 
 
} else {
 
    if( isForm1Submitted() ) {
        $errors = getForm1Errors();
        if( $errors === null) {
              echo getForm2();
        } else {
              echo getForm1();
              echo $errors;
        }
 
    } else {
        echo getForm1();
    }
}
 
[PHP] pobierz, plaintext 

To jest tylko szkic algorytmu - pokombinuj w tym kierunku.

Ten post edytował selecton 14.09.2010, 19:27:42