+- MyBB.de Forum (https://www.mybb.de/forum)
+-- Forum: Anpassungen (https://www.mybb.de/forum/forum-47.html)
+--- Forum: Plugin-Entwicklung (https://www.mybb.de/forum/forum-74.html)
+--- Thema: "Task" > SQL-Befehl funktioniert nicht (/thread-37547.html)
"Task" > SQL-Befehl funktioniert nicht - Gerti - 29.11.2020
Hallo Community,
habe mir eine Task-Datei erstellt aktiviert in welcher ich User der Gruppe "5" (soweit sie länger als 24 Stunden nicht aktiviert haben) wieder gelöscht werden sollen. Irgend etwas klappt mit dem Befehl nicht. Eine Aufgabenplanung welche die Task-Datei aufruft wurde selbstverständlich erstellt.
PHP-Code:
function delete_user_not_activated($task)
{
global $db;
// Löschen aller User welche ihren Account nicht binnen 24 Stunden nach Registrierung aktiviert haben
if ($usergroup = 5 && ($regdate < TIME_NOW-(60*60*24))) {
$sql = "DELETE FROM `mybb_users` WHERE usergroup = 5";
$mysqli->query($sql);
}
add_task_log($task, "Löschung der nicht aktivierten User (>246 Stunden) ausgeführt");
}
?>Nur:
PHP-Code:
DELETE FROM `mybb_users` WHERE usergroup = 5;
funktioniert in phpMyAdmin aber nicht im Task.
Was mache ich falsch?
RE: "Task" > SQL-Befehl funktioniert nicht - itsmeJAY - 29.11.2020
1. Was steht in usergroup drin?
2. Wieso vergleichst du mit einem = anstatt ==
RE: "Task" > SQL-Befehl funktioniert nicht - SvePu - 29.11.2020
Diese Option gibt es standardmäßig doch bereits
Schau mal in den ACP-Einstellungen unter "Benutzer bereinigen"
RE: "Task" > SQL-Befehl funktioniert nicht - Gerti - 29.11.2020
(29.11.2020, 16:17)itsmeJAY schrieb: 1. Was steht in usergroup drin?zu 1 = Inhalt: 5
2. Wieso vergleichst du mit einem = anstatt ==
zu 2 = auch versucht.
(29.11.2020, 16:25)SvePu schrieb: Diese Option gibt es standardmäßig doch bereitsDanke, das war die Lösung!
Schau mal in den ACP-Einstellungen unter "Benutzer bereinigen"
Hätte ich mir den Zeitaufwand ersparen können.
Unabhängig davon würden mich trotzdem die korrekten Befehlsaufbauten interessieren, man hat ja noch weitere Ideen.
RE: "Task" > SQL-Befehl funktioniert nicht - doylecc - 29.11.2020
(29.11.2020, 16:38)Gerti schrieb: Unabhängig davon würden mich trotzdem die korrekten Befehlsaufbauten interessieren, man hat ja noch weitere Ideen.
Auf den ersten Blick fehlt hier ein zweites = Zeichen:
PHP-Code:
if ($usergroup = 5
RE: "Task" > SQL-Befehl funktioniert nicht - itsmeJAY - 29.11.2020
Hatte ich oben schon erwähnt, hehe
RE: "Task" > SQL-Befehl funktioniert nicht - StefanT - 29.11.2020
Ehrliche Meinung: Der Code ist nicht zu retten. Die Variablen $usergroup, $regdate und $mysqli gibt es nicht, die Abfrage ist fehlerhaft und das Datenbank-Query unvollständig.
Zudem ist es ist äußerst unsauber direkt in der Tabelle mybb_users zu löschen, weil es in vielen anderen Tabellen, mehreren Caches und noch im Dateisystem (Avatare) Verweise auf diese gibt. Zum Vergleich hier der Code, der bei einer ordentlichen Lösung ausgeführt wird: https://crossreference.mybb.de/inc/datahandlers/user.php.source.html#l1490
RE: "Task" > SQL-Befehl funktioniert nicht - Gerti - 30.11.2020
Wie von euch erwähnt, genügt es nicht einfach den User inder Tabelle mybb_users zu löschen, da nach der Registrierung die User-ID in insgesamt drei Tabellen enthalten ist. Man müsste somit zuerst mittels einem Array die User-IDs der Gruppe 5 erfassen und dieselbigen dann in allen betroffenen Tabellen löschen.
Nachdem die Funktion, wie ich Dank eures Hinweises entdecken durfte, bereits in MyBB enthalten ist hat sich meine Anfrage erübrigt.
Besten Dank für die Hinweise, Tipps, und Anregungen.