On 4/20/06, Marcel Molina Jr. <[EMAIL PROTECTED]> wrote:
>
> Hey Mike. Thanks for staying on top of the adapter so that it remains up to
> date. Apologies that we haven't gotten around to it sooner. It's quite
> frustrating to put work into something only to see it sit around.

It has been a bit, but it also a learning experience. :-)

> I installed Frontbase and the ruby bindings for it.  At first when I ran the
> test it was complaining that FBSQL_Connect.connect was getting one too many
> arguments passed to it. Just to move forward I took off the extra argument.
> Then when I run the tests I get loads of warnings from Object#type on line
> 511 of the frontbase adapter.

Are you running the newest FrontBase?  I am testing against 4.2.4.  It
definitely
will not work with the 3.x line.

About the extra argument...check that you have the newest Ruby bindings.
The ones on the Frontbase site are *not* the newest.  I had to modify
them to work
with ActiveRecord.  I sent my changes back to them but they haven't
posted them yet.

I have the source to the modified bindings checked in at:

svn://rubyforge.org//var/svn/frontbase-rails/trunk/ruby-frontbase

I added an extra argument to allow naming the database session, so it
sounds like
you are running the old 0.5.2 bindings.  Once I get the adapter code
checked in, I'll look into releasing a gem of it to make things
simpler.

As of right now (revision 4241) it is passing the tests cleanly:

859 tests, 2866 assertions, 0 failures, 0 errors

> On a side note, there is a bit of foo != nil or foo == nil in the code. The
> preferred approach in those cases is to use the nil? predicate method. In
> other words !foo.nil? or foo.nil?.

I'll be glad to change them.  I was learning Ruby in the process of doing this
so I'm sure some of it is still lacking in style :)
_______________________________________________
Rails-core mailing list
Rails-core@lists.rubyonrails.org
http://lists.rubyonrails.org/mailman/listinfo/rails-core

Reply via email to