Fix bug where models weren't being pluralized correctly in page_entries_info #208

Closed
wants to merge 2 commits into
from

Projects

None yet

3 participants

Ryan Hall added some commits Dec 2, 2011
Ryan Hall Fixed pluralization bug in page_entries_info e3b285a
Ryan Hall bug fix: total_entries was returning the wrong value for DISTINCT ON …
…queries. This fix changes count to length when these queries come up.
c16fc52
tobowers commented Jan 9, 2012

+1

Owner
mislav commented Jan 31, 2012

This fixes it only for the English language, so it defies the purpose of having i18n in the first place. Also, no tests.

@mislav mislav closed this Jan 31, 2012

Is there a work around for now?

Owner
mislav commented Jan 31, 2012

Not yet.

@cmeiklejohn cmeiklejohn referenced this pull request in rubygems/rubygems.org Sep 9, 2012
Closed

Incorrect pluralization in Search results page #456

@anenadic anenadic pushed a commit to BioVeL/biodiversitycatalogue that referenced this pull request Nov 14, 2013
Alex Nenadic Added entries for models in config/locales/en.yml as will_paginate ge…
…m does not pluralize models correctly in page_entries_info - see mislav/will_paginate#208.
86fc610
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment