Taxonomy Pathauto coder

+++ b/plugins/coders/facetapi_pretty_paths_coder_taxonomy_pathauto.incundefined
@@ -0,0 +1,88 @@
+    $voc_alias = $this->getVocbularyPathAlias($args['facet']);

Vocbulary? typo ;-)

+++ b/plugins/coders/facetapi_pretty_paths_coder_taxonomy_pathauto.incundefined
@@ -0,0 +1,88 @@
+        // The Search API chains fields with ':', use the original field to

pretty paths should work for both so specific search api logic would undo that.

Apachesolr takes the regular field as 1 facet value. Can you perhaps make this explode optional? Is there a way you can make this work both ways?

Looks good overall, I'd still change the comment so it does not appear as if only search api dependent.

hi, thanks for figuring that out :)

+++ b/facetapi_pretty_paths.moduleundefined
@@ -129,6 +129,14 @@ function facetapi_pretty_paths_admin_form($form, &$form_state) {
+        '#description' => t('If set the term alias from the pathauto settings will be re-used, which avoids term ids in the facet aliases. This setting only works if the default taxonomy path pattern \'[term:vocabulary]/[term:name]\' is used. Changing the value of this checkbox requires a cache clear.'),

i'd say "If set, the term alias..."

can't give it a throughout review atm. would be great if somebody else could provide feedback

+        // Get the alias for this term and extract the term name alias part.
+        $alias = drupal_lookup_path('alias', 'taxonomy/term/' . $term->tid);
+        if ($alias) {
+          $parts = explode('/', $alias);
+          if (count($parts) == 2) {
+            $args['segment']['value'] = $parts[1];
+          }

would be great to document what exactly is going on here: something like taxonomy/term/id is mapped to [facet alias]/[term-name alias] and thus the check count($parts) == 2

+++ b/plugins/coders/facetapi_pretty_paths_coder_taxonomy_pathauto.incundefined
@@ -0,0 +1,79 @@
+      // Rebuild the term alias, get the source and extract the term id.
+      $source = drupal_lookup_path('source', $voc_alias . '/' . $args['value']);
+      if ($source) {
+        $exploded = explode('/', $source);
+        if (count($exploded) == 3) {
+          $args['value'] = $exploded[2];

vice versa here :)

committed to 7.x-1.x, thanks!