Forum PHP.pl > [PHP]System newsów->dodajnews.php

nansss

6.03.2009, 19:09:47

Piszę własny system newsów i mam problem z dodajnews.php
Wygląda on następująco:

Kod

<?php
session_start();
function connect_db()
{
     mysql_connect("localhost", "nazwa_uzytkownika", "haslo") or die(mysql_error());
     mysql_select_db("nazwa_bazy_danych") or die(mysql_error());
}

if($_SESSION['haslo']=="haslo")
{
     if($_POST['news']=="NULL")
     {
             echo "<form action="dodajnews.php" method="post">


<input type="text" name="tytul" value="Tytuł news\'a" size="20">


                   <textarea name="news" cols="20" rows="20">Treść news\'a</textarea>


                   <input type="submit" value="Dodaj">
             </form>";
     }

     else
     {
         $news = $_POST['news'];
         $tytul = $_POST['tytul'];
         $data = date('Y-m-d');
         connect_db();
         mysql_query ("SET NAMES latin2");
         mysql_query("INSERT INTO `news` (`news`,`tytul`,`data`) VALUES ('".$news."','".$tytul."','".$data."') ") or die(mysql_error());

     }

}

?>

Proszę o pomoc co jest źle.
Gdy wrzucam go na serwer to nic nie wyświetla.

Buła

6.03.2009, 19:12:52

Kod

echo "<form action="dodajnews.php" method="post">

Nie wiem, jak u innych, ale na moim serwerze jeśli w echo są " to nie działa. Musi być ' zamiast " przy HTMLowych fragmentach.

Np.

Kod

echo "<form action='dodajnews.php' method='post'>";

nansss

6.03.2009, 19:15:55

Poprawiłem, ale nic nie pomogło

Buła

6.03.2009, 19:20:38

Kod

<?php
session_start();
function connect_db()
{
     mysql_connect("localhost", "nazwa_uzytkownika", "haslo") or die(mysql_error());
     mysql_select_db("nazwa_bazy_danych") or die(mysql_error());
}

if($_SESSION['haslo']=="haslo")
{
     if($_POST['news']=="NULL")
     {
             echo "<form action='dodajnews.php' method='post'>


<input type='text' name='tytul' value='Tytuł newsa' size='20'>


                   <textarea name='news' cols='20' rows='20'>Treść newsa</textarea>


                   <input type='submit' value='Dodaj'>
             </form>";
     }

     else
     {
         $news = $_POST['news'];
         $tytul = $_POST['tytul'];
         $data = date('Y-m-d');
         connect_db();
         mysql_query ("SET NAMES latin2");
         mysql_query("INSERT INTO `news` (`news`,`tytul`,`data`) VALUES ('".$news."','".$tytul."','".$data."') ") or die(mysql_error());

     }

}
?>

Spróbuj tak. W tych krzakach po polskich literach miałeś znak średnika.

nansss

6.03.2009, 19:26:47

W tym momencie także nic nie wyświetla

Mephistofeles

6.03.2009, 19:35:08

A jak ustawiasz 'haslo' w sesji? No i czemu "NULL" a nie null bez cudzysłowów? Albo użyj isset.

nansss

6.03.2009, 19:47:24

A w czym wstawić hasło?

Smoker

6.03.2009, 19:48:33

na wstepie zacznij od zainstalowania sobie programu kolorującego składnie php... polecam keD instalka to jakies 2mb wiec malo a jest przydatny i od razu w nim widac ze w tym co napisales jest błąd

a jaki to juz szukam :] a ty sobei zainstaluj ked'a

spróbuj tak:

[PHP] pobierz, plaintext 
<?php
session_start();
function connect_db()
{
     mysql_connect("localhost", "nazwa_uzytkownika", "haslo") or die(mysql_error());
     mysql_select_db("nazwa_bazy_danych") or die(mysql_error());
}
 
