suggestion: hook_fieldset_helper_rewite_path

Dear Jacob,

First I'd like you to congratulat for this great module. It is very useful for my project.

I'd like to suggest a possibility of rewiting the $current_path variable in the fieldset_helper_state_manager_get_lookup_id() function. There are some pages, which is similar in respect of the layout, only the content is different. One such page is a search result. If I use facets, and I open/close some facet as user, I would like to keep them open/close in the next search as well. The search URLs typically build as the following: [a stable part]/[query term]. The module always use $_GET['q'] as the key for the path, and with my suggestion it would be possible to forget the [query term] part, and using the [stable part] as the key for the state.

so instead of this current code:

  $current_path = $_GET['q'];

I suggest this addition:

  $current_path = $_GET['q'];
  $param =& new stdClass();
  $param->current_path = $current_path;
  module_invoke_all('fieldset_helper_rewite_path', $param);
  $current_path = $param->current_path;

The $param is a reference, and the implementors of the hook could modify the $param->current_path.

I created an implementation for my own need:

function xc_search_fieldset_helper_rewite_path(&$param) {
  $param->current_path = preg_replace('/^(search\/xc_search)\/.*$/', "$1", $param->current_path);
  return TRUE;
}

Hope this suggestion could be useful for others.

Péter