[FIX] Mails nach Deutschem Recht

guest* · July 25, 2012

Es gib bereits ein Post hierüber, ich picke aber speziell diesen Punkt nochmals raus:

Um dem Deutschen Recht gerecht zu werden, müsst Ihr noch in allen Mails euer komplettes Impressum überall ganz unten ergänzen und bei der Bestellbestätigung auch noch zusätzlich ein Link zur Widerrufsbelehrung. Link zu AGB wäre auch empfehlenswert, ist aber keine Pflich. Widerruf ist Pflicht.

silentspeak · July 25, 2012

Ich habe es folgenderweise gelöst als Quick'n Dirty.

Wer will kann es ja noch anpassen und ändern zb als Override wenn möglich.

In classes/Mail.php in Zeile 190 habe ich folgendes Hinzugefügt

####
$templateVars['{imprint_de}'] = "<pre>";
$templateVars['{imprint_de}'] .= "Impressum:\n";
$templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_ADDR1'))."\n";
$templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_ADDR2'))."\n";
$templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_CODE'))." ";
$templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_CITY'))."\n";
$templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_COUNTRY'))."\n";
$templateVars['{imprint_de}'] .= "TEL:".Tools::safeOutput(Configuration::get('PS_SHOP_PHONE'))."\n";
$templateVars['{imprint_de}'] .= "FAX:".Tools::safeOutput(Configuration::get('PS_SHOP_FAX'))."\n";
$templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_DETAILS'))."\n";
$templateVars['{imprint_de}'] .= "</pre>";
####

Jetzt kann man indem man in den E-Mail Templates die ErsetzungsVariable {imprint_de} einfügt, das Impressum bestehend aus den Datenbank Daten welche man unter Kontaktdaten angegeben hat auslesen.

Der obere Code kann auch jederzeit erweitert und ergänz werden mit weiteren Variablen.

guest* · July 26, 2012

Core-Dateien sollte man möglichst ungetastet lassen, weil sonst beim nächsten Upgrade alles wieder verloren geht. Man denkt ja auch nach x-Monaten nicht mehr daran, dass man irgendwo eine Anpassung gemacht hat.

Wer nicht fit im Programmieren ist und keine Override-Datei selbs erstellen kann, der sollte sich meinen Tipp zu Herzen nehmen. So oft ändert man sein Impressum auch wieder nicht... Bin schon über 8 Jahre im e-commerce und die Firmendaten sind noch immer die gleichen...

silentspeak · July 26, 2012

Da magst du Recht haben.

Ich bin beruflich Firmenberater mit Schwerpunkt eCommerce und berate deswegen auch sehr viele Online Shops. Daher bin ich auch immer auf der Suche nach neuen und guten Shop Systemen. (so bin ich hier gelandet )

Ich habe dort schon zigmal erlebt das sich die Fax, Support, Telefonnummer geändert hat, oder Angaben in der Rechtsform, UST ID bei Beendigung einer GbR oder oHG.

Bezüglich des CodeSchnipsels:

Ich werde wenn ich Zeit habe und es noch kein anderer erledigt, es versuchen in eine override einzubauen.

Das oben war ja nur eine Schnelle Lösung.

guest* · July 26, 2012

OK. Danke dir. So haben wir offensichltich einen ähnlichen Job... Aber ich bin vorrangig selbst Händler... Aus Erfahrung weiss ich aber auch, dass die meisten Start-ups nach 3 Jahren garnicht mehr existieren. So kurzlebig ist das e-commerce Geschäft und der große Teil versucht sich darin auch nur sehr halbherzig und mit dem Gedanken - damit werde ich reich...

Leider in 98% der Fälle wird so gedacht. Die Realität ist eine ganz andere. Mein Tag hat sehr oft - und das ist der überwiegende Teil - auch bis zu 20 Std. Arbeitszeit und nur 4 Std. Schlaf.

Ich bin im e-commerce auch schon über 8 fast 9 Jahre tätig, bei mir hat sich noch nichts geändert. Offensichltich war mein Start-up-Berater sehr gut und ich habe von Anfang an alles korrekt aufgebaut. Sicher, es kommen immer wieder neue Gesetze heraus und man muss anpassen, das bleibt uns allen nicht erspart...

silentspeak · July 26, 2012

Ja das kenne ich nur zu gut.

Ich versuche von Anfang an sehr gut zu beraten, vor allem was auch die Arbeitszeiten angeht und die gesamten Prozesse von Bestellung, Lieferung oder auch Lieferanten, Support, Buchhaltung.

