Überarbeitung von janolaw

[Torsten Riemer]

1.) Spricht etwas dagegen die PDF-Verlinkung in der "/includes/external/janolaw/janolaw.php" zu erweitern um HTTP_SERVER?

                  if ($module_name == 'revocation' 
                      && $this->get_configuration('MODULE_JANOLAW_WITHDRAWAL_COMBINE') == 'True' 
                      && isset($this->withdrawal_link[$key])
                      ) 
                  {
                    if ($this->get_configuration('MODULE_JANOLAW_PDF_'.strtoupper($module_name)) == 'True') {
                      $content .= '<br /><br /><a href="'.DIR_WS_CATALOG.$filename.'" target="_blank">PDF - '.$this->document_name[strtoupper($language)][$module_name].'</a>';
                    }
                    $content .= '<br /><a href="'.$this->withdrawal_link[$key].'" target="_blank">PDF - '.$this->document_name[strtoupper($language)]['withdrawal'].'</a>';
                  } else {
                    $content .= '<br /><br /><a href="'.DIR_WS_CATALOG.$filename.'" target="_blank">PDF - '.$this->document_name[strtoupper($language)][$module_name].'</a>';            
                  }

Hintergrund ist der, dass die Links zu den PDFs ansonsten in den Bestellbestätigungs-Mails nicht aufgerufen werden können, da eben HTTP_SERVER im Link fehlt.

2.) Ist es wirklich beabsichtigt, dass in der Datei "/admin/includes/modules/system/janolaw.php" in der Funktion remove() die kompletten Contents gelöscht werden, wenn man das Modul deinstalliert? Hier sollte man dann zumindest bei der Deinstallation einen nicht zu übersehenden Hinweis unterbringen, da es der Shopbetreiber ansonsten evtl. gar nicht mitbekommt, dass er abmahngefährdet ist, wenn er keine Rechtstexte mehr im Shop hat (Impressum, AGB, Widerrufsrecht, Widerrufsformular, Datenschutz).

[Gerhard Waldemair]

Beim Deinstallieren wird bereits folgendes ausgegeben:

ACHTUNG: Alle Moduleinstellungen werden ebenfalls gelöscht!

das sollte reichen. Zudem müssen die Texte lt. Vorgabe von Janolaw gelöscht werden.

[Gerhard Waldemair]

In 10595:

fix #1090