Skip to content

PIOXD-392 - Fixed problem with finish orders cronjob and authorized orders #62

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
FatchipRobert wants to merge 2 commits into
base: main
Choose a base branch
from
Open

PIOXD-392 - Fixed problem with finish orders cronjob and authorized orders #62

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
3951393
PIOXD-392 - Fixed problem with finish orders cronjob and authorized o…
FatchipRobert Feb 20, 2026
1544be2
PIOXD-392 - Fixed problem with finish orders cronjob and authorized o…
FatchipRobert Feb 20, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 4 additions & 7 deletions Application/Model/Cronjob/FinishOrders.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -57,13 +57,12 @@ protected function getPaidUnfinishedOrders()
{
$aOrders = [];

$sProcessingFolder = Payment::getInstance()->getShopConfVar('sMollieStatusProcessing');
$iMollieCronFinishOrdersDays = (int)Payment::getInstance()->getShopConfVar('iMollieCronFinishOrdersDays');
if (empty($iMollieCronFinishOrdersDays)) {
$iMollieCronFinishOrdersDays = 14;
}
$sTriggerDate = date('Y-m-d H:i:s', time() - (60 * 60 * 24 * $iMollieCronFinishOrdersDays));
$sMinPaidDate = date('Y-m-d H:i:s', time() - (60 * 2)); // This will prevent finishing legit orders before the customer does
$sMinTriggerDate = date('Y-m-d H:i:s', time() - (60 * 2)); // This will prevent finishing legit orders before the customer does
$sQuery = " SELECT
OXID
FROM
Expand All @@ -72,11 +71,9 @@ protected function getPaidUnfinishedOrders()
oxstorno = 0 AND
oxpaymenttype LIKE '%mollie%' AND
oxorderdate > ? AND
oxtransstatus = 'NOT_FINISHED' AND
oxfolder = ? AND
oxpaid != '0000-00-00 00:00:00' AND
oxpaid < ?";
$aParams = [$sTriggerDate, $sProcessingFolder, $sMinPaidDate];
oxorderdate < ? AND
oxtransstatus = 'NOT_FINISHED'";
$aParams = [$sTriggerDate, $sMinTriggerDate];
if ($this->getShopId() !== false) {
$sQuery .= " AND oxshopid = ? ";
$aParams[] = $this->getShopId();
Expand Down
6 changes: 3 additions & 3 deletions Application/views/admin_twig/de/mollie_lang.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,8 @@
'SHOP_MODULE_sMollieCronjobLogLevel_extended' => 'Erweitert',
'SHOP_MODULE_sMollieCronjobLogLevel_none' => 'Kein Logging',
'SHOP_MODULE_sMollieCronOrderExpiryActive' => 'Cronjob "Unbezahlte Bestellungen automatisch stornieren" aktiv',
'SHOP_MODULE_sMollieCronFinishOrdersActive' => 'Cronjob "Abschluss von bezahlten aber nicht vollendeten Bestellungen" aktiv',
'SHOP_MODULE_iMollieCronFinishOrdersDays' => 'Zeitfenster über das versucht wird bezahlte aber nicht vollendete Bestellungen abzuschließen',
'SHOP_MODULE_sMollieCronFinishOrdersActive' => 'Cronjob "Abschluss von bezahlten/autorisierten aber nicht vollendeten Bestellungen" aktiv',
'SHOP_MODULE_iMollieCronFinishOrdersDays' => 'Zeitfenster über das versucht wird bezahlte/autorisierte aber nicht vollendete Bestellungen abzuschließen',
'SHOP_MODULE_sMollieCronSecondChanceActive' => 'Cronjob "Versand einer Zahlungs-Erinnerungs-Email" aktiv',
'SHOP_MODULE_iMollieCronSecondChanceTimeDiff' => 'Zeitfenster nach dem Zahlungs-Erinnerungs-Email versendet wird',
'SHOP_MODULE_sMollieCronOrderShipmentActive' => 'Cronjob "Übermittlung von Versandstatus an Mollie" aktiv',
Expand All @@ -48,7 +48,7 @@
'HELP_SHOP_MODULE_sMollieStatusProcessing' => 'Legen Sie den Bestell-Status für Abgeschlossene Zahlungen fest',
'HELP_SHOP_MODULE_sMollieStatusCancelled' => 'Legen Sie den Bestell-Status für stornierte Bestellungen fest',
'HELP_SHOP_MODULE_sMollieCronOrderExpiryActive' => 'Damit dieser Cronjob arbeitet, müssen Sie zusätzlich zu dieser Checkbox noch sicherstellen dass der Mollie Cronjob eingerichtet ist. Mehr dazu erfahren Sie in der README.md Datei des Moduls.',
'HELP_SHOP_MODULE_sMollieCronFinishOrdersActive' => 'Dieser Cronjob hat zur Aufgabe Bestellungen abzuschließen, bei denen der Kunde erfolgreich bezahlt hat, aber scheinbar nicht wieder in den Shop zurückgekehrt ist für den Abschluss des Bestellvorgangs. Der Cronjob finalisiert nur Bestellungen aus dem im Config-Feld "Zeitfenster über das versucht wird bezahlte aber nicht vollendete Bestellungen abzuschließen" konfigurierten Zeitraum.<br><br>Damit dieser Cronjob arbeitet, m&uuml;ssen Sie zus&auml;tzlich zu dieser Checkbox noch sicherstellen dass der Mollie Cronjob eingerichtet ist. Mehr dazu erfahren Sie in der README.md Datei des Moduls.',
'HELP_SHOP_MODULE_sMollieCronFinishOrdersActive' => 'Dieser Cronjob hat zur Aufgabe Bestellungen abzuschließen, bei denen der Kunde erfolgreich bezahlt hat, aber scheinbar nicht wieder in den Shop zurückgekehrt ist für den Abschluss des Bestellvorgangs. Der Cronjob finalisiert nur Bestellungen aus dem im Config-Feld "Zeitfenster über das versucht wird bezahlte/autorisierte aber nicht vollendete Bestellungen abzuschließen" konfigurierten Zeitraum.<br><br>Damit dieser Cronjob arbeitet, m&uuml;ssen Sie zus&auml;tzlich zu dieser Checkbox noch sicherstellen dass der Mollie Cronjob eingerichtet ist. Mehr dazu erfahren Sie in der README.md Datei des Moduls.',
'HELP_SHOP_MODULE_sMollieCronSecondChanceActive' => 'Damit dieser Cronjob arbeitet, müssen Sie zusätzlich zu dieser Checkbox noch sicherstellen dass der Mollie Cronjob eingerichtet ist. Mehr dazu erfahren Sie in der README.md Datei des Moduls.',
'HELP_SHOP_MODULE_sMollieCronOrderShipmentActive' => 'Dieser Cronjob ist nur notwendig, wenn der Versandstatus in Ihrem Shop über ein externes System gesetzt wird und NICHT über den "Jetzt versenden" Button im Oxid-Admin. Damit dieser Cronjob arbeitet, müssen Sie zusätzlich zu dieser Checkbox noch sicherstellen dass der Mollie Cronjob eingerichtet ist. Mehr dazu erfahren Sie in der README.md Datei des Moduls.',
'HELP_SHOP_MODULE_sMollieCronSecureKey' => 'Diese Option ben&ouml;tigen Sie nur, wenn Sie den Cronjob per URL aufrufen m&uuml;ssen anstelle von dem Aufruf über die php cli. Konfigurieren Sie hier einen Sicherheitsschlüssel, den Sie über den GET-Parameter "secureKey" an den Aufruf des Cronjobs anf&uuml;gen. Bsp.: https://YOUR-SHOP-URL-HERE/?cl=MollieCron&secureKey=YOUR-SECURE-KEY',
Expand Down
6 changes: 3 additions & 3 deletions Application/views/admin_twig/en/mollie_lang.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,8 @@
'SHOP_MODULE_sMollieCronjobLogLevel_extended' => 'Extended',
'SHOP_MODULE_sMollieCronjobLogLevel_none' => 'No logging',
'SHOP_MODULE_sMollieCronOrderExpiryActive' => 'Cronjob "Cancel unpaid orders automatically" active',
'SHOP_MODULE_sMollieCronFinishOrdersActive' => 'Cronjob "Completion of paid but unfinished orders" active',
'SHOP_MODULE_iMollieCronFinishOrdersDays' => 'Time window during which attempts are made to complete paid but incomplete orders',
'SHOP_MODULE_sMollieCronFinishOrdersActive' => 'Cronjob "Completion of paid/authorized but not completed orders" active',
'SHOP_MODULE_iMollieCronFinishOrdersDays' => 'Time window during which attempts are made to complete paid/authorized but not completed orders',
'SHOP_MODULE_sMollieCronSecondChanceActive' => 'Cronjob "Dispatch of payment reminder email" aktiv',
'SHOP_MODULE_iMollieCronSecondChanceTimeDiff' => 'Timeframe after which payment reminder email is sent',
'SHOP_MODULE_sMollieCronOrderShipmentActive' => 'Cronjob "Transmission of shipping status to Mollie" active',
Expand All @@ -48,7 +48,7 @@
'HELP_SHOP_MODULE_sMollieStatusProcessing' => 'Set the order status for Completed Payments',
'HELP_SHOP_MODULE_sMollieStatusCancelled' => 'Set the order status for cancelled orders',
'HELP_SHOP_MODULE_sMollieCronOrderExpiryActive' => 'For this cronjob to work, in addition to this checkbox you have to ensure that the Mollie cronjob is set up properly. You can find information on how to set up the cronjob in the README.md of this module.',
'HELP_SHOP_MODULE_sMollieCronFinishOrdersActive' => 'This cronjob has the job to finish orders where the customer paid successfully but seemingly didnt return to the shop to complete the order process. The cronjob only finalizes orders within the time period configured in the setting "Time window during which attempts are made to complete paid but incomplete orders".<br><br>For this cronjob to work, in addition to this checkbox you have to ensure that the Mollie cronjob is set up properly. You can find information on how to set up the cronjob in the README.md of this module.',
'HELP_SHOP_MODULE_sMollieCronFinishOrdersActive' => 'This cronjob has the job to finish orders where the customer paid successfully but seemingly didnt return to the shop to complete the order process. The cronjob only finalizes orders within the time period configured in the setting "Time window during which attempts are made to complete paid/authorized but not completed orders".<br><br>For this cronjob to work, in addition to this checkbox you have to ensure that the Mollie cronjob is set up properly. You can find information on how to set up the cronjob in the README.md of this module.',
'HELP_SHOP_MODULE_sMollieCronSecondChanceActive' => 'For this cronjob to work, in addition to this checkbox you have to ensure that the Mollie cronjob is set up properly. You can find information on how to set up the cronjob in the README.md of this module.',
'HELP_SHOP_MODULE_sMollieCronOrderShipmentActive' => 'This cronjob is only needed if the shipping status in your shop is set by an external service and NOT by the "Ship Now" button. For this cronjob to work, in addition to this checkbox you have to ensure that the Mollie cronjob is set up properly. You can find information on how to set up the cronjob in the README.md of this module.',
'HELP_SHOP_MODULE_sMollieCronSecureKey' => 'This option is only needed if you have to call the cronjob via URL instead of the php cli. Configure your secure key here, which you have to add to the cronjob call as the GET parameter "secureKey". Example: https://YOUR-SHOP-URL-HERE/?cl=MollieCron&secureKey=YOUR-SECURE-KEY',
Expand Down
6 changes: 3 additions & 3 deletions Application/views/admin_twig/es/mollie_lang.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,8 @@
'SHOP_MODULE_sMollieCronjobLogLevel_extended' => 'Ampliado',
'SHOP_MODULE_sMollieCronjobLogLevel_none' => 'Sin registro',
'SHOP_MODULE_sMollieCronOrderExpiryActive' => 'Cronjob «Cancelar automáticamente pedidos no pagados» activo',
'SHOP_MODULE_sMollieCronFinishOrdersActive' => 'Cronjob «Finalización de pedidos pagados pero no terminados» activo',
'SHOP_MODULE_iMollieCronFinishOrdersDays' => 'Ventana de tiempo durante la cual se intenta completar pedidos pagados pero no finalizados',
'SHOP_MODULE_sMollieCronFinishOrdersActive' => 'Cronjob "Finalización de pedidos pagados/autorizados pero no completados" activo',
'SHOP_MODULE_iMollieCronFinishOrdersDays' => 'Ventana de tiempo durante la cual se intenta finalizar pedidos pagados/autorizados pero no completados',
'SHOP_MODULE_sMollieCronSecondChanceActive' => 'Cronjob «Envío de correo electrónico de recordatorio de pago» activo',
'SHOP_MODULE_iMollieCronSecondChanceTimeDiff' => 'Plazo después del cual se envía el correo electrónico de recordatorio de pago',
'SHOP_MODULE_sMollieCronOrderShipmentActive' => 'Cronjob «Transmisión del estado del envío a Mollie» activo',
Expand All @@ -46,7 +46,7 @@
'HELP_SHOP_MODULE_sMollieStatusProcessing' => 'Establecer el estado del pedido para los pagos completados',
'HELP_SHOP_MODULE_sMollieStatusCancelled' => 'Establecer el estado del pedido para los pedidos cancelados',
'HELP_SHOP_MODULE_sMollieCronOrderExpiryActive' => 'Para que este cronjob funcione, además de esta casilla hay que asegurarse de que el cronjob de Mollie está configurado correctamente. Puede encontrar información sobre cómo configurar el cronjob en el README.md de este módulo.',
'HELP_SHOP_MODULE_sMollieCronFinishOrdersActive' => 'Este cronjob tiene la función de finalizar los pedidos en los que el cliente ha pagado correctamente pero supuestamente no ha vuelto a la tienda para completar el proceso de pedido. El cron solo finaliza los pedidos dentro del período configurado en el campo "Ventana de tiempo durante la cual se intenta completar pedidos pagados pero no finalizados".<br><br>Para que este cronjob funcione, además de esta casilla hay que asegurarse de que el cronjob de Mollie está configurado correctamente. Puede encontrar información sobre cómo configurar el cronjob en el README.md de este módulo.',
'HELP_SHOP_MODULE_sMollieCronFinishOrdersActive' => 'Este cronjob tiene la función de finalizar los pedidos en los que el cliente ha pagado correctamente pero supuestamente no ha vuelto a la tienda para completar el proceso de pedido. El cron solo finaliza los pedidos dentro del período configurado en el campo "Ventana de tiempo durante la cual se intenta finalizar pedidos pagados/autorizados pero no completados".<br><br>Para que este cronjob funcione, además de esta casilla hay que asegurarse de que el cronjob de Mollie está configurado correctamente. Puede encontrar información sobre cómo configurar el cronjob en el README.md de este módulo.',
'HELP_SHOP_MODULE_sMollieCronSecondChanceActive' => 'Para que este cronjob funcione, además de esta casilla hay que asegurarse de que el cronjob de Mollie está configurado correctamente. Puede encontrar información sobre cómo configurar el cronjob en el README.md de este módulo.',
'HELP_SHOP_MODULE_sMollieCronOrderShipmentActive' => 'Este cronjob solo es necesario si el estado del envío en su tienda es establecido por un servicio externo y NO por el botón «Enviar ahora». Para que este cronjob funcione, además de esta casilla hay que asegurarse de que el cronjob de Mollie está configurado correctamente. Puede encontrar información sobre cómo configurar el cronjob en el README.md de este módulo.',

Expand Down
6 changes: 3 additions & 3 deletions Application/views/admin_twig/fr/mollie_lang.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,8 @@
'SHOP_MODULE_sMollieCronjobLogLevel_extended' => 'Étendu',
'SHOP_MODULE_sMollieCronjobLogLevel_none' => 'Aucun journalisation',
'SHOP_MODULE_sMollieCronOrderExpiryActive' => 'Cronjob "Annuler automatiquement les commandes non-payées" actif',
'SHOP_MODULE_sMollieCronFinishOrdersActive' => 'Cronjob "Clôture des commandes payées mais non-terminées" actif',
'SHOP_MODULE_iMollieCronFinishOrdersDays' => 'Fenêtre temporelle pendant laquelle on tente de finaliser des commandes payées mais non terminées',
'SHOP_MODULE_sMollieCronFinishOrdersActive' => 'Cronjob "Finalisation des commandes payées/autorisées mais non terminées" actif',
'SHOP_MODULE_iMollieCronFinishOrdersDays' => 'Fenêtre de temps pendant laquelle une tentative est faite de finaliser les commandes payées/autorisées mais non terminées',
'SHOP_MODULE_sMollieCronSecondChanceActive' => 'Cronjob "Envoi des emails de rappel de paiement" actif',
'SHOP_MODULE_iMollieCronSecondChanceTimeDiff' => 'Délai au-delà duquel l\'email de rappel de paiement est envoyé',
'SHOP_MODULE_sMollieCronOrderShipmentActive' => 'Cronjob "Transmission des statuts de livraison à Mollie" actif',
Expand All @@ -47,7 +47,7 @@
'HELP_SHOP_MODULE_sMollieStatusProcessing' => 'Définir le statut de commande pour les paiement achevés',
'HELP_SHOP_MODULE_sMollieStatusCancelled' => 'Définir le statut de commande pour les commandes annulées',
'HELP_SHOP_MODULE_sMollieCronOrderExpiryActive' => 'Pour que ce cronjob fonctionne, en plus de cette case à cocher vous devez vous assurer que le cronjob Mollie est correctement configuré. Vous pouvez trouver comment configurer le cronjob dans le fichier README.md de ce module.',
'HELP_SHOP_MODULE_sMollieCronFinishOrdersActive' => 'Ce cronjob a pour tâche de compléter les commandes où le client a payé avec succès mais qui n\'est visiblement pas retourné à la boutique pour finaliser le processus. Le cron ne finalise que les commandes dans la période définie dans le paramètre "Fenêtre temporelle pendant laquelle on tente de finaliser des commandes payées mais non terminées".<br><br>Pour que ce cronjob fonctionne, en plus de cette case à cocher vous devez vous assurer que le cronjob Mollie est correctement configuré. Vous pouvez trouver comment configurer le cronjob dans le fichier README.md de ce module.',
'HELP_SHOP_MODULE_sMollieCronFinishOrdersActive' => 'Ce cronjob a pour tâche de compléter les commandes où le client a payé avec succès mais qui n\'est visiblement pas retourné à la boutique pour finaliser le processus. Le cron ne finalise que les commandes dans la période définie dans le paramètre "Fenêtre de temps pendant laquelle une tentative est faite de finaliser les commandes payées/autorisées mais non terminées".<br><br>Pour que ce cronjob fonctionne, en plus de cette case à cocher vous devez vous assurer que le cronjob Mollie est correctement configuré. Vous pouvez trouver comment configurer le cronjob dans le fichier README.md de ce module.',
'HELP_SHOP_MODULE_sMollieCronSecondChanceActive' => 'Pour que ce cronjob fonctionne, en plus de cette case à cocher vous devez vous assurer que le cronjob Mollie est correctement configuré. Vous pouvez trouver comment configurer le cronjob dans le fichier README.md de ce module.',
'HELP_SHOP_MODULE_sMollieCronOrderShipmentActive' => 'Ce cronjob est requis uniquement si le statut de livraison dans votre boutique est paramétré par un service externe et PAS par le bouton "Expédier maintenant". Pour que ce cronjob fonctionne, en plus de cette case à cocher vous devez vous assurer que le cronjob Mollie est correctement configuré. Vous pouvez trouver comment configurer le cronjob dans le fichier README.md de ce module.',

Expand Down
Loading