DefaultCommand: ajax event from source not triggered on blur

[haraleib]

DefaultCommand: ajax event from source not triggered on blur

[melloware]

That makes sense. Just remove that re focus line...

[haraleib]

Thank you it is working.

[melloware]

Try this version:
pf-11551.zip

Run mvn clean jetty:run and navigate to http://localhost:8080/primefaces-test/

if (PrimeFaces.widget.DefaultCommand) {
    PrimeFaces.widget.DefaultCommand.prototype.init = function(cfg) {
        PrimeFaces.widget.BaseWidget.prototype.init.call(this, cfg);

        // Cache frequently used elements and values
        this.jqTarget = $(PrimeFaces.escapeClientId(this.cfg.target));
        this.scope = this.cfg.scope ? $(PrimeFaces.escapeClientId(this.cfg.scope)) : null;
        var $this = this;
        var scopeEnterKey = 'keydown.' + this.id;

        // Support container elements such as splitButton
        if (this.jqTarget.is(':not(:button):not(:input):not(a)')) {
            this.jqTarget = this.jqTarget.find('button,a').filter(':visible').first();
        }

        // Attach keypress listener to parent form
        var closestForm = this.jqTarget.closest('form');
        closestForm.off(scopeEnterKey).on(scopeEnterKey, {
            scopeEnter: false
        }, function(e, data) {
            data = data || e.data;
            if (($this.scope && data.scopeEnter && data.scopeDefaultCommandId === $this.id) ||
                (!$this.scope && !data.scopeEnter && (e.key === 'Enter'))) {
                var eventTarget = $(e.target);
                // Do not proceed if target is a textarea, button, link, or TextEditor
                if (eventTarget.is('textarea,button,input[type="submit"],a,.ql-editor')) {
                    return true;
                }
                if (!$this.jqTarget.is(':disabled, .ui-state-disabled')) {
                    // focus default command button and click it
                    $this.jqTarget.trigger('focus').trigger(PrimeFaces.csp.clickEvent($this.jqTarget));
                    // re-focus the original input that pressed ENTER
                    eventTarget.trigger('focus');
                }
                e.preventDefault();
                e.stopImmediatePropagation();
            }
        });

        // Add keydown listener to scope if available
        if (this.scope) {
            this.scope.off(scopeEnterKey).on(scopeEnterKey, function(e) {
                if (e.key === 'Enter') {
                    closestForm.trigger(e, {
                        scopeEnter: true,
                        scopeDefaultCommandId: $this.id
                    });
                    e.stopPropagation();
                }
            });
        }

    }
}

[haraleib]

@melloware I tried it. I think it is a nice idea to return the focus, do you think this can result in errors if the commandButton updates the content or forwards/redirects to another page, so that the eventTarget is not available any more?

Nevertheless, returning the focus results in a loop where, the

• 1 request is from the inputText
• 2 request is from the commandButton
• 3 ... n requests are from the inputText

The line which is returning the focus and causing the loop

[melloware]

Awesome I updated my PR to remove that line as well.