Übergabe eines Custom Fields aus Datenbank in eigenes Feld

jannikjohn98

Hallo zusammen,

ich benötige derzeit in einem eigenen Plugin Hilfe.
Und zwar habe ich in der WordPress Datenbank in der Tabelle prefix.usermeta den meta_key "checksumme". Dort ist beim Benutzer der Wert 1 eingetragen, mithilfe dieser 1 soll im Plugin eine if-Funktion gesteuert werden. (Screenshot des Datenbankeintrags habe ich angehängt). Der richtige Benutzer soll anhand der User-ID erkannt werden.

Der Plugincode an der Stelle sieht wie folgt aus:

/* Get the current user's ID */
add_action('init','getUserID');
function getUserID() {
$user_id = get_current_user_id();
}

function databaseconnect()
{
global $wpdb;
$results = $wpdb->get_var( $wpdb->prepare("SELECT meta_value FROM mittag_rplusermeta WHERE meta_key = \'checksumme\' AND user_id = %d", $user_id )) ;
}

Danach soll mit if($results == 1 ) eine Aktion ausgeführt werden (diese funktioniert bereits).

Der Übergabewert in $results sollte meiner Meinung nach dem Wert 1 entsprechen, aber es greift nicht.

Über eine Idee würde ich mich freuen, Danke im Voraus.

Gruß Jannik

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

wp-entwickler

KLar greift es nicht, weil die variable aus der einen Funktion nicht in der anderen verfügbar ist.
Aber der erste hook ist doch sowieso unnötig.
Nutz doch gleich get_current_user_id() in der zweiten funktion:

Code

function databaseconnect()
{
global $wpdb;
$results = $wpdb->get_var( $wpdb->prepare("SELECT meta_value FROM mittag_rplusermeta WHERE meta_key = \'checksumme\' AND user_id = %d", get_current_user_id())) ;
}

mensmaximus

Code

$checksum = get_user_meta( get_current_user_id(), 'checksumme' , true );

Eigene SQL-Selects sollte man solange vermeiden, solange es eine native WordPress Funktion gibt.

jannikjohn98

Hallo,

danke für beiden Antworten. Ich habe beide Varianten ausprobiert, leider auch hier wieder keine Reaktion.

Die Abfrage im phpMyAdmin (sprich SQL Query) gibt den Wert 1 aus, 1 ist hier ein int falls dies relevant ist. Dies scheint also im Prinzip zu funktionieren.

Meine Vermutung ist, dass er die aktuelle User-ID nicht bekommt. Wenn ich die if-Funktion direkt über die User-ID laufen lasse, bspw.

PHP

if ( $user_id == 1)
{


}

greift es auch nicht.

mensmaximus

Hast Du eventuell eine alte PHP Version?

Code

$user_id = get_current_user_id();
$checksum = get_user_meta( $user_id, 'checksumme' , true );
if ( 1 == intval( $checksum ) ) {
     # do something
}

Das geht natürlich nur, wenn der Benutzer angemeldet ist. Aber das sollte ja klar sein.

jannikjohn98

Die PHP-Version ist aktuell.

Selbstverständlich ist der Benutzer angemeldet, in diesem Fall der Administrator (User-ID 1).

Der oben genannte Code löst leider immer wieder einen fatalen Fehler beim Aktivieren des PlugIns aus, allerdings ohne Fehlerursache. Das würde ich mir vorweg nochmal kurz anschauen.

mensmaximus

Zitat von jannikjohn98

fatalen Fehler beim Aktivieren des PlugIns aus, allerdings ohne Fehlerursache.

Das gibt es nicht. Das php error.log zeigt immer seinen Stacktrace bei einem fatalen Fehler..

jannikjohn98

Fehler ist weg, nur eine fehlende Klammer in der if-Funktion.

mensmaximus

Ich hatte die Klammer im Beitrag ergänzt. Du hast die Funktion wahrscheinlich aus der E-Mail kopiert.

jannikjohn98

Ganz genau.

PHP

