makesites / brisk

Automation for Express.js #express.js #javascript
3 stars 1 forks source link

Remove method param when no method found #34

Closed tracend closed 11 years ago

tracend commented 11 years ago

Currently the URL is split and compared against a controller and its methods.

If a method is not found it reverts to index, but the method name that was looked up is passed to the controller unchanged.

Instead of using the method name as a variable in conditioning it is best if it's removed from the passed parameters and added in the query object.

tracend commented 11 years ago

To avoid any overwriting with existing data in the query, an unlikely yet generic key should be used.

Proposing _key

tracend commented 11 years ago

Closing as addressed