og (organic groups) argument and filter

did you come any further with that?

+1 for an og argument and filter

The idea of providing OG integration is great, however i am not sure if that is something that needs to be part of ApacheSolr Views module, I have started a discussion on a new project proposal ApacheSolr Views Assorted Plugins

Despite of the discussion if this should be part of the module or not, I made a patch for this. The patch provides only group id arguments.
This patch also includes the bugfix as mentioned here: http://drupal.org/node/449190.

This is the wrong patch, please use patch from #5

And now the corresponding patch :)

In general this works fine for me. Thanks!!

+++ apachesolr_views/apachesolr_views.views.inc	2011-03-15 09:34:40.351191994 +0100
@@ -116,9 +119,6 @@ function apachesolr_views_views_plugins(
     ),
-    'argument' => array( 	 
-      'handler' => 'apachesolr_views_handler_argument_search', 	 

This looks like a good change, see below, but a bit out of scope this patch.

+++ apachesolr_views/apachesolr_views.views.inc	2011-03-15 09:34:40.351191994 +0100
@@ -215,6 +214,7 @@ function apachesolr_views_views_data() {
     );
+  ¶

Here is some space added

+++ apachesolr_views/apachesolr_views.views.inc	2011-03-15 09:34:40.351191994 +0100
@@ -245,6 +260,9 @@ function apachesolr_views_views_data() {
+      'argument' => array(     ¶
+        'handler' => 'apachesolr_views_handler_argument_search',     ¶

some space as well, sorry

+++ apachesolr_views/handlers/apachesolr_views_handler_argument_gid.inc	2011-03-15 09:34:40.351191994 +0100
@@ -0,0 +1,19 @@
+// $Id:

this is now not needed anymore with git

In general this looks fine, if there is a rerole of the few things.

Additional i think, that this should perhaps better belong into apachesolr_og

Updated the patch to fix some style problems and set the group context on the argument handler like the original handler.

The facets might shouldn't have () as part of the link.

I recreated the patch form #9 against the latest dev. It also adds a few changes to adhere to coding standards.

This corrects a wrong handler reference in my previous patch