mauricemach / zappa

Node development for the lazy.
zappajs.org
MIT License
949 stars 81 forks source link

Fix problems referencing partial 'global/nav' from layout #103

Closed timshadel closed 12 years ago

timshadel commented 13 years ago

Layout didn't get the app id appended to it, so referencing a partial with a name 'global/nav' resulted in the monkeypatching expecting that the app id was 'global'. Additionally, even if the monkeypatched vesion received a valid raw path, the raw version was never attempted.

This is fixed in two ways:

The tests show both problems.

Of note is the fact that the problem can be fixed by #2 alone, but then it always checks the filesystem twice. The change for #1 avoids that.

timshadel commented 12 years ago

See comments on zappajs/zappajs#8 (needs tests + doc). Closing since I won't get around to that work.