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 -~----------~----~----~----~------~----~------~--~---
