Merge pull request #394 from travis-ci/owner-route-error-handling

Add error handling for the owner route

app/controllers/error.coffee

@@ -0,0 +1,5 @@
+`import Ember from 'ember'`
+
+Controller = Ember.Controller.extend()
+
+`export default Controller`

app/routes/basic.coffee

@@ -2,6 +2,11 @@
 `import Ember from 'ember'`
 
 Route = Ember.Route.extend
+  activate: ->
+    if @routeName != 'error'
+      @controllerFor('error').set('layoutName', null)
+    return @_super.apply(this, arguments)
+
   beforeModel: (transition) ->
     @auth.autoSignIn() unless @signedIn()
 

app/routes/error.js

@@ -0,0 +1,10 @@
+import TravisRoute from 'travis/routes/basic';
+
+export default TravisRoute.extend({
+  resetController(controller, isExiting, transition) {
+    if (isExiting) {
+      controller.set('message', null);
+      controller.set('layoutName', null);
+    }
+  }
+});

app/routes/owner.coffee

@@ -4,7 +4,6 @@
 `import config from 'travis/config/environment'`
 
 Route = TravisRoute.extend
-
   deactivate: ->
     @controllerFor('loading').set('layoutName', null)
 
@@ -19,4 +18,17 @@ Route = TravisRoute.extend
 
     @_super.apply(this, arguments)
 
+  actions:
+    error: (error, transition, originRoute) ->
+      login = transition.params.owner.owner
+      message = if error.status == 404
+        "Couldn't find an owner with login \"#{login}\""
+      else
+        "There was an error while loading data, please try again."
+
+      @controllerFor('error').set('layoutName', 'simple')
+      @controllerFor('error').set('message', message)
+
+      return true
+
 `export default Route`

app/templates/error.hbs

@@ -0,0 +1,5 @@
+{{#if message}}
+  {{message}}
+{{else}}
+  There was an error, please try again.
+{{/if}}

app/templates/owner/error.hbs

@@ -0,0 +1 @@
+foo

app/views/error.coffee

@@ -0,0 +1,9 @@
+`import Ember from 'ember'`
+
+View = Ember.View.extend
+  layoutName: (->
+    if name = @get('controller.layoutName')
+      'layouts/' + name
+  ).property('controller.layoutName')
+
+`export default View`