Fallback für {$PAGINATION} bzw. {$NAVIGATION} erstellen

[Torsten Riemer]

Aktualisiert man einen Shop, dann wird "USE_PAGINATION_LIST" auf "false" gesetzt. Hat man aber ein aktuelles Template, dann ist in den HTML Dateien nur noch "{$PAGINATION}" vorhanden und nicht mehr "{$NAVIGATION}".

Entweder könnte man den folgenden Teil:

  if (USE_PAGINATION_LIST == 'false') {
    $module_smarty->assign('NAVIGATION', '<div class="smallText" style="clear:both;">
                                            <div style="float:left;">'.$listing_split->display_count(TEXT_DISPLAY_NUMBER_OF_PRODUCTS).'</div> 
                                            <div style="text-align:right;">'.TEXT_RESULT_PAGE.' '.$listing_split->display_links(MAX_DISPLAY_PAGE_LINKS, xtc_get_all_get_params(array ('page', 'info', 'x', 'y', 'keywords')).(isset($_GET['keywords'])?'keywords='. urlencode($_GET['keywords']):'')).'</div> 
                                          </div>');
  } else {   
    $module_smarty->assign('DISPLAY_COUNT', $listing_split->display_count(TEXT_DISPLAY_NUMBER_OF_PRODUCTS));
    $module_smarty->assign('DISPLAY_LINKS', $listing_split->display_links(MAX_DISPLAY_PAGE_LINKS, xtc_get_all_get_params(array ('page', 'info', 'x', 'y', 'keywords')).(isset($_GET['keywords'])?'keywords='. urlencode($_GET['keywords']):'')));
    $module_smarty->caching = 0;
    $pagination = $module_smarty->fetch(CURRENT_TEMPLATE.'/module/pagination.html');
    $module_smarty->assign('NAVIGATION', $pagination);
    $module_smarty->assign('PAGINATION', $pagination);
  }

ersetzen durch:

  if (USE_PAGINATION_LIST == 'false') {
    $navigation = '<div class="smallText" style="clear:both;">
                     <div style="float:left;">'.$listing_split->display_count(TEXT_DISPLAY_NUMBER_OF_PRODUCTS).'</div> 
                     <div style="text-align:right;">'.TEXT_RESULT_PAGE.' '.$listing_split->display_links(MAX_DISPLAY_PAGE_LINKS, xtc_get_all_get_params(array ('page', 'info', 'x', 'y', 'keywords')).(isset($_GET['keywords'])?'keywords='. urlencode($_GET['keywords']):'')).'</div> 
                   </div>';
    $module_smarty->assign('NAVIGATION', $navigation);
    $module_smarty->assign('PAGINATION', $navigation);
  } else {   
    $module_smarty->assign('DISPLAY_COUNT', $listing_split->display_count(TEXT_DISPLAY_NUMBER_OF_PRODUCTS));
    $module_smarty->assign('DISPLAY_LINKS', $listing_split->display_links(MAX_DISPLAY_PAGE_LINKS, xtc_get_all_get_params(array ('page', 'info', 'x', 'y', 'keywords')).(isset($_GET['keywords'])?'keywords='. urlencode($_GET['keywords']):'')));
    $module_smarty->caching = 0;
    $pagination = $module_smarty->fetch(CURRENT_TEMPLATE.'/module/pagination.html');
    $module_smarty->assign('NAVIGATION', $pagination);
    $module_smarty->assign('PAGINATION', $pagination);
  }

Oder aber wir sparen uns den Config-Schalter "USE_PAGINATION_LIST" gleich komplett ein und bauen die Abfrage anhand des Vorhandenseins der "pagination.html", also so:

  if (!is_file(DIR_FS_CATALOG.'templates/'.CURRENT_TEMPLATE.'/module/pagination.html')) {
    $module_smarty->assign('NAVIGATION', '<div class="smallText" style="clear:both;">
                                            <div style="float:left;">'.$listing_split->display_count(TEXT_DISPLAY_NUMBER_OF_PRODUCTS).'</div> 
                                            <div style="text-align:right;">'.TEXT_RESULT_PAGE.' '.$listing_split->display_links(MAX_DISPLAY_PAGE_LINKS, xtc_get_all_get_params(array ('page', 'info', 'x', 'y', 'keywords')).(isset($_GET['keywords'])?'keywords='. urlencode($_GET['keywords']):'')).'</div> 
                                          </div>');
  } else {   
    $module_smarty->assign('DISPLAY_COUNT', $listing_split->display_count(TEXT_DISPLAY_NUMBER_OF_PRODUCTS));
    $module_smarty->assign('DISPLAY_LINKS', $listing_split->display_links(MAX_DISPLAY_PAGE_LINKS, xtc_get_all_get_params(array ('page', 'info', 'x', 'y', 'keywords')).(isset($_GET['keywords'])?'keywords='. urlencode($_GET['keywords']):'')));
    $module_smarty->caching = 0;
    $pagination = $module_smarty->fetch(CURRENT_TEMPLATE.'/module/pagination.html');
    $module_smarty->assign('NAVIGATION', $pagination);
    $module_smarty->assign('PAGINATION', $pagination);
  }

Betroffene Dateien:

• /admin/includes/configuration_installer.php (USE_PAGINATION_LIST entfernen)
• /lang/english/admin/configuration.php (USE_PAGINATION_LIST_TITLE & USE_PAGINATION_LIST_DESC entfernen)
• /lang/german/admin/configuration.php (USE_PAGINATION_LIST_TITLE & USE_PAGINATION_LIST_DESC entfernen)
• /includes/classes/split_page_results.php
• /includes/modules/product_listing.php
• /account_history.php
• /product_reviews.php
• /products_new.php
• /reviews.php
• /specials.php

[Gerhard Waldemair]

In 10840:

fix #1238

[Torsten Riemer]

In 10843:

Addition on r10840 (fix #1238)