Joey Server - move html code out of FilesController::wrapper() and into a view.

RESOLVED WONTFIX

Status

RESOLVED WONTFIX
11 years ago
10 years ago

People

(Reporter: dougt, Assigned: clouserw)

Tracking

Details

(Whiteboard: marked_as_wfm_due_to_project_retirement)

(Reporter)

Description

11 years ago
 
(Assignee)

Comment 1

11 years ago
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
(Reporter)

Updated

11 years ago
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.
(Reporter)

Comment 2

10 years ago
Joey has been retired.  Thanks for participating.

http://joey.labs.mozilla.com/
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → WONTFIX
Whiteboard: marked_as_wfm_due_to_project_retirement
You need to log in before you can comment on or make changes to this bug.