Thanks, everythink is fine now ;)
On Jul 31, 7:49 pm, Pat Allan <[email protected]> wrote:
> Right, this time I wrote a failing scenario, then fixed it properly.
> Sorry for the hassle.
>
> --
> Pat
>
> On 31/07/2009, at 5:33 PM, Pierre Valade wrote:
>
>
>
> > In fact, I have the following error when destroying an object indexed
>
> > You have a nil object when you didn't expect it!
> > You might have expected an instance of Array.
> > The error occurred while evaluating nil.*
>
> >http://pastie.org/566888
>
> > Thanks,
> > Pierre
>
> > On Jul 31, 5:49 pm, Pat Allan <[email protected]> wrote:
> >> Pierre: I've reverted the change - let me know if it's not all fine
> >> now :)
>
> >> --
> >> Pat
>
> >> On 31/07/2009, at 2:51 PM, Pat Allan wrote:
>
> >>> Oh... so you're using a column called id as your Sphinx primary key,
> >>> but object.id returns the hash?
>
> >>> --
> >>> Pat
>
> >>> On 31/07/2009, at 11:35 AM, Pierre Valade wrote:
>
> >>>>http://github.com/freelancing-god/thinking-sphinx/commit/b9fac6e0d199
> >>>> ...
>
> >>>> this commit breaks the set_sphinx_primary_key functionnality
>
> >>>> I think that attributes[klass.primary_key_for_sphinx.to_s] was the
> >>>> right way to find the primay_key id and not self.send
> >>>> (self.class.primary_key_for_sphinx) because A.R overwrite the :id
> >>>> method when you use :set_primary_key
>
> >>>> What do you think? I've changed this and this work just fine ;)
>
> >>>> Pierre
>
> >>>> On Jul 31, 10:27 am, Pierre Valade <[email protected]> wrote:
> >>>>> Great.
> >>>>> Thinking Sphinx is better everyday thanks to Pat and contributors.
> >>>>> I'm
> >>>>> now using it in replacement of some mysql queries that were too
> >>>>> complex because they involve a lot of jointures.
>
> >>>>> I'm now doing a "nil search" a lot of times !
> >>>>> Model.search nil, parameters
>
> >>>>> And it's working great!
>
> >>>>> On Jul 30, 8:26 pm, Jeffrey Chupp <[email protected]> wrote:
>
> >>>>>> Thanks for merging it, Pat.
>
> >>>>>> Pierre, I'll be happy to look into any issues you may discover.
> >>>>>> I'm
> >>>>>> not using some of the more complicated aspects of TS, so it isn't
> >>>>>> impossible that I've missed some of the more edge cases.
>
> >>>>>> On Jul 30, 1:12 pm, Pierre Valade <[email protected]>
> >>>>>> wrote:
>
> >>>>>>> Thanks Pat.
> >>>>>>> I've just updated my project with the last version.
> >>>>>>> I will let you know if there's any issues.
>
> >>>>>>> Thanks!
>
> >>>>>>> On Jul 30, 5:26 pm, Pat Allan <[email protected]> wrote:
>
> >>>>>>>> Right, it's in and pushed. Let me know if there's any issues.
>
> >>>>>>>> --
> >>>>>>>> Pat
>
> >>>>>>>> On 30/07/2009, at 2:43 PM, Pat Allan wrote:
>
> >>>>>>>>> ... Or I could actually look at the git repo, and see there's
> >>>>>>>>> recent
> >>>>>>>>> commits.
>
> >>>>>>>>> --
> >>>>>>>>> Pat
>
> >>>>>>>>> On 30/07/2009, at 2:34 PM, Pat Allan wrote:
>
> >>>>>>>>>> Hi Pierre
>
> >>>>>>>>>> Seems plenty of people are asking for this, so I'll
> >>>>>>>>>> definitely
> >>>>>>>>>> try to
> >>>>>>>>>> merge it in. Jeff: going by other emails, are you still using
> >>>>>>>>>> your
> >>>>>>>>>> fork? Are you able to update it to my latest codebase?
>
> >>>>>>>>>> Cheers
>
> >>>>>>>>>> --
> >>>>>>>>>> Pat
>
> >>>>>>>>>> On 29/07/2009, at 9:45 AM, Pierre Valade wrote:
>
> >>>>>>>>>>> Hi Pat,
>
> >>>>>>>>>>> Would it be possible to merge the branch from semanticart
> >>>>>>>>>>> which
> >>>>>>>>>>> allow
> >>>>>>>>>>> to specify a primary_key for sphinx.
> >>>>>>>>>>>http://github.com/semanticart/thinking-sphinx/commits/
> >>>>>>>>>>> alt_primary_key
>
> >>>>>>>>>>> I find it useful because I have models with a uuid key
> >>>>>>>>>>> (which
> >>>>>>>>>>> is set
> >>>>>>>>>>> as the primary key for ActiveRecord).
>
> >>>>>>>>>>> What do you think?
>
> >>>>>>>>>>> Thanks,
>
> >>>>>>>>>>> Pierre
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---
