[Nitro] OG [PATCH]: Avoid setting OGTABLE if already initialized

Mark Van De Vyver mvyver at gmail.com
Wed Sep 19 03:55:13 EDT 2007


On 9/19/07, George Moschovitis <george.moschovitis at gmail.com> wrote:
> I changed the code to use defined? like this:
>
>       klass.const_set "OGTABLE", table(klass) unless defined? klass::OGTABLE
>
> now everything is ok.

Thanks that also removes the error message - still leaves me with a
locked db. Now, where is that happening.... ;)

fingers crossed.
Mark

> thanks for spotting this (and your attention to detail)
>
> regards,
> George.
>
>
> On 9/19/07, George Moschovitis <george.moschovitis at gmail.com> wrote:
> > please be more careful with patches... after I applied this, things
> > stopped working.
> >
> > -g.
> >
> > On 9/19/07, Mark Van De Vyver <mvyver at gmail.com> wrote:
> > > Scenario: In a spec file, an 'OG class' is present outside of a
> > > describe statement.
> > > Effect: Patch eliminates the following warnings from spec's:
> > > .../usr/src/nitro-repo/script/lib/../../og/lib/og/store/sql.rb:268:
> > > warning: already initialized constant OGTABLE
> > >
> > > _______________________________________________
> > > Nitro-general mailing list
> > > Nitro-general at rubyforge.org
> > > http://rubyforge.org/mailman/listinfo/nitro-general
> > >
> > >
> >
> >
> > --
> > http://www.me.gr
> > http://phidz.com
> > http://blog.gmosx.com
> > http://cull.gr
> > http://www.joy.gr
> > http://nitroproject.org
> >
>
>
> --
> http://www.me.gr
> http://phidz.com
> http://blog.gmosx.com
> http://cull.gr
> http://www.joy.gr
> http://nitroproject.org
>


More information about the Nitro-general mailing list