yrgoldteeth / bootstrap-will_paginate

Format will_paginate html to match Twitter Bootstrap styling

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Rails 4.1 PG::SyntaxError: ERROR: syntax error at or near "as"

DazDotOne opened this issue · comments

As documented in O-I/pluck#10 and Casecommons/pg_search#167, there is an issue with will_paginate using the old .count() syntax.

Nazgum has made a branch that seems to have fixed will_paginate at https://github.com/nazgum/will_paginate.git

any chance this could be pulled in?

I've never done a pull request before but would be able to spare some time if you could give me instructions.

Thanks

The .count() syntax is in the rubygems version of will_paginate still I'm guessing and I don't have any control of the upstream will_paginate code, this is just the view helpers to create Twitter Bootstrap particular HTML for the will_paginate views. Or, possibly, you've got an older version of will_paginate installed that meets the dependency for this particular gem.

I will check this against the pure will_paginate rubygem version later today, and if it's fixed upstream I'll look into locking the will_paginate dependency version to current.

commented

As far as I know, the commit necessary to fix this was merged into the master branch of will_paginate. However, there has been no new gem release.

Ah, then yeah, I can't do anything about that, I just maintain the bootstrap-will_paginate gem and not the main will_paginate one.

Nice one guys