Query richtig erstellen

Moin,

Hab grad feststellen müssen, dass in der Datenbank der Bezug zu Show als Spalte tx_show benannt ist.

Das würde ich SOFORT! ändern. Convention over Configuration. Was per Convention nicht passt, muss extra configuriert werden. Je mehr also schon stimmt - durch Einhalten der Convention - umso weniger muss extra und umständlich nach- oder überkonfiguriert werden.
Und wenn die Namen nicht durchgängig sind, stößt du früher oder später immer wieder auf Probleme, oder derjenige, der deinen Code in ein paar Jahren weiterbearbeiten soll - eventuell bist du das selbst.

Der Prefix tx_ deutet eher auf einen DB-Tabellennamen hin als auf eine Spalte.
Stimmt denn dein TCA diesbezüglich?

Nochmal zu diesem Punkt:

$this->view->assign('aktuelleProduktionen', $aktuelleProduktionen);
$this->view->assign('archivierteProduktionen', $archivierteProduktionen);
$this->view->assign('show', $show);

Wenn aktuelleProd und archivierteProd Teil der Show sind (so hab ich es verstanden),
dann reicht es, die show zu übergeben, weil du ja dann aus der View auch auf show.archivProd und show.aktuelleProd zugreifen kannst.
Dazu brauchst du im Show-Model die beiden Get-Funktionen (s.o. im thread), welche dir gesiebte Listen liefern.

Dann zu dem Punkt deiner Überschrift, die ausbleiben soll, wenn kein Element da ist:
Wenn du eine Liste aktuelleProduktionen hast, kannst du abzählen, wieviele Elemente drin sind ... count()
Wenn nicht, kannst du z.B. eine Methode in dein Model packen, namens hasAktuelleProduktionen() (ich bin kein Fan von Denglish, aber sei's drum)
Dort läufst du alle Prods durch und lieferst true, wenn es min. eine aktuelle gibt.

Da fällt mir noch was ein: Du hast schon den beschreibenden Kommentar über den Funktionen des Repository, oder ? Ich bin mir nicht sicher, wie wichtig dieser ist, aber schaden tut er auf keinen Fall.

Ich hab das Gefühl, dass wir ab uns zu an einander vorbei reden. Nur nochmal die Strucktur, damit das ganz klar wird was hier überhaupt passiert. Ich nehme ein Beispiel... W.A. Mozart - Die Zauberflöte

Erste Tabelle der Datenbank...
SHOW (also Die Zauberflöte)
- Titel
- Autor
- Inhalt
- Personen der Handlung
- ...
- PRODUKTIONEN (Die Zauberflöte wird ja öfter mal wo aufgeführt, also sollen die Produktionen auch gelistet sein)
-- Produktionsname
-- Startdatum
-- Enddatum
-- Ensemble
-- Wichtige Vorstellungen

und noch vieles mehr.
Damit ist also Produktionen ein Child von Show.

Jetzt aber zu deinem letzten Post.
Warum die Spalte tx_show heißt weiß ich ehrlich gesagt nicht. Das hat der Extension-Builder so generiert. Hab daran einfach nichts geändert. Ist mir schon einmal aufgefallen, allerdings hab ich nichts daran geändert, weil alles funktioniert hat. Werde ich mir nochmal genauer ansehen.

Ich denke jetzt kommen wir der Sachen langsam näher, weil ich vorab auch immer davon ausgegangen bin, ich müsse im Model etwas machen. Also, wenn ich mir in Show Produktionen listen lassen muss, dann ist dies ja jetzt auch im Model per Get-Funktion geregelt. Die aktuelle Variante scheint mir eher eine für eine ListAction zu sein.

Und genau da brauch ich jetzt konkrete Hilfe. Wie muss ich es nun anstellen, dass ich eine funktionierende Get-Funktion im Model hinbekomme? Eine Query brauch ich natürlich schon, weil ich ja nicht, wie bisher natürlich schon möglich, alle, sondern nur Produktionen nach Richtlinien ausgegeben haben will. Nur wo genau kommt die Query dann hin, und wie konfiguriere ich das nun als Get-Funktion.

Ich will das einfach mal verstehen, damit ich es in Zukunft selbst umsetzen kann. Ich mach ja learning by doing. Für Programmierer ist das ja auch alles von Anfang an klar und ich muss mich da langsam vortasten. Ich bin echt für die Hilfe dankbar.

glg M.

...Damit ist also Produktionen ein Child von Show.

Soweit hab ich das schon verstanden.

Was der ExtensionBuilder so generiert... da musst du schon aufpassen!
Ich weiß nicht, welche Version du hast oder welche aktuell ist. Ich hab hier 2.5.2
Gerade bei 1:n Beziehungen erzeugt der ExtBuilder doppelte Einträge.
Du musst ja zwei Beziehungen aufbauen. Einmal 1:n von Show nach Produktion und einmal n:1 in der anderen Richtung.
Die Show muss ihre Produktionen kennen und die Prod. muss wissen, zu welcher Show sie gehört.

Die doppelten Einträge stehen in der ext_tables.sql und in den TCA-Dateien.
Die doppelten Columns sind die mit passthrough -> Löschen!

Die Klasse Produktion hat dann ein Show-Objekt und die Klasse Show hat dann eine Liste von Produktion(en). In der DB steht bei der N:1 die uid des entsrpechenden Objekts und bei der 1:n steht die Anzahl der referierten Objekte.
Das macht der ExtBuilder dann auch richtig.

...

Genau so ist das. Das ist für mich auch alles klar und logisch.

Und ich möchte nun eben in der Show-Ansicht Produktionen auf drei Varianten listen können.

Einmal ganz normal - das geht jea jetzt auch schon. Also die Liste aller zugeordneten Produktionen.
Dann nochmal gefiltert nach aktuellen Produktionen.
Und eine dritte Variante für archivierte Produktionen.

Die Ausgabe soll jeweils über <f:for each... erfolgen.

Nur wohin platziere ich jetzt den Filter also die Query? Wie bekomme ich es hin, dass ich eben auch {aktuelleProduktionen} ausgeben kann.

...
Du willst also von deiner Klasse Show folgende Funktionen aufrufen:
1. getProduktionen()
2. getAktuelleProduktionen()
3. getArchivierteProduktionen()

1. ist trivial. Diese hat der ExtBuilder schon generiert, sollte genau das tun:
return $this->produktionen;

Für 2 und 3 fallen mir aber zwei Möglichkeiten ein:
i) Entweder du holst (analog zu Punkt 1) alle $this->produktionen, läufst das in einer Schleife durch, prüfst $produktion->getAbgelaufen(), packst diese in ein temporäres Array und lieferst dieses.
ii) Die zweite Möglichkeit ist übers Repository.
Afaik * gehört das aber nicht ins Model sondern in den Controller.
Edit: Das heißt nicht, daß es nicht machbar wäre.

