Added ability to queue up children and register them once their parent is registered

Author: theJoeBiz

src/state.js

@@ -1,7 +1,7 @@
 $StateProvider.$inject = ['$urlRouterProvider', '$urlMatcherFactoryProvider', '$locationProvider'];
 function $StateProvider(   $urlRouterProvider,   $urlMatcherFactory,           $locationProvider) {
 
-  var root, states = {}, $state;
+  var root, states = {}, $state, queue = [];
 
   // Builds state properties from definition passed to registerState()
   var stateBuilder = {
@@ -143,6 +143,13 @@ function $StateProvider(   $urlRouterProvider,   $urlMatcherFactory,           $
     return undefined;
   }
 
+  function queueState(parentName, state) {
+    if (!queue[parentName]) {
+      queue[parentName] = [];
+    }
+
+    queue[parentName].push(state);
+  }
 
   function registerState(state) {
     // Wrap a new object around the state so we can store our private details easily.
@@ -156,6 +163,14 @@ function $StateProvider(   $urlRouterProvider,   $urlMatcherFactory,           $
     if (!isString(name) || name.indexOf('@') >= 0) throw new Error("State must have a valid name");
     if (states[name]) throw new Error("State '" + name + "'' is already defined");
 
+    // If parent is not registered yet, add state to queue and register later
+    if (name.indexOf('.') !== -1) {
+      var parentName = name.substring(0, name.lastIndexOf('.'));
+      if (!states[parentName]) {
+        return queueState(parentName, state.self);
+      }
+    }
+
     for (var key in stateBuilder) {
       if (isFunction(stateBuilder[key])) state[key] = stateBuilder[key](state, stateBuilder.$delegates[key]);
     }
@@ -170,6 +185,13 @@ function $StateProvider(   $urlRouterProvider,   $urlMatcherFactory,           $
       }]);
     }
 
+    // Register any queued children
+    if (queue[name]) {
+      for (var i = 0; i < queue[name].length; i++) {
+        registerState(queue[name][i]);
+      }
+    }
+
     return state;
   }
 

test/stateSpec.js

@@ -25,7 +25,9 @@ describe('state', function () {
       E = { params: [ 'i' ] },
       H = { data: {propA: 'propA', propB: 'propB'} },
       HH = { parent: H },
-      HHH = {parent: HH, data: {propA: 'overriddenA', propC: 'propC'} }
+      HHH = {parent: HH, data: {propA: 'overriddenA', propC: 'propC'} },
+      J = {},
+      JJ = {},
       AppInjectable = {};
 
   beforeEach(module(function ($stateProvider, $provide) {
@@ -45,6 +47,8 @@ describe('state', function () {
       .state('H', H)
       .state('HH', HH)
       .state('HHH', HHH)
+      .state('JJ.J', J)
+      .state('JJ', JJ)
 
       .state('home', { url: "/" })
       .state('home.item', { url: "front/:id" })
@@ -87,7 +91,6 @@ describe('state', function () {
     expect($state.current).toBe(state);
   }
 
-
   describe('.transitionTo()', function () {
     it('returns a promise for the target state', inject(function ($state, $q) {
       var trans = $state.transitionTo(A, {});
@@ -539,6 +542,8 @@ describe('state', function () {
         'H',
         'HH',
         'HHH',
+        'JJ',
+        'JJ.J',
         'about',
         'about.person',
         'about.person.item',