[OOP] WBB-Plugin

Monday · 13. Februar 2015

Ich arbeite derzeit an einem WBB-Plugin, einem PokeBBCode.
Wiefolgt sind meine BBCode-Klasse aus:

PHP

<?php
require_once(WCF_DIR.'lib/data/message/bbcode/BBCodeParser.class.php');
require_once(WCF_DIR.'lib/data/message/bbcode/BBCode.class.php');


class POKEBBCode implements BBCode {


    public function getParsedTag($openingTag, $content, $closingTag, BBCodeParser $parser) {
        if ($parser->getOutputType() == 'text/html') {
            // remove linebreaks
            $content = str_replace("<br />", "", $content);
            $content = StringUtil::encodeHTML($content);            

            $db = WCF::getDB();
            $numbersql = "SELECT * FROM `pokedex` WHERE name_de = '$content'";
            $numberquery = $db->query($numbersql);
            $number = $numberquery->fetch_object();

            return '<img src="www.floinkboard.cwsurf.de/wcf/images/pkmn_icons/'.$number->id.'.gif">';
        }
        else if ($parser->getOutputType() == 'text/plain') {
            return WCF::getLanguage()->get('wcf.bbcode.poke.text', array('$content' => $content));
        }        
    }
}
?>

Alles anzeigen

Allerdings erkennt er den query();-Befehl nicht und deswegen kommt folgende Meldung:

Zitat

Fatal error: Call to undefined method MySQLDatabase::query() in /home/www/cwcity/hosting/f/l/floinkboard/htdocs/wcf/lib/data/message/bbcode/POKEBBCode.class.php on line 15

#

Wie kann ich das ersetzen bzw. was mache ich falsch?

Nuffin · 13. Februar 2015

Die Methode, nach der du suchst, heißt sendQuery, was du mit Google in 5 Minuten selbst hättest rausfinden können. Ansonsten sieht der Code halbwegs okay aus.

Allerdings solltest du den $content nicht für HTML maskieren, sondern für SQL. Niemand hindert mich daran ';DROP TABLE pokedex;-- als Pokemon-Namen einzutragen, was einfach die Tabelle wegschmeißen würde.

Monday · 13. Februar 2015

Zitat

Fatal error: Invalid SQL: SELECT * FROM `pokedex` WHERE name_de = 'Bisasam'

Bei folgendem Code:

PHP

<?php
require_once(WCF_DIR.'lib/data/message/bbcode/BBCodeParser.class.php');
require_once(WCF_DIR.'lib/data/message/bbcode/BBCode.class.php');


class POKEBBCode implements BBCode {


    public function getParsedTag($openingTag, $content, $closingTag, BBCodeParser $parser) {
        if ($parser->getOutputType() == 'text/html') {
            // remove linebreaks
            $content = str_replace("<br />", "", $content);
            $content = StringUtil::encodeSQL($content);            

            $db = WCF::getDB();
            $numbersql = "SELECT * FROM `pokedex` WHERE name_de = '$content'";
            $numberquery = $db->sendQuery($numbersql);
            $number = $numberquery->fetch_object();

            return '<img src="www.floinkboard.cwsurf.de/wcf/images/pkmn_icons/'.$number->id.'.gif">';
        }
        else if ($parser->getOutputType() == 'text/plain') {
            return WCF::getLanguage()->get('wcf.bbcode.poke.text', array('$content' => $content));
        }        
    }
}
?>

Alles anzeigen

Was Zeile 11 betrifft, bin ich mir nicht sicher ob es diese Funktion wirklich gibt^^

Nuffin · 14. Februar 2015

Wenn es die Methode in Zeile 11 nicht gäbe, müsstest du eine Fehlermeldung ähnlich der aus dem Startpost enthalten. Inwiefern das SQL falsch ist, kann ich ohne genauere Kenntnis des Datenbankschemas nicht sagen, für mich sieht das OK aus. Überprüf mal, ob die Tabelle tatsächlich exakt so heißt, und nicht evtl. anders (z.B. "pokemon" o.ä.), und mach das Selbe für den Namen der Spalte.

**TimWolla** · 14. Februar 2015

Zitat von Nuffin

