WBCE CMS – Way Better Content Editing.
Du bist nicht angemeldet.
Ich krieg das Gefühl nicht los, das $template->set_var einfach das JavaScript rauslöscht.
Evt. lässt es sich lösen wenn die modify.phps als Template Files mittels set_file ausgefüher werden. Werde es morgen ausprobieren.
Beitrag geändert von rjgamer (14.11.2017 17:03:00)
Offline
Während dem Duschen eingefallen. Die Ursache liegt mit hoher Wahrscheinlichkeit bei den geschweiften und eckigen Klammern in der Ckeditor-Initialisierung. Die werden einfach entfernt da die Template Engine meint es seien Variablenplatzhalter.
Ich versuch daher die Template Engine beim Regex Teil zu tweaken. Ok?
Beitrag geändert von rjgamer (14.11.2017 19:33:27)
Offline
Klingt plausibel (vielleicht sollte ich auch mal duschen gehen?).
Nur, wenn du jetzt an der Template-Engine rumbastelst und wir dann auf die nächste Inkombatibiltät warten, sind wir halt Ruck-Zuck bei den Bedenken, die Florian in #9 geäußert hat ...
... nein in Europa verwenden wir beim Programmieren nicht € statt $ ...
Offline
Ja leider. Ich probiers zuerst mit set_file.
Offline
Deine Vermutung scheint wirklich zu zu treffen.
Laut Debug-Ausgabe sollte dieser Scriptteil stehen:
<script type="text/javascript">//<![CDATA[ CKEDITOR.replace('content1', {"contentsCss":"http:\/\/localhost\/WBCE_CMS\/wbce\/templates\/wbce_vertal\/editor.css","stylesSet":"wb:http:\/\/localhost\/WBCE_CMS\/wbce\/modules\/ckeditor_dev\/wb_config\/editor.styles.js","templates_files":["http:\/\/localhost\/WBCE_CMS\/wbce\/modules\/ckeditor_dev\/wb_config\/editor.templates.js"],"customConfig":"http:\/\/localhost\/WBCE_CMS\/wbce\/templates\/wbce_vertal\/wb_ckconfig.js","filebrowserBrowseUrl":"\/WBCE_CMS\/wbce\/modules\/ckeditor_dev\/ckeditor\/filemanager\/browser\/default\/browser.html?Connector=\/WBCE_CMS\/wbce\/modules\/ckeditor_dev\/ckeditor\/filemanager\/connectors\/php\/connector.php","filebrowserImageBrowseUrl":"\/WBCE_CMS\/wbce\/modules\/ckeditor_dev\/ckeditor\/filemanager\/browser\/default\/browser.html?Type=Image&Connector=\/WBCE_CMS\/wbce\/modules\/ckeditor_dev\/ckeditor\/filemanager\/connectors\/php\/connector.php","filebrowserFlashBrowseUrl":"\/WBCE_CMS\/wbce\/modules\/ckeditor_dev\/ckeditor\/filemanager\/browser\/default\/browser.html?Type=Flash&Connector=\/WBCE_CMS\/wbce\/modules\/ckeditor_dev\/ckeditor\/filemanager\/connectors\/php\/connector.php","uploader":false,"extraPlugins":"wbdroplets,wblink,wbsave,wbshybutton,autolink,ckawesome,codemirror,lineutils,oembed,textselection,widget,widgetselection,syntaxhighlight","removePlugins":"wsc,link,save","removeButtons":"Font","fontawesomePath":"http:\/\/localhost\/WBCE_CMS\/wbce\/include\/font-awesome\/css\/font-awesome.min.css","height":"350","width":"100%"}); //]]></script>
Im generierten Quelltext bleibt davon lediglich dies übrig:
<script type="text/javascript">//<![CDATA[
CKEDITOR.replace('content1',
);
//]]></script>
Die komplette Konfiguration in {} wird unterschlagen
... nein in Europa verwenden wir beim Programmieren nicht € statt $ ...
Offline
Grade hat was "click" gemacht.
Das könnte die Lösung sein:
$template->set_unknowns('keep');
vor dem ob_start()
* unknowns defines what to do with undefined template variables
* "remove" = remove undefined variables
* "comment" = replace undefined variables with comments
* "keep" = keep undefined variables
"remove" ist default.
Beitrag geändert von bernd (14.11.2017 21:47:17)
... nein in Europa verwenden wir beim Programmieren nicht € statt $ ...
Offline
Bei mir auch Gleichzeitiges denken lohnt sich
Gott preise die set_unknowns-Methode von dem alten, seit Jahrtausenden (oder der Jahrtausendwende ) bestehendem Code in der include/phplib/template.inc
[== PHP ==]
// Disable removing of unknown vars to prevent the deleting of JavaScript array [] or {}
$template->set_unknowns('keep');
Patch-Request: https://github.com/WBCE/WBCE_CMS/pull/310
Beitrag geändert von rjgamer (14.11.2017 21:57:25)
Offline
Bitte die Implementation ausgiebig testen.
Offline
Nur der Vollständigkeit halber:
Nein, ich war nicht duschen
... nein in Europa verwenden wir beim Programmieren nicht € statt $ ...
Offline
Hab den Fix reingemerged
Offline
rjgamer
Danke! Aber leider noch eine Inkompatibilität festgestellt.
Und zwar wenn zB. im Code2 Modul auch ein Template mit der Variable $template instantiiert wird, wird die $template-Variable von admin/pages/modify.php überschrieben. Hab jetzt die Variable einfach umbenannt in $pageModifyTemplate.
Neuer Commit: https://github.com/WBCE/WBCE_CMS/pull/311
Beitrag geändert von rjgamer (15.11.2017 13:54:29)
Offline
#311 ist jetzt auch gemerged
Offline
Danke und sorry für die Umstände.
Da ich aber das Ganze auch parallel in einer neuen Webseite implementiere, sind mir zum Glück auch gleich die Fehler aufgefallen. Lieber jetzt als erst bei einem neuen WBCE-Release.
Gruss
Offline
florian, colinax