Redirect an einen anderen Controller funktioniert nicht

Sven Kalbhenn

Hey Leute,
ich habe gerade ein Problem mit einem "redirect" bei der Entwicklung einer neuen Extension (TYPO3 v12), wo ich im Moment einfach nicht weiterkomme.
Ich habe in meiner Extension ein Plugin "Persons" und ein Plugin "Notes", die mit einer 1:n-Beziehung miteinander verknüpft sind - also eine Person kann beliebig viele Notizen erstellen, aber jede Notiz gehört nur zu einer Person.
Auf der Detail-Ansicht der Person wird eine Liste seiner Notizen angezeigt und neben jeder Notiz gibt es einen Delete-Button, mit dem die jeweilige Notiz gelöscht werden kann.
Nach dem Löschen der Notiz soll jedoch wieder die Detail-Ansicht der Person angezeigt werden und nicht die Listen-Ansicht der Notizen!

Der Button sieht so aus:

<f:link.action pageUid="{settings.pageIDs.notePID}" action="delete" controller="Note" arguments="{note : note}">
  Delete
</f:link.action>

Es wird also die Delete-Funktion des Note-Controller aufgerufen, die wie folgt aussieht:

    public function deleteAction(Note $note)
    {
        $redirectPerson = $note->getPerson();
        $redirectPID = intval($this->settings['pageIDs']['personPID']);
        //DebugUtility::debug($redirectPID, 'PID');
        //DebugUtility::debug($redirectPerson, 'Person');
        $this->addFlashMessage('Die Notiz wurde erfolgreich gelöscht.', '', ContextualFeedbackSeverity::OK);
        $this->noteRepository->remove($note);
        return $this->redirect('show', 'Person', null, ['person' => $redirectPerson], $redirectPID);
    }

Die Notiz wird korrekt gelöscht und im Debug-Output befindet sich in der Variablen $redirectPID die korrekte PID der Person-Detail-View und in der Variablen $redirectPerson befindet sich das korrekte Person-Object, dass wieder angezeigt werden soll.
Trotzdem funktioniert der "redirect" einfach nicht.
Lasse ich hinten die $redirectPID einfach weg, erscheint zwar keine Fehlermeldung aber ich werde auf die Default-Listen-Ansicht des Notizen-Plugins weitergeleitet, statt zurück zur Detail-Ansicht der Person.
Mit der $redirectPID wie oben gezeigt, erscheint folgende Fehlermeldung:

#1297759968 TYPO3\CMS\Extbase\Property\Exception
in /var/www/html/vendor/typo3/cms-extbase/Classes/Property/PropertyMapper.php line 66

            return $result;
        } catch (TargetNotFoundException $e) {
            throw $e;
        } catch (\Exception $e) {
            throw new Exception('Exception while property mapping at property path "' . implode('.', $currentPropertyPath) . '": ' . $e->getMessage(), 1297759968, $e);
    }
    /**

in /var/www/html/vendor/typo3/cms-extbase/Classes/Mvc/Controller/ActionController.php line 840

        }
        $this->propertyMapper->resetMessages();
        try {
            $argument->setValue(
                $this->propertyMapper->convert(
                    $rawValue,
                    $dataType,
                    $argument->getPropertyMappingConfiguration()
                )

Ich habe keine Ahnung was ich damit anfangen soll.
Meines Erachtens habe ich alle Werte im korrekten Format angegeben, weshalb ich nicht verstehe, warum das Property-Mapping da rummeckert.
Und noch weniger ist mir klar, wie ich das Problem lösen soll!?

Hat jemand eine Idee, was an meinem "redirect" falsch ist und wie ich das Problem lösen kann?

Für jeden Tipp wäre ich sehr dankbar!

Eric Harrer

Sven Kalbhenn Zeig mal die exakte URL, auf welche der Redirect erfolgt und unter welcher der Fehler aktuell erscheint. Daraus lässt sich vielleicht das Problem ablesen. Ansonsten würde ich nochmal genau Prüfen, ob die übergebenen Parameter korrekt sind. Die folgenden Parameter werden erwartet:

protected function redirect(
  ?string $actionName,
  ?string $controllerName = null,
  ?string $extensionName = null,
  ?array $arguments = null,
  ?int $pageUid = null,
  $_ = null,
  int $statusCode = 303
): ResponseInterface {
  /* ... */
}

Folgende Werte übergibst du (mit Kommentaren):

return $this->redirect(
  'show', /* → $actionName - Eine `showAction` muss im `PersonController` existieren */
  'Person', /* → $controllerName - Existiert der `PersonController` wirklich oder heißt der Controller anders? */
  null, /* → $extensionName - Warum `null`? Schreib doch den UpperCamelCase ExtensionName hier rein */
  ['person' => $redirectPerson], /* → $arguments - Welche Argumente erwartet deine `showAction` im `PersonController`? Reichen diese Infos aus? */
  $redirectPID /* → $pageUid - Ich nehme an das passt, hast du ja geschrieben, evtl. noch den Typ `int` sicherstellen */
);

