It's inconvenient to track all the different variables being used to track the last nid and changed, and there is no central way to reset them - guess that was a regression.

CommentFileSizeAuthor
#6 add-back-341430-5.patch12.94 KBpwolanin
#5 add-back-341430-4.patch12.9 KBpwolanin
#1 add-back-341430-1.patch10.15 KBpwolanin

Comments

pwolanin’s picture

pwolanin commented
Status: Active » Needs review
StatusFileSize
new add-back-341430-1.patch10.15 KB

like this?

pwolanin’s picture

pwolanin commented
Assigned: Unassigned » pwolanin

note, that without this change back the _nodeaccess module is pretty well broken - there is no easy way for it to require general re-indexing.

An alternative would be to define a new hook to tell all modules to re-index, but I think the way in the patch is more convenient for add-on modules.

paul.lovvik’s picture

paul.lovvik commented
Status: Needs review » Reviewed & tested by the community

Nice work. Definitely an improvement.

febbraro’s picture

febbraro commented

Looks good to me.

pwolanin’s picture

pwolanin commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new add-back-341430-4.patch12.9 KB

actually - it would be better if we had an update path

pwolanin’s picture

pwolanin commented
StatusFileSize
new add-back-341430-5.patch12.94 KB

should also clear out the existing variables.

pwolanin’s picture

pwolanin commented
Status: Needs review » Patch (to be ported)

committed the patch in #6 to 6.x

JacobSingh’s picture

JacobSingh commented

Looks good.

pwolanin’s picture

pwolanin commented
Status: Patch (to be ported) » Closed (fixed)