$user_id = get_current_user_id();
$checksum = get_user_meta( $user_id, 'checksumme' , true );
if ( 1 == intval( $checksum ))
{   
    function custom_shop_page_redirect()
        {
            if(is_cart() || is_checkout())
            {
                echo('<script language="Javascript">alert("Test");</script>');
                exit();
            }
        }
        add_action( 'template_redirect', 'custom_shop_page_redirect' );
}

Alles anzeigen

So ist der Code nun drin, ebenfalls wieder ohne Reaktion.
Bei Zugriff auf den Warenkorb oder auf die Kasse müsste per JavaScript der Alert auslösen. In der Theorie.

mensmaximus

Das funktioniert so nicht. die Abfrage der User ID liegt ja außerhalb der Funktion. Und dann ist ja noch die Frage wann die Anzeige erfolgen soll und was passieren soll. Der template_redirect action hook hat ja eine spezielle Funktion. Erzähle uns doch einmal die gesamte Story. Dann kann man auch richtig helfen.

Und noch ein Tipp: Alert ist Pfui. Schreib Debug Informationen in die Browserkonsole (console.log)

jannikjohn98

Also das Prinzip ist wie folgt:

Im Benutzerprofil ist ein Feld hinterlegt, wo die Checksumme 0 oder 1 eingetragen werden kann.
Die Benutzer mit der Prüfsumme 1 sollen kein Zugriff auf den Warenkorb und die Kasse, also prinzipiell auf die Bestellfunktion bekommen. Dieses soll zusätzlich nur auf die Tage Dienstag und Donnerstag greifen.

Wenn die Bedingungen passen, also es ist Dienstag oder Donnerstag und die Prüfsumme ist gleich 1, soll eine Weiterleitung oder ein Pop-Up-Fenster erscheinen mit einer Information über die Sperre.

Hier nochmal der gesamte Code:

PHP

<?php


defined( 'ABSPATH' ) || exit();


/* Festlegen der Zeitzone und Speichern des UNIX-Zeitstempels in die Variable $timestamp */
date_default_timezone_set('Europe/Berlin');
$timestamp = time();


/* Festlegen des aktuellen Tages */
$tag = date("w");


$user_id = get_current_user_id();
$checksum = get_user_meta( $user_id, 'checksumme' , true );
if ( 1 == intval( $checksum ))
{   
    if ( $tag == 2 || $tag == 4 )
    {
        function custom_shop_page_redirect()
            {
                if(is_cart() || is_checkout())
                {
                echo '<script language="javascript">';
                echo 'alert("Shop ist gesperrt!")';
                echo '</script>';
                }
            }
            add_action( 'template_redirect', 'custom_shop_page_redirect' );
    }
}
?>

Alles anzeigen

mensmaximus

Wie ich schon sagte, eine Abfrage außerhalb der Funktion geht nicht. Das ganze gehört in eine Klasse. Dann kann man die User_Id im Constructor auslesen und für die aktuelle Session speichern. Im Anschluss kann man dann aller weiteren notwendigen Schritte einleiten (Bestellbutton entfernen, Zugriff auf Warenkorb und Kasse entfernen, usw.)

Mehr kann ich leider nicht mehr helfen, da ich gestern eine Anfrage erhalten habe wo es genau darum geht, die Bestellfunktion am Dienstag und Donnerstag zu ermöglichen. Das mag Zufall sein, aber ich muss davon ausgehen, dass es sich genau um diese Anfrage handelt.

jannikjohn98

Danke sehr für den Lösungsansatz, das werde ich mir morgen dann genauer anschauen.
Hier wird es sich tatsächlich um eine Zufall handeln, aber das ist kein Problem. Danke nochmals und einen schönen Abend noch.

r23

Zitat von jannikjohn98

Wenn die Bedingungen passen, also es ist Dienstag oder Donnerstag und die Prüfsumme ist gleich 1, soll eine Weiterleitung oder ein Pop-Up-Fenster erscheinen mit einer Information über die Sperre.

eine Funktion hat leider nichts in einer If Abfrage verloren. Schreibe deine Funktion ausserhalb von deiner IF Abfrage

