Frontend: Daten aus dem Model holen

Aligator13 · 1. März 2016

Hallo zusammen,

ich habe die letzten 2 Jahre damit verbracht, das Backend meiner kleinen Komponente auszubauen. Nun sitze ich am Frontend und stelle fest: Hier läuft einiges doch etwas anders als bisher.

Scheinbar gibt es für die Einzelansicht von Elementen JModelItem im Frontend nicht? Zumindest parent::getItem() scheint nicht zu funktionieren?

Wenn ich hier auf JModelList zurück greife, dann muss ich mir natürlich irgendwie die Parameter "holen", die ich benötige, um wirklich nur den passenden Datensatz anzuzeigen.

Bisher mache ich das wie folgt:

Code

protected function populateState($ordering = null, $direction =  null)    {        $id         =     JRequest::getInt('id');        $this->setState('id', $id);        }

soweit, so gut.

Nun nutze ich getListQuery:

Code

protected function getListQuery($pk = null)
    {
        $db             =     $this->getDbo();
        $query             =     $db->getQuery(true);
        $query->select(
                        $this->getState(
                                        'list.select',
                                        'a.id, a.title, ' .
                                        'a.state, a.catid,' .
                                        'a.content, a.startdate, a.enddate' 
                        ) );
        $query->from($db->quoteName('#__bestia_states').' AS a');

        $query->where('(a.state IN (0, 1))');

        if ($id = $this->getState('id'))
        {
            $query->where('a.id = '.(int) $id);
        }    


        return $query;
    }

Alles anzeigen

Okay, auch das funktioniert, aaaaaaaaber:

Wenn nun

&id=

angegeben wird, erhalte ich natürlich fiese MySQL-Fehler:

Zitat

0 SQLSTATE[42000]: Syntax error or access violation: 1065 Query was empty SQLSTATE[42000]: Syntax error or access violation: 1065 Query was empty

Gibt es da eine elegantere Methode?

Re:Later · 1. März 2016

Zitat

Scheinbar gibt es für die Einzelansicht von Elementen JModelItem im Frontend nicht?

Wirf einen Blick in FE model com_content article.php

Code

class ContentModelArticle extends JModelItem

JModelItem ist also vorgesehen.

Zitat

Zumindest parent::getItem() scheint nicht zu funktionieren?

Das ist korrekt, da die Methode nur in JModelAdmin existiert, die im Normalfall wohl nur im BE "extended" wird.

Code

abstract class JModelAdmin extends JModelForm

Nur so dahergefaselt: JModelAdmin laden und "extenden".

Hast du mal ein Debug
echo (string) $query;
gemacht? Kann mich täuschen, aber Zeilen 6-11 des 2. Codes, basteln die ein korrektes SELECT?
Aber schreibst ja, das funktioniert schon mal.

Und, wenns tatsächlich nur die ID ist, die den SQL-Fehler erzeugt, musst vorher im Controller abfangen, gar keine Abfrage starten oder behelfsweise.

Zitat

$query->where('a.id = ' . (int) $this->getState('id'));

also if rausnehmen. So hast immer einen Wert, ggf. halt 0.

Ob das hier was bringt, aber ein Bsp. aus Modulen, die ein "Fremd-Model" nutzen wollen.
Im Bsp. Frontend-Models articles.php bzw. article.php

Spoiler anzeigen

Statt JPATH_SITE JPATH_ADMINISTRATOR setzen? Und als $com_path deine Komponente etc.?

deGobbis · 1. März 2016

Zitat von ehrenwert

$id = JRequest::getInt('id');
$this->setState('id', $id);
}

Übrigens JRequest ist depricated, besser ist

Code

$app = JFactory::getApplication();
$id = $app->input->get('id', null, 'int');

Damit setzt du auch gleich einen Defaultwert.

Siehe auch http://www.joomla-wiki.de/dokumentation/JInput/get oder https://docs.joomla.org/API17:JRequest

firstlady · 2. März 2016

Code

$query->select(                        $this->getState(                                        'list.select',

Dein select braucht Spaltennamen, keine Inhalte der Session. So muss es heissen:

Code

$query->select(
                        $query->qn(
                                        'list.select', (das muss raus)
                                        'a.id, a.title, ' .

Aligator13 · 2. März 2016

Hallo und vielen Dank.

Darf ich einmal fragen, was "qn" hier bedeutet?

firstlady · 2. März 2016

du darfst - es ist ein alias für quoteName. Praktischerweise quotet das dann gleich alle Namen in der Liste. Du solltest aber immer alles quoten, sicher ist sicher

z.B.

Code

$query->where( $query->qn('a.id') .   ' = ' . (int) $id);

firstlady · 2. März 2016

Da hatte ich noch was übersehen, sry

Code

$this->getState(                                        'list.select',                                        'a.id, a.title, ' .                                        'a.state, a.catid,' .                                        'a.content, a.startdate, a.enddate'

Du muss natürlich jeden Spaltennamen extra quoten:

Code

$query->qn('a.id', 'a.title',  usw  );

Aligator13 · 2. März 2016

Vielen lieben Dank!