geddy / model

Datastore-agnostic ORM in JavaScript
265 stars 55 forks source link

Files in app/controllers are required to have content #170

Open danfinlay opened 10 years ago

danfinlay commented 10 years ago

Right now trying to remove a controller requires removing the controller files as well, which was confusing for me. An empty file in app/controllers shouldn't break the program, right?

mde commented 10 years ago

Did you mean to file this issue over in https://github.com/geddy/geddy?

It might. Geddy assumes everything in controllers is something it can load and look for a constructor on. Init code lives here: https://github.com/geddy/geddy/blob/master/lib/init/controller.js Would love a patch to fix this issue.