Willst du nach einer bestimmten Property filtern, nimmst du am besten die magic findByIrgendwas.
$this->produktionenRepository->findByAbgelaufen('1');
Ohne nachzuschauen, mein ich das Ergebnis ist ein QueryResult.
Ich glaube allerdings, daß so etwas _nicht_ geht...
$this->produktionenRepository->findByAbgelaufen('0')->findByOpenend('1');
... aber ohne Gewähr.
Vielleicht hilft das:
http://stackoverflow.com/questions/23501462/extbase-repository-findby-two-parameters
Du kannst ja mal findAll() ausprobieren. (Selbstredend, was das liefert)
Ebenso kannst du diese zählen mit countBy[Property].
Wenn du in der Richtung weitermachen willst, solltest du dir mal die magischen Funktionen eines Repository zu Gemüte führen.

Bin mir nicht sicher, aber ich denke, dein Problem lässt sich ohne Query-Objekt - allein durch findBy-Methoden lösen.

Gruß
Harald

Danke Harald.

Ich bin wohl einfach zu blöd dafür :/

Also ich hab jetzt einiges ausprobiert.
Ich kann natürlich im Model getArchivierteProduktionen() hinzufügen und bekomme erneut alle Produktionen der Show über return $this->produktionen; heraus. Doch bringt mir das ja nicht viel, weil ich dadurch exakt das selbe Ergebnis wie bereits über getProduktionen() bekomme.

Im Model hab ich keine Filter-Funktion entdeckt, was natürlich besonders schlau wäre. Also zB return $this->produktionen->findByXxx('x'); das mag das System natürlich gar nicht und ich bekomme sofort eine Fehlermeldung.

