diff --git a/src/Form/FacetSourceEditForm.php b/src/Form/FacetSourceEditForm.php index bdc7779..b8680b5 100644 --- a/src/Form/FacetSourceEditForm.php +++ b/src/Form/FacetSourceEditForm.php @@ -107,14 +107,18 @@ class FacetSourceEditForm extends EntityForm { ]; $url_processors = array(); + $url_processors_description = array(); foreach ($this->urlProcessorPluginManager->getDefinitions() as $definition) { $url_processors[$definition['id']] = $definition['label']; + $url_processors_description[] = $definition['description']; } $form['urlProcessor'] = [ - '#type' => 'select', + '#type' => 'radios', '#title' => $this->t('URL Processor'), '#options' => $url_processors, '#default_value' => $facet_source->getUrlProcessorName(), + '#description' => $this->t( + 'The URL Processor defines the url structure used for this facet source.') . '
- ' . implode('
- ', $url_processors_description), ]; // The parent's form build method will add a save button. diff --git a/src/Plugin/facets/processor/UrlProcessorHandler.php b/src/Plugin/facets/processor/UrlProcessorHandler.php index 3ec3819..5a78e00 100644 --- a/src/Plugin/facets/processor/UrlProcessorHandler.php +++ b/src/Plugin/facets/processor/UrlProcessorHandler.php @@ -14,16 +14,16 @@ use Drupal\facets\Processor\PreQueryProcessorInterface; use Drupal\facets\Processor\ProcessorPluginBase; /** - * The URL processor, delegates all of it's work to an actual url processor. + * The URL processor handler triggers the actual url processor. * - * Because we need to be able to decide where in the facet the urlProcessor - * should be triggered there needs to be an actual processor. This farms out all - * the work it does to the actual @FacetsUrlProcessor. + * The URL processor handler allows managing the weight of the actual URL + * processor per Facet. This handler will trigger the actual + * @FacetsUrlProcessor, which can be configured on the Facet source. * * @FacetsProcessor( * id = "url_processor_handler", - * label = @Translation("URL processor handler"), - * description = @Translation("A processor that holds the other url processors (that can be set per facet source)."), + * label = @Translation("URL handler"), + * description = @Translation("Triggers the URL processor, which is set in the Facet source configuration."), * stages = { * "pre_query" = 50, * "build" = 15, diff --git a/src/Plugin/facets/url_processor/QueryString.php b/src/Plugin/facets/url_processor/QueryString.php index 78812b5..9be51c0 100644 --- a/src/Plugin/facets/url_processor/QueryString.php +++ b/src/Plugin/facets/url_processor/QueryString.php @@ -17,8 +17,8 @@ use Symfony\Component\HttpFoundation\Request; * * @FacetsUrlProcessor( * id = "query_string", - * label = @Translation("Query string url processor"), - * description = @Translation("Most simple url processor which uses the query sting.") + * label = @Translation("Query string"), + * description = @Translation("Query string is the default Facets URL processor, and uses GET parameters, e.g. ?f[0]=brand:drupal&f[1]=color:blue") * ) */ class QueryString extends UrlProcessorPluginBase {