Kartal1984 Posted May 28, 2013 Share Posted May 28, 2013 Hallo, ich habe bei der aktuellen Presta Shop Version so einige Fix-Dateien von euch (1-Button-Lösung, Widerrufsbelehrung, Preise exckl. Versand, PayPal) hochgeladen bisher sieht auch alles gut aus bis auf, dass auf der Startseite die Menüleiste oben nach rechts verrutscht ist sowie das Feld der Produktsuche. Was lief hier verkehrt? Link to comment Share on other sites More sharing options...
guest* Posted May 28, 2013 Share Posted May 28, 2013 Link zur Seite wäre hilfreich, damit man helfen kann. Link to comment Share on other sites More sharing options...
Kartal1984 Posted May 28, 2013 Author Share Posted May 28, 2013 Um folgenden Shop geht es Link to comment Share on other sites More sharing options...
guest* Posted May 28, 2013 Share Posted May 28, 2013 Das Problem Navigations-Menu hat nichts mit den Fixen zu tun. Nachdem du die Fixe eingebaut hast, hat sich der smarty-Cache neu generiert und zeigt dir die Anzeige ohne Cache an. Das Problem Navigations-Menu ist ein Bug und findest du hier: http://www.prestasho...1/page__hl__top Link to comment Share on other sites More sharing options...
Kartal1984 Posted May 29, 2013 Author Share Posted May 29, 2013 Aber komisch ist es schon oder nicht? Denn das ganze ist aufgetaucht nachdem ich die Fix-Dateien aufgespielt habe und vorher Testmodus aktivieren musste laut Anleitung von dir/euch was die Fix Dateien einspielen angeht. Ehrlich gesagt finde ich in dem Link was du gepostet hast nur ein hin und her, was genau ist da in den Post nun die Lösung? Deine mit der CSS Änderung? Diese habe ich gemacht, keine Veränderung. Link to comment Share on other sites More sharing options...
eleazar Posted May 29, 2013 Share Posted May 29, 2013 ich habe zu diesem Thema einen Post geschrieben: http://www.prestashop.com/forums/topic/251595-anderung-im-template-in-der-version-1541/ so wie ich die Diskussion verstehe war in den Fixen die header.tpl nicht upgedatet. Es hätte aber auch nicht gereicht, weil das Logo absolut positioniert werden muß, sonst rutscht es hoch, wenn der rechte Teil über die ganze Breite geht. Das sollte in dem Update des Fixes für 1541 und die, die das Thema nicht updaten, auch berücksichtigt werden. Vllt eine Inline-CSS verwenden, oder eine eigene CSS einbinden, die die alte Einstellung überschreibt, jedenfalls scheint es wirklich am Fix zu liegen. Sind aber zu wenige Zeichen, um so viel darüber zu diskutieren Gruß und schönen Somer Link to comment Share on other sites More sharing options...
Kartal1984 Posted May 29, 2013 Author Share Posted May 29, 2013 Hallo eleazar, hat geklappt :-) was ist da passiert? Ich habe festgestellt, dass bei der header.tpl. bei diesem Post http://www.prestashop.com/forums/topic/239639-fixpreisanzeige-inkl-steuer-exkl-versand-ps-15-fancybox/ wegen Versandkostenanzeige die TPL genau so war und das das ganze ausgelöst hat? Link to comment Share on other sites More sharing options...
guest* Posted May 29, 2013 Share Posted May 29, 2013 Habe ich doch schon weiter oben erklärt, dass das absolut nichts mit dem Fix zu tun hat. Wenn du an den Templatest etwas änderst, oder force compile schaltest, dann wird smarty komplett neu kompiliert. Deshalb ist dir erst zu diesem Zeitpunkt der Fehler aufgefallen, weil du keine gecachte Anzeige mehr gesehen hast, sondern die fehlerhafte die schon immer fehlerhaft war, aber eben wegen des caches nicht zum Vorschein kam. Die Fixe haben keine Fehler. Diese sind schon hunderte von Male eingesetzt worden ohne Probleme. Außer die, die User selbst produzieren, weil sie nicht korrekt lesen, oder kein Standard-Template verwenden. Link to comment Share on other sites More sharing options...
Kartal1984 Posted May 29, 2013 Author Share Posted May 29, 2013 Mir ging es ums Prinzip und das ich es verstehe, da ich an dem Template ja nix verändert habe bis auf das ich die Dateien die der Fix-Angelegenheiten dient eingespielt habe. Plus habe ich einen Wert verändern was hier gesagt wurden ist und alles war in Ordnung. Deshalb kann ich der Behauptung das vorher das Template kaputt war nicht nachvollziehen, wenn ich daran nichts gemacht habe. Link to comment Share on other sites More sharing options...
guest* Posted May 29, 2013 Share Posted May 29, 2013 Das Prinzip ist, dass du den Cache abschaltest und ihn dann neu kompilierst und dann wieder einschaltetst. Hat nichts mit dem Fix zu tun. Man sollte sowieso bei Fremdmodule und Fremdthemes bevor Installation immer den Cache ausschalten und die Kompilierung auf erzwingen. So hat man sofort eine korrekte Anzeige von dem was man installiert hat und nicht in der Regel erst Tage später, wenn der Server vom Provider irgendwann mal neu gestartet wurde... Link to comment Share on other sites More sharing options...
Kartal1984 Posted May 29, 2013 Author Share Posted May 29, 2013 Ähhhm, ich habe weder ein Template noch ein fremdes Modul gehabt habe die nackte Presta originla Version mit den Fix-Dateien begonnen zu bearbeiten. Und bin dabei strick der Beschreibung nachgegangen. Aber gut verstehe brauche ich es nicht auch wenn ich das originale Theme mit PResta genommen habe ohne ein fremdes Modul. Link to comment Share on other sites More sharing options...
eleazar Posted May 30, 2013 Share Posted May 30, 2013 (edited) Hi, Kartal1984, du hast übrigens vollkommen recht mit deinem Hinweis auf den Fix Fancybox. Leider scheint sich CD2500 ihre eigenen Veröffentlichungen nicht genau anzusehen. Denn wenn man die header.tpl der fancybox ansieht, dann ist hier derselbe Fehler drin (Zeile 86-88): <div id="header_right" class="grid_6 omega"> {$HOOK_TOP} </div> Und es sollte nun mal, um zu vermeiden, dass das Logo den Header verschiebt, heißen: <div id="header_right" class="grid_9 omega"> Es ist der Fix, der das Problem auslöst. Da könntest du den Cache noch so oft löschen. Edited May 30, 2013 by eleazar (see edit history) Link to comment Share on other sites More sharing options...
guest* Posted May 30, 2013 Share Posted May 30, 2013 Das ist nicht der Fix, der das auslöst. Die header.tpl ist die Original Prestashop mit angepassten Zeilen. Wenn diese Datei einen Fehler von Haus aus enthält, dann kann der Fix nichts dafür. Ist ein Bug von Prestashop und nicht vom Fix. Trotzdem Danke für den Hinweis. Wurde bereits im Fix ergänzt. Link to comment Share on other sites More sharing options...
eleazar Posted May 30, 2013 Share Posted May 30, 2013 Komm, das ist völlig egal, denn er stammt aus der von dir verteilten ZIP-Datei. Ansonsten ... you're welcome. Link to comment Share on other sites More sharing options...
Kartal1984 Posted May 31, 2013 Author Share Posted May 31, 2013 (edited) Edited May 31, 2013 by Kartal1984 (see edit history) Link to comment Share on other sites More sharing options...
Lockesoft Posted May 31, 2013 Share Posted May 31, 2013 Mannomann ist das peinlich. Man müsste sich doch nur mal kurz die beiden Dateien ansehen um zu merken, dass die eigene Aussage falsch ist. Die mit dem Fix verteilte header.tpl hat(te) den Fehler drin, ist also maßgeblich schuld an den Darstellungsfehlern. Stattdessen wird wiederholt behauptet, dass es nicht so ist. Wahre Größe wäre es gewesen sich das wirklich selber mal anzusehen und sich dann zu entschuldigen. Das ein Fehler im Fix war ist sicher nicht das Problem, sowas kann passieren, aber so ein Theater ist einfach nur peinlich und sicher kein guter Dienst für PrestaShop. Link to comment Share on other sites More sharing options...
guest* Posted May 31, 2013 Share Posted May 31, 2013 Was soll denn peinlich sein. Die Datei enthält keinen Fehler, sondern ist eine Original-Datei von Prestashop. Es wäre einmal an der Zeit, dass du eleazar und auch du Lockesoft nicht ständig Dinge moniert, bei welchen ihr weder mitgeholfen habt, noch den Code angepasst habt und von daher 0 Ahnung habt was zur Basis hergenommen wurde (nachträglich Code vergleichen das kann jeder, ohne zu wissen was die Basis war). Die Datei ist Original und es wurden lediglich die Zeilen mit der Fancy-Box ergänzt. Peinlich, und da gebe ich dir Lockesoft Recht, ist vielmehr, dass du Kritik an der Arbeit anderer ausüben mußt ohne konstruktiv dazu irgend etwas beigetragen zu haben. Kannst du es besser, dann bist du ja eingeladen in Zukunft selbst die Fixe in die Hand zu nehmen. Auch scheinst du sehr inkonsequent in deinen Aussagen zu sein lieber Lockesoft. Im Fix selbst, hast du die Funktionsweise überprüft und als alles korrekt übernommen und jetzt kritisierst du hier, dass es nicht funktioniert ? Was nun ? Erst ist es OK, dann wieder nicht ? http://www.prestasho...ost__p__1186776 Entschuldigen kann man sich nur, für etwas wo man selbst einen Fehler gemacht hat. Ist ja hier nicht der Fall weil es eine Original-Datei ist mit Ergänzumg, aber keinerlei Änderungen. Link to comment Share on other sites More sharing options...
Lockesoft Posted June 2, 2013 Share Posted June 2, 2013 Ok, ich soll ein Statement abgeben, dann werde ich das mal tun. Fakt ist, dass die Datei aus dem Installationsarchiv 1.5.4.1 ohne Fehler ist. <!-- Header --> <div id="header" class="grid_9 alpha omega"> <a id="header_logo" href="{$base_dir}" title="{$shop_name|escape:'htmlall':'UTF-8'}"> <img class="logo" src="{$logo_url}" alt="{$shop_name|escape:'htmlall':'UTF-8'}" {if $logo_image_width}width="{$logo_image_width}"{/if} {if $logo_image_height}height="{$logo_image_height}" {/if} /> </a> <div id="header_right" class="grid_9 omega"> {$HOOK_TOP} </div> </div> In der mit dem Fix verteilten Datei und das ist nunmal in einem Posting von cd2500 geschehen, also sehe ich Sie da als Herausgeber an ist der Fehler allerdings vorhanden. <!-- Header --> <div id="header" class="grid_9 alpha omega"> <a id="header_logo" href="{$base_dir}" title="{$shop_name|escape:'htmlall':'UTF-8'}"> <img class="logo" src="{$logo_url}" alt="{$shop_name|escape:'htmlall':'UTF-8'}" {if $logo_image_width}width="{$logo_image_width}"{/if} {if $logo_image_height}height="{$logo_image_height}" {/if} /> </a> <div id="header_right" class="grid_6 omega"> {$HOOK_TOP} </div> </div> Es ist sogar so, dass es sich um Orginalcode handelt, zumindest was diese Stelle im Quellcode angeht, ABER eben nicht von von einer 1.5.4.1, den der ist an der Stelle fehlerfrei. Es ändert aber nichts an der Tatsache, dass mit der fehlerhaften Orginaldatei der Fix ebenfalls fehlerhaft war. Warum das ganze bei mir nicht auftritt ist ganz einfach zu begründen, dass ich mir die Fixe angesehen habe und festgestellt habe, dass dort nichts anderes gemacht wird, als das, was ich mittlerweile seit Jahren in die Kundenshops hier einbaue. Einziger Unterschied war: Es wird ein extra Template eingebunden, wo ich die Texte direkt eingefügt habe. Die Idee finde ich OK, also werde ich meine Änderungen in Zukunft auch so lösen. Ich nutze aber ganz speziell das header.tpl nicht, da ich das Fancybox-JS mit einem Modul in den Header lade. Einem Modul, dass auch direkt die Overrides mit installiert. Derzeit ist da aber noch ein kleiner Fehler drin, darum ist das noch nicht veröffentlicht. Btw. hat BluTiges gerade erst selber ein ähnliches Modul veröffentlicht. Ich bin also durchaus in der Lage zu einem positiven Voting für die Fixe zu kommen. Der Fix funktioniert ja soweit. Nur mal so nebenbei gefragt, ausser dass die Fixe mit Deinen Postings vertrieben werden, was ist Dein Anteil daran cd2500? Für mich lesen sich sämtlich Threads zu den Fixen, als würdest Du ja die Macherin sein, aber de Fakto erledigt BluTiges dann die Arbeit. Desweiteren bleibe ich bei meiner Meinung, dass es peinlich ist, wenn man wiederholt auf etwas aufmerksam gemacht wird und sich nicht ein einziges Mal die Mühe macht nachzusehen, ob da evtl. etwas dran sein könnte. Ganz sicher, wenn das mit einem von mir entwickelten oder vertriebenen Stück Software passieren würde, kannst Du davon ausgehen, das ich der Erste bin, der sich das ganze von allen Seiten betrachtet und wenn der Fehler in dem von mir vertrieben Code steckt sich ebenso so als erster entschuldigt. Das Verhalten erwarte ich umso mehr, desto größer die Sprüche sind um wieviel mehr man ja Ahnung hat In keinem Fall würde ich mich dahin stellen und mit Großen Worten einfach immer wieder behaupten, dass alle anderen lügen und schon mal gar keine Ahnung haben. Der Fehler steckte in dem Fix und selbst wenn es Orginalcode war, die aktuelle Version hat den Fehler nicht mehr drin Was die angeblichen persönlichen Angriffe angeht. Wen soll ich ansprechen? Die Allgemeinheit? Die hat nicht behauptet, dass das alles nicht stimmt. Es war nur eine Person und die habe ich nicht einmal namentlich erwähnt. Und als letzter Gedanke noch..... Moderationsrechte für das deutsche Forum dürften auch alle globalen Moderatoren haben. Zumindest steht bei denen als Bereich überall "Alle Foren" dran. Evtl. kann ein Admin nachvollziehen, wer, wann und warum etwas verschoben hat. Bei den meisten Forensystemen geht sowas. In diesem Sinne einen schönen Abend noch LG Klaus / Lockesoft Link to comment Share on other sites More sharing options...
guest* Posted June 3, 2013 Share Posted June 3, 2013 Stellungnahme bezüglich der Phantomgeschichte und Moderatorrechte erfolgte hier. http://www.prestashop.com/forums/index.php?/topic/248311-update-problem/page__view__findpost__p__1253227 Link to comment Share on other sites More sharing options...
Xavier du Tertre Posted June 3, 2013 Share Posted June 3, 2013 This topic is now closed, for the same reason this one was (link). I hope that is the last one I have to intervene on. Please bury the hatchet, for your own credibility's sake. Link to comment Share on other sites More sharing options...
Recommended Posts