Implement a setting to add "rel=nofollow" to facet links

Thanks,

Changing the 2 functions do the trick:

function theme_facetapi_link_active($variables) {
  $accessible_vars = array(
    'text' => $variables['text'],
    'active' => TRUE,
  );
  $accessible_markup = theme('facetapi_accessible_markup', $accessible_vars);

  // Sanitizes the link text if necessary.
  $sanitize = empty($variables['options']['html']);
  $link_text = ($sanitize) ? check_plain($variables['text']) : $variables['text'];

  // Resets link text, sets to options to HTML since we already sanitized the
  // link text and are providing additional markup for accessibility.
  $variables['text'] = t('(-) !markup', array('!markup' => $accessible_markup));
  $variables['options']['html'] = TRUE;
  $variables['options']['attributes']['rel'] = 'nofollow'; // added the rel="nofollow"
  return theme_link($variables) . $link_text;
}

function theme_facetapi_link_inactive($variables) {
  $accessible_vars = array(
    'text' => $variables['text'],
    'active' => FALSE,
  );
  $accessible_markup = theme('facetapi_accessible_markup', $accessible_vars);

  // Sanitizes the link text if necessary.
  $sanitize = empty($variables['options']['html']);
  $variables['text'] = ($sanitize) ? check_plain($variables['text']) : $variables['text'];

  // Adds count to link if one was passed.
  if (isset($variables['count'])) {
    $variables['text'] .= ' ' . theme('facetapi_count', $variables);
  }

  // Resets link text, sets to options to HTML since we already sanitized the
  // link text and are providing additional markup for accessibility.
  $variables['text'] .= $accessible_markup;
  $variables['options']['html'] = TRUE;
  $variables['options']['attributes']['rel'] = 'nofollow'; // added the rel="nofollow"
  return theme_link($variables);
}

It will be nice to have a option on admin menu to "turn nofollow on or off".

Yes, its critical, I ask about this because on the past I already have problems with googlebot indexing lots and lots of pages. Was adiviced by a ex google quality (spam) team member to add the nofollow tag.

Before that have the same imput on google webmaster forums (http://www.google.com/support/forum/p/webmasters/thread?tid=2594f5a8e956... -is not on english the discussion-) and a google employer says I should add a to the page (that we can do easy with meta tags module) but In my opinion its not necessary because we already have that on links.

So, the correct is having the canonical tag to search page index and have the rel='nofollow' on links.

The canonical is provided by core, so the only change on this module is the rel='nofollow' tag.

Sorry about my english ;)

That depend of use cases...

If you use faceted to taxonomy search you end up with 2 taxonomy page versions (duplicate content), the facet result and the taxonomy page itself.

As an option, its a good feature to have, disabled by default, because fit on more use cases.

In realtion to patch:

git apply -v facetapi-1370342-10.patch                  
facetapi-1370342-10.patch:19: trailing whitespace.
    
Checking patch facetapi.admin.js...
Checking patch plugins/facetapi/widget_links.inc...
Applied patch facetapi.admin.js cleanly.
Applied patch plugins/facetapi/widget_links.inc cleanly.
warning: 1 line adds whitespace errors.

I dont know if this error is do my reverted changes or the patch itself.

The patch works and all chekboxs are on widgets and nofollow added to urls.