Fehlende Überschrift (leerer h1 Tag) in advanced_search_result.php

[Modulfux]

Soeben ist mir aufgefallen, dass im Bereich der advanced_search_result.php das <h1>-Tag leer bleibt.
Fehler wird sein, dass in der Ergebnislistung durch die product_listing.html kein SMARTY-Tag für die Überschrift deklariert wird.

Lösung habe ich jetzt auf die schnelle nicht parat.

Gruß
Ronny



Linkback: https://www.modified-shop.org/forum/index.php?topic=8268.0

[franky_n]

Hallo webkiste,

advanced_search_result.php in Zeile 268:

Code: PHP  [Auswählen]

    $listing_sql = $select_str.$from_str.$where_str;
    require (DIR_WS_MODULES.FILENAME_PRODUCT_LISTING);

ändern in:

Code: PHP  [Auswählen]

    $listing_sql = $select_str.$from_str.$where_str;
    $advanced_search_header = NAVBAR_TITLE2_ADVANCED_SEARCH;
    require (DIR_WS_MODULES.FILENAME_PRODUCT_LISTING);

includes/modules/product_listing.php in Zeile 60 nach:

Code: PHP  [Auswählen]

    $module_smarty->assign('CATEGORIES_IMAGE', $image);
    $module_smarty->assign('CATEGORIES_DESCRIPTION', $category['categories_description']);

das einfügen:

Code: PHP  [Auswählen]

        $module_smarty->assign('ADVANCED_SEARCH_HEADER', $advanced_search_header);

templates/xtc5/module/product_listing/product_listing_v1.html in Zeile 2:

Code: PHP  [Auswählen]

{if $CATEGORIES_HEADING_TITLE}
  <h1>{$CATEGORIES_HEADING_TITLE}</h1>
{else}
  <h1>{$CATEGORIES_NAME}</h1>
{/if}
 

ändern in:

Code: PHP  [Auswählen]

{if $CATEGORIES_HEADING_TITLE}
  <h1>{$CATEGORIES_HEADING_TITLE}</h1>
{elseif $CATEGORIES_NAME}
  <h1>{$CATEGORIES_NAME}</h1>
{else}
  <h1>{$ADVANCED_SEARCH_HEADER}</h1>
{/if}
 

Dann sollte eine Überschrift erscheinen.

Viele Grüße

Franky

[EDIT Tomcraft 04.11.2010: Code korrigiert, Danke an Spritzpistole für die Lösung.]

[speedy]

Was sich selbst nach so vielen Jahren noch alles für Bugs finden, erstaunlich.
Mich wundert es manchmal, dass es früher überhaupt funktionierte.

[Modulfux]

Danke franky, das ist die perfekte Lösung. Danke dafür.

Gruß
Ronny

[koshiro]

Hallo Franky,
super Lösung, jetzt fehlt ja nur noch so eine Lösung für die Hersteller, denn der H1 Tag bleibt ja auch leer.
Gruß
Micha

[franky_n]

Hallo koshiro,

in der /includes/modules/default.php in Zeile 559 vor dem:

Code: PHP  [Auswählen]

  include (DIR_WS_MODULES.FILENAME_PRODUCT_LISTING);

das einfügen:

Code: PHP  [Auswählen]

  $advanced_search_header = NAVBAR_TITLE2_ADVANCED_SEARCH;

EDIT: Wichtig ist das obige Änderung "Überschrift advanced_search" vorher gemacht wurde!

Viele Grüße

Franky

[koshiro]

Hallo franky,
nein macht er so nicht, bleibt leer.
Gruß
Micha

PS: Sorry mein Fehler geht, hatte noch was im Cache. Aber kann man das eventuell hinbekommen, das er bei den Herstellern anstatt Suchergebnisse den Name des Herstellers ausgibt ?

[koshiro]

Hallo franky,
ich weiss nicht ob du meinen Betrag nochmal gelesen hattest, da ich den im Nachhinein noch mal editiert hatte.
Kann man das bei den Herstellern auch so lösen, dass der Herstellername anstatt Suchergebnisse ausgegeben wird?
Gruß
Micha

[franky_n]

Hallo koshiro,

hab's gelesen aber kann derzeit nur alle 30 Minuten was posten. Andere sind ja auch noch dran!

Die letzte Änderung muss dann erweitert werden.

In der /includes/modules/default.php in Zeile 559 das dann wieder