Ich muss sagen, dass ich das mit der Schleife nicht verstehe (also i) ), dazu fehlt mir wohl die Kenntnis was da genau gemacht werden muss. Hab hier auch nicht wirklich was im Netz gefunden.

Und dann Methode ii)
Das ist für mich ja alles logisch, aber ich bekomme niemals eine Ausgabe.

Mir ist bewusst, dass ich das im Controller angehen muss. Ich will jetzt einfach mal alle Produktionen in der Datenbank ausgeben. Also mit findAll().

Dazu muss ich mal im Controller das nötige Repository injecten:

	/**
     * produktionenRepository
     *
     * @var \Musicalplanet\MpDatenbank\Domain\Repository\ProduktionenRepository
     * @inject
     */
    protected $produktionenRepository;

Soweit so gut, nun muss ich zur korrekten Action und dort noch die Ausgabemöglichkeit definieren.

    /**
	* action produktionen
	*
	* @param \Musicalplanet\MpDatenbank\Domain\Model\Show $show
	* @return void
	*/
	public function produktionenAction(\Musicalplanet\MpDatenbank\Domain\Model\Show $show) {
        $archivierteProduktionen = $this->produktionenRepository->findAll();
        $this->view->assign('show', $show);
        $this->view->assign('archivierteProduktionen', $archivierteProduktionen);
	}

Nur bekomme ich damit rein gar nichts an Produktionen ins Fluid.

Hab mir dann irgendwann gedacht, dass wohl eine Listung über <f:for each="{show.archivierteProduktionen}" as="produktion"> nicht funktionieren wird, da ja dies gar nicht unter show läuft und hab die abfrage auf <f:for each="{archivierteProduktionen}" as="produktion"> geändert. Kein Ergebnis.

Und über den <f:debug> sehe ich auch keine Ausgabe mit archivierteProduktionen.

Es ist grad echt zum verzweifeln. :/

Wenn das mal klar ist, dann wir das System zu 99 % fertig. Ich werd nämlich öfter mal solche Filter benötigen. :/

lg M.

Na da sind wir doch endlich mal ein Stück weiter!

Hab mir dann irgendwann gedacht, dass wohl eine Listung über <f:for each="{show.archivierteProduktionen}" as="produktion"> nicht funktionieren wird, da ja dies gar nicht unter show läuft und hab die abfrage auf <f:for each="{archivierteProduktionen}" as="produktion"> geändert.

Soweit richtig.

findAll() liefert alle vorhandenen Objekte. Unabhängig wessen Kind sie sind.
Eventuell gilt es noch die pid zu beachten. Und selbverständlich müssen diese Objekte dann auch die entsprechnde pid haben. Überprüf doch mal in deiner DB, ob die pid überall stimmt!

Und achte auf deine Fehlermeldungen, wenn du sie mir schon nicht mitteilen willst.

Ich muss sagen, dass ich das mit der Schleife nicht verstehe (also i) ), dazu fehlt mir wohl die Kenntnis was da genau gemacht werden muss.

Du hast doch schon gesagt, was du machen willst. (Oder hab ichs gesagt?) Diesen Wunsch gilt es in PHP zu formen.
So aus der Hüfte raus geschossen würde ich das in etwa so machen:

Model/Show.php

/**
 *** geht glaub so nicht: @return \TYPO3\CMS\Extbase\Persistence\ObjectStorage<\Musicalplanet\MpDatenbank\Domain\Model\Produktion>
 *** aber so müsste es gehen:
 * @return array
 */
public function getArchivierteProduktionen() {
    $temp = array();
    foreach($this->produktionen as $p)
       if ($p->isArchiviert())
           $temp[] = $p;
    return $temp;
}

Model/Produktion.php

/**
 * @return boolean
 */
public function isArchiviert(){
  return $this->archiviert;
}

Entsprechend dazu das gleiche für die Aktuellen Prod.

Guten Morgen!

P.S.: Die magischen find-Methoden sind im Repository.
Auf ObjectStorage (oder gar Array) kannst du die nicht anwenden.

Hi Danke gleich mal.

Also zu den Fehlermeldungen muss ich sagen, dass ich auf Development bin, aber nicht wirklich Fehlermeldungen bekomme. Zumindest nicht dann wenn ich weiß, so sollte es funktionieren. Nur wenn ich dann mal herumprobiere kommt mal was - aber das ist auch klar. Hätte mich eher gewundert, wenn da ne Ausgabe erfolgt.