if($_SESSION['haslo'] == 'haslo')
{
     if(!isSet($_POST['news']))
     {
             echo "<form action=\"dodajnews.php\" method=\"post\">
                  
 
<input type=\"text\" name=\"tytul\" value=\"Tytuł newsa\" size=\"20\">
                  
 
                   <textarea name=\"news\" cols=\"20\" rows=\"20\">Treść newsa</textarea>
                  
 
                   <input type=\"submit\" value=\"Dodaj\">
             </form>";
     }
    
     else
     {
         $news = $_POST['news'];
         $tytul = $_POST['tytul'];
         $data = date('Y-m-d');
         connect_db();
         mysql_query ("SET NAMES latin2");
         mysql_query("INSERT INTO `news` (`news`,`tytul`,`data`) VALUES ('".$news."','".$tytul."','".$data."') ") or die(mysql_error());
    
     }  
    
}
?>
[PHP] pobierz, plaintext

I jedno pytanie... hasło masz kodowane ? bo ze skrytu wynika że nie więc proponuje kodować hasło w md5() a jeśli masz to problem tkwi w tym ze nic nie wyswietla bo hasło sie nie zgadza

Spawnm

6.03.2009, 19:55:01

Cytat(Smoker @ 6.03.2009, 19:48:33 )

na wstepie zacznij od zainstalowania sobie programu kolorującego składnie php... polecam keD instalka to jakies 2mb wiec malo a jest przydatny i od razu w nim widac ze w tym co napisales jest błąd

a jaki to juz szukam :] a ty sobei zainstaluj ked'a

albo jakikolwiek normalny edytor ,np: notepad++.

if($_POST['news']=="NULL") zmień na if($_POST['news']==NULL) lub if( empty($_POST['news']) )

Mephistofeles

6.03.2009, 19:55:18

No ale czy hasło w ogóle gdzieś istnieje? Bo wnioskując z

Cytat("nansss")

A w czym wstawić hasło?

to mogę sobie pomyśleć, że nie...

nansss

6.03.2009, 19:57:46

Istnieje w panel.php

Smoker

6.03.2009, 19:58:45

hasło istnieje i z tego co widze to jest zapisane w $_SESSION['haslo'] ale zawsze jest takie same

i to jest dziwne

Ps. probowales mojego rozwiazania ?

Mephistofeles

6.03.2009, 20:02:13

Tak jak mówi Smoker, formularz się raczej nie wyświetli, bo $_POST['news'] na 99 % nie ma wartości "NULL"

.

Smoker

6.03.2009, 20:04:01

w moim rozwiazaniu uzylem if(!isSet($_POST['news'])) zeby wyswietlal formularz jak nie dostal newsa w zmiennej POST

bo masz racje ze NULL bedzie dopiero jak wysle formularz z pustą zmienną

nansss

6.03.2009, 20:05:50

Hasło jest takie samo, ponieważ tylko jedna osoba będzie dodawała newsy

Cytat(Smoker @ 6.03.2009, 19:58:45 )

Ps. probowales mojego rozwiazania ?

TAK PRÓBOWAŁEM

Smoker

6.03.2009, 20:09:25

no to widocznie twoje hasło nie pasuje :]

Ps wstawilem tam
$haslo = 'haslo';
if($haslo == 'haslo')
{...

I mi dziala :] wiec bankowo to jest sprawa hasla

Spawnm

6.03.2009, 20:11:42

pokaż kod który ustawia sesje z hasłem

nansss

6.03.2009, 20:19:17

Tutaj jest pięć kodów: panel.php, dodajnews.php, usunnews.php, index.php i news.php

panel.php

Kod

<?php
session_start();
if($_GET['wyloguj'])
{
     session_unset();
     echo 'Wylogowano';
     exit;
}

else
{

     if($_POST["haslo"]=="haslo")
     {
         $_SESSION['haslo'] = $_POST["haslo"];
         echo "Zalogowano..";
         echo '<p />';
     }

     else
     {
         echo "

         <form method='post' action='panel.php'>

         Hasło:

         <input type='password' value='' name='haslo' /> <input type='submit' value='Zaloguj' />

         </form>

         ";
     }

}

