Unterschiedliche Behandlung von ctype_digit() und strval() (PHP-Funktionen)

Sergeij_Molotow

Hallo liebe Gemeinde,

ich habe ein Problem mit einer PHP-Funktion, die ich in einem Seitentemplate von Wordpress benutze. Ich weiß, ich sollte die Frage in einem PHP-Forum stellen, aber vielleicht findet sich jemand, der mir auf Anhieb einen Tipp geben kann.

Ich habe ein Seitentemplate, an dessen Beginn ich einen REQUEST abfange. Hauptsächlich erfasse ich dort verschiedene IDs, die ich mit der GET-Methode an die gleiche Seite sende.

PHP

switch ($_SERVER['REQUEST_METHOD']) {
    case 'GET':
        $the_request = $_GET;
        switch ($the_request) {
            case isset($the_request['deleteuser_ID']):
                $deleteuser_ID = strip_tags(trim($the_request['deleteuser_ID']));
                if (ctype_digit(strval($deleteuser_ID)) == TRUE) {
                    if (user_exists_by_id($deleteuser_ID) == TRUE) {
                        $deleteuser_ID;
                    } else {
                        $deleteuser_ID = FALSE;
                    }
                } else {
                    $deleteuser_ID = FALSE;
                }
            case isset($the_request['canceluser_ID']):
                $canceluser_ID = strip_tags(trim($the_request['canceluser_ID']));
                if (ctype_digit(strval($canceluser_ID)) == TRUE) {
                    if (user_exists_by_id($canceluser_ID) == TRUE) {
                        $canceluser_ID;
                    } else {
                        $canceluser_ID = FALSE;
                    } 
                } else {
                    $canceluser_ID = FALSE;
                }
            case isset($the_request['message_ID']):
                $message_ID = (string) strip_tags(trim($the_request['message_ID']));
                if (ctype_digit(strval($message_ID)) == TRUE) { // Überprüft, ob der String eine Ziffer enthält.
                    if (user_exists_by_id($message_ID) == TRUE) { // Überprüft, ob die ID tatsächlich einem User gehört
                        if ($message_ID <= $message_count -1 AND  $message_ID >= 0) { // $message_count zählt ein Array mit  verschiedenen Nachrichten. Zur Reproduktion kann man es gegen "4"  tauschen.
                            $message_ID;
                        } else {
                            $message_ID = FALSE;
                        }
                    } else {
                        $message_ID = FALSE;
                    }
                } else {
                    $message_ID = FALSE;
                }
        }
    case 'POST':
        break;
}

Alles anzeigen

Das Problem befindet sich hier:

PHP

if (ctype_digit(strval($message_ID)) == TRUE)

Nach mehreren Tests, mit var_dump und manuellem Debugging, gibt mir diese Überprüfung - trotz das in der URL eine Zahl für diese Variable steht - ein FALSE aus. Bei allen anderen Überprüfung klappt es.

Die URL kommt von wp_redirect(), etwas später aus dem Skript. Zum Beispiel hier:

PHP

