Modify

Opened 10 years ago

Closed 9 years ago

Last modified 9 years ago

#696 closed Aufgabe (fixed)

xtc_php_mail Funktion anpassen

Reported by: INETvisio Owned by: somebody
Priority: niedrig Milestone: modified-shop-2.0.2.0
Component: Shop Version: 2.0.0.0
Keywords: Cc:
Blocked By: Blocking:

Description

Hallo modified,

In der xtc_php_mail Funktion gibt es ein "exit;", wenn die Emailversand fehlschlägt.
Könnte man den Exit hier rausnehmen und mit einem Rückgabe Wert vorsehen? Wie z.B. eine true bei Erfolg und eine false bei "Mailer Error".

Mailer Errors kommen meistens nicht vor, sobald man ein SMTP einsetzt, können diese Fehler spontan vorkommen und bei der Bestellprozess sieht es schrecklich aus.

Dann wäre es Möglich, bei der Modulentwicklung ein messageStack einzusetzen und den Admin informieren, dass Mail nicht versendet wurde.

Gruß,
Jürgen H.

Attachments (0)

Change History (9)

comment:1 by Torsten Riemer, 10 years ago

In Shopversion 1.06 rev 4642 SP2 sah der Code so aus:

  if (!$mail->Send()) {
    echo "Message was not sent <p>";
    echo "Mailer Error: ".$mail->ErrorInfo."</p>";
    exit;
  }

Für Shopversion 2.00 haben wir das bereits wie folgt überarbeitet:

  if (!$mail->Send()) {
    trigger_error('Mailer Error - '.$mail->ErrorInfo, E_USER_WARNING);
  }
Last edited 10 years ago by Torsten Riemer (previous) (diff)

comment:2 by Torsten Riemer, 10 years ago

Version: 1.062.0

comment:3 by Torsten Riemer, 10 years ago

Resolution: fixed
Status: newclosed

comment:4 by INETvisio, 10 years ago

Rückgabe wie true/false wäre in der Situation auch praktisch. Könnte man das noch erweitern?

in reply to:  1 comment:5 by INETvisio, 10 years ago

Beispiel:

  if (!$mail->Send()) {
    trigger_error('Mailer Error - '.$mail->ErrorInfo, E_USER_WARNING);
    return false;
  }
  return true;

comment:6 by INETvisio, 10 years ago

Resolution: fixed
Status: closedreopened

comment:7 by Torsten Riemer, 10 years ago

Milestone: modified-shop-2.00modified-shop-2.10
Version: 2.02.10

comment:8 by Gerhard Waldemair, 9 years ago

Resolution: fixed
Status: reopenedclosed

In 10512:

fix #696

comment:9 by Torsten Riemer, 9 years ago

Milestone: modified-shop-2.1.0.0modified-shop-2.0.1.1
Version: 2.0.1.02.0.0.0

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.