suzanne.soy/travis-web
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Find job for clearing logs only if it's already in store

Browse Source 
Otherwise we make ajax request every time job is created.
This commit is contained in:
Piotr Sarnacki 2013-01-23 22:08:17 +01:00
parent ee9f9d934c
commit 2723ae5c06
2 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
assets/scripts/app/pusher.coffee
View File

@ -41,8 +41,8 @@ $.extend Travis.Pusher.prototype,
# TODO remove job:requeued, once sf-restart-event has been merged # TODO remove job:requeued, once sf-restart-event has been merged
# TODO this also needs to clear logs on build:created if matrix jobs are already loaded # TODO this also needs to clear logs on build:created if matrix jobs are already loaded
if event == 'job:created' || event == 'job:requeued' if event == 'job:created' || event == 'job:requeued'
job = Travis.Job.find(data.job.id) if Travis.store.isInStore(Travis.Job, data.job.id)
job.clearLog() if job Travis.Job.find(data.job.id).clearLog()
Ember.run.next -> Ember.run.next ->
Travis.app.store.receive(event, data) Travis.app.store.receive(event, data)

3
assets/scripts/app/store.coffee
View File

@ -67,6 +67,9 @@ Travis.Store = DS.Store.extend
{ id: id, clientId: clientId } { id: id, clientId: clientId }
isInStore: (type, id) ->
!!@typeMapFor(type).idToCid[id]
receive: (event, data) -> receive: (event, data) ->
[name, type] = event.split(':') [name, type] = event.split(':')