I'm still not clear on what FilesController::Wrapper() is even supposed to be doing. It looks like the only thing linking to it is in the iphone template. If this is it's own page, why are we not just using an existing template, and rendering the appropriate things inside of it? Also, there is a comment in the function that says "// Must be careful with this because we aren't checking for ownership here." -- Why aren't we checking ownership?
OS: Mac OS X → All
Hardware: PC → All
Summary: move html code out of FilesController::wrapper() and into a view. → Joey Server - move html code out of FilesController::wrapper() and into a view.
Joey has been retired. Thanks for participating. http://joey.labs.mozilla.com/
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.