Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Pfadanzeige im Backend bei Inhaltselementen falsch #7566

Closed
Samson1964 opened this issue Jan 14, 2015 · 11 comments
Closed

Pfadanzeige im Backend bei Inhaltselementen falsch #7566

Samson1964 opened this issue Jan 14, 2015 · 11 comments
Labels

Comments

@Samson1964
Copy link

Den nachfolgenden Fehler konnte ich in meinem 3.2.16 als auch in der Onlinedemo 3.4.0 nachstellen.

1.) Ein Inhaltselement in einem Artikel erstellen oder bearbeiten. Z.B. in der Seite "Home". Sobald Version 2 davon angelegt ist, erscheint die Version im Log auf der BE-Startseite.
2.) Jetzt im Log auf der BE-Startseite das Inhaltselement eines anderen Artikels zum Bearbeiten wählen.

Im Pfad wird "Artikel -> Home -> ..." angezeigt, obwohl gar kein Inhaltselement aus dem Artikel "Home" bearbeitet wird. Contao verwendet offensichtlich den aus der letzten regulären Bearbeitung gespeicherten(?) Pfad und zeigt diesen an.

@leofeyer leofeyer added this to the 3.2.17 milestone Jan 16, 2015
@leofeyer
Copy link
Member

Kann ich nicht reproduzieren. Ich habe es mit einem Artikel aus "Home" sowie aus "The academy" versucht und jedes Mal war der Pfad korrekt.

@leofeyer leofeyer removed this from the 3.2.17 milestone Jan 20, 2015
@bennie99
Copy link

Genau wie oben beschrieben ist das zu reproduzieren, auch in der Online-Demo.

@xchs
Copy link
Contributor

xchs commented Jan 25, 2015

Ich kann das ebenfalls unter Contao 3.4.2 reproduzieren.

@Samson1964
Copy link
Author

Nochmal im Detail:
1.) Einloggen in der Demo
2.) Auf BE-Startseite ein Element tl_content bearbeiten
3.) Im Pfad steht z.B. "Articles » Edit content element ID 173"
4.) Jetzt in den Artikeln, z.B. den Artikel "Content elements" auswählen
5.) Ein neues Element anlegen und speichern
6.) Evtl. noch ein zweites Mal das Element bearbeiten und speichern.
7.) Auf BE-Startseite das Element aus Pkt. 2 noch einmal zum Bearbeiten auswählen.
8.) Im Pfad steht jetzt im Unterschied zu Pkt. 3: "Articles » Content Elements » Edit content element ID 173", also ein falscher Pfad

@leofeyer leofeyer added this to the 3.2.19 milestone Jan 31, 2015
@leofeyer leofeyer reopened this Jan 31, 2015
@leofeyer
Copy link
Member

Könnte das ein Fehler in der Onlinedemo sein? Denn ich kann es hier lokal definitiv nicht reproduzieren.

@xchs
Copy link
Contributor

xchs commented Mar 19, 2015

Ich beschreibe nochmals kurz, wie ich das gegenständliche Problem reproduziert habe:

  • Contao 3.4.4 & COD
  • "Systemwartung" > "Versionen löschen" > "Daten bereinigen"
  • "Inhalte" > "Artikel" > Artikel "Home" (Artikel-ID: 1) bearbeiten
  • Inhaltselement ID 1 bearbeiten, den Text beliebig modifizieren und die Änderung speichern
  • "Inhalte" > "Artikel" > Artikel "Content Elements" (Artikel-ID: 63) bearbeiten
  • Inhaltselement ID 163 bearbeiten, den Text beliebig modifizieren und die Änderung speichern
  • Auf der Backend-Startseite werden nun unter "Zuletzt bearbeitet" zwei Einträge betreffend die zuvor gemachten Änderungen angezeigt
  • Auf der Backend-Startseite unter "Zuletzt bearbeitet" das Element mit der ID 1 (!) bearbeiten:
    versions
  • Die Headline zeigt nun im Pfad anstelle "Artikel » Home » Inhaltselement ID 1 bearbeiten" fälschlicherweise "Artikel » Content Elements » Inhaltselement ID 1 bearbeiten" an:
    path

@leofeyer
Copy link
Member

Danke @xchs, jetzt konnte ich es auch reproduzieren.

Das Problem entsteht durch die Konstante CURRENT_ID, die nach einem bestimmten System gesetzt wird, um verschiedene Probleme zu lösen. Normalerweise muss man ja erstmal einen Artikel aufrufen, um ein Inhaltselement zu bearbeiten, und dabei merkt sich Contao in der Session, welchen Wert CURRENT_ID hat. Beim Aufruf über die Versionsliste auf der Startseite fehlt der Weg über das Artikelmodul, daher hat CURRENT_ID einfach den Wert des Artikels, der zuletzt (ohne direkten Zugriff) aufgerufen wurde.

@contao/developers Eine Idee, wie man das beheben könnte?

Aus meiner Sicht ist das Problem viel zu unwichtig, als dass wir da Energie investieren sollten. Gerade die Konstante CURRENT_ID ist in ihrer Verwendung relativ komplex, so dass hier Fehlerpotential besteht.

@Toflar
Copy link
Member

Toflar commented Mar 19, 2015

Dito.

@xchs
Copy link
Contributor

xchs commented Mar 19, 2015

Stimme ich ebenfalls zu.

@aschempp
Copy link
Member

Aus meiner Sicht ist das Problem viel zu unwichtig, als dass wir da Energie investieren sollten.

👍

@Samson1964
Copy link
Author

Okay, muß ich halt mit leben. So oft benutze ich das an der Stelle nicht.

@leofeyer leofeyer removed this from the 3.2.20 milestone Mar 20, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

6 participants