[CLOSED] Re-write Ext.preg call as Ext.define - Extjs5 migration

[betamax]

We use many plugins and have traditionally used this method to register them:

    if (typeof Ext !== 'undefined') {
    var AutomatedTesting = Ext.extend(Object, {
        constructor: function (config) {
            config = config || {};
            Ext.apply(this, config);
        },
        init: function (parent) {
            var parentFn = parent.view.getRowClass || Ext.emptyFn;
 
            parent.view.getRowClass = function (record, rowIndex, rp, ds) {
                var cls = ' qa-row-' + record.id;
                var parentCls = parentFn(record, rowIndex, rp, ds);
                if (parentCls != undefined) {
                    cls += ' ' + parentCls;
                }
                return cls;
            };
        }
    });
    Ext.preg('AutomatedTesting', AutomatedTesting);
};

On my Ext3 pages I am now getting the js error:
"Ext.preg is not a function"
According to the Extjs migration page,http://www.manilaforwarders.com/asse...migration.html, I should be using Ext.define(...), but I have not found any clear examples that would apply, to the type of code above, using define().
Can you give me some guidance?

[betamax]

if (typeof Ext !== 'undefined') {
    vms.extVer = parseFloat(Ext.version || Ext.getVersion().version);
    if (vms.extVer < 4){
        var AutomatedTesting = Ext.extend(Object, {
            constructor: function(config){
                config = config || {};
                Ext.apply(this, config);
            },
            init: function(parent){
                var parentFn = parent.view.getRowClass || Ext.emptyFn;
 
                parent.view.getRowClass = function(record, rowIndex, rp, ds){
                    var cls = ' qa-row-' + record.id;
                    var parentCls = parentFn(record, rowIndex, rp, ds);
                    if (parentCls != undefined){
                        cls += ' ' + parentCls;
                    }
                    return cls;
                };
            }
        });
        Ext.preg('AutomatedTesting', AutomatedTesting);
    } else{
        Ext.define('vms.AutomatedTesting', {
            extend: "Object",
            alias: "AutomatedTesting",
            constructor: function(config){
                config = config || {};
                Ext.apply(this, config);
            },
            init: function (parent) {
                var parentFn = parent.view.getRowClass || Ext.emptyFn;
 
                parent.view.getRowClass = function (record, rowIndex, rp, ds) {
                    var cls = ' qa-row-' + record.id;
                    var parentCls = parentFn(record, rowIndex, rp, ds);
                    if (parentCls != undefined) {
                        cls += ' ' + parentCls;
                    }
                    return cls;
                };
            }
        });
    };
};

[Daniil]

Hi @betamax,

Your solution appears to be fine.

[betamax]

hi Danill,

Should I need to leave the original .preg for backwards compat to Ext v1.8, or will the .define work for both version 1.x and v3.x?
Remember I am running multiple versions of Ext.

[Daniil]

I would expect that Ext.define() should work with v1. Does it work for you? As for Ext.preg(), I think it might be still required for v1. So, I would try to leave Ext.define() for both versions, but call Ext.preg() additionally for v1.

It is theoretically. I have not tested.