if($_SESSION['haslo']=='haslo')
{
     echo "
         <a style='color: red; text-decoration: none;' href='dodajnews.php'> Dodaj aktualność </a>


         <a style='color: red; text-decoration: none;' href='index.php?strona=panel&wyloguj=1'> Wyloguj się </a>
     ";
}

?>

dodajnews.php

Kod

<?php
session_start();
function connect_db()
{
     mysql_connect("host", "uzytkownik", "haslo") or die(mysql_error());
     mysql_select_db("nazwa") or die(mysql_error());
}

if($_SESSION['haslo'] == 'haslo')
{
     if(!isSet($_POST['news']))
     {
             echo "<form action=\"dodajnews.php\" method=\"post\">


<input type=\"text\" name=\"tytul\" value=\"Tytuł newsa\" size=\"20\">


                   <textarea name=\"news\" cols=\"20\" rows=\"20\">Treść newsa</textarea>


                   <input type=\"submit\" value=\"Dodaj\">
             </form>";
     }

     else
     {
         $news = $_POST['news'];
         $tytul = $_POST['tytul'];
         $data = date('Y-m-d');
         connect_db();
         mysql_query ("SET NAMES latin2");
         mysql_query("INSERT INTO `news` (`news`,`tytul`,`data`) VALUES ('".$news."','".$tytul."','".$data."') ") or die(mysql_error());

     }

}
?>

usunnews.php

Kod

<?php
session_start();
function connect_db()
{
mysql_connect("host", "uzytkownik", "haslo") or die(mysql_error());
mysql_select_db("nazwa") or die(mysql_error());
}

if($_SESSION['haslo']=="haslo")
{
connect_db();

mysql_query ("SET NAMES latin2");

$id = $_GET['id'];
mysql_query(' DELETE FROM `news` WHERE `id`="'.$id.'" ') or die(mysql_error());

echo 'Usunięto news';

}

?>

index.php

Kod

<?php

function connect_db()
{
     mysql_connect("host", "uzytkownik", "haslo") or die(mysql_error());
     mysql_select_db("nazwa") or die(mysql_error());
}
connect_db();

mysql_query ("SET NAMES latin2");

$query = mysql_query(" SELECT * FROM `news` ORDER BY DATA DESC LIMIT 1");

while($fetch=mysql_fetch_array($query))
{
$tekst  .= '<b><h2>'.$fetch[1]. '</h2>
</b>'.$fetch[2].'<br/><a href="index.php?strona=news&id='.$fetch[0].'">Czytaj wiecej...</A>';
}
echo $tekst;
mysql_close();

?>

news.php

Kod

<?php

function connect_db()
{
     mysql_connect("host", "uzytkownik", "haslo") or die(mysql_error());
     mysql_select_db("nazwa") or die(mysql_error());
}
connect_db();

mysql_query ("SET NAMES latin2");

$query = mysql_query(" SELECT * FROM `news` ORDER BY `id` DESC ");

while($fetch=mysql_fetch_array($query))
{
    echo '<table>';
    echo '<tr>';
    echo '<td width="430px"> <p class="style4 style5 style6">'.$fetch['tytul'].'</p> </td>'
    .'<td align="left"> <p class="style4 style5" style="font-size: 10px;">'.'Dodano: '.$fetch['data'].'</p> </td>';

    if($_SESSION['haslo'])
    {
    echo '<td>';
    echo '<a  class="style4 style5" style="font-size: 10px; color: red; text-decoration: none;" href="index.php?strona=usunnews&id='.$fetch['id'].' "> Usuń </a> ';
    echo '</td>';
    echo '</tr>';
    }
    else
    {
        echo'</tr>';
    }
    echo '<tr><td>';
    echo '<hr>';
    echo '<p class="style4 style5">'. $fetch['news'].'</p>';
    echo '

';
    echo '</td></tr>';

    echo '</table>';
}

