Make `register` more flexible

publicclass / express-partials

Express 3.x Layout & Partial support.

182 stars 146 forks source link

Make `register` more flexible #3

Closed shimaore closed 12 years ago

shimaore commented 12 years ago

I didn't figure out we could have register use different types of arguments until I tried to use it for real. :(

This makes register incrementally a little bit more clever about its second argument. It now accepts a render function (same as previously), a template module (or some object with a render member), or a template module name.

Updated tests are provided.

shimaore commented 12 years ago

Hmmm, github is nicely mixing stuff up. To clarify:

c365887e9d52f544d8f188a03b419ced7bc52e84 contains the changes to register() I described above.
cef6e19620b0f092494857364c12335070668234 are the additions to the test suite that test for: "you don't need to register a template engine if the extension is the same as the engine's name." (This is true of express-partials but not of Express 3.x at this time, though.)