Newsscript edit funk. [anfänger]

JuSTAx

Hallo zusammen^^
also ich lese mich grad ein bissien in PhP ein und versuche mit hilfe eines ebooks ein kleines newsscript zu machen.

news adden und löschen hab ich hinbekommen aber leider hängts editieren.
wenn ich auf die edit seite gehe zeigts mir das formular zwar an aber zeigt mir nicht den eintrag zum editieren.

PHP

<?php
require_once ('conf.php'); 
$db_link = @mysql_connect (MYSQL_HOST, MYSQL_BENUTZER, MYSQL_KENNWORT); 
if ( ! $db_link ) { 
die('keine Verbindung zur Zeit möglich - später probieren '); 
}
$db_sel = mysql_select_db( MYSQL_DATENBANK ) or die("Auswahl der Datenbank fehlgeschlagen");
//
function anzeige_daten_zum_bearbeiten ($id) { 
$sql = " SELECT * FROM news WHERE id = '$id' "; 
$db_erg = mysql_query( $sql ); 
if ( ! $db_erg ) { 
die('Ungültige Abfrage: ' . mysql_error()); 
} 
$zeile = mysql_fetch_array( $db_erg, MYSQL_ASSOC); 
echo '<form name="" action="" method="POST" enctype="text/html">'; 
echo '<p>Autor:<br />'; 
echo '<input type="text" name="name" value="'; 
echo $daten['name']; 
echo '" size="10" maxlength="10" />'; 
echo '</p>'; 
echo '<p>Titel:<br />'; 
echo '<input type="text" name="email" value="'; 
echo $daten['email']; 
echo '" size="50" maxlength="255" />'; 
echo '</p>'; 
echo '<p>Text:<br />'; 
echo '<textarea name="eintrag" rows="10" cols="80">'; 
echo $daten['eintrag']; 
echo '</textarea></p>'; 
echo '<input type="hidden" name="id" value="'; 
echo $daten['id']; 
echo '" />'; 
echo '<input type="hidden" name="vorgang" value="update" />'; 
echo '<input type="Submit" name="" value="&Auml;nderungen speichern" />'; 
echo '</form>'; 
echo '<hr />'; 
} 
if ( $_POST['vorgang'] == 'update' ) { update_daten ( $_POST['daten'] ); }
// Daten updaten 
function update_daten ( $daten ) { 
// hier sollte noch eine Validierung des Datums stattfinden
$sql = "UPDATE news SET 
`name` = '". $daten['name'] ."', 
`email` = '". $daten['email'] ."',  
`eintrag` = '". $daten['eintrag'] ."'
WHERE id = '". $daten['id'] ."' "; 
//
//
$db_erg = mysql_query( $sql ); 
if ( ! $db_erg ) { die("Ungültige Abfrage: $sql <hr />" . mysql_error()); 
} else { 
echo "<h2>News updatet</h2>"; 
} 
}
// Daten zum Ändern anzeigen 
if ( $_GET['vorgang'] == 'aendern' AND $_POST['vorgang'] <> 'update' ) { anzeige_daten_zum_bearbeiten ( $_GET['id'] ); }
?>
<body style="color: #FFFFFF; background-color: #000000">

Alles anzeigen

auf die seite komm ich über einen link -> Team%20Divinity/adm/newsedit.php?vorgang=aendern&id=58 also die ID steht im oberen link.

