Hi!

Even if it is possible to override hidden settings, the overriden class file is not included by Feeds. So this overriden class will never exists.

This patch suggests a solution for feeds_scheduler_class and feeds_source_class.

CommentFileSizeAuthor
feeds_hidden_settings.patch3.77 KBb-prod

Comments

alex_b’s picture

alex_b commented
Title: Override hidden settings » Load alternative classes
Status: Needs review » Postponed

The solution is to have proper autoloading (coming in D7) or to use CTools plugin API like we did this for fetcher/parser/processor plugins. The proposed solution strikes me as an odd third way.

In the meantime, alternative classes always need to be loaded -- the performance penalty isn't too bad, I hope this is an option for you.

Postponed?

b-prod’s picture

b-prod commented

I will apply my patch manually, awaiting a proper way to do that (autoloading in D7 sounds really good). Actually I don't like to include useless files, load unused classes...

But I understand there is no omission or bug in your code. Thank for your answer.

twistor’s picture

twistor commented
Issue summary: View changes
Status: Postponed » Closed (works as designed)