Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix template and component attribute refresh for long paths - fix #125 #127

Closed
wants to merge 1 commit into from

Conversation

b-laporte
Copy link
Member

This PR fixes issue #125.
As you can see in the fix, there were actually 2 problems:

  • the observers were not recalculated (I forgot this use case!)
  • and the scope has to be switched to the container scope - this is due to a specificity of sub-templates and sub-components that have to manage two scopes: the scope of the container, and the scope of the sub-template. Current implementation has only one scope object, which is assigned to the template scope and not to the container scope, thus the fix..

@b-laporte b-laporte closed this in c2cc2e6 Apr 15, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant