Hier kommt ein weiterer Test vom Plugin BookStore_XH (vorläufige Version 1.2.1alpha1 von
https://github.com/cmsimple-xh/bookstore (Stand: 27.11.2017) in der schon einige Fehler von cmb beseitigt worden sind).
Weiterhin wurden einige Anmerkungen aus vorangegangenen Threads als Anhang hinzugefügt, damit diese für die Weiterentwicklung des Plugins leichter auffindbar sind.
Zweiter Test von Plugin BookStore_XH
Getestet lokal unter
Xampp
PHP-Version 7.1.1
CMSimple_XH 1.7.1
Template fhs-simple
Plugin in der Sytem-Info sichtbar? => Ja (Bookstore )
Plugin im Admin-Menü sichtbar? => Ja
Plugin im Admin-Menü aufrufbar? => Ja
Debug-Modus-Fehlermeldungen beim Aufruf einer Seite mit dem Plugin Aufruf " ({{{PLUGIN:bookstore();}}}) " des jungfräulichen Bookstores (Bookstore Eingabe Formular sichtbar):
- WARNING: end() expects parameter 1 to be array, null given
- C:\xampp\htdocs\plugins\bookstore\index.php:271
- NOTICE: Undefined offset: 4
- C:\xampp\htdocs\plugins\bookstore\index.php:273
- WARNING: array_reverse() expects parameter 1 to be array, null given
- C:\xampp\htdocs\plugins\bookstore\index.php:278
- WARNING: array_slice() expects parameter 1 to be array, null given
- C:\xampp\htdocs\plugins\bookstore\index.php:434
- WARNING: Invalid argument supplied for foreach()
- C:\xampp\htdocs\plugins\bookstore\index.php:450
Anmerkung: Wenn mindestens ein Buch im Katalog eingetragen ist, erscheinen diese Fehlermeldungen nicht mehr.
Ein neues Buch anlegen und speichern: Funktioniert ohne Fehlermeldungen im Debug-Modus.
Ein vorhandener Bucheintrag änderbar: Ja, ohne Fehlermeldungen (Aufrufbar über das Plugin Adminmenü)
Aufruf eines Links der in der Buchbeschreibung hinterlegt wurde: Ja, funktioniert (Bei der Eingabe der URL darauf achten, dass die URL OHNE "https://" eingetragen wird, sonst steht es 2x in diesem Feld)
Über die XH-Suche nach einem Buch suchen: Funktioniert mit folgenden Fehlermeldungen im Debug-Modus:
- NOTICE: Undefined offset: 2
- C:\xampp\htdocs\plugins\bookstore\index.php:249
Ergebnis:
CMSimple_XH 1.6 Kompatibilität: Ja
CMSimple_XH 1.7 Kompatibilität: Ja*
Kompatibilität PHP 7.0: Ja
Kompatibilität PHP 7.1: Ja
Anmerkungen zu dem BookStore_XH Test (ToDo-Liste für BookStore_XH):
Im Eingabeformular eines neuen Buches wird das Template fehlerhaft dargestellt.
Frase wrote:
Das "Layout zerschossen"-Problem liegt an der verschiebbaren Eingabemaske für neue Bücher.
Die benutz (!) jQuery UI und bekommt von irgendwoher z-index: 101;
Ab z-index: 200; läge es oben.
cmb wrote:
Ah, danke fürs untersuchen. Ich frage mich halt, warum hier überhaupt ein JS basierter Dialog verwendet wird; könnte doch eigentlich ein normales HTML-Formular auf einer eigenen "Seite" sein. Jedenfalls würde ich fhs-simple deswegen nicht anpassen (z-index ist sowieso immer so eine Sache; hatte erst kürzlich eine Anfrage, wo z-index:1000 nicht genug war).
Fehlerhafte Darstellung im Admin-Modus:
Frase wrote:
Im Admin-Modus kommt aber wieder die UI-Body-Geschichte und der XH-html-Margin-Hack zusammen.
Das kann man umgehen, indem man in der CMS-Konfiguration mal "Bearbeitungsmenü (Admin-Menü) - Scrollen" anklickt.
Anmerkung: in der Doku darauf hinweisen.
frase wrote:
Aber über den html-Hack sollten wir wirklich nochmal nachdenken.
Ich glaube allerdings verstanden zu haben, was du meinst: jQueryUI nur einsetzen, wenn es nicht anders geht - und dann auch nur sparsam.
Dafür gibt es wirklich (einige) handfeste Gründe.
Diverse (viele) BookStore Style-Probleme beheben:
Frase wrote:
Es gibt auch noch viele Style-Probleme.
Z.B. sowas:
Code: Select all
<textarea class="bs_area" id="bs_edit_description_area" name="book_edit_description" rows="3" required="" style="display: block; overflow: hidden;" cols="100"></textarea>
cols="100" muss jedes Layout zerschießen.
cmb wrote:
Soweit ich weiß, wird das cols-Attribut von CSS width übersteuert. Da könnte man zumindest leicht nachbessern. Allgemein sollte man heutzutage vielleicht besser auf rows und cols verzichten (auch wenn ich die Defaults 2 und 20 fraglich finde). Unter HTML 4.01 waren die Attribute allerdings verpflichtend, wenn ich nicht irre.
Anzeige Mailadresse:
cmb wrote:
Anderes Problem: beim Hinzufügen eines neuen Buches steht im E-Mail-Eingabefeld "wird nicht angezeigt". Das passiert zumindest bei den Voreinstellungen dennoch, was rechtlich gesehen gar nicht gut ist.
Weitere Anmerkungen:
Hartmut wrote:
Um Bookstore in seinem Anwendungs Portfolio "etwas breiter zu positionieren" müsste noch etwas Zeit investiert werden. (bspw. Labels der Felder über eine Sprachdatei änderbar, Style-Probleme (soweit mit geringen Aufwand), konfigurierbare Forms für die Eingabe/Ansicht reduzieren, ...)
cmb wrote:
Ich würde empfehlen, das Plugin erst mal wenigstens von den Kinderkrankheiten zu befreien. Gibt man zum Beispiel bei einem neuen Buch als Benutzername <cmb an, wird ein quasi leerer Datensatz angelegt, aber nicht gelistet. Gibt man als Titel <my life ein, werden fast alle weiteren Angaben ignoriert, aber das Buch wird gelistet. Das Problem ist, dass strip_tags() auf den gesamten Datensatz angewendet (
https://github.com/cmsimple-xh/bookstor ... x.php#L688) wird, statt auf einzelne Felder.
cmb wrote:
Okay, ich habe Bookstore nun geforkt (
https://github.com/cmsimple-xh/bookstore), Version 1.2 eingespielt, und einige der besprochenen Probleme behoben. Besonders bzgl. der jQueryUI-Probleme bitte ich andere, sich darum zu kümmern.
Ich hoffe das ich nichts übersehen habe.
Viele Grüße
Hartmut