Applying pretty facet paths on node view pages (make constructPath() a public method)

Hm.. That would mean I would need to map every single node alias?!

That could work. But a simple test of the inbound_alter workaround makes me think its not going to work as intended anyway. For example:

  if (preg_match('|^frame/flex-mx929/(.*)|', $path, $matches)) {
    $path = 'node/1065/' . $matches[1];
  }

This works, to an extent. I can access /frame/flex-mx929/brand/Flex and it redirects to node/1065/brand/Flex showing me the node with the facets applied. So it means I can access the nodes at their aliased urls with pretty facet paths appended. But instead of showing me the content at that aliased URL, the path in the URL bar gets rewritten to the node/[nid] base form where really, I want the alias to be retained otherwise it kinda defeats the point.

Ahh, wait. It will work, but there is an issue with globalredirect #774950: Incompatible with hook_url_inbound_alter(). If I disable globalredirect it works as intended.

For the record, this works for my use case:

  // Allow facetapi_pretty_paths to work on aliased node view pages.
  // See http://drupal.org/node/1519680 for conversation.
  if (preg_match('|^(frames/.*?)(/.*)|', $path, $matches)) {
    $source = drupal_lookup_path('source', $matches[1]);
    $pretty_path_part = $matches[2];
    $path = $source . $pretty_path_part;
  }

So... I have managed to do what I needed to do, by replacing the original code as per my original post above, with some slightly more complex looking code (although should also be a little more robust) that does the same things, but using pretty paths.

In order to make this work, I needed to be able to call facetapi_pretty_paths' constructPath() method externally (see below). Is there any specific reason why this is a protected method? And, would you consider making it public so that it could be used for use cases like my own?

// Get handler on the facetapi adapter, processor, and facet definitions.
$adapter = facetapi_adapter_load('search_api@optics_node_index');
$processor = new FacetapiUrlProcessorPrettyPaths($adapter);
$facets = $adapter->getEnabledFacets();

$nid = isset($row->_entity_properties['nid'])? $row->_entity_properties['nid'] : '';
$title = isset($row->_entity_properties['title'])? $row->_entity_properties['title'] : '';

$params = $processor->fetchParams();

$segments = array();

// Add in a facet for function (optical / suns)
if (isset($view->filter['field_product_field_optics_function']->value)) {
  if (isset($facets['field_product:field_optics_function'])) {
    $facet_path = $processor->getFacetPath($facets['field_product:field_optics_function'], array(), NULL);
    $alias = $processor->getFacetPrettyPathsAlias($facets['field_product:field_optics_function']);
    $segments[] = array(
      'alias' => $alias,
      'value' => $view->filter['field_product_field_optics_function']->value,
      'facet' => $facets['field_product:field_optics_function'],
    );
  }
}

// Add in a facet for gender
if (isset($view->filter['field_product_field_optics_audience']->value)) {
  if (isset($facets['field_product:field_optics_audience'])) {
    $facet_path = $processor->getFacetPath($facets['field_product:field_optics_audience'], array(), NULL);
    $alias = $processor->getFacetPrettyPathsAlias($facets['field_product:field_optics_audience']);
    $segments[] = array(
      'alias' => $alias,
      'value' => $view->filter['field_product_field_optics_audience']->value,
      'facet' => $facets['field_product:field_optics_audience'],
    );
  }
}

$path = $processor->constructPath('node/' . $nid, $segments);

// Add additional facets from the views arguments.
// convert views args into facetapi_pretty_paths style url.
$facetapi_pretty_path = implode('/', $view->args);
$path .= '/' . $facetapi_pretty_path;

// Add the base path of the original search as a paramater that we can use
// later to add a 'back to search' link.
$query = drupal_get_query_parameters(NULL, array('q', 'f'));
$query['search_path'] = $params['q'];

$output = l($output, $path, array('query' => $query, 'html' => TRUE));

If so, here is a patch to that effect.