Traumflug Posted November 22, 2017 Posted November 22, 2017 Eigentlich wollte ich gestern diese Macke im AEUC-Modul angehen, bei der irgendwie doppelte Texte erscheinen. Oder sowas Ähnliches. Selber habe ich das noch nie gesehen und ich kann es auch nicht reproduzieren, siehe https://github.com/thirtybees/thirtybees/issues/104#issuecomment-346155268 Deswegen wäre es grossartig, wenn mir jemand helfen könnte, diese Macke ans Tageslicht zu befördern. Man kann nur Macken beheben, die sich auch reproduzieren lassen. Es braucht wohl ein paar mehr Details, wie man zu diesen doppelten Texten kommt. Besten Dank für die Hilfe. Bildschirmfotos, Schritt-für-Schritt-Anleitungen für einen frisch installierten Shop, alles ist willkommen. Reproduzierbarkeit muss her. Ähm, nicht bei Euch, sondern bei mir :-)
zimmer-media Posted November 23, 2017 Posted November 23, 2017 @traumflug Ich hatte das Problem bei Prestashop mit AEUC, ich kann mich momentan aber leider nicht mehr daran erinnern, was ich genau getan habe. Da weiß ich jetzt nur noch, das ich im PS-Forum eine Lösung gefunden hatte und diese in den betroffenen Shops umgesetzt habe.
Traumflug Posted November 24, 2017 Author Posted November 24, 2017 Prima, vielen Dank! Wenigstens mal Einer der bestätigt, dass es das Problem überhaupt gibt. Ich war schon kurz davor, das Github Issue einfach als erledigt zu markieren. Wenn das niemand reproduzieren kann, gibt es auch kein Problem. Jetzt werden wir wohl warten müssen bis sich mal jemand daran erinnert, wie und wo das Problem überhaupt auftaucht. Und das dann auch noch hier oder auf Github beschreibt.
zimmer-media Posted November 24, 2017 Posted November 24, 2017 @traumflug ich habe mal nachgeschaut, bin mir aber nicht mehr sicher welche Lösung ich angewandt habe https://www.prestashop.com/forums/topic/338872-anzeige-versandkosten-mwst-doppelt-layoutfehler/ https://www.prestashop.com/forums/topic/541316-incl-mwst-weg-zzgl-versand-weg-lieferzeit-doppelt-auf-startseite/
Traumflug Posted November 24, 2017 Author Posted November 24, 2017 Der zweite Link ist ziemlich gut, vielen Dank. Da wird mal ein wenig die Ursache erörtert, nicht nur billiges Heftpflaster verteilt.
DRMasterChief Posted November 25, 2017 Posted November 25, 2017 Bestätigen ich diesen Fehler kann :) Ich habe eine Zeit lang mit PS 1.6.1.15 + AEUC rumgespielt. Der SmartyCache musste ausgeschaltet bleiben, sonst hat man diesen Fehler umgehend. Das schwierige daran scheint zu sein, daß auch das Entwicklerteam die genaue Ursache bisher nicht ergründet hat oder sich darum kümmern mag. Der Fehler "springt" auch ein wenig im Dreieck, denn mal geht es einige Stunden/Tage ohne Fehler und plötzlich taucht er im Shop auf, geht dann auch nicht mehr weg. Neukompilieren oder Cache löschen hat da teilweise auch Einfluss drauf.
Traumflug Posted November 25, 2017 Author Posted November 25, 2017 Wegens der Bestätigung: wir haben hier nicht PS 1.6.irgendwas, sondern thirty bees :-) Nachdem ich mich tief genug in diese Geschichte hinein gebohrt habe, habe ich das hier gefunden: https://github.com/thirtybees/advancedeucompliance/commit/46ad6f900e4d2c2b629ff8ffe20f17795db55d6b Sprich: die Macke wurde schon vor einem Monat behoben, es hat nur niemand das zugehörige Github Issue geschlossen. Das habe ich jetzt nachgeholt. Da ich die ganze Sache schon untersucht hatte bevor ich den Fix entdeckt habe, gibt es jetzt auch eine ausführliche Erklärung, was los war und welche alternativen Lösungsmöglichkeiten es gibt: https://github.com/thirtybees/thirtybees/issues/104#issuecomment-346943339 Fall erledigt. Besten Dank für die Mithilfe!
DRMasterChief Posted November 25, 2017 Posted November 25, 2017 Hatte ich auch nicht gesehen daß TB da dran ist, sorry daher.... also sollten wir advancedeucompliance/views/templates/hook/hookDisplayProductPriceBlock.tpl austauschen und das wars dann? Wäre ja zu schön :)
Traumflug Posted November 25, 2017 Author Posted November 25, 2017 Man braucht gar nichts machen. Diese geänderte Datei kommt automatisch mit dem Update.
DRMasterChief Posted November 25, 2017 Posted November 25, 2017 Jo dann wirds aber gruselig, habe das gerade bei mir im 1.0.3 versucht, und ich habe den Fehler sobald ich die Datei ändere :( (vorher nicht), allerdings verwende ich ein anderes Theme. Dann würde mir ein TB-Update wohl Probleme dahingehend machen, ohje muss ich gleich mal gucken was da los sein könnte.
Traumflug Posted November 25, 2017 Author Posted November 25, 2017 Welche Version hat denn das AEUC-Modul? Es sollte die Version 3.1.1 sein. Der Fix ist seit Version 3.0.1 drin. Kann man unter Backoffice -> Modules & Services sehen, da steht die Version bei jedem Modul. Es ist natürlich möglich, dass Dein Theme eine eigene Datei hookDisplayProductPriceBlock.tpl hat. Die wird dann anstatt der gleichnamigen Datei im Modul genommen. Wenn das so ist, diese Datei entfernen oder wie in diesem Commit bearbeiten: https://github.com/thirtybees/advancedeucompliance/commit/46ad6f900e4d2c2b629ff8ffe20f17795db55d6b Das ist die erste grüne und die letzte grüne Zeile, die hinzu gefügt wird. Der Rest der Änderung in dieser Datei ändert nur den Whitespace, der spielt für die Funktion keine Rolle.
DRMasterChief Posted November 26, 2017 Posted November 26, 2017 Ja habe die AEUC 3.1.1 Mein Theme hat KEINE eigene Datei hookDisplayProductPriceBlock.tpl Wenn ich als Kunde eingeloggt bin, funktioniert ohne Smartycache alles wie es soll. Wenn ich Smartycache einschalte, sehe ich auf der Startseite keine Angaben mehr zu inkl. MwSt. zzgl. Versandkosten (in den Kategorien und Artikeldetails aber schon noch). Werde da also mal noch basteln müssen.
Traumflug Posted November 26, 2017 Author Posted November 26, 2017 Du kannst ja mal in die hookDisplayProductPriceBlock.tpl im Modul schauen, ob das {nocache} da drin ist. Ist gleich die erste Zeile nach dem Lizenz-Kommentar. Ansonsten will ich nicht ausschliessen, dass es da noch eine zweite Macke gibt.
Traumflug Posted November 26, 2017 Author Posted November 26, 2017 Habe auch nochmal ein wenig probiert, das wird ja immer lustiger. Bei mir ist es genau umgekehrt. Schalte ich den Cache ein, sieht es gut aus. Schalte ich ihn aus, fehlen die Mwst.-Angaben. Wenn ich dann beim Modul auf "Configure" klicke, bekomme ich eine DB Exception: Table 'thirtybees-dev.tb_aeuc_email' doesn't exist Da muss ich wohl nochmal ran.
DRMasterChief Posted November 26, 2017 Posted November 26, 2017 Ja das {nocache} ist drin, es ist die unveränderte Datei aus dem AEUC 3.1.1 Zum "probieren" musst du auch mal versuchen wie es sich verhält wenn du als Admin und/oder Kunde angemeldet bzw. Gast (nur "Zuschauer") bist, denn da erhält man durchaus unterschiedliche Anzeigen. Wurde bei PS im Forum auch schonmal so angesprochen, macht das natürlich nicht einfacher und wo da der Hase im Pfeffer liegt wusste bisher keiner. Ist bei Dir das Verhalten als Kunde gleich wie bei Admin?
DRMasterChief Posted February 10, 2018 Posted February 10, 2018 Neue Lösung in Sicht ?? https://www.prestashop.com/forums/topic/662685-download-aeuc-europ%C3%A4ische-rechtssicherheit-verbessertes-modul-ohne-cachefehler/ Fehler in der foreach -Schleife behoben?? Vielleicht auch für mdekker interessant, da der Fehler in TB scheinbar noch immer vorhanden ist: $newhook->name = $hookname; $new_hook->title = $hook['name']; //$newhook->title = $hookname; $new_hook->title = $hook['name']; Achtung, ich muss sagen daß ich diese "Lösung" nicht versucht habe, da es damit teilweise auch noch Probleme zu geben scheint. Was da genau die Ursache für die erneuten Probleme ist kann ich nicht sagen, weiß ich nicht. Sollte jeder selbst lesen und ergründen und zuletzt in seiner eigenen Konfiguration testen (vorher auf jeden Fall Backups aller dieser Dateien machen um schnell wieder zurück zu können) ! ! ! !
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now