Zugriff auf pid in dca

[TomH]

Ich bin schon ein gutes Stück weiter gekommen, aber jetzt hackt es an einer anderen Stelle:

Ich rufe die Funktion über load_callback auf (soweit kein Problem):

$GLOBALS['TL_DCA']['tl_newsletter']['fields']['sender']['load_callback'][] = array('xNewsletterSender', 'senderAdress'); 

Folgende Funktion wird aufgerufen (funktioniert ebenfalls):

class xNewsletterSender extends Newsletter {
    public function senderAdress($row, $label) {
        $obj = $this->Database->prepare("SELECT * FROM tl_newsletter_channel WHERE id=?")->limit(1)->execute(???);
    }
} 

Zum Problem: Wie kann ich in dieser Funktion auf die pid (oder id) zugreifen, sodass ich sie in execute() eintragen kann?

Vielen Dank,
Tom

Falls sich jemand fragt, was das werden soll: Eine Erweiterung, die es ermöglicht jedem Newsletterarchiv einen Absender zuzuweisen.

[TomH]

Habe einen Weg gefunden auf die id zu zugreifen:

$this->Input->get('id'); 

Vielleicht könnte mal jemand über meinen SQL-Code schauen, um an die pid zu kommen, weil mit dem stimmt auch etwas noch nicht:

$obj = $this->Database->prepare("SELECT *, (SELECT * FROM tl_newsletter_channel WHERE tl_newsletter_channel.id=tl_newsletter.pid) AS archive FROM tl_newsletter WHERE id=?")
->limit(1)
->execute($this->Input->get('id')); 

Fehlermeldung:

Fatal error: Uncaught exception Exception with message Query error: Operand should contain 1 column(s) (SELECT *, (SELECT * FROM tl_newsletter_channel WHERE tl_newsletter_channel.id=tl_newsletter.pid) AS archive FROM tl_newsletter WHERE id='5' LIMIT 0,1) thrown in /system/libraries/Database.php on line 519

[cbeier]

ich verstehe gerade den Zweck deines SQL-Statements noch nicht ganz. Aber ich glaube mit einem Join kommst du da weiter, denn du willst ja die Daten von zwei Tabellen in einer Abfrage haben. Oder?

SELECT *
FROM tl_newsletter_channel AS channel
INNER JOIN tl_newsletter AS archive
  ON channel.id = archive.pid
WHERE id=?

[TomH]

Der Sinn ist ganz simpel:

• Ich lese per Input->get die id des Newsletters aus
• Mit dieser id lese ich per Db-Abfrage die pid (id des Newsletterarchives) aus
• Über die pid bekomme ich die gespeicherten Daten aus tl_newsletter_channel

Nachtrag: Als Vorlage diente mir folgender Code:

$objArticlesStmt = $this->Database->prepare("SELECT *, (SELECT title FROM 
tl_news_archive WHERE tl_news_archive.id=tl_news.pid) AS archive, (SELECT 
displayed_name FROM tl_news_archive WHERE tl_news_archive.id=tl_news.pid) 
AS displayed_name, (SELECT jumpTo FROM tl_news_archive WHERE 
tl_news_archive.id=tl_news.pid) AS parentJumpTo, (SELECT name FROM tl_user 
WHERE id=author) AS author FROM tl_news WHERE pid IN(" . implode(',', $this->news_archives)
 . ")" . ($this->news_featured ? " AND featured=1" : "") . (!BE_USER_LOGGED_IN 
? " AND (start='' OR start<?) AND (stop='' 
OR stop>?) 
AND published=1" : "") . " ORDER BY date DESC");

[cbeier]

Oje, SQL-Code in einer Zeile ist sehr schwer zu lesen und nachzuvollziehen.

Der INNER JOIN sollte genau das machen was du bezweckst. Hast du das mal probiert? Der Asterix (*) wird da aber wohl nicht funktionieren. Ich finde es aber sowieso besser, wenn im SELECT nur die Datenfelder aus der DB geholt werden, die wirklich gebraucht werden.

Beispiel:

SELECT channel.name, channel.email, archive.date ...

[TomH]

Du hattest bei WHERE id=? das archive.id vergessen - ansonsten funktioniert es wunderbar! Vielen Dank.

Hier der endgültige Code für alle, die so etwas in die Richtung auch mal gebrauchen:

$obj = $this->Database->prepare("
SELECT tl_newsletter_channel.sender, tl_newsletter_channel.senderName, tl_newsletter_channel.visibility, tl_newsletter_channel.overwrite 
FROM tl_newsletter_channel 
INNER JOIN tl_newsletter 
ON tl_newsletter_channel.id = tl_newsletter.pid 
WHERE tl_newsletter.id=?")
->limit(1)
->execute($this->Input->get('id'));

[cbeier]

Du hattest bei WHERE id=? das archive.id vergessen - ansonsten funktioniert es wunderbar! Vielen Dank.

Ja, das passiert schon mal am späten Abend.

[FloB]

TomH, nutze bitte überall die Code-Tags (auch bei SQL – einfach [ CODE ]) – macht das ganze übersichtlicher .

[andreas.schempp]

Deine Funktion ist etwas verwirrend zu lesen, denn es müsste nicht

public function senderAdress($row, $label) 

sondern besser

public function senderAdress($value, $dc) 

heissen.

$dc ist nämlich der Datacontainer, und darin kannst du mit $dc->id auf die ID zugreifen. Verwende immer besser den $dc als Input->get() (habe ich früher auch gemacht). Aber beim mehrfachen bearbeiten bleibt die GET-ID immer stehen und die $dc->id sollte dann stimmen.

[TomH]

Vielen Dank für den Tipp! Habe ich angepasst. Ich war von der get-Variante auch nicht wirklich begeistert, aber hatte übersehen, dass die id auch im dc gespeichert ist.

@FloB: Antwort per PN, damit wir hier beim Thema bleiben.

[TomH]

Eine (hoffentlich vorerst letzte) Frage habe ich noch:

Meine Klasse ist folgendermaßen definiert:

class xNewsletterSender extends Newsletter 

Wenn ich nun in einer Funktion, in der $dc verfügbar ist, in der Klasse, folgenden Befehl aufrufe:

parent::send($dc); 

sind in der Funktion send im DataContainer keine Werte mehr gespeichert.

Woran liegt das und wie lässt es sich beheben?

[andreas.schempp]

Hallo Tom,

Eigentlich kann das nicht sein.

Hast du die Namen des Parameters auch $dc benannt?

Beachte dass $dc meist nur die ID enthält, und keine anderen Werte (wie table oder pid).

[TomH]

Eigentlich kann das nicht sein.

Ist auch nicht so war wohl gestern Abend etwas spät.

Ja, dc für DataContainer finde ich gut.