Eigene Komponente: Fehler beim Speichern eines Eintrags im Backend

markowski · 3. März 2020

Hallo,

ich habe ein kleines Problem beim Programmieren einer eigenen Komponente. Die Komponente soll dazu dienen eine Speiseplan für eine Schule zu erstellen.

Kurz was zum Aufbau:

- 3 Datenbanktabellen

- 3 Listenansichten im Backend inkl. den entsprechenden Formularansichten für die Daten

Jetzt zum Problem:

Bin ich in einer Formularansicht und betätige den Button "speichern" werden die Formulardaten zwar korrekt in die entsprechende Datenbank geschrieben das Formular aber leer neu geladen. Normalerweise sollten aber die gerade gespeicherten Daten angezeigt werden.

Ich vermute den Fehler im Model beim Laden der Formulardaten aus der Datenbank:

Code

protected function loadFormData() {
        /* Daten aus dem Sitzungsspeicher holen sofern vorhanden */
        $app  = JFactory::getApplication();
        $data = $app->getUserState('com_meals.edit.ingredient.data', array());

        /* ggf. Datensatz aus der Tabelle einlesen */
        if (empty($data)) {

            /* Die Id des Datensatzes, den der Benutzer angeklicht hat in der Liste */
            $requested_id = $app->input->get('id', 0, 'int');
            
            $db = $this->getDbo();
            $query = $db->getQuery(true);
            $query->select('*');
            $query->from('#__meals_ingredients');
            $query->where('ingredient_id = '.$requested_id);
        
            $db->setQuery($query);
            $data = $db->loadObject();
        
        }

        return $data;
    }

Alles anzeigen

Die Variable "$data" ist nach dem Speichern leer.

Wie bekomme ich die gerade gespeicherten Daten in diese Variable?

Mirko

firstlady · 3. März 2020

Welchen Task hast du auf dem Button speichern? apply?

markowski · 3. März 2020

In der "view.html.php" steht folgendes:

Code

JToolBarHelper::apply('ingredient.apply', 'JTOOLBAR_APPLY');
JToolBarHelper::save('ingredient.save', 'JTOOLBAR_SAVE');
JToolBarHelper::cancel('ingredient.cancel', 'JTOOLBAR_CANCEL');

firstlady · 3. März 2020

Dann müssten beim apply die Daten im Form stehen bleiben, wenn du das übliche Schema anwendest.

Bei save müsste es in die Übersicht zurück gehen.

Eigentlich geht das automatisch wenn man nach dem üblichen Schema arbeitet.

Es kommt jetzt daruf an was du anders machst. Kannst du mal mit einer der core Komponente vergleichen?

markowski · 3. März 2020

Hab gerade mal geschaut bei com_content und com_contact, sieht eh alles etwas anders aus, konnte aber nichts auffälliges finden.

Welcher Teil der Komponente setzt eigentlich das

$app->getUserState('com_meals.edit.ingredient.data', array());

?

Hier mal die vollständige Model Datei für die Formularansicht:

PHP

<?php defined('_JEXEC') or die;
jimport('joomla.application.component.modeladmin');

class MealsModelIngredient extends JModelAdmin {
 
    public function getTable($type = 'meals', $prefix = 'MealsTable', $config = array()) {
        
        return JTable::getInstance($type, $prefix, $config);
        
    }

    public function getForm($data = array(), $loadData = true) {
      
        $form = $this->loadForm('meals', 'ingredient', array('control' => 'jform', 'load_data' => $loadData));
        if (empty($form)) {
            return false;
        }

        return $form;
      }
      
    public function save($data) {
        
            // lädt die Daten des Formulars in ein Objekt
            $data = (object) $data;
            $data->allergens = json_encode($data->allergens);

            // neuen Datensatz einfügen
            if ($data->ingredient_id == '' || $data->ingredient_id == '0') {
                $db = $this->getDbo();
                $query = $db->getQuery(true);
                $query->insert($db->quoteName('#__meals_ingredients'));
                $query->columns('
                        title, 
                        alias,
                        published,
                        description, 
                        allergens'
                );
                $query->values(
                        $db->quote($data->title).', '
                        .$db->quote($data->alias).', '
                        .$db->quote($data->published).', '
                        .$db->quote($data->description).', '
                        .$db->quote($data->allergens)
                );
                $db->setQuery($query);
                
                try { $db->query(); } 
                catch (Exception $e) { }
            }
            
            else {
        
                // update der Datenbank
                $db = $this->getDbo();
                $db->updateObject('#__meals_ingredients', $data, 'ingredient_id');
        
            }
            
            return true;
    }