mysql_close();

?>

Spawnm

6.03.2009, 20:25:35

[PHP] pobierz, plaintext 
<?php
if($_POST["haslo"]=="haslo")
     {
         $_SESSION['haslo'] = $_POST["haslo"];
         echo "Zalogowano..";
         echo '<p />';
     }
?>
[PHP] pobierz, plaintext

ciekawy html

daj echo $_SESSION['haslo'] ;
w pliku w którym ci nie działa hasło, ewentualnie w strlen , może jakaś biała spacja ci się zakradła .

nansss

6.03.2009, 20:27:47

Cytat(Spawnm @ 6.03.2009, 20:25:35 )

daj echo $_SESSION['haslo'] ;

Gdzie mam to dać?

Spawnm

6.03.2009, 20:29:19

w pliku co ci nie działa zaraz przed sprawdzeniem poprawności (czyli na samym początku po session_start)

Smoker

6.03.2009, 20:31:30

w dodajnews.php gdzies na gorze i zobacz jakie hasło Ci pokaze... czy 'haslo' czy jakies inne czy wywali blad ze nie ma hasla :]

Mephistofeles

6.03.2009, 20:33:27

[PHP] pobierz, plaintext 
<?php
function connect_db()
{
     mysql_connect("host", "uzytkownik", "haslo") or die(mysql_error());
     mysql_select_db("nazwa") or die(mysql_error());
}
connect_db();
?>
[PHP] pobierz, plaintext

A co to jest? Nie dało się normalnie, bez funkcji? I tak w każdym pliku masz to samo. No i po co dajesz warunek $_SESSION['haslo'] == 'haslo', skoro zaraz później już robisz to bez sprawdzania zgodności? Wywal lepiej te == 'haslo' i wtedy sprawdź.

nansss

6.03.2009, 20:33:55

Dałem w tym miejscu co mówiłeś i nie wyskakuje mi, że złe hasło, a dodajnews.php dalej nie dziala

Usunąłem to =='haslo' i bylo jeszcze gorzej tj. zniknal mi favicon i tytyl

Mephistofeles

6.03.2009, 20:36:13

Daj echo 'costam' zaraz po sprawdzeniu hasła z sesji, i drugi raz po sprawdzeniu POSTa. I powiedz, który się wyświetla, albo czy w ogóle się wyświetlają

.

Spawnm

6.03.2009, 20:38:05

nie wyskakuje że złe czyli wyświetla się dobre hasło , tak? a dałeś strlen czy są białe spacje , ewentualnie daj
echo '"'.$_SESSION['haslo'].'"';
jeśli masz w sesji spacje to będzie odstęp między hasłem a " i wiadomo gdzie błąd

bo po za tym to nie widać

Smoker

6.03.2009, 20:38:09

Mephistofeles jak to wywali to kazdemu sie pokaze dodawanie newsa a nie tylko "adminowi"

a co do połączenia to masz racje

ja to zawsze robie w osobnym pliku i tylko includuje jak jest taka potrzeba :]

nansss

6.03.2009, 20:39:28

Żaden się nie wyświetlił, a tylko usunął mi favicon i tytuł

Smoker

6.03.2009, 20:43:36

powinienes wogole przerobic ten skrypt

wstawic w bazie kolumne admin i nadac wartosc 1 dla admina i 0 dla usera

i w panelu admina sprawdzic czy ma adma czy nie jesli nie ma to header("Location: index.php"); a jesli jest to dalsza czesc skryptu i nie musialbys teraz szukac jakiegos malego durnelo bledu w kodzie

Ps. jesli haslo pokazal dobre to haslo == haslo i nie widze innej mozliwosc a wg twojego skryptu haslo !== haslo :]

nansss

6.03.2009, 20:45:52

Kurde nie moge skminić ;/

Mephistofeles

6.03.2009, 20:46:49

