Patch for 3 small errors - $id, get_filters(), and apachesolr_modify_query/OG problem

so the last change doesn't work. This is actually because the $id field was just made protected in the last apachesolr project commit. Workin on a workaround here soon. Your fix 'works' because $id is always 0. So it really doesn't work...

The other two changes are correct though.

shouldn't apachesolr_views_query::get_filters() look more like:

  /**
   * return an array of filters
   *
   * Compabatiablity layer with Solr_Base_Query
   */
  function get_filters($name = NULL) {
    $filters = array();
    foreach ($this->_facets as $type => $fields) {
      foreach ($fields as $data) {
        $filters['#name'] = ($data['exclude']) ? "NOT $type" : $type;
        $filters['#value'] = $data['value'];
      }
    }

    if (empty($name)) {
      return $filters;
    }
    $matches = array();
    foreach ($filters as $filter) {
      if ($filter['#name'] == $name) {
        $matches[] = $filter;
      }
    }
    return $matches;
  }

seems silly to totally ignore $name param

also after applying this patch I'm getting a new error:
Fatal error: Call to undefined method apachesolr_views_query::get_available_sorts() in modules/apachesolr/apachesolr.module on line 463

It looks like the apachesolr_block() function calls $query->get_available_sorts() but the function isn't part of the Drupal_Solr_Query_Interface interface. I can't tell if we should just implement get_available_sorts() or kick it over to the apachesolr queue to have them straighten out their interface.

Need the method added to the interface and this class.

Ignoring the name isn't the intention, just not ready to handle it yet :-D. apachesolr moving to fast :-D

see #442198: Add get_available_sorts() to the Drupal_Solr_Query_Interface interface and #443252: Make protected id public again

ok apply those two patches and do a new CVS checkout. Here is the patch that i just committed