G. Schurkin

Moin, ab Version 11 soll der return-Type immer htmlResponse sein. Versuch mal bitte das: $this->forward('show'); statt $this->redirect('show').

Hier ist ein Link dazu: https://docs.typo3.org/c/typo3/cms-core/main/en-us/Changelog/11.0/Deprecation-92815-ActionControllerForward.html

Eric Harrer

G. Schurkin $this->forward('show'); ist ja laut der von dir verlinkten Dokumentation der veraltete Weg. Wenn dann müsste auf return new ForwardResponse('show'); umgestellt werden.

Da sich die Frage aber nicht auf das Forwarding sondern auf einen echten Redirect bezieht, muss man vermutlich schon bei $this->redirect('show') bleiben. Das wird auch in der offiziellen Dokumentation immernoch so genannt:
https://docs.typo3.org/m/typo3/reference-coreapi/12.4/en-us/ExtensionArchitecture/Extbase/Reference/Controller/ActionController.html#extbase_class_hierarchy-actions
Neu ist, dass $this->redirect in v12 vom Typ ResponseInterface ist:
https://github.com/TYPO3/typo3/blob/39bc29cd2a35c5bf0612fe79e36f3636e86ccd50/typo3/sysext/extbase/Classes/Mvc/Controller/ActionController.php#L703
während dies in v11 noch void (also ohne Rückgabe) war:
https://github.com/TYPO3/typo3/blob/b437207ce36e126884772f40e347b58f9169a337/typo3/sysext/extbase/Classes/Mvc/Controller/ActionController.php#L961

G. Schurkin

Oh nein, ich muss mich entschuldigen. Ich habe völligen Unsinn erzählt. Ich muss die Dokumentation etwas genauer lesen.

Sven Kalbhenn

@Eric Harrer
Ich habe den redirect Befehl jetzt wie folgt geändert:

return $this->redirect('show', 'Person', 'Leseohren', ['person' => $redirectPerson], $redirectPID, null, 303);

"show" lautet die Action, "Person" ist der Controller, "Leseohren" ist der Name der Extension, $redirectPerson beinhaltet die korrekte Person, die angezeigt werden soll. Trotzdem wird leider die selbe Fehlermeldung angezeigt.

Die generierte URL sieht so aus:

https://leseohren.ddev.site/test-pages/person?tx_leseohren_personen%5Baction%5D=show&tx_leseohren_personen%5Bcontroller%5D=Person&tx_leseohren_personen%5Bperson%5D%5B0%5D=18&cHash=1141a2d13956e8dd85378c16d8f836e5

Die 18 ist die korrekte uid der übergebenen Person.
Aus meiner Sicht stimmt also die URL.

Die showAction Funktion im Person-Controller sieht so aus:

    /**
     * action show
     *
     * @return ResponseInterface
     */
    public function showAction(Person $person): ResponseInterface
    {
        $this->view->assign('person', $person);
        return $this->htmlResponse();
    }

Es wird also nur die anzuzeigende Person erwartet, die ich ja übergebe.

Leider bin ich kein Stück weiter gekommen.
Ich verstehe einfach nicht, was der PropertyMapper von mir will.

Für jeden weiteren Tipp, wäre ich sehr dankbar!!

Eric Harrer

Sven Kalbhenn Ich habe die Redirect-URL mal decodiert und in ihre Bestandteile zerlegt:

https://leseohren.ddev.site/
test-pages/person + ?
tx_leseohren_personen[action]=show + &
tx_leseohren_personen[controller]=Person + &
tx_leseohren_personen[person][0]=18 + &
cHash=1141a2d13956e8dd85378c16d8f836e5

https://leseohren.ddev.site/ ist die Base URL aus deiner Site Configuration. test-pages/person ist sicherlich der normale lesbare Seiten-Pfad, der verwendet wird, wenn man die Seite mit der uid aus der Variable $redirectPID aufruft.

Alle GET-Parameter der Adresse, die im Extbase-Kontext verwendet werden (also alle außer cHash), beginnen mit tx_leseohren_personen. leseohren entspricht dabei, nach Konvention, dem klein geschriebenem Extension-Key und personen entspricht dem klein geschriebenem Plugin-Namen. Ich nehme mal an, dass du das Extbase Plugin unter dem Bezeichner Personen per ExtensionUtility::configurePlugin() in der ext_localconf.php konfiguriert hast.

Unter dem Namensraum tx_leseohren_personen werden nun in der Array-Schreibweise in eckigen Klammern die Schlüssel action und controller geführt mit von dir bereits bestätigten Werten show für die Action und Person für den Controller.

Darüber hinaus wird ein Argument person unter tx_leseohren_personen[person][0]=18 angegeben. Auffällig ist hier, dass das Argument person offenbar ein Array enhält und unter dem Array-Index 0 den Wert 18 enthält. Es müsste doch eigentlich tx_leseohren_personen[person]=18 lauten, um direkt, auf Basis des übergebenen Wertes, auf das Person Domain Model mappen zu können oder?

