Hi there,

I was considering joining this issue into #820836: ApacheSolr not creating proper RSS feeds - but decided this issue is most likely unrelated.

Using the latest stable apachesolr and the latest 'stable' opensearch module, results in RSS feeds all returning 404 page not found.

Simple fix was to comment out a check that seems to never pass:

/**
 * Presents an opensearch results page.
 * @param $info
 *   An array describing the search hook for which visualize information.
 *
 * @return
 *   The XML content for the Opensearch feed, or the error page 404.
 */
function opensearch_view($info) {
  //if ($info->feed) {
    // Retrieve and log the search keywords.
    $keywords = trim(search_get_keys());
    watchdog('opensearch', '%keys (@type).', array('%keys' => $keywords, '@type' => module_invoke($info->type, 'search', 'name')), WATCHDOG_NOTICE, l(t('results'), 'search/' . $info->type . '/' . $keywords));

    // OpenSearch pages are 1-indexed.
    if (isset($_GET['page'])) {
      $_GET['page']--;
    }

    // Collect the search results and display the feed.
    $results = module_invoke($info->type, 'search', 'search', $keywords);
    opensearch_feed($info, $keywords, $results);
  //}
  //else {
  //  drupal_not_found();
  //}
}

Any ideas on when you could commit this and roll a new release?

Thanks
Sean

Comments

Anonymous’s picture

Priority: Critical » Normal

Using the latest stable apachesolr and the latest 'stable' opensearch module, results in RSS feeds all returning 404 page not found.

Simple fix was to comment out a check that seems to never pass:

The fix is to enable the RSS feed for Apache Solr; differently, the IF-statement will always fail. Removing the IF-statement is not the fix.

Anonymous’s picture

Status: Active » Closed (works as designed)

I am changing status as per my previous comment. The IF-statement is necessary because the module allows to set for which modules implementing hook_search() the RSS feed must be enabled; if you get an error 404 is because the feed for that module has not been enabled.

Anonymous’s picture

Title: RSS feeds do not work for apachesolr » RSS feeds do not work for ApacheSolr