Zeitraum für Product Download

[Volker Strähle]

Ich bastel zur Zeit an einer Downloadfunktion. Dabei ist mir in der download.php aufagefallen, dass je nach eingegebenen Werten ein Download gar nicht möglich ist. Der Grund ist die Art der Berechnung wie lange ein Download erlaubt ist. Es wird immer vom Tag der Bestellung gerechnet und nicht vom Tag der Downloadfreigabe.

Bsp.:
gekauft 03.04.2014 (Karfreitag, Feiertag)
als Downloadzeit ist 3 Tage eingestellt.
Freigabe erfolgt aber erst am 8.4. weil der Kunde mit Vorkasse zahlen wollte
=> download trotz Freigabe nicht möglich.

Abhilfe: Tabelle orders_products_download um Spalte Freigabe-Datum erweitern und von diesem auf Gültigkeit des Downloads prüfen.

[Volker Strähle]

Noch was: ich bin der Meinung, dass in download.php unnötige Abfragen ausgeführt werden. Ich habe da mal so geändert:

status allowed for download

$allowed_status = join("','",explode(',', DOWNLOAD_MIN_ORDERS_STATUS));

$downloads_query = xtc_db_query("SELECT opd.orders_products_filename

FROM ".TABLE_ORDERS_PRODUCTS_DOWNLOAD." opd

JOIN ".TABLE_ORDERS." o

ON o.orders_id = opd.orders_id

WHERE o.orders_id = '".(int)$_GET['order']."'

AND o.orders_status IN ('".$allowed_status."')

AND opd.orders_products_download_id = '".(int)$_GET['id']."'

AND opd.orders_products_filename != ''

AND o.date_purchased >= DATE_SUB(CURDATE(), INTERVAL opd.download_maxdays DAY)

AND opd.download_count > '0'

AND opd.download_key = '".xtc_db_input($_GET['key'])."'

AND (o.customers_id = '".(int)$_SESSION['customer_id']."' OR o.customers_email_address = '".xtc_db_prepare_input($_POST['email_address'])."')");

/*

Check that order_id, download_id and key match

$check_status_query = xtc_db_query("SELECT o.orders_id,

o.orders_status,

o.date_purchased,

opd.orders_products_download_id

FROM ".TABLE_ORDERS_PRODUCTS_DOWNLOAD." opd

JOIN ".TABLE_ORDERS." o

ON o.orders_id=opd.orders_id

WHERE opd.orders_id = '".(int)$_GET['order']."'

AND opd.orders_products_download_id = '".(int)$_GET['id']."'

AND opd.download_key = '".xtc_db_input($_GET['key'])."'");

if (xtc_db_num_rows($check_status_query) > 0) {

$check_status = xtc_db_fetch_array($check_status_query);

$allowed_status = explode(',', DOWNLOAD_MIN_ORDERS_STATUS);

if (in_array($check_status['orders_status'], $allowed_status)) {

status allowed for download

$downloads_query = xtc_db_query("SELECT opd.orders_products_filename

FROM ".TABLE_ORDERS_PRODUCTS_DOWNLOAD." opd

JOIN ".TABLE_ORDERS." o

ON o.orders_id = opd.orders_id

WHERE o.orders_id = '".$check_status['orders_id']."'

AND opd.orders_products_download_id = '".$check_status['orders_products_download_id']."'

AND opd.orders_products_filename != ''

AND DATE_SUB(CURDATE(), INTERVAL opd.download_maxdays DAY) <= '".$check_status['date_purchased']."'

AND opd.download_count > '0'

AND opd.download_key = '".xtc_db_input($_GET['key'])."'

AND o.customers_id = '".(int)$_SESSION['customer_id_download']."'");

*/

[Gerhard Waldemair]

In 8352:

fix #466

[Gerhard Waldemair]

In 8353:

fix #466