PHP

function custom_shop_page_redirect()
            {
                if(is_cart() || is_checkout())
                {
                echo '<script language="javascript">';
                echo 'alert("Shop ist gesperrt!")';
                echo '</script>';
                }
            }

Bitte im PHP Handbuch nachlesen

https://www.php.net/manual/de/functions.user-defined.php

Wenn deine Bedingungen erfüllt sind, kannst du deine Funtion aufrufen

PHP

if ( 1 == intval( $checksum ))
{
    if ( $tag == 2 || $tag == 4 )
    {
       custom_shop_page_redirect();
    }
 }

Script jetzt nicht geprüft...

wp-entwickler

1. ist "checksumme" ein denkbar dämlicher Name für deinen user meta eintrag, weil es hier 1. nicht um eine checksum geht und der Name einfach nichts aussagt.

2. ist die user anmeldung wahrscheinlich in deiner plugin-datei noch gar nicht durch (ich gehe jetzt mal davon aus dass der code aus deiner plugin-datei ist) und deswegen wird dort get_current_user_id() nicht funktionieren.
Du brauchst einen passenden hook.
Bau doch einfach den check auf die user meta IN die funktion im hook ein, also "custom_shop_page_redirect".
Du machst es dir unnötig kompliziert,

3. Solltest du die basics des debuggens lernen, damit du mal weisst, warum dein code nicht funktioniert.

jannikjohn98

Danke nochmals für die vielen Tipps.

Habe es jetzt wie folgt hinbekommen:

PHP

efined( 'ABSPATH' ) || exit();


/* Festlegen der Zeitzone und Speichern des UNIX-Zeitstempels in die Variable $timestamp */
date_default_timezone_set('Europe/Berlin');
$timestamp = time();


/* Festlegen des aktuellen Tages */
$tag = date("w");


//Weiterleitung Dienstag und Donnerstag wenn Prüfsumme = 1
if ( $tag == 2 || $tag == 4 ) {
            function custom_shop_page_redirect() {
                $user_id = get_current_user_id();
                $checksum = get_user_meta( $user_id, 'checksumme' , true );
                if ( 1 == intval( $checksum ) ) {
                    if ( is_cart() || is_checkout() ) {
                    $link = "https://www.google.de";
                    wp_redirect( $link );
                    exit();
                    }
                }
            }
add_action( 'template_redirect', 'custom_shop_page_redirect' );
}

Alles anzeigen

Die URL google.de ist nur Platzhalter.

b3317133

Am Rande bemerkt, die Nutzung von [FONT=Courier New]intval()[/FONT] ist ggf. zu überdenken, hier ein paar Beispiele, die alle [FONT=Courier New]1[/FONT] ergeben:

PHP

$checksum = 1;
var_dump( intval( $checksum ) );


$checksum = 1.123;
var_dump( intval( $checksum ) );


$checksum = "1";
var_dump( intval( $checksum ) );


$checksum = "1.456";
var_dump( intval( $checksum ) );


$checksum = array(true);
var_dump( intval( $checksum ) );


$checksum = true;
var_dump( intval( $checksum ) );


$checksum = array(false, true);
var_dump( intval( $checksum ) );

Alles anzeigen

mensmaximus

Zitat von b3317133

Am Rande bemerkt, die Nutzung von [FONT=Courier New]intval()[/FONT] ist ggf. zu überdenken, hier ein paar

Laut TO gibt es nur die Werte 0 und 1. intval() ist daher genau der richtige Weg.

b3317133

Wenn es nur 0 oder 1 gäbe, bräuchte man intval() gar nicht... beim Tagesvergleich wird auch String mit int durcheinandergeworfen, dort scheint es ja zu klappen.

Wenn man ein Custom Field als "bool" missbrauchen will, sollte man eher [FONT=Courier New]!empty()[/FONT] verwenden.

Willkommen! Melden Sie sich an oder registrieren Sie sich.

Übergabe eines Custom Fields aus Datenbank in eigenes Feld

Jetzt mitmachen!