ViewController must call parent constructor before creating observers.

Author: brian428

packages/deft/src/coffee/mvc/Observer.coffee

@@ -140,7 +140,7 @@ Ext.define( 'Deft.mvc.Observer',
 		hostTarget = @locateTarget( host, target )
 		return false if not hostTarget?
 
-		if hostTarget.isObservable? or hostTarget.mixins?.observable?
+		if hostTarget.isObservable? or hostTarget.isUtilObservable? or hostTarget.mixinId is "observable" or hostTarget.mixins?.observable?
 			return true
 		else
 			hostTargetClass = Ext.ClassManager.getClass( hostTarget )

packages/deft/src/coffee/mvc/ViewController.coffee

@@ -166,9 +166,10 @@ Ext.define( 'Deft.mvc.ViewController',
 			@companions = Ext.merge( @companions, config.companions )
 			delete config.companions
 
-		@initConfig( config ) # Ensure any config values are set before creating observers.
+		# Core Sencha VC logic, including initConfig() call, happens in superclass constructor, so call it first.
+		@callParent( arguments )
 		@createObservers()
-		return @callParent( arguments )
+		return @
 
 
 	###*

packages/deft/test/coffee/mvc/ViewController.coffee

@@ -257,7 +257,7 @@ describe( 'Deft.mvc.ViewController', ->
 
 		specify( 'attaches listeners to observed objects in a ViewController with no subclasses', ->
 			eventData = { value1: true, value2: false }
-			
+
 			Ext.define( 'ExampleViewController',
 				extend: 'Deft.mvc.ViewController'
 
@@ -275,11 +275,11 @@ describe( 'Deft.mvc.ViewController', ->
 			sinon.spy( ExampleViewController.prototype, 'messageHandler' )
 
 			messageBus = Ext.create( 'Ext.util.Observable' )
-			
+
 			viewController = Ext.create( 'ExampleViewController',
 				messageBus: messageBus
 			)
-			
+
 			expect( hasListener( messageBus, 'message' ) ).to.be.true
 
 			messageBus.fireEvent( 'message', eventData )