wellrad-Shop 1.3 (cmb)

Ein CMSimple Support Forum für deutsch sprechende Nutzer und Entwickler
albert
Posts: 526
Joined: Sun Mar 07, 2010 8:01 pm
Location: Germany
Contact:

Re: wellrad-Shop 1.3 (cmb)

Post by albert » Fri Jun 30, 2017 10:44 am

frase wrote: Probiert: Keine Änderung.
bei mir sah es so aus, als würde der Pfad zu den Bildern nicht gefunden

cmb
Posts: 14225
Joined: Tue Jun 21, 2011 11:04 am
Location: Bingen, RLP, DE
Contact:

Re: wellrad-Shop 1.3 (cmb)

Post by cmb » Fri Jun 30, 2017 10:48 am

frase wrote:Ich hatte allerdings auch vor, den Code a bissl "aufzuräumen". Alte Kommentare entfernen, "/>" alte XH-Ends beseitigen usw.
Ich denke, ist es grundsätzlich gut alles in kleinen Häppchen zu machen, so dass man eine brauchbare Commit-Message hat, und die Änderungen nachvollziehen kann.

Ich schlage vor, erst mal nur die Umbennung fertig zu machen (falls noch nicht passiert), und dann mir das ZIP zu schicken – ich stell's dann auf Github ein, und dann kann man weitersehen.
frase wrote:Es ist wohl auch eher der Filebrowser (oder FileManager oder wie der heißt ...).
Okay, ich schau's mir an.
Christoph M. Becker – Plugins for CMSimple_XH

cmb
Posts: 14225
Joined: Tue Jun 21, 2011 11:04 am
Location: Bingen, RLP, DE
Contact:

Re: wellrad-Shop 1.3 (cmb)

Post by cmb » Fri Jun 30, 2017 10:52 am

cmb wrote:
frase wrote:Es ist wohl auch eher der Filebrowser (oder FileManager oder wie der heißt ...).
Okay, ich schau's mir an.
Hmm, auch in XH 1.7 mit TinyMCE 4 klappt's bei mir. Ich schlage vor, das Thema ruhen zu lassen, bis wir eine gemeinsame Shop-Version haben.
Christoph M. Becker – Plugins for CMSimple_XH

frase
Posts: 5085
Joined: Thu Apr 21, 2016 6:32 am
Location: Saxony
Contact:

Re: wellrad-Shop 1.3 (cmb)

Post by frase » Fri Jun 30, 2017 10:52 am

Filebrowser:
Console meldet:
GET
http : ... /assets/css/stylesheet.css

frase
Posts: 5085
Joined: Thu Apr 21, 2016 6:32 am
Location: Saxony
Contact:

Re: wellrad-Shop 1.3 (cmb)

Post by frase » Fri Jun 30, 2017 11:13 am

Lösung für den Filebrowser (unter XH 1.7):
Datei:
..\plugins\filebrowser\tpl\editorbrowser.html

Zeile 8 ändern:

Code: Select all

<link rel="stylesheet" href="<?php echo "{$pth['folder']['corestyle']}xhstyles.css"?>" type="text/css">
Plötzlich kann man Bilder einfügen!

cmb
Posts: 14225
Joined: Tue Jun 21, 2011 11:04 am
Location: Bingen, RLP, DE
Contact:

Re: wellrad-Shop 1.3 (cmb)

Post by cmb » Fri Jun 30, 2017 11:27 am

frase wrote:Lösung für den Filebrowser (unter XH 1.7):
Ah, danke! Manu hatte das in einem PR nebenbei gefixt, und den hatte ich gerade ausgecheckt. Mir war nicht klar, das es dabei um einen Bugfix ging – ist jetzt behoben.
Christoph M. Becker – Plugins for CMSimple_XH

frase
Posts: 5085
Joined: Thu Apr 21, 2016 6:32 am
Location: Saxony
Contact:

Re: wellrad-Shop 1.3 (cmb)

Post by frase » Fri Jun 30, 2017 11:31 am

frase wrote:Plötzlich kann man Bilder einfügen!
Korrektur:
Man kann zwar einfügen - funktioniert aber nicht.
In data/catalog.php wird aus dem Link:

Code: Select all

<p><a class=\\\"zoom_g\\\" title=\\\"Brötchen\\\" href=\\\"userfiles/images/shop/bread.jpg\\\"><img title=\\\"Brötchen\\\" src=\\\"userfiles/images/shop/preview_bread.jpg\\\" alt=\\\"Brötchen\\\" width=\\\"100\\\"></a></p>';
Und das ist Mist.
Da wird irgendwo wieder escaped oder so.

cmb
Posts: 14225
Joined: Tue Jun 21, 2011 11:04 am
Location: Bingen, RLP, DE
Contact:

Re: wellrad-Shop 1.3 (cmb)

Post by cmb » Fri Jun 30, 2017 11:40 am

frase wrote:Da wird irgendwo wieder escaped oder so.
Ja, siehe https://cmsimpleforum.com/viewtopic.php ... =20#p60529.
Christoph M. Becker – Plugins for CMSimple_XH

frase
Posts: 5085
Joined: Thu Apr 21, 2016 6:32 am
Location: Saxony
Contact:

Re: wellrad-Shop 1.3 (cmb)

Post by frase » Fri Jun 30, 2017 11:48 am

cmb wrote:Ja, siehe viewtopic.php?f=16&t=12593&start=20#p60529.
Ich erinnere mich. Mist.

cmb
Posts: 14225
Joined: Tue Jun 21, 2011 11:04 am
Location: Bingen, RLP, DE
Contact:

Re: wellrad-Shop 1.3 (cmb)

Post by cmb » Fri Jun 30, 2017 11:58 am

frase wrote:
cmb wrote:Ja, siehe viewtopic.php?f=16&t=12593&start=20#p60529.
Ich erinnere mich. Mist.
Wenn wir mindestens PHP 5.4.0 voraussetzen (was IMO kein Problem sein sollte, und auch andere Vorteile hätte), dann könnten einfache alle stripslashes() entfernt werden, und man müsste die paar addslashes() mal durchgehen, und entweder entfernen, oder zu addcslashes() ändern. Der Aufwand hält sich dann wohl in Grenzen.

PS: Oder einfach var_export() verwenden, um catalog.php zu beschreiben; das ist ohnehin viel einfacher – sieht halt etwas anders aus.
Christoph M. Becker – Plugins for CMSimple_XH

Post Reply