Das zweite schau ich mir mal gleich an, aber bei der ersten Sache verstehe ich das mit PID nicht so ganz.
Klar hab ich in meinem TS die PID für die jeweiligen Ausgaben deklariert. Doch dachte ich, dass das rein für Fluid interessant ist, weil ich hier bei Links die PID mitgebe, damit der Link auf die richtige Seite geroutet wird. Das ist in meinen Augen wichtig für die jeweiligen Einstellungen und auch realurl, der ja tw einen anderen pfad dadurch bekommt.

Hier verstehe ich nicht was die PID damit zu tun hat. Aber vielleicht redest du auch nicht von der PID im TS (settings) sondern von einer Einstellung die ich bisher noch nicht entdeckt habe, oder gerade nicht im Kopf habe.

glg M.

Da ist irgendwo der Hund begraben.

Ich hab verstanden was das alles macht und hab jetzt etwas herum probiert. Wenn ich die Schleife für aktuelle Produktionen baue und mal einfach filtere nach Produktionen die Openend sind - was direkt in der Tabelle deklariert ist - funktioniert das.

Aber die Filterung nach Abgelaufen geht leider nicht.

Das hab ich in meinem ProduktionenModel

    /**
    * @return $abgelaufen
    */
    public function getAbgelaufen() {
        return ($this->getEndtag() < new \DateTime());
    }
    
	/**
	* Sets the abgelaufen
	*
	* @param boolean $openend
	* @return void
	*/
	public function setAbgelaufen($abgelaufen) {
		$this->abgelaufen = $abgelaufen;
	}

    /**
    * Returns the boolean state of abgelaufen
    *
    * @return boolean
    */
    public function isAbgelaufen() {
        return $this->abgelaufen;
    }

Im Fluid kann ich aber mit <f:if condition="{produktion.abgelaufen}==1"> filtern. Also funktioniert das da korrekt. Doch per $p->isAbgelaufen() findet die Schleife genau gar nichts. Irgendwas stimmt da nicht, ich seh aber keinen Fehler. :/

Da hab ich übrigens noch eine kleine Frage.

    /**
    * @return $abgelaufen
    */
    public function getAbgelaufen() {
        return ($this->getEndtag() < new \DateTime());
    }

Gibt mir TRUE wenn der Endtag kleiner ist als das aktuelle Datum. Da aber natürlich nur ein Datum eingegeben wird, verschwindet die Produktion schon, wenn der heutige Tag ist. Meine Theorie - das geht ja alles in Sekunden. Also ist Endtag Mitternacht. \DateTime ist aber natürlich die jetzige Zeit auch in Minuten und Sekunden, weshalb gleich mal nach Mitternacht Abgelaufen zu TRUE wird, auch wenn die Vorstellung - für gewöhnlich - erst am Abend stattfindet. Mein Ansatz war, bin ja nicht blöd, dann müssen wir hald nen Tag dran hängen, weil eigentlich hätte ich gerne, dass immer am darauf folgenden Tag Abgelaufen TRUE wird. Also getEndtag + 86400 (Tag in Sekunden). Nur hab ich nirgends gefunden wie ich so was bauen kann. Ich hoffe das ist nicht wieder mal komplizierter als es scheint.

glg M.

Zu dem unteren Thema dachte ich es funktioniert irgendwie so... aber bekomm es nicht hin:

    /**
	* Returns the abgelaufen
	*
    * @return boolean $abgelaufen
    */
    public function getAbgelaufen() {
        $folgetag = $this->getEndtag() + 86400;
        return ($folgetag < new \DateTime());
    }

Hab was entdeckt!!

Ich hab im ProduktionenModel ja folgendes:

	/**
	* abgelaufen
	*
	* @var boolean
	*/
	protected $abgelaufen = FALSE;


    /**
	* Returns the abgelaufen
	*
    * @return boolean $abgelaufen
    */
    public function getAbgelaufen() {
        return ($this->getEndtag() < new \DateTime());
    }
    
	/**
	* Sets the abgelaufen
	*
	* @param boolean $abgelaufen
	* @return void
	*/
	public function setAbgelaufen($abgelaufen) {
		$this->abgelaufen = $abgelaufen;
	}

    /**
    * Returns the boolean state of abgelaufen
    *
    * @return boolean
    */
    public function isAbgelaufen() {
        return $this->abgelaufen;
    }

