search

1602 / compound

MVC framework. Built on Node.JS. Works on server and browser.
http://compoundjs.com
1.6k stars 182 forks source link

Parameter interpolation error #506

Closed aleksandrmelnyk closed 11 years ago

aleksandrmelnyk commented 11 years ago

Am using non-eval controllers mode. Controller action code looks like: IndexController.prototype.workroom = function(c) { this.roles = 'ADMIN_ROLE'; c.render(); }

And view code use 'roles' parameter: { ... roles: '#{roles}' ... }

And sometimes I got error: "roles is undefined". Most often this happens when i started application with process.env.NODE_ENV == 'production' and before started with development. Did anybody face this before? Might that be connected anyhow with cache?

anatoliychakkaev commented 11 years ago

It was fixed in stable branch and released.

On Tue, Apr 9, 2013 at 10:38 AM, Oleksandr Melnyk notifications@github.com wrote:

Am using non-eval controllers mode. Controller action code looks like: IndexController.prototype.workroom = function(c) { this.roles = 'ADMIN_ROLE'; c.render(); } And view code use 'roles' parameter: { ... roles: '#{roles}' ... } And sometimes I got error: "roles is undefined". Most often this happens when i started application with process.env.NODE_ENV == 'production' and before started with development. Did anybody face this before?

Might that be connected anyhow with cache?

Reply to this email directly or view it on GitHub: https://github.com/1602/compound/issues/506

aleksandrmelnyk commented 11 years ago

Yup, I have just updated compund module and didn't experience the issue.

Thank you much!

anatoliychakkaev commented 11 years ago

To what version you've updated?

On Tue, Apr 9, 2013 at 1:12 PM, Oleksandr Melnyk notifications@github.comwrote:

Yup, I have just updated compund module and didn't experience the issue.

Thank you much!

— Reply to this email directly or view it on GitHubhttps://github.com/1602/compound/issues/506#issuecomment-16102066 .

Thanks, Anatoliy Chakkaev

aleksandrmelnyk commented 11 years ago

1.1.6

anatoliychakkaev commented 11 years ago

Yep, that's fine. It's current stable. Or you can use unstable 1.1.7-x if you need some new stuff i'm working on.

On Tue, Apr 9, 2013 at 1:23 PM, Oleksandr Melnyk notifications@github.comwrote:

1.1.6

— Reply to this email directly or view it on GitHubhttps://github.com/1602/compound/issues/506#issuecomment-16102596 .

Thanks, Anatoliy Chakkaev

aleksandrmelnyk commented 11 years ago

Thank you Anatoliy, appreciate.