[regression] TextBox: on("keypress", callback) never fires callback

[bill]

Starting in 1.9.1 (see #17154), TextBox?.on("keypress", ...) never fires.

According to Dylan, this fix actually breaks a pretty common pattern with ValidationTextBox?, which is to prevent/mask invalid input. Prior to this change, you could extend ValidationTextBox? with something like:

require([
    "dojo/_base/declare",
    "dijit/form/ValidationTextBox",
    "dojo/domReady!"
], function (declare, ValidationTextBox) {
    function isNumber(n) {
        return !isNaN(parseFloat(n)) && isFinite(n);
    }
    var InputBox = declare([ValidationTextBox], {
        buildRendering: function () {
            this.inherited(arguments);
            this.on("keypress", function (event) {
                var value = this.get("value") + event.charOrCode;
                if ((/\s/).test(event.charOrCode) || !isNumber(value)) {
                    // if value is not a number, ignore it
                    event.preventDefault();
                }
            });
        },
        validator: isNumber
    });
    new InputBox().placeAt(document.body);
});

[Bill Keese <[email protected]…>]

In 289f96b2592e3bb23a714256ab1d48bcbfaa3bce/dijit:

Error: Processor CommitTicketReference failed

Unsupported version control system "git": Can't find an appropriate component, maybe the corresponding plugin was not enabled? 

[Bill Keese <[email protected]…>]

In 073c1a07a3fc26901b63bda9f1e195fdaf7e9499/dijit:

Error: Processor CommitTicketReference failed

Unsupported version control system "git": Can't find an appropriate component, maybe the corresponding plugin was not enabled?