kann mir da jemand helfen?
wenn ich so was eingebe und auf speichern drücke steht da zwar News updatet aber weder auf der HP noch in der mysql db ist geändert :(^^

crAzywuLf

Diese Zeile:

PHP

if ( $_POST['vorgang'] == 'update' ) { update_daten ( $_POST['daten'] ); }

sieht so aus als müsste sie noch innerhalb dieser Funktion sein, also quasi so:

PHP

function anzeige_daten_zum_bearbeiten ($id)
{ 
    $sql = " SELECT * FROM news WHERE id = '$id' "; 
    $db_erg = mysql_query( $sql ); 
    if ( ! $db_erg ) 
    { 
        die('Ungültige Abfrage: ' . mysql_error()); 
    } 
    $zeile = mysql_fetch_array( $db_erg, MYSQL_ASSOC); 
    echo '<form name="" action="" method="POST" enctype="text/html">'; 
    echo '<p>Autor:<br />'; 
    echo '<input type="text" name="name" value="'; 
    echo $daten['name']; 
    echo '" size="10" maxlength="10" />'; 
    echo '</p>'; 
    echo '<p>Titel:<br />'; 
    echo '<input type="text" name="email" value="'; 
    echo $daten['email']; 
    echo '" size="50" maxlength="255" />'; 
    echo '</p>'; 
    echo '<p>Text:<br />'; 
    echo '<textarea name="eintrag" rows="10" cols="80">'; 
    echo $daten['eintrag']; 
    echo '</textarea></p>'; 
    echo '<input type="hidden" name="id" value="'; 
    echo $daten['id']; 
    echo '" />'; 
    echo '<input type="hidden" name="vorgang" value="update" />'; 
    echo '<input type="Submit" name="" value="&Auml;nderungen speichern" />'; 
    echo '</form>'; 
    echo '<hr />'; 
    if ( $_POST['vorgang'] == 'update' ) { update_daten ( $_POST['daten'] ); }
}

Alles anzeigen

Wieso benutzt du da überhaupt Funktionen?

EDIT:
Hä?
Mal ganz davon abgesehen woher nimmst du das ass. Array "daten"?
Vielleicht währe es sinnvoller erstmal den ganzen Code zu posten...

Unregistriert

hä?
hatte ich eigdl xD
also nochmal die edit datei:

PHP

<?php
require_once ('conf.php'); 
$db_link = @mysql_connect (MYSQL_HOST, MYSQL_BENUTZER, MYSQL_KENNWORT); 
if ( ! $db_link ) { 
die('keine Verbindung zur Zeit möglich - später probieren '); 
}
$db_sel = mysql_select_db( MYSQL_DATENBANK ) or die("Auswahl der Datenbank fehlgeschlagen");
//
function anzeige_daten_zum_bearbeiten ($id) { 
$sql = " SELECT * FROM news WHERE id = '$id' "; 
$db_erg = mysql_query( $sql ); 
if ( ! $db_erg ) { 
die('Ungültige Abfrage: ' . mysql_error()); 
} 
$zeile = mysql_fetch_array( $db_erg, MYSQL_ASSOC); 
echo '<form name="" action="" method="POST" enctype="text/html">'; 
echo '<p>Autor:<br />'; 
echo '<input type="text" name="name" value="'; 
echo $daten['name']; 
echo '" size="10" maxlength="10" />'; 
echo '</p>'; 
echo '<p>Titel:<br />'; 
echo '<input type="text" name="email" value="'; 
echo $daten['email']; 
echo '" size="50" maxlength="255" />'; 
echo '</p>'; 
echo '<p>Text:<br />'; 
echo '<textarea name="eintrag" rows="10" cols="80">'; 
echo $daten['eintrag']; 
echo '</textarea></p>'; 
echo '<input type="hidden" name="id" value="'; 
echo $daten['id']; 
echo '" />'; 
echo '<input type="hidden" name="vorgang" value="update" />'; 
echo '<input type="Submit" name="" value="&Auml;nderungen speichern" />'; 
echo '</form>'; 
echo '<hr />'; 
} 
if ( $_POST['vorgang'] == 'update' ) { update_daten ( $_POST['daten'] ); }
// Daten updaten 
function update_daten ( $daten ) { 
// hier sollte noch eine Validierung des Datums stattfinden
$sql = "UPDATE news SET 
`name` = '". $daten['name'] ."', 
`email` = '". $daten['email'] ."',  
`eintrag` = '". $daten['eintrag'] ."'
WHERE id = '". $daten['id'] ."' "; 
//
//
$db_erg = mysql_query( $sql ); 
if ( ! $db_erg ) { die("Ungültige Abfrage: $sql <hr />" . mysql_error()); 
} else { 
echo "<h2>News updatet</h2>"; 
} 
}
// Daten zum Ändern anzeigen 
if ( $_GET['vorgang'] == 'aendern' AND $_POST['vorgang'] <> 'update' ) { anzeige_daten_zum_bearbeiten ( $_GET['id'] ); }
?>

Alles anzeigen

und von dieser datei komm ich zum edit script^^:

PHP

<?php 
require_once ('conf.php'); 
// Daten zum Ändern anzeigen 
if ( $_GET['vorgang'] == 'aendern' ) { anzeige_daten_zum_bearbeiten ( $_GET['id'] ); }
$db_link = @mysql_connect (MYSQL_HOST, MYSQL_BENUTZER, MYSQL_KENNWORT); 
// Nutzen von Datenbank (Name ist hinterlegt in Konstante MYSQL_DATENBANK 
$db_sel = mysql_select_db( MYSQL_DATENBANK ) or die("Auswahl der Datenbank fehlgeschlagen"); 
$sql = " SELECT * FROM news "; 
$db_erg = mysql_query( $sql ); 
if ( ! $db_erg ) { die('Ungültige Abfrage: ' . mysql_error()); 
} 
while ($daten = mysql_fetch_array( $db_erg, MYSQL_ASSOC)) { 
// Aushabe der Daten 
echo '
<table class="style1" style="width: 100%">
	<tr>
		<td class="style3"><strong>Titel: '.$daten['email'].'</strong></td>
		<td class="style5">Autor: '.$daten['name'].' - '.$daten['datum'].'</td>
		<td class="style5"><a href="newsdel.php?id='. $daten['id'] .'">delete</a></td>
		<td class="style5"><a href="newsedit.php?vorgang=aendern&id='. $daten['id'] .'">edit</a></td> 
	</tr>
</table>';
echo '<p class="style4">Text: '.$daten['eintrag'].' </p>';
echo '</br>'; 
echo '</br>'; 
}
echo "</table>"; 
mysql_free_result( $db_erg ); 
?>

Alles anzeigen

hoffe nu is alles da xD

sry nochmal^^
und danke für deine antwort *gg

JuSTAx

ups sry war von mir.
vergessn einzuloggen.

<.<

@mod kannst du das in diesen beitrag reinpacken? xD
doppelposts stinken <.<

danköö

Pion

Sagt mal weil mir das wieder auffällt vor allem macht ja auch http://tut.php-quake.net/de/guestbook.html das so, für jede neue Zeile ein echo , finde ich sowas von unsinnvoll?

mfg

JuSTAx

wie gesagt ich lern das grad aus sonem anfänger ebook.
aber werd aus dem edit nicht schlau deswegen frag ich ja hier^^

crAzywuLf

Also ich hab mal alles zusammengeworfen, ist zwar nicht gestestet, aber schau mal ob es jetzt vll klappt:

PHP

<?php 
require_once ('conf.php'); 
$db_link = @mysql_connect (MYSQL_HOST, MYSQL_BENUTZER, MYSQL_KENNWORT);
$db_sel = mysql_select_db( MYSQL_DATENBANK ) or die("Auswahl der Datenbank fehlgeschlagen");


if ($_GET['vorgang'] == 'aendern' || $_POST['vorgang'] == 'update')
{
    if ($_POST['vorgang'] == 'update')
    {
//
// News Eintrag updaten
//
        $daten = $_POST;
        // Daten updaten 

        // hier sollte noch eine Validierung des Datums stattfinden ??? hä
        $db_erg = mysql_query("UPDATE `news` SET `name` = '". $daten['name'] ."', `email` = '". $daten['email'] ."', `eintrag` = '". $daten['eintrag'] ."' WHERE `id` = '". (int) $daten['id'] ."'"); 
        if (!$db_erg)
        {
            die("Ungültige Abfrage: " . mysql_error());
        }
        else
        { 
            echo "<h2>News updatet</h2>"; 
        }
        mysql_free_result($db_erg);
        unset($daten);
    }


//
// News Eintrag anzeigen zum editieren
//
    $id = $_GET['id'];
    $db_erg = mysql_query("SELECT * FROM `news` WHERE `id` = '" . (int) $id . "'"); 
    if (!$db_erg) die('Ungültige Abfrage: ' . mysql_error()); 
    $news_eintrag = mysql_fetch_array($db_erg, MYSQL_ASSOC);
    echo '
<form action="newsedit.php" method="POST"> 
<p>
    Autor:<br /> 
    <input type="text" name="name" value="' . $news_eintrag['name'] . '" size="10" maxlength="10" />
</p> 
<p>
    Titel:<br /> 
    <input type="text" name="email" value="' . $news_eintrag['email'] . '" size="50" maxlength="255" />
</p>
<p>
    Text:<br />
    <textarea name="eintrag" rows="10" cols="80">' . $news_eintrag['eintrag'] . '</textarea>
</p>
<input type="hidden" name="id" value="' . $news_eintrag['id'] . '" /> 
<input type="hidden" name="vorgang" value="update" />
<input type="Submit" name="" value="&Auml;nderungen speichern" /> 
</form> 
<hr />';
    mysql_free_result($db_erg);
}
else
{


//
// News anzeigen
//
    $db_erg = mysql_query("SELECT * FROM `news`"); 
    if (!$db_erg) die('Ungültige Abfrage: ' . mysql_error());
    while ($daten = mysql_fetch_array($db_erg, MYSQL_ASSOC))
    { 
        // Aushabe der Daten 
        echo '
<table class="style1" style="width: 100%">
    <tr>
        <td class="style3"><strong>Titel: ' . $daten['email'] . '</strong></td>
        <td class="style5">Autor: ' . $daten['name'] . ' - ' . $daten['datum'] . '</td>
        <td class="style5"><a href="newsdel.php?id=' . $daten['id'] . '">delete</a></td>
        <td class="style5"><a href="newsedit.php?vorgang=aendern&id=' . $daten['id'] . '">edit</a></td> 
    </tr>
</table>
<p class="style4">Text: ' . $daten['eintrag'] . ' </p>
<br/><br/>';
    }
    mysql_free_result($db_erg);
}


?>

Alles anzeigen

währ halt die newsedit.php

Afrael

Da musst du die gesamten $daten validieren. mysql_error() an User auszugeben sollte man außerdem vermeiden.

JuSTAx

ja hehe^^
das ist ja nur für mich um fehler zu sehen^^
kommt danach raus.^^

danke euch *gg

Edit: Klappt wunderbar!!! Super danke xD"
hätte aber nochne kleine frage^^
Also mein Addscript funzt auch^^ aber wenn ich die add seite aufrufe, wird automatisch eine leere mysql zeile in die db eingetragen
und ich weiss nit wie ich des beim aufrufen blocken kann

PHP

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">


<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type" />
<title>Ihr Name</title>
</head>


<body style="color: #FFFFFF; background-color: #000000">


</body>


</html>


<form name="" action="<?php echo $_SERVER['PHP_SELF']; ?>" 
method="POST" enctype="text/html"> 
<p>Ihr Name:<br /> 
<input type="text" name="name" value="" size="50" maxlength="150" /> 
</p>
<p>Titel:<br /> 
<input type="text" name="email" value="" size="50" maxlength="150" /> 
</p> 
<p>Eintrag:<br /> 
<textarea name="eintrag" rows="10" cols="50"></textarea> 
</p> 




<input type="Submit" name="" value="speichern" /> 
</form>

<?php 
mysql_set_charset('utf8');
///
require_once ('conf.php');
//
if ( $_POST['eintrag'] != "" ) { 
echo "<h2>Eintrag speichern</h2>";
} 


//SQL COMMANDS
$sql = " INSERT INTO news "; 
$sql .= " SET ";
$sql .= " name ='". $_POST['name'] ."', "; 
$sql .= " email ='". $_POST['email'] ."', "; 
$sql .= " datum ='". date("Y-m-d H:i:s") ."', ";
$sql .= " eintrag ='". $_POST['eintrag'] ."' ";


echo "<hr />SQL: $sql<hr />";
$db_link = @mysql_connect (MYSQL_HOST, MYSQL_BENUTZER, MYSQL_KENNWORT); 
if ( ! $db_link ) { 
// hier sollte dann später dem Programmierer eine 
// E-Mail mit dem Problem zukommen gelassen werden 
// die Fehlermeldung für den Programmierer sollte
// das Problem ausgeben mit: mysql_error() 
die('keine Verbindung zur Zeit möglich - später probieren '); 
}


$db_sel = mysql_select_db( MYSQL_DATENBANK ) or die("Auswahl der Datenbank fehlgeschlagen");
// ausführen des mysql-Befehls 
$db_erg = mysql_query( $sql ); 
if ( ! $db_erg ) { die('Ungültige Abfrage: ' . mysql_error());
 }
 exit;
  ?>

Alles anzeigen

phore

Du prüfst auch nicht, ob schon was eingegeben wurde. Dein Script wird also immer 2x ausgeführt: 1x beim Laden der Seite und 1x beim Speichern (= neuladen) der Seite. Prüfe also, ob das Formular überhaupt bearbeitet wurde, bevor du was speicherst:

PHP

<?php 
if (isset($_POST))
{
  mysql_set_charset('utf8');
  ///
  require_once ('conf.php');
  //
  if ( $_POST['eintrag'] != "" ) { 
  echo "<h2>Eintrag speichern</h2>";
  } 

  //SQL COMMANDS
  $sql = " INSERT INTO news "; 
  $sql .= " SET ";
  $sql .= " name ='". $_POST['name'] ."', "; 
  $sql .= " email ='". $_POST['email'] ."', "; 
  $sql .= " datum ='". date("Y-m-d H:i:s") ."', ";
  $sql .= " eintrag ='". $_POST['eintrag'] ."' ";

  echo "<hr />SQL: $sql<hr />";
  $db_link = @mysql_connect (MYSQL_HOST, MYSQL_BENUTZER, MYSQL_KENNWORT); 
  if ( ! $db_link ) { 
  // hier sollte dann später dem Programmierer eine 
  // E-Mail mit dem Problem zukommen gelassen werden 
  // die Fehlermeldung für den Programmierer sollte
  // das Problem ausgeben mit: mysql_error() 
  die('keine Verbindung zur Zeit möglich - später probieren '); 
  }

  $db_sel = mysql_select_db( MYSQL_DATENBANK ) or die("Auswahl der Datenbank fehlgeschlagen");
  // ausführen des mysql-Befehls 
  $db_erg = mysql_query( $sql ); 
  if ( ! $db_erg ) { die('Ungültige Abfrage: ' . mysql_error());
   }
   exit;
}
  ?>

Alles anzeigen

Dein Script ist so übrigens etwa so unsicher wie irgend möglich. Ich hoffe du informierst dich zuerst über PHP Sicherheit bevor du sowas ins Internet stellst.

JuSTAx

jop da hab ich schon reingeguckt^^
ich will erstmal das ass alles geht bevor ich sowas dann reinmache xD

hab nochn kleines member script gemacht mit session & rang check.. dürft das reichen?^^ xD

phore

Was ist mit den ganzen xD?

Ein Login reicht ja, wenn es jedoch so geschrieben ist, wie dein Newsscript, dann hab ich meine Zweifel ob es wirklich sicher ist.

JuSTAx

könntest du mir noch verraten wieso?^^

jaa.. dieses ganze und xD is ne krankheit.
hoffe des stört keenen

**edit: btw
nach was mussichn googlen wenn ich über einen klein 2 seiten öffnen will? <:(^^

phore

Naja weil das Newsscript nicht guter Code ist. Da hab ich meine Zweifel ob denn dein Login wirklich was taugt. Aber vielleicht täusch ich mich.

Zitat von JuSTAx

**edit: btw
nach was mussichn googlen wenn ich über einen klein 2 seiten öffnen will? <:(^^

Was heisst "über einen klein 2 seiten öffnen will?"?

JuSTAx

joa also hab das bei mir so gemacht das nachm einloggen die mit den benötigen rang die admin seite öffnen können.
Diese öffnet sich auf der index seite in einem kleinen frame xD

bei meiner logout funktion wird man nachm ausloggen auf die loginseite gebracht aber des adminpanel im frame ist dann noch offen...

man kann zwar nixmehr benutzen weil man nicht eingeloggt ist, aber ich hätt gern das dess dann weg geht xD

edit:
öhm wers im prinzip nicht mit meta machbar?

PHP

<?php 
// Wird ausgeführt um mit der Ausgabe des Headers zu warten. 
ob_start (); 


session_start (); 
session_unset (); 
session_destroy (); 
echo '<meta http-equiv="refresh" content="1" target="n1;url=navi-not.php"><center>...</center></meta>';


header ("Location: formular.php"); 
ob_end_flush (); 
?>

Alles anzeigen

aber da wird zwar dauernd ein refresh gemacht aber es öffnet sich nix in dem inlineframe

Tags