Jump to content

Problem mit Navigationsleiste nach Neukompilierung des Themas


Recommended Posts

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

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

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

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

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

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

Ä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

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 by eleazar (see edit history)
Link to comment
Share on other sites

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

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

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

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

Guest
This topic is now closed to further replies.
×
×
  • Create New...