Code: PHP  [Auswählen]

$advanced_search_header = NAVBAR_TITLE2_ADVANCED_SEARCH;

ändern in:

Code: PHP  [Auswählen]

  if (isset ($_GET['manufacturers_id'])) {
    $manufacturers_name = xtDBquery("select manufacturers_name from ".TABLE_MANUFACTURERS." where manufacturers_id = '".(int) $_GET['manufacturers_id']."'");
    $manufacturers_name = xtc_db_fetch_array($manufacturers_name,true);
    $manufacturers_name = $manufacturers_name['manufacturers_name'];
    $advanced_search_header = $manufacturers_name;
  } else {
    $advanced_search_header = NAVBAR_TITLE2_ADVANCED_SEARCH;
  }
 

Viele Grüße

Franky

[koshiro]

Hallo Franky,
perfekt, funktioniert einwandfrei. Tausend  
Das Problem besteht schon so lange, aber noch nie eine Lösung für gefunden, und jetzt ist es einfach top.
Gruß
Micha

[koshiro]

So bei meinem einen neuen Shop funktioniert es nicht, obwohl ich genau die gleichen Änderungen gemacht habe wie bei den anderen.
Hier mal der Code von den Dateien.

>>> Code entfernt <<<

Shopversion 1.05

Gruß
Micha

[EDIT Tomcraft 26.09.2010: ellenlangen Code entfernt, bitte als Dateien anhängen.]

[franky_n]

Hallo koshiro,

bitte nimm den langen Quelltext wieder raus indem Du den Post noch mal bearbeitest (geht bis zu 1 Stunde nach Veröffentlichung) und ersetze ihn durch 4 php Dateien gezippt als Anhang.
Ansonsten ist das hier ja nicht mehr wirklich lesbar.
Oder Du gibst Zeilen an und postest nur den Quelltext Ausschnitt, aber bitte nicht die ganze Datei.

Also ich sehe zur Zeit keinen wirklichen Fehler.
Aber bist Du Dir sicher das Du diese Änderungen:

Code: PHP  [Auswählen]

{if $CATEGORIES_HEADING_TITLE}
  <h1>{$CATEGORIES_HEADING_TITLE}</h1>
{elseif $ADVANCED_SEARCH_HEADER}
  <h1>{$ADVANCED_SEARCH_HEADER}</h1>
{else}
  <h1>{$CATEGORIES_NAME}</h1>
{/if}

im richtigen Template gemacht hast?

Viele Grüße

Franky

[koshiro]

Hallo Franky,
ja ich habe es zu spät gesehen, und konnte dann meinen Beitrag nicht mehr editieren. Ich werde nachher mal Torsten anschreiben ob er den Quelltext raus nimmt, damit das wieder übersichtlich aussieht.

Ich habe das Problem gelöst, und zwar habe ich bei mir einmal eine _product_listing_v1.html und eine product_listing_v1.html, die ich eigentlich auch nur nutze. Erst nachdem ich die änderungen auch in der _product_listing_v1.html vorgenommen hatte funktionierte es, was ich eigentlich nicht verstehe, aber egal hauptsache es geht jetzt. Ich danke Dir nochmal für Deine super Lösung.  

Gruß
Micha

[franky_n]

Hallo koshiro,

das liegt daran, dass das default Template immer das ist was in der Dateireihenfolge vorne liegt.
Ein _product_listing_v1.html liegt wenn Du den Explorer öffnest dateimäßig vor der normalen product_listing_v1.html und wird daher als Default Template genommen.
Dein Shop nutzt daher also das _product_listing_v1.html als Template.
(Darum fragte ich ob Du Dir sicher bist das es im richtigen Template ist)

Viele Grüße

Franky

[Spritzpistole]

Super, wollte genau zu der Problematik gerade ein neues Thema erstellen, konnte dann in den Suchergebnissen aber doch noch fündig werden.

Der "Workaround" funktioniert einwandfrei.

Gruß
Thomas

EDIT: Allerdings steht jetzt auch Suchbegriffe in der Ergebnisliste da, wenn ich durch die Kategorienavigation klicke. Lediglich bei Kategorien mit Unterkategorien erscheint beim Klick auf die Oberkategorie deren Bezeichnung. Erreicht man die letzte Kategorieebene oder gibt es nur eine Ebene, erscheint eben "Suchergebnisse". Lässt sich daran noch etwas ändern?