12spokes/tandem — comment on commit 36a196a105ce40a55427a9d0b49044954d55e59a
File: app/helpers/tandem/pages_helper.rb line 164
By treybean:

Yeah, that's along the same way I'd approach it-break it all down into
simpler methods. Probably have a grouped_pages method and then a page_link
method and then iterate away. I just didn't have time this morning and
wanted to get this patch in so Colleen doesn't get blocked by a refactor.
I'll create a chore for it when I get in.

On Tuesday, February 14, 2012, Ethan Vizitei <
reply@reply.github.com>
wrote:
> looking at the helper, is there any reason we can't define "iterate" and
"generate" as protected methods and set "page_groups" as an instance
variable for the duration? I think that would do away with the inline
definition of procs and such, but maybe there's an obstruction I haven't
noted
>
> ---
> Reply to this email directly or view it on GitHub:
>
https://github.com/12spokes/tandem/commit/36a196a105ce40a55427a9d0b49044954d55e59a#commitcomment-964773
>

--
Trey Bean
President, Co-Founder
12 Spokes
(801) 736-4716 x 700
(888) 422-7741 x 700