Jak wywali sprawdzanie z sesji, to raczej nic się nie zmieni (pomijając bezpieczeństwo), bo dopóki zmienna w sesji nie istnieje to nikomu admina nie pokaże. Ja tak miałem dłuższy czas u siebie, i działało, sprawdzałem tylko czy istnieje zmienna.
A skoro usunął ci się tytuł, to może coś jest nie tak z HTMLem?
Okrężna droga - daj tam jakiegoś INSERTA do bazy, i zobacz czy się dodało

. Albo lepiej nie, chociaż spróbować nie zaszkodzi.
A jak spróbujesz takiego:

[PHP] pobierz, plaintext 
<?php
session_start();
function connect_db()
{
     mysql_connect('host', 'uzytkownik', 'haslo') or die(mysql_error());
     mysql_select_db('nazwa') or die(mysql_error());
}
echo 'debug 1 ;)';
echo $_SESSION['haslo'];
if($_SESSION['haslo'] == 'haslo')
{
echo 'debug 2';
     if(!isSet($_POST['news']))
     {
echo 'debug 3';
             echo '<form action="dodajnews.php" method="post">
                  <input type="text" name="tytul" value="Tytuł newsa" size="20">
                   <textarea name="news" cols="20" rows="20">Treść newsa</textarea>
                   <input type="submit" value="Dodaj">
             </form>';
     }
    
     else
     {
echo 'debug 4';
         $news = $_POST['news'];
         $tytul = $_POST['tytul'];
         $data = date('Y-m-d');
         connect_db();
         mysql_query ('SET NAMES latin2');
         mysql_query('INSERT INTO `news` (`news`,`tytul`,`data`) VALUES ('.$news.','.$tytul.','.$data.');') or die(mysql_error());
    
     }  
    
}
?>
[PHP] pobierz, plaintext

Sprawdziłem u siebie, i działa.

Smoker

6.03.2009, 20:49:47

i napisz co CI pokazał...

Ps... po echo pojedyńczy cudzysłow ? a nie wyswietli wtedy calego input itd jako tekst zamiast to przerabiac jako html ?

nansss

6.03.2009, 20:50:49

Wyskoczyło debug 1

ale usunęło tytuł i favicon

Mephistofeles

6.03.2009, 20:54:22

Ja zawsze w PHP stosuję apostrofy, a cudzysłowy tylko do HTMLa (w SQLu takie koło tyldy

).

Smoker

6.03.2009, 20:55:28

czyli w zmiennej $_SESSION['haslo'] nie ma wartosci haslo :]

sproboj
if($_SESSION['haslo']=='haslo')

zamienic na:

if(strlen($_SESSION['haslo'])=='haslo')

Spawnm

6.03.2009, 20:56:14

Cytat(Smoker @ 6.03.2009, 20:49:47 )

i napisz co CI pokazał...

Ps... po echo pojedyńczy cudzysłow ? a nie wyswietli wtedy calego input itd jako tekst zamiast to przerabiac jako html ?

ktoś tu chyba nie zna podstaw php

różnica między ' a " jest taka że jak w " dasz zmienną to wyświetli ci jej wartość a w ' wyświetli co stringa (nazwę zmiennej)
html nie ma tu nic do rzeczy zawsze tak samo jest traktowany

-----------

ale usunęło tytuł i favicon

a gdzie ty deklarujesz w tym pliku <title> i ikonkę ? bo jakoś nie widzę

nansss

6.03.2009, 20:58:00

Zmieniłem na:

if(strlen($_POST["haslo"]=="haslo"))

i wyskoczył mi napis haslo

EDIT;
Ja tylko wysyłam kod php

Spawnm

6.03.2009, 20:58:58

pokaż jak wygląda teraz twój kod

Smoker

6.03.2009, 21:00:20

spawnm to nie kazdy jest idealny

ja zawsze uzywalem ' do tekstów a " gdy mialem jakis kod htmla :]ale najwazniejsze ze mi dzialalo

ps. ja tam zle napisalem ten kod... juz go popawilem... teraz sproboj

