sasallys/dashboardinator — pull request #1 by zenchild (2013-02-27 05:11:44+00:00)
Title: PortalsController#show changes

Hey Robert,

I had a bit of time tonight to look at your code tonight. In particular the piece you pointed out to me earlier. I added a few changes I thought might make it a bit easier to deal with in the future. I moved the SQL to find the ordered metrics to the Portal model and used ARel syntax instead. That way it will return an ActiveRecord::Relation and you can tack on additional queries or do a #reload on the returned object to freshen its data. You're not obligated by any means to use this Pull but I thought I'd send it on anyway.

Cheers,

Dan