Correcting error #130 - changing @stop to pause()

[Art1Sec8]

This appears to fix issue #130:
#130

@Stop is depreciated in iron-router 0.7.0 and going forward we should use @Pause(). When @Stop is used in the entrySignOut route, BlazeLayout throws the following error in the console (client side):

Exception from Deps recompute function: Error: BlazeLayout: Sorry, couldn't find a template named entrySignOut. Are you sure you defined it?

Replacing @Stop with @Pause seems to fix the issue.

[cinjon]

I'm getting an error on this: "Uncaught TypeError: undefined is not a function" at line 27 in router.coffee, i.e. where this switch occurred. Are you guys not getting that?

[hellogerard]

Hmm,

You have to change line 22 from:

    onBeforeAction: ->

to

    onBeforeAction: (pause) ->

[cinjon]

Thanks for the quick reply Gerard. That didn't work though (it's also not on the router.coffee for this commit). Any other ideas?

[hellogerard]

Sorry then. I didn't actually try this. I just know it was a change in
blaze.

On Tuesday, April 15, 2014, cinjon notifications@github.com wrote:

Thanks for the quick reply Gerard. That didn't work though (it's also not
on the router.coffee for this commit). Any other ideas?

Reply to this email directly or view it on GitHubhttps://github.com//pull/143#issuecomment-40535109
.

Sent from my mobile device.

[Art1Sec8]

@cinjon I wasn't getting this error before, but I am getting it now. I am not sure what changed. But, I will try to dig into it when I can.

[matthewfieger]

I am also getting "Uncaught TypeError: undefined is not a function" on sign-out. I have @Pause() in line 27 of router.coffee.

[dkoo761]

+1 I am also getting this error now.

[Art1Sec8]

Ok; I think I may have gotten this one now. Take a look at iron-meteor/iron-router#620

Looks like the the issue is that pause needs to be passed as a parameter like so:

 @route 'entrySignOut',
    path: '/sign-out'
    onBeforeAction: (pause)->
      Session.set('entryError', undefined)
      if AccountsEntry.settings.homeRoute
        Meteor.logout () ->
          Router.go AccountsEntry.settings.homeRoute
        pause();

I put together a PR: #168

[matthewfieger]

That worked for me. Thanks!

[dgarzon]

Thank you @Art1Sec8 that fixed the problem!