search

alnv / contao-catalog-manager-bundle

Catalog Manager
https://catalog-manager.org/
4 stars 0 forks source link

Breadcrumb. `pageTitle` der Seite mit Modul "Detailansicht" wird nicht zurückgesetzt. #38

Closed grummbeer closed 4 weeks ago

grummbeer commented 1 month ago

Wenn auf einer Seite die Detailansicht gezeigt wird, fügt sich in der breadcrumb der Eintrag für das Katalog-Element ein. Die Contao-Seite mit dem Modul hat nun den Titel der Katalog-Entität.

Seiten:

Breadcrumb

Wird jetzt ein Katalog-Elmente aufgerufen sieht die breadcrumb so aus:

Der Titel der Seite "Island" wurde hier https://github.com/alnv/contao-catalog-manager-bundle/blob/4df2f7f98c8eeee27715bef327c266058981a6cd/src/EventListener/GeneratePageListener.php#L26

mit den Daten aus dem Katalog überschrieben, aber hier, beim Einfügen des letzten Elementes in die Breadcrumb nicht wieder herstellst.

https://github.com/alnv/contao-catalog-manager-bundle/blob/4df2f7f98c8eeee27715bef327c266058981a6cd/src/EventListener/GenerateBreadcrumbListener.php#L63

Es sieht so aus, als sollte hier, neben dem URL auch der pageTitle wiederhergestellt werden.

if (($objPage = PageModel::findByPk($strPageId)) && !$blnRequireItem) {
      try {
          $arrItem['href'] = $objPage->getFrontendUrl();
          $arrItem['title'] = $objPage->pageTitle;
       // ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      } catch (\Exception $objError) {
          //
      }
  }
grummbeer commented 1 month ago

@alnv wenn das bestätigt ist und mein Lössungsvorschlag passend ist, würde ich einen PR dafür einreichen.

alnv commented 1 month ago

Ich werde es die kommende Tage kontrollieren …

grummbeer commented 1 month ago

Danke, vielleicht auch noch den Schlüssel isRoot für das Katalog-Element hinzufügen.

Der Wert sollte vermutlich immer false sein, da er ja hinten drangenhängt wird.

$arrItem['isRoot'] = false;
alnv commented 1 month ago

Danke … wir werden es in die v3.2 packen.

grummbeer commented 1 month ago

Mercy!

isRoot

weil: PHP Warning: Undefined array key "isRoot" in … nav template

grummbeer commented 4 weeks ago

Vielen Dank!