Wenn ich die ersten Zeilen rausnehme - also protected $abgelaufen = FALSE; - dann seh ich im Debug, dass es gar keine Variable Abgelaufen mehr gibt. Spannenderweise funktioniert die Abfrage über Fluid aber einwandfrei. Also hier kann ich dennoch über <f:if condition="{produktion.abgelaufen}"> korrekt filtern.

Was mir jetzt nicht mehr logisch erscheint. Umso mehr scheint es mir logisch, dass so meine Schleife nicht funktioniert.

Wenn ich jetzt, des Spaßes halber, protected $abgelaufen = FALSE; auf protected $abgelaufen = TRUE; ändere, dann sind auch alle Datensätze einfach Abgelaufen = TRUE (im Debug).

Also scheint an der Variable Abgelaufen mein

    /**
	* Returns the abgelaufen
	*
    * @return boolean $abgelaufen
    */
    public function getAbgelaufen() {
        return ($this->getEndtag() < new \DateTime());
    }
    
	/**
	* Sets the abgelaufen
	*
	* @param boolean $abgelaufen
	* @return void
	*/
	public function setAbgelaufen($abgelaufen) {
		$this->abgelaufen = $abgelaufen;
	}

    /**
    * Returns the boolean state of abgelaufen
    *
    * @return boolean
    */
    public function isAbgelaufen() {
        return $this->abgelaufen;
    }

absolut nichts zu ändern.

Mich verwirrt vor allem, dass es aber über Fluid dennoch funktioniert. Also wo ist der Fehler?! :/

Vorhin ging es mal um die PID.

Wenn hier die PID der Datensätze gemeint war - da ist alles aus der Datenbank in einem Systemordner.

glg M.

Huiuiuiuiui!
Eieieieiei!

Also... mir ist jetzt nicht ganz klar, was du am gesamten framwork extbase/fluid verstanden hast oder was nicht...

Wo fang ich da am besten an...?
Für jede deiner Properties - egal welchen Typs - sollte es ein Paar Get/Set-Methoden geben.

Das sieht dann im Model immer so aus:

/**
 * foo
 *
 * @var <TYPDEKLARATION IST WICHTIG>
 */
 protected $foo;

public function getFoo() {
        return $this->foo;
    }

public function setFoo(<TYP> $foo) {
        $this->foo = $foo;
    }

Das zum einen.
Zum anderen:
Wenn du folgendes fluid.html hast:

<f:anyviewhelper param="{foo.bar}">

...muss 1. dieses Skript die Variable (Objekt) foo kennen
und 2. dieses Objekt eine Methode getBar() haben.

Wenn du das soweit verstanden hast, sollte dir auch ein Licht aufgehen, warum trotz fehlerhaftem Code etwas 'richtiges' rauskommt.

Also entweder du hast Properties, die du in der get-Methode direkt returnst.
Oder z.B. eine Methode getAlter(), welche die Differenz zwischen Heute und Geburt liefert.

Ich hoffe, das war soweit verständlich.
Bis dann

Zu deinem Get/Set-Paar:
Was du gemacht hast, war...
1. Eine Property $abgelaufen defniert
2. Eine passende Set-Methode dazu.
3. Eine Get-Methode, die einen Wert völlig unabhängig von der eigentlichen Property ausgiebt.

Eventuell willst du get/Set für $endtag.

Hallo Harald.

Danke.
Ich denke, dass ich das alles verstanden hab. Und das ist mir ja auch alles klar.

Mein Ziel ist es ja eine Variable zu bekommen, die es im Grunde gar nicht gibt. Die heißt dann Abgelaufen.

Wenn ich mir aus der Tabelle Daten holen muss und dann ausgebe ist mir das System klar, aber hier will ich ja eigentlich nur eine Variable erstellen die direkt im Model eine Ausgabe anhand Prüfung mit anderen Variablen anstellt.

Und dann ist mir das System nicht mehr ganz klar.

Also rein mein Hausverstand sagt mir, dass der Getter mal sich Daten holt und anschließend der Setter die Daten dann in der Variablen ausgiebt. Wozu eigentlich Is dann ist ist mir hier nicht ganz klar.

