denny/ShinyCMS — comment on commit 4c4b68f03313cac72318d305771e545a914dd1e5
File:  line 0
By bobtfish:

erm?

I mean, the logic in the model, AWESOME - but the previous code _EPICLY_ fails to do that...

Well, sure - the logic is technically in the model, but this also means the model has in-depth knowledge of what $ctx is, which just isn't appropriate outside web environments.

I appreciate that I've front-loaded here (i.e. moving things back into controllers herre, but given we are talking about an app which considers $rs->search(%really_complex_stuff) to be a sane thing to do, and it was passing the entire $ctx (rather than select parts from in) into the model that was being built, I think this is a step forwards!