nansss

6.03.2009, 21:00:39

[PHP] pobierz, plaintext 
<?php
session_start();
echo $_SESSION['haslo'] ;
function connect_db()
{
    mysql_connect("host", "uzytkownik", "haslo") or die(mysql_error());
    mysql_select_db("nazwa") or die(mysql_error());
}
if(strlen($_POST["haslo"]=="haslo"))
 
{
    if(!isSet($_POST['news']))
 
    {
            echo "<form action=\"dodajnews.php\" method=\"post\">
                 
 
<input type=\"text\" name=\"tytul\" value=\"Tytuł newsa\" size=\"20\">
                 
 
                  <textarea name=\"news\" cols=\"20\" rows=\"20\">Treść newsa</textarea>
                 
 
                  <input type=\"submit\" value=\"Dodaj\">
            </form>";
    }
   
    else
    {
        $news = $_POST['news'];
        $tytul = $_POST['tytul'];
        $data = date('Y-m-d');
        connect_db();
        mysql_query ("SET NAMES latin2");
        mysql_query("INSERT INTO `news` (`news`,`tytul`,`data`) VALUES ('".$news."','".$tytul."','".$data."') ") or die(mysql_error());
   
    }  
   
}
?>
[PHP] pobierz, plaintext

Smoker

6.03.2009, 21:02:38

zamiast
if(strlen($_POST["haslo"]=="haslo"))
ma byc
if(strlen($_SESSION['haslo'])=='haslo')

ps ide zapalic bo zaraz nie wytrzymam :]

pyro

6.03.2009, 21:04:55

Cytat(Smoker @ 6.03.2009, 21:02:38 )

zamiast
if(strlen($_POST["haslo"]=="haslo"))
ma byc
if(strlen($_SESSION['haslo'])=='haslo')

ps ide zapalic bo zaraz nie wytrzymam :]

strlen() zawsze zwroci int mundrolu

// edit

ide zapalic bo zaraz z Toba nie wytrzymam

// edit 2

chociaz nie pale...

Spawnm

6.03.2009, 21:05:09

zamiast

[PHP] pobierz, plaintext 
<?php
if(strlen($_POST["haslo"]=="haslo"))
 
{
   if(!isSet($_POST['news']))
 
   {
?>
[PHP] pobierz, plaintext

daj:

[PHP] pobierz, plaintext 
<?php
$pass=trim($_POST["haslo"]);
if($pass=='haslo')
 
{
   if(!empty($_POST['news']))
 
   {
?>
[PHP] pobierz, plaintext

nansss

6.03.2009, 21:05:47

Nic to nie dało...
ehh

Spawnm

6.03.2009, 21:09:32

dziwne , powinno działać

a powiedz czy echo $_SESSION['haslo'] ; wyświetla ci haslo czy hasło ;]

jak haslo to się poddaje

Smoker

6.03.2009, 21:11:05

a tam ma byc $_POST['haslo'] a nie $_SESSION['haslo']

?

nansss

6.03.2009, 21:11:36

haslo

Spawnm

6.03.2009, 21:12:39

Cytat(Smoker @ 6.03.2009, 21:11:05 )

a tam ma byc $_POST['haslo'] a nie $_SESSION['haslo']

?

racja

daj:

[PHP] pobierz, plaintext 
<?php
$pass=trim($_SESSION["haslo"]);
if($pass=='haslo')
 
{
  if(!empty($_POST['news']))
 
  {
?>
[PHP] pobierz, plaintext

mój błąd

Smoker

6.03.2009, 21:13:33

to mam ostateczny pomysl :] skasuj caly kod z dodajnewsa.php i zrob tak... najpierw input w ktorym koles ma wpisac haslo i jesli bedzie to haslo admina to pokaz mu dodawanie postow a jak nie to nic ma nie pokazac...

jak chcesz to ja CI to napisze

nansss

6.03.2009, 21:15:21

Napisałbyś cały dodajnews?