Clear cache on changing order of terms

Committed to 7.x-1.x-dev so it should be available in the next development release (>"7.x-1.3+4-dev").

Committed and created a new release: 7.x-1.4.

I'm using 7.x-1.6 and am experiencing this. Is it possible the problem was re-introduced?

Same here with 7.x-1.6, clearing the cache is needed to get the correct structure.

Please update 7.x-1.6

I will check into this more and update with a patch if needed. Clients definitely needing terms to instantly appear in these forms in the correct position.

I've found the issue.

Ultimately, it lies in taxonomy_get_tree() caching taxonomy trees – https://api.drupal.org/api/drupal/modules%21taxonomy%21taxonomy.module/f....

When you submit the taxonomy sort form (admin/structure/taxonomy/vocabulary_foo), SHS tried to update it's cache of the vocabulary options. It looks like it's replacing the cache with the result of taxonomy_get_tree(), which, even tho the form's submit handler updated all of the term weights / hierarchy, the terms have already been loaded into memory during this bootstrap cycle and thus their new weight isn't taken into consideration.

Patch unsets this cache via drupal_static_reset('taxonomy_get_tree');. Tested and working for me.

Patch #8 seemed to work at first sight, still had issues. Fixed it by adding this code in a custom module named "shs fix"

<?php

/*
 * implements hook_taxonomy_term_insert
 */
function shs_fix_taxonomy_term_insert($term) {
  _cache_clear($term);
}

/*
 * implements hook_taxonomy_term_update
 */
function shs_fix_taxonomy_term_update($term) {
  _cache_clear($term);
}

/*
 * implements hook_taxonomy_term_delete
 */
function shs_fix_taxonomy_term_delete($term) {
  _cache_clear($term);
}

function _cache_clear($term) {
  //machine names of the vocabularies that use SHS filtering
  $vocabularies = array('veelgestelde_vragen_categorieen');

  if(in_array($term->vocabulary_machine_name, $vocabularies)) {
    // Reset taxonomy_get_tree()'s static cache.
    drupal_static_reset('taxonomy_get_tree');
  }
}

If anyone has a better way to solve this, please feel free to share.

I was also still running into the issue where re-ordering a 2nd level item never appeared in SHS until a full cache clear. Turns out the cache key that was being set only cared about the first level of items. See this code from ~ line 583:

      // Get term children (only first level).
      $tree = taxonomy_get_tree($vid, $parent, 1);
      foreach ($tree as $term) {
            ...

Because we only have a generic cache key of "shs:[vid]" then trees below level 1 are never cached. I appended the parent ID to the cache like so:

$cache_key = "shs:{$vid}:{$parent}";

and it's working as expected for me now. Can we discuss getting this stuff committed?

Hi csdco,
Can you assist a little further. Not 100% sure what to do with the code above. Can you explain further or provide a patch? Am I correct in that $cache_key = "shs:{$vocabulary_cache_key}"; will be replaced by $cache_key = "shs:{$vid}:{$parent}"; (line 774). Is that correct? Thank you.

Still running into this problem. Created a module from #9, but after editing a list and moving a child term back to root, the term does not appear when editing a node with this term. Running 7.x-1.6+56-dev.

Fixed with commit to #2344653: SHS cache system doesn't work correctly.