Yes it works now! Thank you Pat!
Greets -act On 27 Jul., 13:32, Pat Allan <[email protected]> wrote: > Okay, change is live - let me know if that fixes the problem. > > Cheers > > -- > Pat > > On 27/07/2009, at 11:56 AM, Rocco Di Leo wrote: > > > Great, thanks Pat! > > > Greets > > -act > > > On Mon, Jul 27, 2009 at 12:40 PM, Pat Allan <p...@freelancing- > > gods.com> wrote: > > > I've got a fix ready (you don't want to use .to_i if it's a blank > > string, or nil, because the default value of 1 is never reached) - > > just need to find a connection which is happy with SSH so I can push > > to GitHub. Give me a few hours :) > > > -- > > Pat > > > On 27/07/2009, at 11:29 AM, ACTRAiSER wrote: > > > > Sorry for Spamming. It has nothing to do with Rails 2.3.3. > > Pagination > > > works fine when downgraden to TS 1.1.24 again .. so i am still > > > lost. :-) Last Message for now. > > > > Greets > > > -act > > > > On 27 Jul., 12:25, ACTRAiSER <[email protected]> wrote: > > >> Uhm ... i just changed that line and noticed that now will_paginate > > >> complains with the same error. Its probably related to the > > upgrade to > > >> Rails 2.3.3 and not to TS/WillPaginate at all. I have to check if > > >> Url- > > >> Params are handled differently now with 2.3.3. If somebody already > > >> knows, please le me know. > > > >> Greets > > >> Rocco > > > >> On 27 Jul., 12:20, ACTRAiSER <[email protected]> wrote: > > > >>> Hi there, > > > >>> i upgraded to TS 1.2.0 yesterday and noticed that Pagination > > fails. > > >>> The URL in Question which is accessed > > >>> ishttp://development.myproject.com/clubs?page=2 > > > >>> It seems that it lacks a ".to_i"-Call in the current_page- > > method of > > >>> search.rb: > > > >>> def current_page > > >>> @options[:page] || 1 > > >>> end > > > >>> shoud probably be > > > >>> def current_page > > >>> @options[:page].to_i || 1 > > >>> end > > > >>> I checked the older version of TS and it handles the conversion > > >>> around > > >>> Line 464 in the method search_results > > > >>> client.limit = options[:per_page].to_i if options[:per_page] > > >>> page = options[:page] ? options[:page].to_i : 1 > > >>> page = 1 if page <= 0 > > > >>> For the sake of completeness, here is the Error output with 1.2.0: > > > >>> undefined method `-' for "2":String > > > >>> /opt/ruby-enterprise-1.8.6-20080810/lib/ruby/gems/1.8/gems/ > > >>> freelancing- > > >>> god-thinking-sphinx-1.2.0/lib/thinking_sphinx/search.rb:158:in > > >>> `offset' > > >>> [...] > > > >>> Greets > > >>> -act --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Thinking Sphinx" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/thinking-sphinx?hl=en -~----------~----~----~----~------~----~------~--~---
