makeusabrew/jaoss

move all assigns in controller construct to render

Closed this issue · 1 comments

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.

moving tpl var assignment logic to correct place (closed by 9e74f86), fixing bug where render was always invoked even on redirect