mojito Calling Do.Halt in mojito plugins prevents other plugins from hooking into the RS

Calling Do.Halt in mojito plugins prevents other plugins from hooking into the RS

Open jarnoux opened this issue 11 years ago • 4 comments

I want to access yui-lang type resources and their yui.name but this: https://github.com/yahoo/mojito/blob/develop/lib/app/addons/rs/yui.js#L545 won't let me. Please find another way to return the resource and prevent the host method from executing. Maybe neutralize the host method? Maybe use Do.Prevent?

Oct 10 '13 02:10 jarnoux

/cc @drewfish

Oct 10 '13 15:10 caridy

I'm investigating a way to plug into the yui plugin but it proves tricky as afterHostMethod cannot be used on methods that are already used to wrap around a host method: https://github.com/yui/yui3/issues/1297

Oct 10 '13 16:10 jarnoux

@jarnoux You can solve your problem by using after('addResourceVersion') yes? If so that lowers the priority of this issue, though we should still look at why addons/rs/yui is doing a Do.Halt.

Oct 10 '13 21:10 drewfish

addResourceVersion solves my problem

On Thu, Oct 10, 2013 at 2:14 PM, Drew Folta [email protected] wrote:

@jarnoux You can solve your problem by doing after('addResourceVersion') yes? If so that lowers the priority of this issue, though we should still look at why addons/rs/yui is doing a Do.Halt.

Reply to this email directly or view it on GitHub: https://github.com/yahoo/mojito/issues/1261#issuecomment-26092353

Oct 11 '13 02:10 jarnoux

mojito
mojito copied to clipboard

Calling Do.Halt in mojito plugins prevents other plugins from hooking into the RS

@jarnoux You can solve your problem by doing `after('addResourceVersion')` yes? If so that lowers the priority of this issue, though we should still look at why addons/rs/yui is doing a `Do.Halt`.

mojito mojito copied to clipboard

Calling Do.Halt in mojito plugins prevents other plugins from hooking into the RS

@jarnoux You can solve your problem by doing after('addResourceVersion') yes? If so that lowers the priority of this issue, though we should still look at why addons/rs/yui is doing a Do.Halt.

mojito
mojito copied to clipboard

@jarnoux You can solve your problem by doing `after('addResourceVersion')` yes? If so that lowers the priority of this issue, though we should still look at why addons/rs/yui is doing a `Do.Halt`.