Memory Limit überschritten in admin/mail.php

[Volker Rehn]

Die admin/mail.php überschreitet das memory limit, wenn der shop eine große Anzahl Kunden hat, bei meinen Tests ab 100000 Kunden. Das ist natürlich konfigurationsabhängig, sollte aber trotzdem nicht datenabhängig ein Limit erreichen.

Versendet man aus dem Adminbereich eine Mail an einen ausgewählten Kunden (über die Kundenliste), so wird neben den Kundengruppen die gesamte Kundentabelle abgefragt und ins Dropdown des Mailformulars übertragen.

Da der Kunde bereits ausgewählt ist, sollte dessen Mailadresse (sofern sie übergeben wurde) an dieser Stelle als Filter benutzt werden - und sonst keine weitere Liste für $customers erzeugen. Zeile 207, statt

// End customers Status 1.x
$mail_query = xtc_db_query("select customers_email_address, customers_firstname, customers_lastname from " . TABLE_CUSTOMERS . " order by customers_lastname");
while($customers_values = xtc_db_fetch_array($mail_query)) {
  $customers[] = array('id' => $customers_values['customers_email_address'],
                                   'text' => $customers_values['customers_lastname'] . ', ' . $customers_values['customers_firstname'] . ' (' . $customers_values['customers_email_address'] . ')');
}

zb so:

// End customers Status 1.x
$selected_customer = isset($_GET['customer']) ? $_GET['customer'] : $_POST['customers_email_address'];
if (!empty($selected_customer)) {
  $mail_query = xtc_db_query("select customers_email_address, customers_firstname, customers_lastname from " . TABLE_CUSTOMERS . " 
                where customers_email_address = '$selected_customer'
                order by customers_lastname");
  while($customers_values = xtc_db_fetch_array($mail_query)) {
    $customers[] = array('id' => $customers_values['customers_email_address'],
                                     'text' => $customers_values['customers_lastname'] . ', ' . $customers_values['customers_firstname'] . ' (' . $customers_values['customers_email_address'] . ')');
  }
}  

[Gerhard Waldemair]

In 11769:

fix #1536