Bug in MysqliDriver.php?

Ulricus · 18. Oktober 2019

Hallo zusammen,

ich habe evtl. einen Bug im MysqliDriver.php gefunden, bin mir aber nicht sicher, könnte auch ein Fehler meinerseits sein. In Joomla 3 hat derselbe Code aber funktioniert. Version ist Alpha12. Folgendes Verhalten:

- Ich habe ein Formular mit teilweise optionalen, daher leeren Werten, siehe Screenshot. Rechts sieht man mein Table-Objekt $object mit den drei leeren Werten "amount_..." rot eingerahmt. Ich befinde mich in MysqliDriver.php, Funktion insertObject.

- Nun laufe ich durch die foreach-Schleife bis zur Auswertung des Wertes amount_small. Folgender Code ist das:

Code

// Iterate over the object variables to build the query fields and values.
        foreach (get_object_vars($object) as $k => $v)
        {
            // Skip columns that don't exist in the table.
            if (!array_key_exists($k, $tableColumns))
            {
                continue;
            }

            // Only process non-null scalars.
            if (\is_array($v) || \is_object($v) || $v === null)
            {
                continue;
            }

            // Ignore any internal fields.
            if ($k[0] === '_')
            {
                continue;
            }

            // Ignore null datetime fields.
            if ($tableColumns[$k] === 'datetime' && empty($v))
            {
                continue;
            }

            // Prepare and sanitize the fields and values for the database query.
            $fields[] = $this->quoteName($k);
            $values[] = $this->quote($v);
        }

Alles anzeigen

- Am Ende der Schleife, bei // Prepare and sanitize ..., werden alle Werte mit Hochkommata umgeben. Auch leere Werte, aus denen wird "''", siehe folgenden Screenshot:

- Wird nun die Query ausgeführt, kommt folgende Fehlermeldung von der Datenbank:

Incorrect integer value: '' for column 'amount_small' at row 1

Hier noch meine Datenbankdefinition:

Code

CREATE TABLE `#__easyfpus` (
    `id`       INT(11)     NOT NULL AUTO_INCREMENT,
    `asset_id` INT(10)     NOT NULL DEFAULT '0',
    `created`  DATETIME    NOT NULL DEFAULT '0000-00-00 00:00:00',
    `created_by`  INT(10) UNSIGNED NOT NULL DEFAULT '0',
    `name` VARCHAR(100) NOT NULL,
    `favorite` BOOLEAN NOT NULL DEFAULT FALSE,
    `calories` FLOAT NOT NULL DEFAULT '0.0',
    `carbs` FLOAT NOT NULL DEFAULT '0.0',
    `amount_small` int(11) DEFAULT '0',
    `amount_medium` int(11) DEFAULT '0',
    `amount_large` int(11) DEFAULT '0',
    `comment_small` VARCHAR(100) NOT NULL,
    `comment_medium` VARCHAR(100) NOT NULL,
    `comment_large` VARCHAR(100) NOT NULL,
    `published` tinyint(4) NOT NULL DEFAULT '1',
    `params`   VARCHAR(1024) NOT NULL DEFAULT '',
    PRIMARY KEY (`id`)
)
    ENGINE =InnoDB
    AUTO_INCREMENT =0
    DEFAULT CHARSET =utf8mb4
    DEFAULT COLLATE =utf8mb4_unicode_ci;

SET sql_mode = '';

Alles anzeigen

Ist dort ggf. etwas falsch eingestellt?

Vielen Dank für Eure Hinweise!

Re:Later · 18. Oktober 2019

Schätze mal, die Antwort wirst du in diesen Zeilen der MySliDriver.php finden (im Konstruktor).

Code

        /**
         * sql_mode to MySql 5.7.8+ default strict mode minus ONLY_FULL_GROUP_BY because it's inconvenient for some.
         *
         * @link https://dev.mysql.com/doc/relnotes/mysql/5.7/en/news-5-7-8.html#mysqld-5-7-8-sql-mode
         */
        $sqlModes = [
            'STRICT_TRANS_TABLES',
            'ERROR_FOR_DIVISION_BY_ZERO',
            'NO_AUTO_CREATE_USER',
            'NO_ENGINE_SUBSTITUTION',
        ];

Alles anzeigen

Du übergibst einen Leer-String an ein Integer-Feld. Bin nicht ganz sicher, ob man dann 0, '0' oder NULL übergibt.

Gibt ja noch mehr "Fiesheiten" in dieser Art, die daher rühren, dass man die Default-Einstellungen von MySQL zusätzlich für Joomla erzwingt (seit Joomla 4).

Re:Later · 18. Oktober 2019

Seh aber auch ein paar Zeilen drunter, dass man die sqlModes wohl überschreiben kann. Bin aber nicht sicher. Mach derzeit kein Joomla 4.

Harmageddon · 19. Oktober 2019

Wie sieht denn der Code aus, mit dem du die Daten in die Tabelle einträgst? Bzw. sind da die amounts im array/Objekt enthalten und mit welchen Werten? null, 0, '0' oder ''?

Edit: Ok, hab den Screenshot kapiert. Du übergibst einen leeren String als amount. Wenn da nichts eingetragen ist, würde ich beim Erstellen von $object dieses Feld gar nicht hinzufügen, dann sollte eigentlich der in der Tabelle definierte Standardwert genommen werden. Oder du setzt es explizit auf 0.

In Joomla! 4 wird meines Wissens der MySQL-Modus geändert. Bei J!3 wurde so ziemlich alles ignoriert, womit MySQL noch irgendwie klarkommt. Das hat zu unschönen Datenbankstrukturen und -abfragen geführt, weshalb jetzt der Modus etwas strikter gesetzt wird und man das alles schön macht. Ich würde daher davon abraten, den Modus händisch umzustellen, nur damit die eigenen Queries wieder funktionieren, sondern lieber diese so umbauen, dass sie sauber sind.

Ulricus · 19. Oktober 2019

Danke, Harmageddon, ich override jetzt die validate-Funktion in meiner Model-Klasse wie folgt und es funktioniert nun wunderbar:

Code

    /**
     * Method to cast the form data into the types expected by the database.
     *
     * @param   Form    $form   The form to validate against.
     * @param   array   $data   The data to validate.
     * @param   string  $group  The name of the field group to validate.
     *
     * @return  array|boolean  Array of filtered data if valid, false otherwise.
     *
     * @since   0.11.0
     */
    public function validate($form, $data, $group = null)
    {
        // First validate the data, then - if valid - cast them
        if ($validData = parent::validate($form, $data, $group))
        {
            $castData = array();
            
            // The data seem to be valid, so now cast to expected types
            foreach($validData as $k => $v)
            {
                switch ($k)
                {
                    // If 'id' is 0 it's a new record, so leave it away,
                    // the 'amount_...' fields are optional, so also leave away if empty,
                    // otherwise these values are int
                    case 'id':
                    case 'amount_small':
                    case 'amount_medium':
                    case 'amount_large':
                        if ($v != '' && $v != '0') $castData[$k] = (int) $v;
                        break;
                    
                    // 'carbs' and 'calories' are float type
                    case 'calories':
                    case 'carbs':
                        $castData[$k] = (Float) $v;
                        break;
                    
                    // All other values: pass them on 1:1
                    default:
                        $castData[$k] = $v;
                }
            }
            
            return $castData;
        } else {
            return false;
        }
    }

Alles anzeigen

Danke für den Hinweis!

Tags