Prüfe mal welchen Typ $redirectPerson besitzt. Der URL nach wäre das ja dann ein Array an einer Stelle an der ein Integer (die uid des Person Domain Model Datensatzes) erwartet wird.

Im Code deines Eingangs-Posts steht:

$redirectPerson = $note->getPerson();

Evtl. gibt es ein Problem im Rückgabewert von getPerson() im Note Domain Model. Wird hier eventuell ein ObjectStorage mit potenziell mehreren Person Objekten zurückgegeben statt nur eines Objektes? Zeig am besten mal diese Methode.

Falls sich meine Vermutung bewahrheitet, könnte folgender Code ein einfacher Workaround sein, der das erste Person-Objekt des ObjectStorage zurück gibt:

return $this->redirect('show', 'Person', 'Leseohren', ['person' => $redirectPerson[0]], $redirectPID, null, 303);

Mehr Aufschluss wird dir ansonsten auch der Property Mapper selbst geben können, in dem der Fehler ausgeworfen wird (vendor/typo3/cms-extbase/Classes/Property/PropertyMapper.php in Zeile 66). Schreib doch dort, bevor der Fehler geworfen wird, ein paar Zeilen zum debugging hinein, um beim nächsten Frontend-Aufruf mehr über den Fehler-Kontext zu erfahren:

...
} catch (\Exception $e) {
    \TYPO3\CMS\Core\Utility\DebugUtility::debug($source, 'SOURCE');
    \TYPO3\CMS\Core\Utility\DebugUtility::debug($targetType, 'TARGET TYPE');
    \TYPO3\CMS\Core\Utility\DebugUtility::debug($currentPropertyPath, 'CURRENT PROPERTY PATH');
    throw new Exception('Exception while property mapping at property path "' . implode('.', $currentPropertyPath) . '": ' . $e->getMessage(), 1297759968, $e);
}
...

Randnotiz:
Da du meine PM an dich offenbar bisher noch nicht bemerkt hast hier kurz zur Info: Um die Lesbarkeit der von dir geteilten mehrzeiligen Code-Blöcke zu verbessern, macht es Sinn, den Code mit drei Backticks ``` (je in einer eigenen Zeile vor und nach dem Code) zu umschließen. Außerdem könntest du per Angabe des Code-Typs für eine farbliche Optimierung sorgen. Mehr dazu hier. Die normale "Code einfügen"-Funktion verwendet nur den Inline Code-Style. Bei mehrzeiligem Code macht die Block Schreibweise aber mehr Sinn, da sonst die Zeileneinrückung verloren geht. Ich habe mir erlaubt deine letzten Posts in dieser Hinsicht zu bearbeiten.

Sven Kalbhenn

@Eric Harrer Vielen Dank für Deine Hilfe!
Ich hatte tatsächlich übersehen, dass die Person als Array übergeben wird. Mit $redirectPerson[0] funktioniert es nun.
Nochmal tausend Dank für den Hinweis!

Mit freundlichen Grüßen... Sven

Eric Harrer

Sven Kalbhenn super, freut mich, dass diese Auffälligkeit direkt die Lösung des Problems war!

Etwas unsicher war ich mir noch, ob $redirectPerson->toArray()[0] oder einfach $redirectPerson[0] geschrieben werden kann. Schließlich handelt es sich ja (vermutlich) nicht direkt um ein Array, sondern um die Klasse TYPO3\CMS\Extbase\Persistence\ObjectStorage. Da du geschrieben hast, dass es mit $redirectPerson[0] funktionierte habe ich mich hierzu nochmal informiert.

Die ObjectStorage-Klasse wird wir folgt definiert:

class ObjectStorage implements \Countable, \Iterator, \ArrayAccess, ObjectMonitoringInterface

ArrayAccess klang zunächst vielversprechend. Die PHP-Dokumentation bestätigt nun, dass auf Klassen, die ArrayAccess implementieren wie auf Arrays zugegriffen werden kann. Der Zugriff per $redirectPerson[0], wie dies bei Arrays möglich ist, ist also auch mit dem ObjectStorage möglich. Wieder was gelernt ☺️

Ansonsten solltest du dir auch nochmal die Frage beantworten, ob ein ObjectStorage für Person-Objekte im Note Domain Model überhaupt sinnvoll ist. Schließlich hast du ja eingangs geschrieben:

jede Notiz gehört nur zu einer Person

Sinnvoller wäre hier vermutlich direkt mit einem Person-Objekt zu arbeiten. Dann sparst du dir auch den Zugriff auf das erste Element einer (potenziellen) Objekte-Sammlung im Redirect-Kontext.

Unterstütze das Forum

Hilf mit, den Community Hub für TYPO3 werbefrei zu halten und die Betriebskosten zu decken. Werde Official Community Hub Supporter für nur 5 Euro im Monat (zzgl. USt) und erhalte ein exklusives Supporter-Badge. Jeder Beitrag zählt!

Jetzt Supporter werden

Impressum - Datenschutz - Kontakt - Netiquette - RSS