Ich mache das jetzt seit 1996, damals überwiegend Ladengeschäfte mit Werbung und Optimierung deren Prozesse. Heute nur noch eine Handvoll Ladengeschäfte und überwiegend Online Auftritte bzw Liefer oder Lagerfirmen.

Berate von Gründung bis hin zur versuchten Rettung einer abwärts fallenden Firma die den Anschluss verpasst hat.

Manche glauben halt die stellen nen Online Shop Online und das Geld kommt von fast alleine.

guest* · July 26, 2012

Manche glauben halt die stellen nen Online Shop Online und das Geld kommt von fast alleine.

Glauben ? Nein die meisten sind davon sogar sehr überzeugt... Die Hype war vor langer, langer Zeit. Neben e-commerce sind meine Stärken: Aufbau von neuen und Sanierung von maroden Unternehmen. Also sehr änhlich zu deinem Job.

guest* · July 26, 2012

Habe eine Bitte, damit das nicht zu off-topic wird. Wenn du einen Fix hast, dann editiere Post #2 so bleibt das im Zusammenhang und User müssen nicht unseren small-talk lesen.

Pete78 · July 26, 2012

Hi,

bin neu mit presta unterwegs und ziehe gerade den ersten Shop damit hoch. Die Funktion mit dem Impressum etc. benötige ich auch. Hier also mein erster Beitrag fürs Forum :-)

Ich hoffe, ich habe das mit den Overrides richtig gemacht, jedenfalls einfach folgenden Code in eine Mail.php kopieren und diese dann in overrides/classes/Mail.php kopieren.

Bei den Mail Templates kann man dann wie oben beschrieben dann einfach {imprint_de} verwenden.

Wenn man Overrides so nicht benutzt, bin ich gerne für Verbesserungen offen :-)

<?php
/*
Overrides the core Mail class to add a new imprint_de field in the template VARs
*/
class Mail extends MailCore
{
public static function Send($id_lang, $template, $subject, $templateVars, $to,
 $toName = null, $from = null, $fromName = null, $fileAttachment = null, $modeSMTP = null, $templatePath = _PS_MAIL_DIR_, $die = false)
{

 $templateVars['{imprint_de}'] = "<pre>";
 $templateVars['{imprint_de}'] .= "Impressum:\n";
 $templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_ADDR1'))."\n";
 $templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_ADDR2'))."\n";
 $templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_CODE'))." ";
 $templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_CITY'))."\n";
 $templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_COUNTRY'))."\n";
 $templateVars['{imprint_de}'] .= "TEL:".Tools::safeOutput(Configuration::get('PS_SHOP_PHONE'))."\n";
 $templateVars['{imprint_de}'] .= "FAX:".Tools::safeOutput(Configuration::get('PS_SHOP_FAX'))."\n";
 $templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_DETAILS'))."\n";
 $templateVars['{imprint_de}'] .= "</pre>";
 #### 
 parent::Send($id_lang,$template,$subject,$templateVars,$to,$toName,$from,$fromName,$fileAttachment,$modeSMTP,$templatePath,$die);
}
}
?>

guest* · July 26, 2012

Vorerst sage ich mal Danke Pete78. Zum testen komme ich aber im Moment leider nicht. Soeben ein private Release für PS 1.5. reingekommen mit deadline für Bugmeldungen und sonst ist ja auch noch meine eigene Arbeit da...

Zoomby · July 30, 2012

@ Pete78: Wo muss ich das denn genau einfügen? Muss ich noch was ändern oder löschen? Bekomme es irgendwie nicht hin.

Danke!

guest* · July 30, 2012

Steht im Post #9 doch klar erklärt. Datei erstellen und unter overrides einfügen und dann anstatt den Text als Hardcode einfügen wie im Post #1, an gewünschter Stelle in den Mails die Variable {imprint_de} einfügen. Am Besten in allen Mails ganz unten wo die {shop_URL} oder auch teilweise als {shop_Name} Variable übergeben wird. Auf jeden Fall bei allen Mails ganz unten und darunter also mit einem Zeilenumbruch nach {shop_URL}, bzw. {shop_Name} dazwischen

Pete78 · September 11, 2012

Hi,

ist noch ein kleiner Fehler drinnen. Aufgefallen war es mir be der "Neues Passwort anfordern" Funktion.

