I remember a while back that somebody had patches to make Facebooker work on Rails 2.2. Are those still around? I''d like to get them merged over the next couple of days. Mike -- Mike Mangino http://www.elevatedrails.com
You already merged my changes that make Rails 2.2 happy. My app seems
to work on 2.2RC2.
There''s one outstanding issue that doesn''t have a nice, clear
solution. As of 2.2, concat complains that it doesn''t need a binding
argument anymore, and the old use is deprecated. On the other hand,
removing that argument will break anyone who''s not on 2.2 yet.
Possible solutions:
1) define our own method that calls concat and does the right thing
depending on the version of rails
2) make a git branch for rails 2.2 changes, make the change below in this branch
3) remove the binding argument everywhere and make people upgrade
4) do nothing
Preferences?
Victor
On Wed, Nov 19, 2008 at 10:10 AM, Mike Mangino
<mmangino at elevatedrails.com> wrote:> I remember a while back that somebody had patches to make Facebooker work
on
> Rails 2.2. Are those still around? I''d like to get them merged
over the next
> couple of days.
>
> Mike
>
> --
> Mike Mangino
> http://www.elevatedrails.com
>
>
>
> _______________________________________________
> Facebooker-talk mailing list
> Facebooker-talk at rubyforge.org
> http://rubyforge.org/mailman/listinfo/facebooker-talk
>
On Nov 19, 2008, at 2:59 PM, Victor Costan wrote:> You already merged my changes that make Rails 2.2 happy. My app seems > to work on 2.2RC2.Okay, that''s good to know. It looks like I just need to fix the tests then.> > > There''s one outstanding issue that doesn''t have a nice, clear > solution. As of 2.2, concat complains that it doesn''t need a binding > argument anymore, and the old use is deprecated. On the other hand, > removing that argument will break anyone who''s not on 2.2 yet. > > Possible solutions: > 1) define our own method that calls concat and does the right thing > depending on the version of rails > 2) make a git branch for rails 2.2 changes, make the change below in > this branch > 3) remove the binding argument everywhere and make people upgrade > 4) do nothingThat''s annoying. I''m okay with the warnings for now. If anyone feels strongly about it, they can create another level of indirection. Mike> > > Preferences? > Victor > > > > On Wed, Nov 19, 2008 at 10:10 AM, Mike Mangino > <mmangino at elevatedrails.com> wrote: >> I remember a while back that somebody had patches to make >> Facebooker work on >> Rails 2.2. Are those still around? I''d like to get them merged over >> the next >> couple of days. >> >> Mike >> >> -- >> Mike Mangino >> http://www.elevatedrails.com >> >> >> >> _______________________________________________ >> Facebooker-talk mailing list >> Facebooker-talk at rubyforge.org >> http://rubyforge.org/mailman/listinfo/facebooker-talk >>-- Mike Mangino http://www.elevatedrails.com
I''m having an issue with using link_to from the publisher after upgrading to 2.2. Error is: can''t convert String into Hash c:/ruby/lib/ruby/gems/1.8/gems/actionpack-2.2.2/lib/action_controller/url_rewriter.rb:130:in `merge'' c:/ruby/lib/ruby/gems/1.8/gems/actionpack-2.2.2/lib/action_controller/url_rewriter.rb:130:in `url_for'' c:/ruby/lib/ruby/gems/1.8/gems/actionpack-2.2.2/lib/action_view/helpers/url_helper.rb:228:in `link_to'' app/helpers/application_helper.rb:23:in `show_link'' app/models/notification_publisher.rb:19:in `notification_text'' Line 23 of application_helper.rb is: link_to h(org.name), organization_url(org) The problem appears to be caused by rails hitting url_for in url_rewriter, when it should be hitting url_for in url_helper. I dug into the problem a little bit but couldn''t find a trivial fix. I worked around this by just writing the html for a link. On Wed, Nov 19, 2008 at 12:38 PM, Mike Mangino <mmangino at elevatedrails.com>wrote:> > On Nov 19, 2008, at 2:59 PM, Victor Costan wrote: > > You already merged my changes that make Rails 2.2 happy. My app seems >> to work on 2.2RC2. >> > > Okay, that''s good to know. It looks like I just need to fix the tests then. > > >> >> There''s one outstanding issue that doesn''t have a nice, clear >> solution. As of 2.2, concat complains that it doesn''t need a binding >> argument anymore, and the old use is deprecated. On the other hand, >> removing that argument will break anyone who''s not on 2.2 yet. >> >> Possible solutions: >> 1) define our own method that calls concat and does the right thing >> depending on the version of rails >> 2) make a git branch for rails 2.2 changes, make the change below in this >> branch >> 3) remove the binding argument everywhere and make people upgrade >> 4) do nothing >> > > That''s annoying. I''m okay with the warnings for now. If anyone feels > strongly about it, they can create another level of indirection. > > Mike > > > >> >> Preferences? >> Victor >> >> >> >> On Wed, Nov 19, 2008 at 10:10 AM, Mike Mangino >> <mmangino at elevatedrails.com> wrote: >> >>> I remember a while back that somebody had patches to make Facebooker work >>> on >>> Rails 2.2. Are those still around? I''d like to get them merged over the >>> next >>> couple of days. >>> >>> Mike >>> >>> -- >>> Mike Mangino >>> http://www.elevatedrails.com >>> >>> >>> >>> _______________________________________________ >>> Facebooker-talk mailing list >>> Facebooker-talk at rubyforge.org >>> http://rubyforge.org/mailman/listinfo/facebooker-talk >>> >>> > -- > Mike Mangino > http://www.elevatedrails.com > > > > _______________________________________________ > Facebooker-talk mailing list > Facebooker-talk at rubyforge.org > http://rubyforge.org/mailman/listinfo/facebooker-talk >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://rubyforge.org/pipermail/facebooker-talk/attachments/20081201/1577fd0a/attachment-0001.html>
Sorry for the delay in getting this fixed. There was a lot of magic going on in the publisher that needed to be reworked. The good news is that the changes appear to work in 2.0, 2.1 and 2.2. Please test your application well before deploying with this release. We have reasonable test coverage of the publisher, but there is a lot of magic in there. Mike On Dec 1, 2008, at 4:21 PM, George Deglin wrote:> I''m having an issue with using link_to from the publisher after > upgrading to 2.2. > > Error is: > can''t convert String into Hash > > c:/ruby/lib/ruby/gems/1.8/gems/actionpack-2.2.2/lib/ > action_controller/url_rewriter.rb:130:in `merge'' > c:/ruby/lib/ruby/gems/1.8/gems/actionpack-2.2.2/lib/ > action_controller/url_rewriter.rb:130:in `url_for'' > c:/ruby/lib/ruby/gems/1.8/gems/actionpack-2.2.2/lib/action_view/ > helpers/url_helper.rb:228:in `link_to'' > app/helpers/application_helper.rb:23:in `show_link'' > app/models/notification_publisher.rb:19:in `notification_text'' > > Line 23 of application_helper.rb is: > link_to h(org.name), organization_url(org) > > The problem appears to be caused by rails hitting url_for in > url_rewriter, when it should be hitting url_for in url_helper. > > I dug into the problem a little bit but couldn''t find a trivial fix. > I worked around this by just writing the html for a link. > > > > > On Wed, Nov 19, 2008 at 12:38 PM, Mike Mangino <mmangino at elevatedrails.com > > wrote: > > On Nov 19, 2008, at 2:59 PM, Victor Costan wrote: > > You already merged my changes that make Rails 2.2 happy. My app seems > to work on 2.2RC2. > > Okay, that''s good to know. It looks like I just need to fix the > tests then. > > > > > There''s one outstanding issue that doesn''t have a nice, clear > solution. As of 2.2, concat complains that it doesn''t need a binding > argument anymore, and the old use is deprecated. On the other hand, > removing that argument will break anyone who''s not on 2.2 yet. > > Possible solutions: > 1) define our own method that calls concat and does the right thing > depending on the version of rails > 2) make a git branch for rails 2.2 changes, make the change below in > this branch > 3) remove the binding argument everywhere and make people upgrade > 4) do nothing > > That''s annoying. I''m okay with the warnings for now. If anyone feels > strongly about it, they can create another level of indirection. > > Mike > > > > > Preferences? > Victor > > > > On Wed, Nov 19, 2008 at 10:10 AM, Mike Mangino > <mmangino at elevatedrails.com> wrote: > I remember a while back that somebody had patches to make Facebooker > work on > Rails 2.2. Are those still around? I''d like to get them merged over > the next > couple of days. > > Mike > > -- > Mike Mangino > http://www.elevatedrails.com > > > > _______________________________________________ > Facebooker-talk mailing list > Facebooker-talk at rubyforge.org > http://rubyforge.org/mailman/listinfo/facebooker-talk > > > -- > Mike Mangino > http://www.elevatedrails.com > > > > _______________________________________________ > Facebooker-talk mailing list > Facebooker-talk at rubyforge.org > http://rubyforge.org/mailman/listinfo/facebooker-talk >-- Mike Mangino http://www.elevatedrails.com
Thanks for working on this. I was just about to start digging into it by myself. I''m still having the same problem as I described earlier. I''m including a custom helper module into my publisher, and then calling a function inside the helper module. The function calls link_to and generates the error I pasted earlier. This appears to be due to link_to inside of url_helper calling url_for inside of ActionController::UrlWriter instead of ActionView::Helpers::UrlHelper. On Wed, Dec 3, 2008 at 12:51 PM, Mike Mangino <mmangino at elevatedrails.com>wrote:> Sorry for the delay in getting this fixed. > > There was a lot of magic going on in the publisher that needed to be > reworked. The good news is that the changes appear to work in 2.0, 2.1 and > 2.2. > > Please test your application well before deploying with this release. We > have reasonable test coverage of the publisher, but there is a lot of magic > in there. > > Mike > > > On Dec 1, 2008, at 4:21 PM, George Deglin wrote: > > I''m having an issue with using link_to from the publisher after upgrading >> to 2.2. >> >> Error is: >> can''t convert String into Hash >> >> c:/ruby/lib/ruby/gems/1.8/gems/actionpack-2.2.2/lib/action_controller/url_rewriter.rb:130:in >> `merge'' >> c:/ruby/lib/ruby/gems/1.8/gems/actionpack-2.2.2/lib/action_controller/url_rewriter.rb:130:in >> `url_for'' >> c:/ruby/lib/ruby/gems/1.8/gems/actionpack-2.2.2/lib/action_view/helpers/url_helper.rb:228:in >> `link_to'' >> app/helpers/application_helper.rb:23:in `show_link'' >> app/models/notification_publisher.rb:19:in `notification_text'' >> >> Line 23 of application_helper.rb is: >> link_to h(org.name), organization_url(org) >> >> The problem appears to be caused by rails hitting url_for in url_rewriter, >> when it should be hitting url_for in url_helper. >> >> I dug into the problem a little bit but couldn''t find a trivial fix. I >> worked around this by just writing the html for a link. >> >> >> >> >> On Wed, Nov 19, 2008 at 12:38 PM, Mike Mangino < >> mmangino at elevatedrails.com> wrote: >> >> On Nov 19, 2008, at 2:59 PM, Victor Costan wrote: >> >> You already merged my changes that make Rails 2.2 happy. My app seems >> to work on 2.2RC2. >> >> Okay, that''s good to know. It looks like I just need to fix the tests >> then. >> >> >> >> >> There''s one outstanding issue that doesn''t have a nice, clear >> solution. As of 2.2, concat complains that it doesn''t need a binding >> argument anymore, and the old use is deprecated. On the other hand, >> removing that argument will break anyone who''s not on 2.2 yet. >> >> Possible solutions: >> 1) define our own method that calls concat and does the right thing >> depending on the version of rails >> 2) make a git branch for rails 2.2 changes, make the change below in this >> branch >> 3) remove the binding argument everywhere and make people upgrade >> 4) do nothing >> >> That''s annoying. I''m okay with the warnings for now. If anyone feels >> strongly about it, they can create another level of indirection. >> >> Mike >> >> >> >> >> Preferences? >> Victor >> >> >> >> On Wed, Nov 19, 2008 at 10:10 AM, Mike Mangino >> <mmangino at elevatedrails.com> wrote: >> I remember a while back that somebody had patches to make Facebooker work >> on >> Rails 2.2. Are those still around? I''d like to get them merged over the >> next >> couple of days. >> >> Mike >> >> -- >> Mike Mangino >> http://www.elevatedrails.com >> >> >> >> _______________________________________________ >> Facebooker-talk mailing list >> Facebooker-talk at rubyforge.org >> http://rubyforge.org/mailman/listinfo/facebooker-talk >> >> >> -- >> Mike Mangino >> http://www.elevatedrails.com >> >> >> >> _______________________________________________ >> Facebooker-talk mailing list >> Facebooker-talk at rubyforge.org >> http://rubyforge.org/mailman/listinfo/facebooker-talk >> >> > -- > Mike Mangino > http://www.elevatedrails.com > > > >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://rubyforge.org/pipermail/facebooker-talk/attachments/20081203/f1309282/attachment-0001.html>
Looks like I just got it working actually. I moved ''include ActionController::UrlWriter'' in publisher.rb above the block where the helpers are included. Not sure if this might have any unintended side effects. On Wed, Dec 3, 2008 at 1:08 PM, George Deglin <george at xapblog.com> wrote:> Thanks for working on this. I was just about to start digging into it by > myself. > > I''m still having the same problem as I described earlier. I''m including a > custom helper module into my publisher, and then calling a function inside > the helper module. The function calls link_to and generates the error I > pasted earlier. > > This appears to be due to link_to inside of url_helper calling url_for > inside of ActionController::UrlWriter instead of > ActionView::Helpers::UrlHelper. > > > > > On Wed, Dec 3, 2008 at 12:51 PM, Mike Mangino <mmangino at elevatedrails.com>wrote: > >> Sorry for the delay in getting this fixed. >> >> There was a lot of magic going on in the publisher that needed to be >> reworked. The good news is that the changes appear to work in 2.0, 2.1 and >> 2.2. >> >> Please test your application well before deploying with this release. We >> have reasonable test coverage of the publisher, but there is a lot of magic >> in there. >> >> Mike >> >> >> On Dec 1, 2008, at 4:21 PM, George Deglin wrote: >> >> I''m having an issue with using link_to from the publisher after upgrading >>> to 2.2. >>> >>> Error is: >>> can''t convert String into Hash >>> >>> c:/ruby/lib/ruby/gems/1.8/gems/actionpack-2.2.2/lib/action_controller/url_rewriter.rb:130:in >>> `merge'' >>> c:/ruby/lib/ruby/gems/1.8/gems/actionpack-2.2.2/lib/action_controller/url_rewriter.rb:130:in >>> `url_for'' >>> c:/ruby/lib/ruby/gems/1.8/gems/actionpack-2.2.2/lib/action_view/helpers/url_helper.rb:228:in >>> `link_to'' >>> app/helpers/application_helper.rb:23:in `show_link'' >>> app/models/notification_publisher.rb:19:in `notification_text'' >>> >>> Line 23 of application_helper.rb is: >>> link_to h(org.name), organization_url(org) >>> >>> The problem appears to be caused by rails hitting url_for in >>> url_rewriter, when it should be hitting url_for in url_helper. >>> >>> I dug into the problem a little bit but couldn''t find a trivial fix. I >>> worked around this by just writing the html for a link. >>> >>> >>> >>> >>> On Wed, Nov 19, 2008 at 12:38 PM, Mike Mangino < >>> mmangino at elevatedrails.com> wrote: >>> >>> On Nov 19, 2008, at 2:59 PM, Victor Costan wrote: >>> >>> You already merged my changes that make Rails 2.2 happy. My app seems >>> to work on 2.2RC2. >>> >>> Okay, that''s good to know. It looks like I just need to fix the tests >>> then. >>> >>> >>> >>> >>> There''s one outstanding issue that doesn''t have a nice, clear >>> solution. As of 2.2, concat complains that it doesn''t need a binding >>> argument anymore, and the old use is deprecated. On the other hand, >>> removing that argument will break anyone who''s not on 2.2 yet. >>> >>> Possible solutions: >>> 1) define our own method that calls concat and does the right thing >>> depending on the version of rails >>> 2) make a git branch for rails 2.2 changes, make the change below in this >>> branch >>> 3) remove the binding argument everywhere and make people upgrade >>> 4) do nothing >>> >>> That''s annoying. I''m okay with the warnings for now. If anyone feels >>> strongly about it, they can create another level of indirection. >>> >>> Mike >>> >>> >>> >>> >>> Preferences? >>> Victor >>> >>> >>> >>> On Wed, Nov 19, 2008 at 10:10 AM, Mike Mangino >>> <mmangino at elevatedrails.com> wrote: >>> I remember a while back that somebody had patches to make Facebooker work >>> on >>> Rails 2.2. Are those still around? I''d like to get them merged over the >>> next >>> couple of days. >>> >>> Mike >>> >>> -- >>> Mike Mangino >>> http://www.elevatedrails.com >>> >>> >>> >>> _______________________________________________ >>> Facebooker-talk mailing list >>> Facebooker-talk at rubyforge.org >>> http://rubyforge.org/mailman/listinfo/facebooker-talk >>> >>> >>> -- >>> Mike Mangino >>> http://www.elevatedrails.com >>> >>> >>> >>> _______________________________________________ >>> Facebooker-talk mailing list >>> Facebooker-talk at rubyforge.org >>> http://rubyforge.org/mailman/listinfo/facebooker-talk >>> >>> >> -- >> Mike Mangino >> http://www.elevatedrails.com >> >> >> >> >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://rubyforge.org/pipermail/facebooker-talk/attachments/20081203/490d0762/attachment.html>
Can you try the most recent version? I forgot to remove an include when I extracted my fix. Mike On Dec 3, 2008, at 4:08 PM, George Deglin wrote:> Thanks for working on this. I was just about to start digging into > it by myself. > > I''m still having the same problem as I described earlier. I''m > including a custom helper module into my publisher, and then calling > a function inside the helper module. The function calls link_to and > generates the error I pasted earlier. > > This appears to be due to link_to inside of url_helper calling > url_for inside of ActionController::UrlWriter instead of > ActionView::Helpers::UrlHelper. > > > > On Wed, Dec 3, 2008 at 12:51 PM, Mike Mangino <mmangino at elevatedrails.com > > wrote: > Sorry for the delay in getting this fixed. > > There was a lot of magic going on in the publisher that needed to be > reworked. The good news is that the changes appear to work in 2.0, > 2.1 and 2.2. > > Please test your application well before deploying with this > release. We have reasonable test coverage of the publisher, but > there is a lot of magic in there. > > Mike > > > On Dec 1, 2008, at 4:21 PM, George Deglin wrote: > > I''m having an issue with using link_to from the publisher after > upgrading to 2.2. > > Error is: > can''t convert String into Hash > > c:/ruby/lib/ruby/gems/1.8/gems/actionpack-2.2.2/lib/ > action_controller/url_rewriter.rb:130:in `merge'' > c:/ruby/lib/ruby/gems/1.8/gems/actionpack-2.2.2/lib/ > action_controller/url_rewriter.rb:130:in `url_for'' > c:/ruby/lib/ruby/gems/1.8/gems/actionpack-2.2.2/lib/action_view/ > helpers/url_helper.rb:228:in `link_to'' > app/helpers/application_helper.rb:23:in `show_link'' > app/models/notification_publisher.rb:19:in `notification_text'' > > Line 23 of application_helper.rb is: > link_to h(org.name), organization_url(org) > > The problem appears to be caused by rails hitting url_for in > url_rewriter, when it should be hitting url_for in url_helper. > > I dug into the problem a little bit but couldn''t find a trivial fix. > I worked around this by just writing the html for a link. > > > > > On Wed, Nov 19, 2008 at 12:38 PM, Mike Mangino <mmangino at elevatedrails.com > > wrote: > > On Nov 19, 2008, at 2:59 PM, Victor Costan wrote: > > You already merged my changes that make Rails 2.2 happy. My app seems > to work on 2.2RC2. > > Okay, that''s good to know. It looks like I just need to fix the > tests then. > > > > > There''s one outstanding issue that doesn''t have a nice, clear > solution. As of 2.2, concat complains that it doesn''t need a binding > argument anymore, and the old use is deprecated. On the other hand, > removing that argument will break anyone who''s not on 2.2 yet. > > Possible solutions: > 1) define our own method that calls concat and does the right thing > depending on the version of rails > 2) make a git branch for rails 2.2 changes, make the change below in > this branch > 3) remove the binding argument everywhere and make people upgrade > 4) do nothing > > That''s annoying. I''m okay with the warnings for now. If anyone feels > strongly about it, they can create another level of indirection. > > Mike > > > > > Preferences? > Victor > > > > On Wed, Nov 19, 2008 at 10:10 AM, Mike Mangino > <mmangino at elevatedrails.com> wrote: > I remember a while back that somebody had patches to make Facebooker > work on > Rails 2.2. Are those still around? I''d like to get them merged over > the next > couple of days. > > Mike > > -- > Mike Mangino > http://www.elevatedrails.com > > > > _______________________________________________ > Facebooker-talk mailing list > Facebooker-talk at rubyforge.org > http://rubyforge.org/mailman/listinfo/facebooker-talk > > > -- > Mike Mangino > http://www.elevatedrails.com > > > > _______________________________________________ > Facebooker-talk mailing list > Facebooker-talk at rubyforge.org > http://rubyforge.org/mailman/listinfo/facebooker-talk > > > -- > Mike Mangino > http://www.elevatedrails.com > > > >-- Mike Mangino http://www.elevatedrails.com
Yeah, that worked. Thanks! On Wed, Dec 3, 2008 at 1:18 PM, Mike Mangino <mmangino at elevatedrails.com>wrote:> Can you try the most recent version? I forgot to remove an include when I > extracted my fix. > > Mike > > > On Dec 3, 2008, at 4:08 PM, George Deglin wrote: > > Thanks for working on this. I was just about to start digging into it by >> myself. >> >> I''m still having the same problem as I described earlier. I''m including a >> custom helper module into my publisher, and then calling a function inside >> the helper module. The function calls link_to and generates the error I >> pasted earlier. >> >> This appears to be due to link_to inside of url_helper calling url_for >> inside of ActionController::UrlWriter instead of >> ActionView::Helpers::UrlHelper. >> >> >> >> On Wed, Dec 3, 2008 at 12:51 PM, Mike Mangino <mmangino at elevatedrails.com> >> wrote: >> Sorry for the delay in getting this fixed. >> >> There was a lot of magic going on in the publisher that needed to be >> reworked. The good news is that the changes appear to work in 2.0, 2.1 and >> 2.2. >> >> Please test your application well before deploying with this release. We >> have reasonable test coverage of the publisher, but there is a lot of magic >> in there. >> >> Mike >> >> >> On Dec 1, 2008, at 4:21 PM, George Deglin wrote: >> >> I''m having an issue with using link_to from the publisher after upgrading >> to 2.2. >> >> Error is: >> can''t convert String into Hash >> >> c:/ruby/lib/ruby/gems/1.8/gems/actionpack-2.2.2/lib/action_controller/url_rewriter.rb:130:in >> `merge'' >> c:/ruby/lib/ruby/gems/1.8/gems/actionpack-2.2.2/lib/action_controller/url_rewriter.rb:130:in >> `url_for'' >> c:/ruby/lib/ruby/gems/1.8/gems/actionpack-2.2.2/lib/action_view/helpers/url_helper.rb:228:in >> `link_to'' >> app/helpers/application_helper.rb:23:in `show_link'' >> app/models/notification_publisher.rb:19:in `notification_text'' >> >> Line 23 of application_helper.rb is: >> link_to h(org.name), organization_url(org) >> >> The problem appears to be caused by rails hitting url_for in url_rewriter, >> when it should be hitting url_for in url_helper. >> >> I dug into the problem a little bit but couldn''t find a trivial fix. I >> worked around this by just writing the html for a link. >> >> >> >> >> On Wed, Nov 19, 2008 at 12:38 PM, Mike Mangino < >> mmangino at elevatedrails.com> wrote: >> >> On Nov 19, 2008, at 2:59 PM, Victor Costan wrote: >> >> You already merged my changes that make Rails 2.2 happy. My app seems >> to work on 2.2RC2. >> >> Okay, that''s good to know. It looks like I just need to fix the tests >> then. >> >> >> >> >> There''s one outstanding issue that doesn''t have a nice, clear >> solution. As of 2.2, concat complains that it doesn''t need a binding >> argument anymore, and the old use is deprecated. On the other hand, >> removing that argument will break anyone who''s not on 2.2 yet. >> >> Possible solutions: >> 1) define our own method that calls concat and does the right thing >> depending on the version of rails >> 2) make a git branch for rails 2.2 changes, make the change below in this >> branch >> 3) remove the binding argument everywhere and make people upgrade >> 4) do nothing >> >> That''s annoying. I''m okay with the warnings for now. If anyone feels >> strongly about it, they can create another level of indirection. >> >> Mike >> >> >> >> >> Preferences? >> Victor >> >> >> >> On Wed, Nov 19, 2008 at 10:10 AM, Mike Mangino >> <mmangino at elevatedrails.com> wrote: >> I remember a while back that somebody had patches to make Facebooker work >> on >> Rails 2.2. Are those still around? I''d like to get them merged over the >> next >> couple of days. >> >> Mike >> >> -- >> Mike Mangino >> http://www.elevatedrails.com >> >> >> >> _______________________________________________ >> Facebooker-talk mailing list >> Facebooker-talk at rubyforge.org >> http://rubyforge.org/mailman/listinfo/facebooker-talk >> >> >> -- >> Mike Mangino >> http://www.elevatedrails.com >> >> >> >> _______________________________________________ >> Facebooker-talk mailing list >> Facebooker-talk at rubyforge.org >> http://rubyforge.org/mailman/listinfo/facebooker-talk >> >> >> -- >> Mike Mangino >> http://www.elevatedrails.com >> >> >> >> >> > -- > Mike Mangino > http://www.elevatedrails.com > > > >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://rubyforge.org/pipermail/facebooker-talk/attachments/20081203/981bda34/attachment-0001.html>