[DEV] SchockCMS - Fanseiten CMS [DEV]

Gefällt mir

Bis aufs Housekeeping gefällt es mir nicht so.

Ich finde es gut mach weiter so und hau rein !

Wichtig ist dabei zu sagen, dass ich dir den Register und Loginsystem als du noch nicht falsch warst, programmiert habe. Lieber SchokoBrunnen.

Find ich super. Weiter so

Wieso sollten wir uns via. PM melden, um nur einige Snippets vom Code zusehn?
Könntest du nicht einfach einige Snippets aus dem Code zeigen?

Gefällt mir soweit, bis auf das Adminpanel. Ich würde mir eher sowas wie ADMINLte nehmen.
Screenshot at 15:54:14 | ScreenCloud

Zitat von Acid

weil der Code schrecklich aussieht.

Es ist doch besser, wenn wir dir eben Tipps geben oder nicht?

Gefällt mir sehr, weiter so.

Zitat von Acid

Okey.

Ich würde das mysql_* weg lassen, lieber auf MySQLi oder PDO umsteigen.
Ich verstehe nicht wieso du count($_POST) > 0 machst, wenn du isset($_POST) hast.

So würde ich eher den Code gestalten.

$ban_user = mysql_real_escape_string($_POST['user']);$ban_grund = mysql_real_escape_string($_POST['reason']);
if(isset($_POST['submit'])) {	if(empty($_POST['user'] || empty($_POST['reason']) {		echo "Bitte fülle alle Felder aus";	} else {		$update_ban = mysql_query("UPDATE user SET ban_grund = '".$ban_grund." WHERE name = '".$ban_user."');		$update_status = mysql_query("UPDATE user SET status = '0' WHERE name = '".$ban_user."'");		echo '<div class="successful">Die Einstellungen wurden erfolgreich geändert.</div>';	}}

Zitat von Madly

Ich würde das mysql_* weg lassen, lieber auf MySQLi oder PDO umsteigen.
Ich verstehe nicht wieso du count($_POST) > 0 machst, wenn du isset($_POST) hast.

So würde ich eher den Code gestalten.

PHP

$ban_user = mysql_real_escape_string($_POST['user']);$ban_grund = mysql_real_escape_string($_POST['reason']);
if(isset($_POST['submit'])) {	if(empty($_POST['user'] || empty($_POST['reason']) {		echo "Bitte fülle alle Felder aus";	} else {		$update_ban = mysql_query("UPDATE user SET ban_grund = '".$ban_grund." WHERE name = '".$ban_user."');		$update_status = mysql_query("UPDATE user SET status = '0' WHERE name = '".$ban_user."'");		echo 'Die Einstellungen wurden erfolgreich geändert.';	}}

Jop, vor allem da MySQL in der neuen PHP Version als "veraltet" gilt.

Zitat von Acid

Ich möchte vorerst mit mysql_ arbeiten, komme damit besser zurecht. Sobald ich dann Fortschritte in PHP mache werde ich auch zu mysqli umsteigen

Du solltest gleich mit mysqli anfangen, ist viel viel einfacher, schneller & es macht mehr "spaß" mit zu coden.

Zitat von Acid

Ich möchte vorerst mit mysql_ arbeiten, komme damit besser zurecht. Sobald ich dann Fortschritte in PHP mache werde ich auch zu mysqli umsteigen

Jedem das seine. Am besten schaust du mal nebenbei nach MySQLi und programmierst noch mit MySQL. Nur das du dich schon einmal darauf vorbereiten kannst.

Schau mal, waechsle doch lieber zu PDO anstatt dieses bloede MySQL zu verwenden. Schon nur alleine "prepared statement" hilft dir gegen SQL Injection. Das heisst du musst nichtmehr die Eingaben von Inputs escapen (mysq_real_escape_string). Ausserdem ist PDO viel staerker Aber egal... War ja nur ein Tipp. P.s sieht gut aus und fuer PDO muss man bischen OOP koennen.