move all assigns in controller construct to render
Closed this issue · 1 comments
makeusabrew commented
Having the assigns in construct is pointless - we might never get through to actually rendering anything (redirect or whatever). This has a particularly adverse affect when messages are present on the flash message stack as they're popped off and then potentially lost.
Move to render / render tpl / json - whichever is appropriate.
makeusabrew commented
moving tpl var assignment logic to correct place (closed by 9e74f86), fixing bug where render was always invoked even on redirect