metamodels / attribute_decimal Goto Github PK

Checklist before I submit this issue report

I confirm that:

• I have tested this with the latest version available
• I have read documentation @ http://metamodels.readthedocs.org/en/ or http://metamodels.readthedocs.org/de/
• I have checked the Contao community forums for references https://community.contao.org/
• I have checked existing issues for duplicates and found none @ https://github.com/MetaModels/attribute_decimal/issues?q=is%3Aissue

My environment is:

(Please fill in the actual values from your environment)

Issue description

When the attribute value is an empty string (''), it will be saved as "0" in the database. Expected behavior would be saving null.

Steps to reproduce

/** @var \MetaModels\IItem $item */
$item->set('decimal-attr', '');
$item->save();

This could be added to the attribute class.

    public function setDataFor($arrValues)
    {
        // Translate all '' values to null, so they will not set as `0` in the database.
        $arrValues = array_map(function ($value) {
            return '' === $value ? null : $value;
        }, $arrValues);

        parent::setDataFor($arrValues);
    }

I'm not sure, whether this is a issue, because this does not happen when using the dcg (in the back end).
When this issue will be confirmed, I could provide a PR.

Same for attribute_numeric.

Hallo,

bei den Templates ist kein xhtml-Template vorhanden.
Vielleicht könnte das noch zugefügt werden. Es reicht wohl aus, ein html5-Template zu kopieren und in .xhtml umzubenennen.

VG und danke
André

SQL-Standard-Wert ist aber NULL

siehe auch MetaModels/attribute_numeric#8

when declared as mandatory in the input-mask the Asterisk is missing.
On submission you will get no error-message - instead it fills in a "0" silently.

Ein Komma (,) wird grundsätzlich in einen Punkt umgewandelt. Das Komma muss aber für Beträge erhalten bleiben, damit man zum Bsp. Euro Beträge darstellen kann.

Über den Attribut-Typ Decimal erfasse ich Preise. Mit Preisen wie 1.22 oder 100.99 funktioniert das bestens, jedoch bei Preisen wie 1.20 oder 100.90 wird die letzte 0 einfach nicht gespeichert. Ändere ich in der DB die Tabellen-Eigenschaft bei der entsprechenden Tabelle von „double NULL“ in „double(12,2) NULL“, wird auch die letzte 0 korrekt gespeichert.

Es wäre begrüßenswert, wenn man dem attribute_decimal ein Price-Tag mitgeben könnte, um im BE Preisformatierung mit Komma, Tausenderpunkt und abschließende Null zu realisieren. Redakteure können dann im BE Preise wie 1.234,00 erfassen und sich anzeigen lassen.

Im FE formatiere ich den Preis im entsprechenden Template mittels number_format.

Möchte ich einen Betrag von 19,90 darstellen

Das Attribut "decimal" kürzt die "Null" ohne Hinweis einfach weg. Für Eurobeträge sind Nachkommastellen aber notwendig!