Ext.app.Controller bug in getModuleClassName ?

[robl]

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



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;
},

[Jamie Avins]

Thank you for the report.

[rake]

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.


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:


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

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

[edspencer]

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