search

cmb69 / pagemanager_xh

Page management for CMSimple_XH
GNU General Public License v3.0
4 stars 3 forks source link

Unpublished pages get "EMPTY HEADING" #53

Closed frase-git closed 7 years ago

frase-git commented 7 years ago

(Version 3.0beta3: hoffentlich nicht schon gemeldet)

Werden Seiten über Pagedata "unveröffentlicht" geschaltet, erhalten Sie im Pagemanager das Label: "LEERE ÜBERSCHRIFT".

BTW: Wenn "unveröffentlicht", dann sollte auch automatisch "In der Navigation anzeigen? = AUS" geschaltet werden.

cmb69 commented 7 years ago

Stimmt. Das liegt daran, dass der Pagemanager die Seitendaten fälschlicherweise im Ansichtsmodus anfragt; um diesen Fehler zu beheben, müsste nur diese Zeile geändert werden zu:

                ->with('action', 'plugin_data')->with('edit', '')

BTW: Wenn "unveröffentlicht", dann sollte auch automatisch "In der Navigation anzeigen? = AUS" geschaltet werden.

Hm, bin unsicher. Im Reiter "Seite" sind die beiden Einstellungen ja auch unabhängig voneinander, und ohne weitere Vorkehrungen würde beim Speichern im Pagemanager der Wert überschrieben; oder man dürfte die Checkbox dann nicht umschalten können. Ich denk mal drüber nach.

cmb69 commented 7 years ago

Ich denk mal drüber nach.

Ich sehe keine saubere Möglichkeit, das zu lösen. Eigentlich bräuchte der Pagemanager zwei Checkboxen, aber das gibt die API nicht her. (Und noch eigentlicher sollte es gar keine zwei Booleschen Flags geben, sondern eine Aufzählung, da unveröffentlicht immer versteckt impliziert, aber das kann aus Gründen der Abwärtskompatibilität nicht einfach geändert werden.) Ich werd's also so lassen wie es jetzt ist.

Vielleicht schreibt ja jemand mal einen alternativen Pagemanager, der ein anderes Widget verwendet (evtl. nestedSortable), das mehr Flexibilität bietet.