Mysteriöses Problem - DB wird nicht geupdatet

[fanrpg]

Also ich habe eine Datei geschrieben die bis gestern problemlos funktioniert hat aber heute tut sie es nicht mehr!

Also ich habe geguckt und der Updatet nicht mehr die DB und wenn ich auf Absenden klicken dann leitet der mich auch nicht weiter auf die index.php sondern belässt es bei der Seite hier ohne up zu daten...

PHP-Code:

$sql = "SELECT * FROM `config`";
$query = mysql_query($sql) or die(mysql_error());
$row = mysql_fetch_assoc($query);
if(isset($submit_new)) {
$siteurlsql = "UPDATE `config` SET `sitename` = '".$_POST['newsitename']."', `siteurl` = '".$_POST['newsiteurl']."', `userwrite` = '".$_POST['newuserwrite']."'";
$siteurlquery = mysql_query($siteurlsql) or die(mysql_error());
echo "<meta http-equiv=\"refresh\" content=\"0; URL=index.php?userid=$_COOKIE[nemgengallerie]&username=$_COOKIE[nemgengallerieusername]\">";
}
?>
<form method="post">
<table border="1">
<tr><td>Seitenname: <input type="text" name="newsitename" value="<?= $row['sitename'] ?>"><br>
Seitenurl: (Wichtig für Uploadscript!!!)<input type="text" value="<?= $row['siteurl'] ?>" name="newsiteurl">&nbsp;&nbsp;<br>
Usern erlauben Bilder hochzuladen: <select name="newuserwrite">
<?
if($row['userwrite'] == "0" or $row['userwrite'] == "")
{
?>
<option value="0">Verbieten</option>
<option value="1">Erlauben</option>
<?
 }
if($row['userwrite'] == "1")
{
?>
<option value="1">Erlauben</option>
<option value="0">Verbieten</option>
<?
}
?>
</select>
<br>
<input type="submit" name="submit_new" value="Absenden">
</td></tr></table></form>

Hat jemand ne Idee?

[fanrpg]

Das oben hat sich erledigt hab ne komplett neue Datei gecodet!

Jetzt aber ne neue Frage ist das sinnvoll?

index.php

PHP-Code:

<?php
include("config.php");
include("header.php");
include("functions.php");

while(sites("sitename", "login"))
{
sites_url("login", sites_inhalt("login"))
}

admin_panel($_GET['adminname']);
?>

functions.php

PHP-Code:

function sites($orderby, $ordervalue)
{
$sql_sites = "SELECT * FROM `sites` ORDER BY '$orderby' = '$ordervalue'";
$query_sites = mysql_query($sql_sites);
while($row_sites = mysql_fetch_assoc($query_sites))
{
echo $row_sites['sitename'];
}
}

function sites_url($sitename, $inhalt)
{
if($_GET['site'] == "$sitename")
{
 echo $inhalt;
}

function sites_inhalt($site_name)
{
$sql_sites2 = "SELECT * FROM `sites` WHERE 'sitename' = '$site_name'";
$query_sites2 = mysql_query($sql_sites2);
while($row_sites2 = mysql_fetch_assoc($query_sites2))
{
echo $row_sites2['siteinhalt'];
}

function admin_panel($adminname)
{
$sql_admin = "SELECT * FROM `user` WHERE `adminrights` = '1' AND `username` = '$adminname'";
$query_admin = mysql_query($sql_admin);
$row_admin = mysql_fetch_assoc($query_admin);
if( $row_admin['adminrights'] == "1" ) {
echo "<a href=\"admin/\">Adminbereich</a>";
}
} 

Also ist das sinnvoll fast die ganze index.php in functions zu packen und die einfach in der index.php aufzurufen oder sowas sinnlos?

[Manni]

Ich bin das kein großer Fan von. Es hat den Nachteil, das man immer in zwei verschiedenen Dateien nache einenm Fehler suchen muss. Irgendwie lohnt es sich dann auch garnicht mehr, eine index.php zu schreiben
Es ggibt bei dir auch Funktionen, die egentlich nur zwei Zelen Code haben. Diese Funktionen sind imo nicht besonders sinnvoll. Du schreibst mehr Code und das ganze wird imo unübersichtlicher.
Was imo schon Sinn macht, sind Funktionen, die mehr Code haben und auch evtl. in mehreren anderen Dateien bzw. an unterschiedlichen Stellen im Code notwendig wären, wo also der Code unter Umständen mehrmals stehen müsste. Da spart man sich überflüssigen Code und erhäht durchaus die Übersichtlichkeit...

(BTW, der Code kann doch so garnicht funktionieren *verwirrt ist*

PHP-Code:

function sites_url($sitename, $inhalt)
{
if($_GET['site'] == "$sitename")
{
echo $inhalt;
} 

Die Bedingung wird bei dieser Funktion zB garnicht geschlossen.
Des weiteren ist diese Zeile nicht wirklich sinnvoll

PHP-Code:

while(sites("sitename", "login")) 

, weil die Funktion sites() keinen Rückgabewert hat...)

[fanrpg]

@Manni

der Code hat aber wie gewünscht funktioniert

[fanrpg]

So neues Problem

Also der Server meldet bei einer SQL Abfrage:

Warning: Wrong parameter count for mysql_query() in /var/www/web39/html/beta/submit_user.php on line 8

...

und das erscheint wenn ich $sql ausgeben lasse:

INSERT INTO `user` (`email`, `pw`) VALUES ('webmaster@fanrpg.gut4me.de', '692805')

...

Un dich frage mich was da falsch ist o.ô

btw. hier die Datei:

PHP-Code:

<?php
if( $_COOKIE['adminrpggxbeta'] == "1" )
{
if(isset($_POST['submit'])) {
$rand_pw = rand("3500", "1000000");
$sql = "INSERT INTO `user` (`email`, `pw`) VALUES ('".$_POST['email']."', '".$rand_pw."')";
echo $sql;
$query = mysql_query($sql) or die(mysql_error());
$message = "Sie haben sich bei rpggx.de registrieren lassen, 
ihr Account name ist: ".$_POST['email']."
Ihr Passwort ist: ".$rand_pw."
Es wird empfohlen das Passwort sicher aufzubewahren";
mail($_POST['email'], "Ihr Account bei rpggx.de", $message);
}
?>
<form method="post">
E-Mail: <input type="text" name="email"><br />
<input type="submit" name="submit">
<?
}
else
{
die("Keine Berechtigung User einzutragen");
}
?>

[Manni]

Das ist keine MySQL-Fehlermeldung, sondern eine von PHP. Du hast eine falsche Parameteranzahl bei mysql_query()
Das könnte daran liegen, das du keine Verbindung zur Datenbank herstellst bzw. keine offene MySQL-Server-Verbindung gefunden wird. Mysql_query() erwartet afaik 2 Parameter, wenn keine Datenbankverbindung gefunden wird...