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

Check if record is in a record array before adding it

Browse Source
This commit is contained in:
Piotr Sarnacki 2013-07-31 14:58:43 +02:00
parent 27da51fcc1
commit 07d2d1ff2a
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
assets/scripts/app/app.coffee
View File

@ -99,7 +99,9 @@ unless window.TravisApplication
result = @loadOrMerge(type, json[root]) result = @loadOrMerge(type, json[root])
if result && result.id if result && result.id
record = type.find(result.id) record = type.find(result.id)
type.addToRecordArrays(record) # TODO: find a nicer way to not add record to record arrays twice
if !type._findAllRecordArray || !type._findAllRecordArray.contains(record)
type.addToRecordArrays(record)
# we get other types of records only in a few situations and # we get other types of records only in a few situations and
# it's not always needed to update data, so I'm specyfing which # it's not always needed to update data, so I'm specyfing which
@ -108,7 +110,8 @@ unless window.TravisApplication
result = @loadOrMerge(Travis.Repo, json.repository || json.repo) result = @loadOrMerge(Travis.Repo, json.repository || json.repo)
if result && result.id if result && result.id
record = Travis.Repo.find(result.id) record = Travis.Repo.find(result.id)
Travis.Repo.addToRecordArrays(record) if !Travis.Repo._findAllRecordArray || !Travis.Repo._findAllRecordArray.contains(record)
Travis.Repo.addToRecordArrays(record)
loadOrMerge: (type, hash, options) -> loadOrMerge: (type, hash, options) ->
options ||= {} options ||= {}