Hi Josh

Probably best to not do that, because then you'll be defining multiple indexes 
on the subclasses, and Sphinx might find that a tad confusing. Thinking Sphinx 
too, for that matter.

-- 
Pat

On 09/01/2010, at 10:32 AM, Josh French wrote:

> I've given this a ponder and there may be a workaround, although I'll
> need Pat or someone else with a better knowledge of Sphinx internals
> to weigh in.
> 
> Are there any disadvantages to moving from something like this:
> 
>    Page.class_eval do
>        include IndexDefinition
>    end
> 
> ...to something like this?
> 
>   [Page, *Page.subclasses].each do |klass
>      klass.class_eval do
>          include IndexDefinition
>      end
>   end
> 
> This way any pre-loaded subclasses get the behavior as well, but I'm
> not sure if defining multiple indexes like this will interfere with
> TS's own subclass handling.
> 
> Any insight?
> 
> Thanks,
> j
> 
> On Jan 8, 2:09 pm, Josh French <[email protected]> wrote:
>>> Any chance to change the code of sphinx-search in a
>>> way to get rid of at least one of those requirements
>> 
>> Not really, unfortunately. I'd just end up creating the same kind of
>> load-order problem for someone else's extension. In your case, you
>> could do:
>> 
>>     config.extensions = [:sphinx_search, :all, :file_system]
>> 
>> What's important is that sphinx_search get loaded before any
>> extensions that add Page subclasses, not that it come strictly first.
>> So you've got some leeway in how you order your extensions.
>> 
>> j
> -- 
> 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.
> 
> 

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


Reply via email to