Anhand meines Beispieles will ich im Getter Daten von einer anderen Variablen holen und mal mit dem aktuellen Datum vergleichen. Das dürfte funktionieren, sonst würde das ja im Fluid niemals funktionieren können.

So jetzt müsste ich das alles mit dem Setter an die Variable weitergeben, da scheint was nicht zu funktionieren.

Ich denke mal, dass mein Getter ganz ok ist, aber dann hackt es. Bin ich da mal auf dem richtigen Weg?

Jetzt gibt's wahrscheinlich was zu lachen..

Rein aus der Logik würde ich das ja so machen, aber so passiert genau das selbe wie bisher. :/

    /**
	* Returns the abgelaufen
	*
    * @return boolean $abgelaufen
    */
    public function getAbgelaufen() {
        $temp = $endtag < new \DateTime();
        return $temp;
    }
    
	/**
	* Sets the abgelaufen
	*
	* @param boolean $abgelaufen
	* @return void
	*/
	public function setAbgelaufen($abgelaufen) {
		$temp = $abgelaufen;
	}

    /**
    * Returns the boolean state of abgelaufen
    *
    * @return boolean
    */
    public function isAbgelaufen() {
        return $temp;
    }

Obwohl wohl $temp nicht über die Funktionen hinweg gültig sein wird...

Dann möglicherweise so...

    /**
	* Returns the abgelaufen
	*
    * @return boolean $abgelaufen
    */
    public function getAbgelaufen() {
        $temp = ($this->getEndtag() < new \DateTime());
        return $temp;
    }
    
	/**
	* Sets the abgelaufen
	*
	* @param boolean $abgelaufen
	* @return void
	*/
	public function setAbgelaufen($abgelaufen) {
		$this->getAbgelaufen() = $abgelaufen;
	}

    /**
    * Returns the boolean state of abgelaufen
    *
    * @return boolean
    */
    public function isAbgelaufen() {
        return $this->abgelaufen;
    }

Wobei ich ne Fehlermeldung bekomme...
Fatal error: Can't use method return value in write context in .../Classes/Domain/Model/Produktionen.php on line 660

Da ist aber einiges im Argen.

Nix für ungut...

Und dann ist mir das System nicht mehr ganz klar.

...aber war es das schon mal?

Du solltest dir mal die Grundlagen der objektorientierten Programmierung aneignen.
Sonst spreche ich hier gegen die Wand.

Du kannst doch keine Variable setzen, die du gar nicht hast.
.
Punkt 1: Es gibt Get+Set für alle Variablen = Properties, die es auch im Model gibt.
Dafür musst du sorgen!
Punkt 2: Alle Get-Methoden kannst du aus fluid aufrufen.

Zum Verständnis ein anderes Beispiel:
Person hat $vorname, $nachname
Du willst aber

 <p>person.fullname</p>

Also schreibst du der Klasse Person eine Methode getFullname().
Diese kann dann noch prüfen, ob Herr oder Frau oder Prof. Dr. sonstwas zum Namen gehört und liefert das gewünschte Ergebnis.
.
Oben hab ich ein Beispiel getAlter() erwähnt.
.
Ich will die jetzt nicht deinen Code schreiben. Aber was heißt das jetzt für dich?
Du hast die property $endtag (glaub ich). Dafür machst du ein Paar Get/Set-Methoden.
Wenn getAbgelaufen() vom Verhältnis $endtag zu heute abhängt, prüfst du das in eben genau dieser Methode, und kannst dann diese aus fluid mit {objektname.abgelaufen} ansprechen.

Bei Boolschen Werte geht evtl. auch isAbgelaufen() - ohne Gewähr - probier's aus wenn du Lust hast.

Hi Harald.

Du hast natürlich voll recht. Ich kenn mich Nüsse aus.

Und dennoch hab ich das jetzt alles tatsächlich geschafft. Danke für deine Hilfe, du hast mir - auch wenn das sicher ur Mühsam war - absolut auf den richtigen Weg gebracht.

Danke dir!!

Einziges was ich noch nicht hab ist, dass ich in getEnddatum das Datum um einen Tag erweitern muss. Aber das werd ich auch noch hinbekommen 😉

glg M.