Opened 12 months ago

Closed 8 months ago

Last modified 8 months ago

#1636 closed Aufgabe (fixed)

Auswahl List- und Box-Ansicht unabhängig von PRODUCT_LIST_FILTER

Reported by: noRiddle Owned by: somebody
Priority: normal Milestone: modified-shop-2.0.5.0
Component: Shop Version: 2.0.4.2

Description

Momentan wird das Erscheinen der Umschalt-Links für Listen- oder Box-Ansicht im Produkt-Listing abhängig von der Einstellung PRODUCT_LIST_FILTER in der Konfiguration gemacht (/includes/modules/listing_filter.php).
Das finde ich nicht richtig so. Die genannten Links haben nicht wirklich etwas mit den Filtern zu tun und sollten unabhängig von diesen anzeigbar sein. Das würde die Flexibilität erhöhen.

Gruß,
noRiddle

Attachments (0)

Change History (9)

comment:1 Changed 12 months ago by Tomcraft

Ja da gebe ich dir Recht und wenn ich das richtig sehe, dann hält sich der Aufwand stark in Grenzen. Es sind lediglich die folgenden Code-Zeilen aus der Klammer if (PRODUCT_LIST_FILTER == 'true') { ... } zu ziehen:

$filter_smarty->assign('language', $_SESSION['language']);
$filter_smarty->assign('LINK_DISPLAY_LIST', xtc_href_link(basename($PHP_SELF), xtc_get_all_get_params(array('show')).'show=list', 'NONSSL'));
$filter_smarty->assign('LINK_DISPLAY_BOX', xtc_href_link(basename($PHP_SELF), xtc_get_all_get_params(array('show')).'show=box', 'NONSSL'));
$filter_smarty->caching = 0;
if (is_file(DIR_FS_CATALOG.'templates/'.CURRENT_TEMPLATE.'/module/listing_filter.html')) {
  $module_filter = $filter_smarty->fetch(CURRENT_TEMPLATE.'/module/listing_filter.html');
}

comment:2 Changed 12 months ago by Tomcraft

  • Milestone set to modified-shop-2.0.5.0

comment:3 Changed 12 months ago by anonymous

So sehe ich das auch.

Gruß,
noRiddle

comment:4 Changed 11 months ago by GTB

  • Resolution set to fixed
  • Status changed from new to closed

In 11900:

fix #1636

comment:5 Changed 9 months ago by anonymous

Im Zusammenhang mit diesem Ticket sehe ich im SVN den Fix, daß folgender Code aus der Condition

if (PRODUCT_LIST_FILTER == 'true') {

heraus genommen wurde:

$filter_smarty->assign('language', $_SESSION['language']);
$filter_smarty->assign('LINK_DISPLAY_LIST', xtc_href_link(basename($PHP_SELF), xtc_get_all_get_params(array('show')).'show=list', 'NONSSL'));
$filter_smarty->assign('LINK_DISPLAY_BOX', xtc_href_link(basename($PHP_SELF), xtc_get_all_get_params(array('show')).'show=box', 'NONSSL'));
$filter_smarty->assign('LINK_FILTER_RESET', xtc_href_link(basename($PHP_SELF), xtc_get_all_get_params(array('filter', 'show', 'filter_id')), 'NONSSL'));

Die letzte Zeile sollte jedoch innerhalb der Condition stehen.
Ansonsten würde der Filter-Reset auch angezeigt wenn man die Filter nicht anzeigen läßt.
Der Reset setzt aber ja DISPLAY_LIST oder DISPLAY_BOX nicht zurück, weil die Werte dafür auch in der Session stehen, somit das Entfernen der GET-Parameter in der URL daran nichts ändert
und somit die Ansicht BOX oder LIST nicht zurückgesetzt wird.

In dem Zusammenhang würde ich übrigens aus Performance-Gründen auch das

{if count($FILTER_TAG) > 0}

aus dem Template entfernen und statt dessen das Smarty im PHP conditional definieren, sodaß im Template ein

{if $FILTER_TAG}

oder

{if isset($FILTER_TAG)}

reichen würde.

Gruß,
noRiddle

comment:6 Changed 9 months ago by anonymous

Wenn an /includes/modules/listing_filter.php ohnehin gearbeitet wird, gibt es in diesem Zusammenhang auch noch dieses neue Ticket #1671.

Gruß,
noRiddle

Last edited 9 months ago by Tomcraft (previous) (diff)

comment:7 Changed 9 months ago by Tomcraft

  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:8 Changed 8 months ago by GTB

  • Resolution set to fixed
  • Status changed from reopened to closed

In 12110:

fix #1636

comment:9 Changed 8 months ago by Tomcraft

  • Reporter changed from anonymous to noRiddle

Add Comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.