Revert "move log subscrition to a log controller"

This was causing the log to not refresh sometimes when switching from
one repo to another, I'm reverting for now, we will have to have another
look.

This reverts commit 2e8b74160c.

Conflicts:
	assets/scripts/app/controllers/build.coffee
	assets/scripts/app/models/build.coffee
	assets/scripts/app/templates/builds/show.hbs
	assets/scripts/app/views/log.coffee
	assets/scripts/vendor/log.js

assets/scripts/app/controllers.coffee

@@ -29,7 +29,6 @@ require 'controllers/builds'
 require 'controllers/flash'
 require 'controllers/home'
 require 'controllers/job'
-require 'controllers/log'
 require 'controllers/profile'
 require 'controllers/repos'
 require 'controllers/repo'

assets/scripts/app/controllers/build.coffee

@@ -1,9 +1,8 @@
 Travis.BuildController = Ember.Controller.extend
-  needs: ['repo', 'log']
+  needs: ['repo']
   repoBinding: 'controllers.repo.repo'
   buildBinding: 'controllers.repo.build'
   commitBinding: 'build.commit'
-  jobBinding: 'controllers.log.job'
 
   currentItemBinding: 'build'
 
@@ -22,7 +21,3 @@ Travis.BuildController = Ember.Controller.extend
   urlCommitter: (->
     Travis.Urls.email(@get('commit.committerEmail'))
   ).property('commit.committerEmail')
-
-  hasLoaded: (->
-    @set('controllers.log.job', @get('build.firstJob')) if @get('build.firstJob') && !@get('build.isMatrix')
-  ).observes('build.id', 'build.firstJob')

assets/scripts/app/controllers/job.coffee

@@ -1,5 +1,5 @@
 Travis.JobController = Em.Controller.extend
-  needs: ['repo', 'log']
+  needs: ['repo']
 
   jobBinding: 'controllers.repo.job'
   repoBinding: 'controllers.repo.repo'
@@ -16,7 +16,3 @@ Travis.JobController = Em.Controller.extend
   urlCommitter: (->
     Travis.Urls.email(@get('commit.committerEmail'))
   ).property('commit.committerEmail')
-
-  hasLoaded: (->
-    @set('controllers.log.job', @get('job'))
-  ).observes('job.id')

assets/scripts/app/controllers/log.coffee

@@ -1,15 +0,0 @@
-Travis.LogController = Ember.Controller.extend
-  needs: ['repo']
-
-  logObserver: (->
-    @unsubscribe()
-    @subscribe(@get('job'))
-  ).observes('job.logId')
-
-  subscribe: (job) ->
-    job = @get('job')
-    job.subscribe() if job && !job.get('isFinished')
-
-  unsubscribe: ->
-    job = @get('subscribed')
-    job.unsubscribe() if job

assets/scripts/app/models/build.coffee

@@ -30,10 +30,6 @@ require 'travis/model'
     @get('jobs.length') > 1
   ).property('jobs.length')
 
-  firstJob: (->
-    @get('jobs').objectAt(0)
-  ).property('jobs.length')
-
   isFinished: (->
     @get('state') in ['passed', 'failed', 'errored', 'canceled']
   ).property('state')

assets/scripts/app/routes.coffee

@@ -112,7 +112,6 @@ Travis.JobRoute = Ember.Route.extend Travis.LineNumberParser,
       controller.set('lineNumber', lineNumber)
 
     repo = @container.lookup('controller:repo')
-    # console.log model.toString()
     repo.set('job', model)
     repo.activate('job')
 

assets/scripts/app/templates/builds/show.hbs

@@ -53,7 +53,7 @@
       {{view Travis.JobsView jobsBinding="build.requiredJobs" required="true"}}
       {{view Travis.JobsView jobsBinding="build.allowedFailureJobs"}}
     {{else}}
-      {{view Travis.LogView jobBinding="job"}}
+      {{view Travis.LogView jobBinding="build.jobs.firstObject"}}
     {{/if}}
   {{else}}
     There are no builds for this repository.

assets/scripts/app/views/build.coffee

@@ -44,7 +44,6 @@ Travis.reopen
 
   BuildView: Travis.View.extend
     templateName: 'builds/show'
-    jobBinding: 'controller.job'
     classNameBindings: ['color', 'loading']
 
     loadingBinding: 'controller.loading'

assets/scripts/app/views/log.coffee

@@ -157,13 +157,16 @@ Travis.OrderedLogEngineMixin = Ember.Mixin.create
       url = Travis.Urls.plainTextLog(@get('log.id'))
       this.$("#log").append $("<p class=\"cut\">Log was too long to display. Download the <a href=\"#{url}\">the raw version</a> to get the full log.</p>")
 
-
 Travis.reopen
   LogView: Travis.View.extend
     templateName: 'jobs/log'
     logBinding: 'job.log'
     contextBinding: 'job'
 
+    didInsertElement: ->
+      job = @get('job')
+      job.subscribe() if job && !job.get('isFinished')
+
     willDestroyElement: ->
       job = @get('job')
       job.unsubscribe() if job