    public function delete(&$pks) {
    
        $pks = (array) $pks;
        $db = $this->getDbo();
        $query = $db->getQuery(true);
        $query->delete($db->quoteName('#__meals_ingredients'));
        $query->where('ingredient_id IN ('. implode(',', $pks) .')');
        $db->setQuery($query);     
        try { $db->query(); } 
        catch (Exception $e) { }
        return true;
    }

    protected function loadFormData() {
        /* Daten aus dem Sitzungsspeicher holen sofern vorhanden */
        $app  = JFactory::getApplication();
        $data = $app->getUserState('com_meals.edit.ingredient.data', array());

        /* ggf. Datensatz aus der Tabelle einlesen */
        if (empty($data)) {

            /* Die Id des Datensatzes, den der Benutzer angeklicht hat in der Liste */
            $requested_id = $app->input->get('id', 0, 'int');
            
            $db = $this->getDbo();
            $query = $db->getQuery(true);
            $query->select('*');
            $query->from('#__meals_ingredients');
            $query->where('ingredient_id = '.$requested_id);
        
            $db->setQuery($query);
            $data = $db->loadObject();
        
        }

        return $data;
    }
}

Alles anzeigen

markowski · 5. März 2020

Hat jemand noch eine Idee oder einen Tipp für mich? Irgendwie komme ich hier nicht weiter, vermutlich sehe ich den Wald vor lauter Bäumen gerade nicht.

Mirko

Harmageddon · 5. März 2020

Zitat von markowski

Welcher Teil der Komponente setzt eigentlich das

$app->getUserState('com_meals.edit.ingredient.data', array());

?

Da werden Formulareingaben temporär gespeichert, wenn die Eingabe noch nicht fertig ist. Also z.B. wenn du was ins Formular eingetragen hast, aber ein Feld falsch oder nicht ausgefüllt hast. Dann wirst du ja wieder zum Formular zurückgeleitet und willst nicht wieder alles von vorn eingeben. Dafür ist das da. Wenn ich das richtig im Kopf habe, wird der im Controller befüllt.

Ich würde übrigens noch überlegen, anstatt die save-Methode komplett in deinem Model zu machen, über die Methode des übergeordneten Models parent::save zu gehen, da da schon viele Sachen abgefangen werden.

Zu deinem Problem: Stimmt denn der Redirect, d.h. wirst du nach dem Speichern auf die richtige Seite (mit der neuen ID in der URL) geleitet? Und was steht in der Variable $requested_id in loadFormData?

firstlady · 5. März 2020

Ich war mal abwesend.

Es ist etwas schwierig, aus Bruchstücken was zu erkennen Soweit ich sehe machst du sachen (delete, getItem) die eigentlich vom parent gemacht werden.

Ich würde das loadFormData etwa so schreiben:

Code

protected function loadFormData()
{
// Check the session for previously entered form data.
$app = Factory::getApplication();
$data = $app->getUserState('com_ingredients.edit.ingredient.data', array());


if (empty($data))
{
$data = $this->getItem();
}


$this->preprocessData('com_ingredients.ingredient', $data);


return $data;
}

Alles anzeigen

markowski · 5. März 2020

Zitat

Zu deinem Problem: Stimmt denn der Redirect, d.h. wirst du nach dem Speichern auf die richtige Seite (mit der neuen ID in der URL) geleitet? Und was steht in der Variable $requested_id in loadFormData?

Klicke ich in der Übersichtstabelle der View "ingredients" auf einen Eintrag bekomme ich mit $requested_id die Id des Eintrags. Speichere ich den Eintrag ist die $requested_id leer, bzw. 0. Und der Redirect gibt mir eine URL ohne Id aus (option=com_meals&view=ingredient&layout=edit).