Bitte obigen Mail.php Code wie folgt benutzen (einfach noch ein return hinzufügen).

<?php
/*
Overrides the core Mail class to add a new imprint_de field in the template VARs
*/
class Mail extends MailCore
{
public static function Send($id_lang, $template, $subject, $templateVars, $to,
 $toName = null, $from = null, $fromName = null, $fileAttachment = null, $modeSMTP = null, $templatePath = _PS_MAIL_DIR_, $die = false)
{

 $templateVars['{imprint_de}'] = "<pre>";
 $templateVars['{imprint_de}'] .= "Impressum:\n";
 $templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_ADDR1'))."\n";
 $templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_ADDR2'))."\n";
 $templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_CODE'))." ";
 $templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_CITY'))."\n";
 $templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_COUNTRY'))."\n";
 $templateVars['{imprint_de}'] .= "TEL:".Tools::safeOutput(Configuration::get('PS_SHOP_PHONE'))."\n";
 $templateVars['{imprint_de}'] .= "FAX:".Tools::safeOutput(Configuration::get('PS_SHOP_FAX'))."\n";
 $templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_DETAILS'))."\n";
 $templateVars['{imprint_de}'] .= "</pre>";
 return parent::Send($id_lang,$template,$subject,$templateVars,$to,$toName,$from,$fromName,$fileAttachment,$modeSMTP,$templatePath,$die);
}
}
?>

guest* · November 19, 2012

Die Code-Anpassung ist nicht für PS 1.5. getestet. Bitte für diese Version so vorgehen wie im Einganspost geschrieben - überall händisch ergänzen.

thefelix · November 21, 2012

Ich kriegs leider nicht hin mit dem Fix

Habs etliche Male probiert.. meine Vorgehensweise:

*) Mail.php erstellt

Inhalt: <?php

/*

Overrides the core Mail class to add a new imprint_de field in the template VARs

*/

class Mail extends MailCore

{

public static function Send($id_lang, $template, $subject, $templateVars, $to,

$toName = null, $from = null, $fromName = null, $fileAttachment = null, $modeSMTP = null, $templatePath = _PS_MAIL_DIR_, $die = false)

{

$templateVars['{imprint_de}'] = "<pre>";

$templateVars['{imprint_de}'] .= "Impressum:\n";

$templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_ADDR1'))."\n";

$templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_ADDR2'))."\n";

$templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_CODE'))." ";

$templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_CITY'))."\n";

$templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_COUNTRY'))."\n";

$templateVars['{imprint_de}'] .= "TEL:".Tools::safeOutput(Configuration::get('PS_SHOP_PHONE'))."\n";

$templateVars['{imprint_de}'] .= "FAX:".Tools::safeOutput(Configuration::get('PS_SHOP_FAX'))."\n";

$templateVars['{imprint_de}'] .= Tools::safeOutput(Configuration::get('PS_SHOP_DETAILS'))."\n";

$templateVars['{imprint_de}'] .= "</pre>";

return parent::Send($id_lang,$template,$subject,$templateVars,$to,$toName,$from,$fromName,$fileAttachment,$modeSMTP,$templatePath,$die);

}

?>

*) Die mail.php in das Verzeichnis overrides/classes/ kopiert

*) Ins Verzeichnis classes gehen die mail.php öffnen und dann stehe ich an, wo muss dich jetzt was einfügen ? Hab was probiert, dann kamen garkeine Mails mehr an.

harleymeier · November 25, 2012

Leider klappt es bei mir auch nicht. Könnte vielleicht jemand eine mail.php anhängen?

Nach 1000 Versuchen gehen entweder die mail nicht oder es ändert sich nichts.

Version jetzt:1.5.2

Danke im Voraus!

guest* · November 25, 2012

Post#15 gelesen ? Für 1.5. manuell den Text in jeder einzelnen Mail einpflegen, wie im Post #1 angegeben einpflegen.

harleymeier · November 25, 2012

Gelesen ja, aber es verstehen ist eine andere Sache :-/

Warum gibts es die override/classes/mail.php? Überschreiben irgendwas....?!

Den Text in jede einzelne mail einpflegen? Was ist d mit gemeint?

Die classes/mail.php sieht jetzt so aus. (von Zeile 184 bis 207, für den Überblick)

/* Get templates content */

$iso = Language::getIsoById((int)$id_lang);

if (!$iso)

{

Tools::dieOrLog(Tools::displayError('Error - No ISO code for email'), $die);

return false;

####