Ext.app.Controller bug in getModuleClassName ?

I'm not sure but I think there is a bug in getModuleClassName. this.name should be namespace.

Code:

---

    getModuleClassName: function(name, type) {
        var namespace = Ext.Loader.getPrefix(name);


        if (namespace.length > 0 && namespace !== name) {
            return name;
        }


        return this.name + '.' + type + '.' + name;  // IS THIS A BUG?
        // Shouldn't it be:  return namespace +'.' + type + '.' + name;
    },

---

Thank you for the report.

Looking at this in PR3, getModuleClassName() is no longer in Ext.app.Controller, but has been moved to Ext.app.Application. But has exact same definition as what was classified here as a bug.

Code:

---

getModuleClassName: function(name, type) {
        var namespace = Ext.Loader.getPrefix( name );
        if (namespace.length > 0 && namespace !== name) {
            return name;
        }
        return this.name + '.' + type + '.' + name;
    }

---

Other thing that looks fishy is that Ext.app.Controller appears to be aliased to Ext.Application. Beginning of Ext.app.Controller is as follows:

Code:

---

Ext.define( 'Ext.app.Controller', {
    alternateClassName: 'Ext.Application'

---

Perhaps some hasty "cut-n-paste" refactoring??

This has all been reworked for PR4. I'm closing the ticket because it works in a different way internally now