AjaxStatus wird nach einem fehlerhaften Request nicht wieder angezeigt

Christopher Walz · Nov 10th 2021

Folgender Code:

JavaScript

    require(['Ajax', 'Ui/Confirmation', 'WoltLabSuite/Core/Ajax/Status'], function(Ajax, UiConfirmation, AjaxStatus) {
        document.querySelectorAll('.jsSendEmails').forEach(function(jsSendEmails) {
            jsSendEmails.addEventListener('click', function(e) {
                if (jsSendEmails.classList.contains('disabled')) {
                    return;
                }

                UiConfirmation.show({
                    confirm: (function () {
                        jsSendEmails.classList.add('disabled');
                        AjaxStatus.show();

                        Ajax.apiOnce({
                            data: {
                                actionName: 'sendEmails',
                                className: 'seo\\data\\commission\\period\\CommissionPeriodAction',
                                objectIDs: [ elData(jsSendEmails, 'object-id') ]
                            },
                            silent: true,
                            finalize: function () {
                                AjaxStatus.hide();
                            }
                        });
                    }).bind(this),
                    message: elData(jsSendEmails, 'confirm-message-html'),
                    messageIsHtml: true
                });
            });
        });
    });

Display More

Wenn der erste Request fehlschlägt (weil eine PermissionDeniedException geworfen wird), wird beim zweiten Request (andere object-id) kein AjaxStatus mehr angezeigt. In der Console wird nichts angezeigt.

**Alexander Ebert** · Nov 10th 2021

Dein Code ergibt irgendwie keinen Sinn. Warum nimmst du die direkten Aufrufe von AjaxStatus nicht raus und entfernst das silent: true?

Christopher Walz · Nov 10th 2021

Weil selbes Problem

JavaScript

    require(['Ajax', 'Ui/Confirmation', 'WoltLabSuite/Core/Ajax/Status'], function(Ajax, UiConfirmation, AjaxStatus) {
        document.querySelectorAll('.jsSendEmails').forEach(function(jsSendEmails) {
            jsSendEmails.addEventListener('click', function(e) {
                if (jsSendEmails.classList.contains('disabled')) {
                    return;
                }

                UiConfirmation.show({
                    confirm: (function () {
                        jsSendEmails.classList.add('disabled');
                        //AjaxStatus.show();

                        Ajax.apiOnce({
                            data: {
                                actionName: 'sendEmails',
                                className: 'seo\\data\\commission\\period\\CommissionPeriodAction',
                                objectIDs: [ elData(jsSendEmails, 'object-id') ]
                            },
                            /*silent: true,
                            finalize: function () {
                                AjaxStatus.hide();
                            }*/
                        });
                    }).bind(this),
                    message: elData(jsSendEmails, 'confirm-message-html'),
                    messageIsHtml: true
                });
            });
        });
    });

Display More

**Alexander Ebert** · Nov 10th 2021

Das kann ich so auf Basis des Codes nicht nachvollziehen, es gibt keinen Code-Path, bei dem im Fehlerfall der Spinner nicht ausgeblendet wird. Das von dir geschilderte Verhalten würde nur Sinn ergeben, wenn der Counter in AjaxStatus negativ wäre. Hast du mal per Step-Debugger überprüft, ob der Counter von einer anderen Stelle aus beeinflusst wird? In Chrome lässt sich JS auch on-the-fly bearbeiten, praktisch um schnell mal eine Debug-Ausgabe einzubauen.

Christopher Walz · Nov 10th 2021

Habe nur kurz getestet: _timeoutShow scheint in Status::show nicht null zu sein, weswegen beim zweiten Request die Klasse active nicht hinzugefügt wird und dadurch der Spinner nicht sichtbar wird.

P.S. Ich nutze das WSC 3.1!

Christopher Walz · Nov 10th 2021

Ah, _timeoutShow wird in hide nie auf null gesetzt

**Alexander Ebert** · Nov 10th 2021

Ja, das hatte ich dann auch bemerkt.

Im folgenden auch mal der JS-Code von dir in "modern". Das ist mit dem IE11 nicht kompatibel, aber das war der Code vorher auch nicht.

JavaScript

require(['Ajax', 'Ui/Confirmation'], (Ajax, UiConfirmation) => {
    document.querySelectorAll('.jsSendEmails').forEach((jsSendEmails) => {
        jsSendEmails.addEventListener('click', (event) => {
            event.preventDefault();

            if (jsSendEmails.classList.contains('disabled')) {
                return;
            }

            UiConfirmation.show({
                confirm() {
                    jsSendEmails.classList.add('disabled');
                    
                    Ajax.apiOnce({
                        data: {
                            actionName: 'sendEmails',
                            className: 'seo\\data\\commission\\period\\CommissionPeriodAction',
                            objectIDs: [ jsSendEmails.dataset.objectId ]
                        }
                    });
                },
                message: jsSendEmails.dataset.confirmMessageHtml,
                messageIsHtml: true
            });
        });
    });
});

Display More

**Alexander Ebert** · Nov 10th 2021

Incorrect reset of the timer to show the loading indicator · WoltLab/WCF@f9a326d

The missing reset caused repeated calls to `show()` being effectively ignored, because the check `_timeoutShow === null` would be always true if the callback…

github.com

AjaxStatus wird nach einem fehlerhaften Request nicht wieder angezeigt

Alexander Ebert Nov 10th 2021

Alexander Ebert Nov 10th 2021

Alexander Ebert Nov 10th 2021

Alexander Ebert Nov 10th 2021

Alexander Ebert Nov 10th 2021

Tim Düsterhus Nov 11th 2021

Participate now!

Share

Similar Threads

"Eingeloggt bleiben" im Login Screen wieder implementieren

Frage zur Styleinstallation

woltlab 5.4 fehlermeldung

Users Viewing This Thread