Skip to content
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.

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

Send status is not changed after paid. #5

Open
RelvisRamone opened this issue Aug 22, 2024 · 7 comments
Open

Send status is not changed after paid. #5

RelvisRamone opened this issue Aug 22, 2024 · 7 comments

Comments

@RelvisRamone
Copy link

Initial situation: Paid order is set to dispatched in JTL Wawi. Data is sent to JTL Shop. The status is not changed in JTL Shop.

Problem does not occur if payment plugin is deactivated.

Tested with: Plugin versions 1.027 and 1.029
Tested with JTL 5.3.2 and JTL 5.3.3
Template Nova and Salepix Fire latest versions
PHP 8.1/8.2 and 8.3

@Dominik-Niziol
Copy link

Dear Relvis

Thank you for your message.

We will gladly take a closer look at your issue. As already mentioned in the issue #3 which you have created we would need the more informations which you can send us via mail to [email protected]

Thank you in advance for your feedback.

wallee support

@RelvisRamone
Copy link
Author

RelvisRamone commented Aug 22, 2024

Hello,

what is the logic or function of the following code from line 149 in Bootstrap.php:

        $dispatcher->listen('shop.hook.' . \HOOK_BESTELLUNGEN_XML_BEARBEITESET, function ($args) use ($handler) {
            $order = $args['oBestellung'] ?? [];
            if ((int)$order->cStatus === \BESTELLUNG_STATUS_BEZAHLT) {
                $order = new Bestellung($args['oBestellung']->kBestellung);

                $paymentMethodEntity = new Zahlungsart((int)$order->kZahlungsart);
                $moduleId = $paymentMethodEntity->cModulId ?? '';
                $paymentMethod = new Method($moduleId);
                $paymentMethod->setOrderStatusToPaid($order);

                Shop::Container()
                    ->getDB()->update(
                        'tbestellung',
                        ['kBestellung',],
                        [$args['oBestellung']->kBestellung],
                        (object)['cAbgeholt' => 'Y']
                    );
            }
        });

@mkis84
Copy link

mkis84 commented Sep 18, 2024

Same problem with in our shop since we updated to JTL Shop 5 and use the wallee plugin.

It also effects all orders paid with other payment plugins.

This is really suboptimal.

@RelvisRamone
Copy link
Author

The code says do not change the status, but hold it. I can put you in touch with a contact who may be able to solve the problem quickly. Wallee doesn't do anything, although we have drawn their attention to it.

@Alcpaone
Copy link

Alcpaone commented Sep 27, 2024

@Dominik-Niziol
Same issue here, also with the latest version 1.0.32. Order Status doesnt update.

@thibault-mambour
Copy link

@Alcpaone do you think you could share the log trace of the error ? Or is it related to the webhook problem you raised ?

@Alcpaone
Copy link

@thibault-mambour i would say issue resolved with 1.0.33

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants