id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc	blockedby	blocking
1090	Überarbeitung von janolaw	Torsten Riemer	Gerhard Waldemair	"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)."	Frage	closed	normal	modified-shop-2.0.2.1	Module	2.0.1.0	fixed