Modify

Opened 4 years ago

Closed 4 years ago

#2083 closed Erweiterung (wontfix)

multilang.php für weitere Module updatesicher machen

Reported by: Karsta Owned by: somebody
Priority: normal Milestone:
Component: Shop Version: 2.0.6.0
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by Torsten Riemer)

shoproot\includes\extra\modules\set_language_sessions\multilang.php

Möchte man weitere Seiten hinzufügen z.B. für das News- oder Blog-Modul, fehlt hier ein auto_include oder im Admin unter > Meta-Tags /Suchmaschinen die Möglichkeit, ähnlich wie für die "seo_url_shopstat", die neue "multilang_geaendert" auszuwählen, um das Überschreiben dieser Datei zu verhindern.

Attachments (0)

Change History (6)

comment:1 by Torsten Riemer, 4 years ago

Component: AdminShop
Description: modified (diff)
Type: Bug/FehlerErweiterung

comment:2 by noRiddle, 4 years ago

Irre ich mich oder kann man nicht auch jetzt schon mittels einer weiteren Datei in
/includes/extra/modules/set_language_sessions/
den gewünschte Ergebnis erreichen ?

if (SEARCH_ENGINE_FRIENDLY_URLS == 'true'
      && defined('SEO_URL_MOD_CLASS')
      && defined('MODULE_MULTILANG_STATUS')
      && MODULE_MULTILANG_STATUS == 'true'
      && !defined('RUN_MODE_ADMIN')
      )
  {

    if (basename($PHP_SELF) == 'BLOGSEITE_ODER_NEWSSEITE.php') {
      if (!isset($_GET['language'])) {
        $_GET['language'] = DEFAULT_LANGUAGE;
      }

      if(EVENTUELLE_WEITERE_CONDITIONS) {
        $redirect_link = xtc_href_link(basename($PHP_SELF), xtc_get_all_get_params(), $request_type);
        $redirect_link = str_replace(array(HTTP_SERVER,HTTPS_SERVER), '', preg_replace("/([^\?]*)(\?.*)/", "$1", $redirect_link));
        $current_link = preg_replace("/([^\?]*)(\?.*)/", "$1", $_SERVER['REQUEST_URI']);

        if ($current_link != $redirect_link) {
          header('HTTP/1.1 301 Moved Permanently' );
          header('Location: '.preg_replace("/[\r\n]+(.*)$/i", "", $redirect_link));
          exit();      
        }
      }
    }  
  }

...womit ich nicht sagen will, daß die Möglichkeit der Erweiterung der multilang.php nicht sinnvoll sein kann. Allerdings dürfte das nicht so einfach zu verwirklichen sein, da man evtl. nötige Conditions nicht vorhersehen und sie deshalb auch schlecht implementieren kann. Eine Möglichkeit der schlichten Erweiterung des Arrays $seo_url_sites dürfte nicht für jeden Fall ausreichen.

Gruß,
noRiddle

comment:3 by Karsta, 4 years ago

Auf Grundlage deines Vorschlags, habe ich das updatesicher jetzt so gelöst, dass ich eine weitere Datei ins Verzeichnis includes\extra\modules\set_language_sessions\ geschoben habe, mit folgendem Inhalt:

if (SEARCH_ENGINE_FRIENDLY_URLS == 'true'
      && defined('SEO_URL_MOD_CLASS')
      && defined('MODULE_MULTILANG_STATUS')
      && MODULE_MULTILANG_STATUS == 'true'
      && !defined('RUN_MODE_ADMIN')
      )
  {
    $seo_url_sites[] = FILENAME_PRODUCTS_NEW;
  }

Was bisher ohne Probleme funktioniert.

comment:4 by Karsta, 4 years ago

Sorry, natürlich statt FILENAME_PRODUCTS_NEW die neue Datei eingesetzt.

comment:5 by noRiddle, 4 years ago

Mmh, wenn die Datei vor der multilang.php includiert würde sollte es nicht funktionieren, da das Array dann in der multilang.php überschrieben würde.
Wenn die Datei nach multilang.php geladen würde verstehe ich aber auch nicht wie es funktionieren sollte, da die Condition

if (in_array(basename($PHP_SELF), $seo_url_sites)) {

dann ja bereits abgearbeitet sein müsste.

Gruß,
noRiddle

comment:6 by Gerhard Waldemair, 4 years ago

Milestone: modified-shop-2.0.6.1
Resolution: wontfix
Status: newclosed

wir werden keine autoincludes für ein autoinclude Modul erstellen. Du kannst einfach eine zusätzliche Datei ablegen.

Modify Ticket

Action
as closed The owner will remain somebody.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.