Sutty/trabajo-afectivo
5
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Fixed race condition of forced model.retrieve(). Added trigger for local model changes vi model.subscribe().

Browse source
This commit is contained in:
Martin Edenhofer 2014-02-09 02:18:06 +01:00
parent 47841fbbac
commit b4cd8db227
1 changed files with 15 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
app/assets/javascripts/app/models/_application_model.js.coffee
View file

@ -136,9 +136,12 @@ class App.Model extends Spine.Model
delete @RETRIEVE_CALLBACK[ record.id ][ key ]
if _.isEmpty @RETRIEVE_CALLBACK[ record.id ]
delete @RETRIEVE_CALLBACK[ record.id ]
@fetchActive = false
# fetch object
console.log 'debug', 'loading..' + @className + '..', id
if !@fetchActive
@fetchActive = true
App[ @className ].fetch( id: id )
return true
return false
@ -183,11 +186,22 @@ class App.Model extends Spine.Model
if !App[ @constructor.className ]['SUBSCRIPTION_ITEM']
App[ @constructor.className ]['SUBSCRIPTION_ITEM'] = {}
# subscribe and render data after local change
@bind(
'refresh change'
=>
for key, callback of App[ @constructor.className ]['SUBSCRIPTION_ITEM'][ @id ]
callback(@)
)
# subscribe and render data after server change
events = "#{@constructor.className}:created #{@constructor.className}:updated #{@constructor.className}:destroy"
App.Event.bind(
events
(record) =>
if @id.toString() is record.id.toString()
# load only once from server
for key, callback of App[ @constructor.className ]['SUBSCRIPTION_ITEM'][ @id ]
App[ @constructor.className ].retrieve( @id, callback, true )
'Item::Subscribe::' + @constructor.className