was einfach die Tabelle wegschmeißen würde.

Nein. PHP unterstützt schon seit Jahren nur eine Abfrage pro Abfrage. Semikola haben - aus Sicherheitsgründen - keine besondere Bedeutung in PHP. Der üblichen Verdächtige ist eher nen UNION ALL, um Daten aus anderen Tabellen zu suchen.

Zitat von Nortius

Bei folgendem Code:

Die Fehlermeldung ist ganz offensichtlich nicht vollständig, da unter anderem der Stacktrace fehlt.

Zitat von Nortius

Was Zeile 11 betrifft, bin ich mir nicht sicher ob es diese Funktion wirklich gibt^^

Nein, gibt es nicht. Genau so wenig wie $numberquery ein Objekt ist, sondern ein Integer. Und darauf kannst du natürlich keine Methoden ausführen. Meinst du, dass du mit "Raten" ans Ziel kommst?

Monday · 14. Februar 2015

Es war nicht die ganze Fehlermeldung, dachte aber diese Zeile wäre genug.
Und ja Sie heißt genauso.
bin leider Bis morgen abend nicht in der Nähe eines pcs deswegen kann ich nix hochladen etC. könntet aber hier die Dateien anschauen
www.floinkboard.cwsurf.de/de.floinkboard.poke.bbcode.tar.gz

Monday · 15. Februar 2015

Also, wenn du es weisst dann kannst du mir auch gleich sagen, was ich besser machen soll, denn ich habe schon stundenlang gegoogled und ihr müsst nicht denken ich hätte nicht vorher versucht diese Fehler selbstständig zu beseitigen.
Im Anhang jetzt die komplette Fehlermeldung falls noch jemand denkt er könnte wirklich produktiv helfen.

Marv · 15. Februar 2015

Steht da doch was das Problem ist.
In der Datenbank sagecrown gibt es die Tabelle pokemon nicht.
Bevor du dich über andere beschwerst erstmal an die eigene Nase fassen.

Monday · 15. Februar 2015

Und alle fragen sich, wie zur Hölle kommt der auf die Datenbank auf sagecrowne?
Diese Seite existiert bei meinem Hoster, aber das ist ein völlig anderer Webspace und ich habe nie irgendwelche Daten von diesem im WBB angegeben?

Marv · 15. Februar 2015

Offensichtlich ja doch.
Bin nicht großartig mit dem WCF vertraut, aber

PHP

$db = WCF::getDB();

Klingt für mich danach dass hier die Datenbankverbindung hergestellt wird.
Dementsprechend musst du irgendwo diese DB angegeben haben, die kommt ja nicht aus dem nichts.

Monday · 15. Februar 2015

Läuft, vielen Dank an alle :)
//e:

Code

Fatal error: Call to a member function fetch_object() on a non-object in /home/www/cwcity/hosting/f/l/floinkboard/htdocs/wcf/lib/data/message/bbcode/POKEBBCode.class.php on line 16

Diese Fehlermeldung übrigens noch :/
also ist das object leer, etwas ähnlich sagte @TimWolla ja bereits, aber wie kann ich das beheben?

immu · 16. Februar 2015

Es soll hilfreich sein, sich die Dokumentation zu den Frameworks, die man verwendet, durchzulesen. Hier, ich habs für dich rausgesucht: https://www.woltlab.com/downloads/manual/wcfdoc_en.pdf

Und ums dir noch leichter zu machen, hier der relevante Teil:

Zitat

The method sendQuery() sends an SQL-query to the database server and returns the result identifier.

Sprich: Die Methode gibt dir nicht das Query-Ergebnis zurück, sondern einen Identifier, mit dem du dieses ansprechen kannst. Du musst das dann noch abholen, etwa mit der Methode fetchArray. Oder du verwendest gleich getFirstRow.

Btw., "SELECT *" sollte man außer für ad-hoc Abfragen nicht verwenden.

[OOP] WBB-Plugin

Ähnliche Themen

Bisafans.de – Mobile Ansicht

Diverse Fehler und Probleme bei Laptop und PC

Ausblick auf Neuerungen im BisaBoard

Benutzer online in diesem Thema