search

amateescu / search_api_solr

11 stars 14 forks source link

Use state for the last_optimize value. #30

Closed Berdir closed 9 years ago

Berdir commented 9 years ago

Using config for this kind of data is wrong and it was broken because it didn't call save.

I think there is more similar stuff in the module right now, where it is changing config on a regular basis, but that can be fixed separately.

amateescu commented 9 years ago

Makes sense but do you have any idea why this would break the tests on Travis?

Berdir commented 9 years ago

I assume tests are already broken I don't think this is related.

amateescu commented 9 years ago

Yeah, I thought so too but https://travis-ci.org/amateescu/search_api_solr/builds doesn't seem to agree :) Meh, the change is good so I'll merge it anyway, the failures can be investigated separately.