Kann es sein, dass das Problem bei JTable liegt?

Die Komponente besitzt ja drei SQL Tabellen sozusagen für jeden View eine. Im "tables" Ordner liegt aber nur eine Datei "meals.php".

Hier mal die Dateistruktur der Komponente (siehe Anhang).

Mirko

Harmageddon · 5. März 2020

Zitat von markowski

Und der Redirect gibt mir eine URL ohne Id aus (option=com_meals&view=ingredient&layout=edit).

Dann ist das das Problem. Hast du die save-Methode im Controller auch selbst eingerichtet oder benutzt du da die von FormController? In ersterem Fall müsstest du dir überlegen, woher du die ID des neu gespeicherten Datensatzes herbekommst, und diese an den Redirect anhängen. Wenn du die von FormController verwendest, sollte das dort schon drin sein, vorausgesetzt, das Model legt die ID im State ab. AdminModel macht das z.B. hier: https://github.com/joomla/joom…odel/AdminModel.php#L1302

Spricht denn etwas dagegen, nach deiner Vorverarbeitung (json_encode etc) die Daten im Model über parent::save($data) zu speichern?

markowski · 5. März 2020

Danke für deine Rückmeldung, werd mir das morgen mal in Ruhe anschauen. Und das Model umbauen.

markowski · 6. März 2020

So habs mir mal soweit angeschaut und eine Frage dazu:

Im Admin Model steht folgende Zeile:

Code

if (isset($table->$key)) {
    $this->setState($this->getName() . '.id', $table->$key);
}

Ich nehme mal an das $table->$key aus der Funktion ...

Code

public function getTable($type = 'meals', $prefix = 'MealsTable', $config = array()) {
    return JTable::getInstance($type, $prefix, $config);
}

..gespeist wird?

Wenn ja, würde er ja vermutlich – entsprechend der Funktion oben – auf die Tabelle #__meals_items zugreifen. Die währe aber nicht korrekt, sondern die Tabelle #__meals_ingredients.

Sehe ich das richtig?

firstlady · 6. März 2020

Ja, siehst du richtig.

Wenn du in dem view ingredients mit einer Tabelle meals_ingredients arbeiten willst dann muss das wohl type='ingredients' sein.

Übrigens, die Dateien index.html kannst du alle löschen. Sie sind nicht mehr notwendig und blähen das Verzeichnis auf.

markowski · 6. März 2020

Zweite Sache parent::save():

Ich hab jetzt im Model mal folgendes eingetragen – meinst du das?

Code

public function save($data) {
    $data['allergens'] = json_encode($data['allergens']);
    return parent::save($data);
}

Das funktioniert nur bedingt, da der Datensatz dann statt in #__meals_ingredients in die Haupttabelle #__meals_items gespeichert wird.

markowski · 6. März 2020

Danke für den Hinweis mit der index.html

firstlady · 6. März 2020

Ich kann das so remote nicht beurteilen, leider. In welchem Model bist du?

Wenn in meals, dann speichert oder ändert das jetzt einen Satz "meals" mir einer Anzahl von Allergenen im Feld param.

bembelimen · 6. März 2020

Zitat von firstlady

Es ist etwas schwierig, aus Bruchstücken was zu erkennen

Nutze (J)Table Klassen
Nutze getItem
Nutze die Joomla! Core Methoden
Hör auf irgendwo zwischendrin mit eigenem Code irgendwas erreichen zu wollen, was dir Joomla! schon liefert

Und wichtig: Pack mal deinen kompletten Code (als ZIP) hier in den Beitrag, damit man dir helfen kann.

markowski · 6. März 2020

Ich habs!!!

Folgendes musste ich anpassen:

1. im Ordner "tables" eine ingredients.php anlegen.

2. Model der ingredient.php anpassen

3. Datenbank #__meals_ingredients die Spalte mit der Id von "ingredient_id" auf "id" ändern

4. kleinere Anpassungen durch die Änderung der Id-Spalte in der Datenbank.

Danke an alle für die Hilfestellungen! Das hat auf jeden Fall mein Verständnis für einige Abläufe verbessert.

markowski · 6. März 2020

Ich schließe den Beitrag mal ab. Danke noch mal an alle.

Tags