if (current_user_can('manage_options') && $deleteuser_ID != FALSE && $assignuser_ID != FALSE) {
    $user_deleted = FALSE;
    $userdata = get_user_meta($deleteuser_ID);
    // Documentation: http://codex.wordpress.org/Function_Reference/wp_delete_user
    require_once(ABSPATH.'wp-admin/includes/user.php' );
    $user_deleted = wp_delete_user($deleteuser_ID , $assignuser_ID);
    // Documentation: http://codex.wordpress.org/Function_Reference/get_page_link
    $page_url = get_page_link(get_the_ID());
    // Documentation: http://codex.wordpress.org/Function_Reference/wp_redirect
    if ($user_deleted == TRUE) {
         wp_redirect($page_url.'?message_ID=1&firstname='.$userdata['first_name'][0].'&lastname='.$userdata['last_name'][0]);
        exit;
    } else {
        wp_redirect($page_url.'?message_ID=0');
        exit;
    }

Alles anzeigen

Ich bin relativ unerfahren damit, bin also für fachliche Hinweise hoffen, die mir das Problem erklären... Ich hoffe, das Problem kannst du reproduzieren. Falls du mehr Infos brauchst, lass es mich bitte wissen.

Ich sitze schon seit 4 Stunden dran und komme nicht weiter...

Danke für deine Hilfe im Voraus.

Gruß
Sergeij

Werbepost

Hallo!

Wenn du gerade an deiner Website arbeitest oder dein aktuelles Hosting überdenkst: Wir betreiben mit NetzLiving eine Hosting-Plattform, die speziell auf Performance, Sicherheit und einfache Verwaltung ausgelegt ist.

✔️ Schnelle Ladezeiten (optimiert für WordPress & Co.)
✔️ Deutsche Server & DSGVO-konform
✔️ Persönlicher Support (kein 0815-Ticket-System)

Mehr erfahren

Melewo

So würde ich das nicht schreiben, eher so:

PHP

if (isset($_GET["message_ID"]) and !empty($_GET["message_ID"])) {


    $the_request_id = trim($_GET["message_ID"];
    $the_request_id = preg_replace("/[^0-9]/", "", $the_request_id);
}

Mit

PHP

add_filter( 'query_vars', 'fuege_parameter_hinzu' );

und einer add_rewrite_rule brauchtest Du die auch nicht mehr als GET-Parameter in einem QueryString zu übergeben, sondern als Permalink. Beispiel:

http://www.coder-welten.com/die-wp-rewrite…nd-umschreiben/

Melewo

Deine Problem-Codezeile ist aber richtig, wenn das Ergebnis nicht wie erwartet, so liegt ein anderer Fehler vor:

PHP

<?php error_reporting(E_ALL);
/**
 * Seite unter http://localhost/a.php speichern
 * Seite mit http://localhost/a.php?message_ID=22 aufrufen
 */ 
$the_request = $_GET;


echo gettype($the_request['message_ID'])."<br>\n";   
// liefert string


echo strval($the_request['message_ID'])."<br>\n";   
// liefert 22


echo gettype(strval($the_request['message_ID']))."<br>\n";   
// liefert string


echo is_string(strval($the_request['message_ID']))."<br>\n";   
// liefert 1


echo ctype_digit(strval($the_request['message_ID']))."<br>\n";   
// liefert 1


if (ctype_digit(strval($the_request['message_ID']))) 
    echo "True?<br>\n";
else
    echo "False?<br>\n";
// liefert True?


if (ctype_digit(strval($the_request['message_ID'])) == true) 
    echo "True?<br>\n";
else 
    echo "False?<br>\n";
// liefert True?


/* Ausgabe:


string
22
string
1
1
True?
True?
*/
?>

Alles anzeigen

r23

PHP

$message_ID = (string) strip_tags(trim($the_request['message_ID']));
                if (ctype_digit(strval($message_ID)) == TRUE) { // Überprüft, ob der String eine Ziffer enthält.

grummle ist dein code grausam zu php

PHP

if (isset($_GET['message_ID']) && is_numeric($_GET['message_ID'])) {

hth

Sergeij_Molotow

Hallo Melewo und r23,

r23: Bezüglich des Codes: ich steck ja grad in der Entwicklung. Kannst du mir noch sagen, was grausam ist? Ich habe so eine Ahnung, was du meinst, würde es aber genau wissen wollen.
Is_numeric akzeptiert auch Kommazahlen (lt. PHP-Doku), was ich als ID niemals übergeben werde. Ich will auf positive Ganzzahlen überprüfen, deshalb auch der Umweg über strval(). :)

Melewo: Ok, also bei mir ist die Message ID entweder 0,1,2 oder 3 und er liefert mir nur(!) bei dieser Überprüfung trotz Zahl ein False. Ich versteh das nicht, weil bei den anderen ID-Prüfungen, die die ID mit der selben Methode übergeben bekommen funktioniert es.

Also ich sehe bei euch eher die Tendenz IF-Schleifen zu benutzen. Ist das Switch deshalb eher ungeeignet?

Danke an euch!
Gruß
Sergeij

Sergeij_Molotow

Ich vermute den Fehler ganz ehrlich auch eher bei wp_redirect(). Denn die anderen GET-Variablen (canceluser_ID etc.) werden über Links gesendet, während ich die Message_ID mit wp_redirect() übergebe... :/

Gruß
Sergeij

r23

Zitat von Sergeij_Molotow

Bezüglich des Codes: ich steck ja grad in der Entwicklung. Kannst du mir noch sagen, was grausam ist?

man nimmt nichts - rein gar nichts - ungeprüft an

Beispiel
[COLOR=#000000][COLOR=#007700][/COLOR][COLOR=#0000BB]$the_request [/COLOR][COLOR=#007700]= [/COLOR][COLOR=#0000BB]$_GET[/COLOR][COLOR=#007700];[/COLOR][/COLOR]

Die OWASP Top 10 ist mittlerweile anerkannter Branchenstandard und eine der bekanntesten Übersichten über Sicherheitsschwachstellen von Webanwendungen

OWASP Top 10
http://blogrettung.de/allgemein/owas…uch-in-deutsch/

Zitat von Sergeij_Molotow

Is_numeric akzeptiert auch Kommazahlen (lt. PHP-Doku), was ich als ID niemals übergeben werde. Ich will auf positive Ganzzahlen überprüfen

Dafür gibt es dann is-int in php...

http://www.php.net/manual/de/function.is-int.php

Sergeij_Molotow

Hallo r23,

vielen Dank für die Hinweise. Ich werde die Punkte mal ändern.

Is_int hatte ich auch schon im Visir, war mir nur unsicher, weil es dem Variablentyp prüft, aber nicht ganz genau den Inhalt (was ctype_digit() in Verbindung mit Strval() macht).

Das $_Request wird dann bald aus meinem Code verschwinden... :)

Gruß
Sergeij

Sergeij_Molotow

Leute,

ich habe meinen Fehler vom Startpost gefunden. Und zwar steckt er hier

PHP

if (user_exists_by_id($message_ID) == TRUE)

Fuck Copy & Paste! :) Ich lasse die Message ID darauf prüfen, ob es einen passenden User dafür gibt. :) Klar, dass das ab und zu ein False ergibt... :)

Gruß
Sergeij

Willkommen! Melden Sie sich an oder registrieren Sie sich.

Unterschiedliche Behandlung von ctype_digit() und strval() (PHP-Funktionen)

Jetzt mitmachen!