New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Eigene Seite inkludierbar über PAGE_CONTENT_HEADER #166
Conversation
hm geht das hiermit nicht? http://www.redaxo.org/de/doku/extension-points/?mid=17 kann auch sein dass ich nicht genau verstanden habe, was du suchst. |
also, der screenshot sollte genau zeigen, was ich mir wünsche :) (hat nix mit dem slice menu zu tun...) |
Gregor weiss sonst auch was ich meine...Hab ihn schon zugetextet gehabt damit... |
Praktisch fänd' ich das auch ... |
Sollte über den Extension Point PAGE_CONTENT_MENU möglich sein |
Erstmal ja, deshalb hab ich den EP auch im Titel dieses Issues verwendet ;) Aber wenn ich das richtig verstanden habe kann ich darüber nur einen neuen Menüpunkt inkl. Link adden. Es wäre cool da aber ne PHP Datei reinklemmen zu können :) |
Wie Rudi schon sagte, haben wir uns darüber beim Bochum-Treffen unterhalten. Er hat schon Recht, man kann den EP zurzeit nicht wirklich sinnvoll nutzen. Man kann zwar einen Menüpunkt hinzufügen, aber nicht sagen, welche Datei geladen werden soll. Ich hatte nach dem Treffen auch noch bisschen drüber nachgedacht, ich probier da demnächst mal was. |
ahhhh, endlich einer der mich versteht :))) cooool! wollte gerade auch noch über den multiuploader schreiben und die ganzen gepatchten config.inc.php's die da mitgeliefert werden... |
Hab den EP bislang nie benutzt, hätte aber genau das erwartet.. das ich ne eigene page aufrufen kann.. insofern: klares +1 ;) |
@jdlx: man könnte damit dann rexseo sogar ne eigene seo-page spendieren...und bräuchte nicht mehr ein haufen metafelder erzeugen... @gharlan: ich glaube wenn man es ganz sauber lösen will muss man vielleicht die einzelnen unterseiten in extra dateien exkludieren, denn momentan stehen ziemlich fiese hartcodierte sachen in der content.inc.php drin wie z.B. if ($mode != 'meta')
$mode = 'edit'; und man muss auch schauen das das be_search addon sauber funzt mit den geaddeten pages, so dass ich z.B. den artikel über die schnellnavigation wechseln kann und gleichzeitig auf meiner seo-page bleibe (die aus dem screenshot). |
Ich habe mir jetzt mal etwas überlegt, was aber nur bei den Medienpool-Subpages funktioniert. Beim Medienpool wird der EP jetzt gar nicht mehr gebraucht, dafür ist es sehr einfach auch in anderen Addons einsetzbar. |
ok, cool :) da wird sich @hirbod bestimmt freuen. Wichtig ist halt dass die Lösung(en) so ausgelegt sind, dass die bestehenden Addons hier keine Umwege mehr über Patches/MetaInfos/Link zu Addon mehr machen müssen. |
Wobei es auch erst mal nur R5 ist, R4 ist noch mal wieder ein anderes Thema, da muss ich sowieso noch schauen, ob da was sinnvolles geht, ohne bc breaks. |
oh ja, dass issue ist ja zweimal vertreten...dachte gerade ich wäre hier in R4 :) |
würde ich in separate Pages aufteilen, also edit, meta und metafuncs |
👍 macht absolut Sinn, dann wäre es im Redaxo Stil |
Das können wir gerne in einzelne Dateien aufteilen. Allerdings ist das für das Thema hier weniger das Problem. Das Problem ist eher, dass es keine normalen Subpage-Objekte (rex_be_page) sind, die Navi sitzt ja auch nicht normal oben, sondern zwischendrin, und sie hat links und rechts Tabs. Eine zusätzliche Hürde ist noch, dass es für die Editseite teilweise nur einen Tab gibt, teilweise aber mehrere (pro Ctype). |
@@ -11,6 +11,7 @@ class rex_be_page implements rex_be_page_container | |||
$itemAttr = array(), | |||
$linkAttr = array(), | |||
$path, | |||
$subPath, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hier müssen wir irgendwie nen besseren namen finden, oder falls wir keinen finden die properties in der Klasse kommentieren
Die Ctypes wären die 3te Ebene, so dass der linke Tab immer "Editiermodus" heißen könnte und die Ctypes sich darunter anordnen. Wie im Installer, wo bereits eine 3te Ebene dabei ist. Vorteil wäre auch, dass andere Pages ebenfalls die 3te Ebene nutzen können. |
@tbaddade Ja stimmt, ctypes als dritte Ebene würde das schon mal sehr erleichtern. Dritte Ebene über normale Page-Objekte würde dann aber automatisch bedeuten, wir hätten dann Pages ala page=content/edit/ctype2. Fände ich aber ok. |
👍 |
👍 da es konsequent zu allen anderen Seiten ist |
Ich bin hier durch, @staabm ist dir noch irgendwas aufgefallen, oder kann ich mergen? |
Habs mir nicht im detail angesehen und nicht getestet, die grunsätzliche Idee find ich aber gut. Prinzipiell 👍 |
case 'sync' : $file = 'sync.php'; break; | ||
default : $file = 'media.php'; break; | ||
$pageObj = rex_be_controller::getCurrentPageObject()->getPage(); | ||
if ($pageObj->hasSubPath()) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
das hier find ich noch ein wenig ungeschickt... diese subPath Geschichte ist also ein Weg den der Core erlaubt, der aber nur in Addons funktioniert die dies hier mit diesen paar Zeilen Code dann auch immer vorsehen müssen?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ich finde es ok, dass Addons das explizit vorsehen müssen. Vor allem habe ich aber auch keine bessere Idee. :D
Die Addons haben über ihre Page nun mal die Kontrolle, daher müssen sie ja irgendwas machen, damit dann die Subpage auch eingebunden wird…
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Vllt vom Namen her nochn vorschlag: hasFixedSubPath
oder hasOverriddenSubPath
oder hasPathOverride
Idee zu folgendem aus redaxo/redaxo4#94
In der Verwaltung der "Artikel Metainfos" ein select anzeigen lassen, welche Pages sich als "Tab" registriert haben (Kein Registrierungen -> kein select -> Defaultpage "Metadaten"). Metainfo selbst verarbeitet dann die Felder automatisch. |
Es wäre total cool, wenn man sich nicht nur mit einem Link in die Content Header Menüleiste einhängen könnte, sondern mit einer eigenen PHP-Seite. Dann könnte man z.B. eine eigene SEO Page realisieren, so wie hier bereits per content.inc.php Hack geschehen:
https://www.dropbox.com/s/w8guzpvt4x02k7n/redaxo_seo_page.jpg
Oder z.B. das UrlReplace Addon könnte dann mit einer eigenen Page direkt im Artikel-Bereich vertreten sein und nicht wie bisher nur mit einem Link auf die Addon-Seite.