"Node: Nid" and node title fields cannot be hidden in View

Thanks for posting this, I didnt see this covered in any documentation and without it no worky.

Yep I can confirm this problem, hopefully see a fix soon.

Node id can now be excluded from the display but it has to be provided as a view field. When the Node id is not included in the display the node id will be retrieved from the view result.

Check out the current development snapshot for testing.

I also have a small issue with this.

Scenario #1:
1) A 'default' view with basic settings that mirror the nodereference_explorer_default view, with the Style: Table (Selectable). The Node: NID field is excluded from the display.
2) A 'Grid' view with basic settings that mirror the nodereference_explorer_default view, with the Style: Grid, and Row Style: Fields (Selectable). The Node: NID field is excluded from the display.

This works as you might think, and allows you to properly select your fields/items. It returns the correct [nid:123] into the autocomplete box.

Scenario #2:
1) A 'default' view with basic settings of Style: Grid, and Row Style: Fields (Selectable). The Node: NID is excluded from the display.
2) A 'Grid' view with basic settings of Style: Grid, and Row Style: Fields (Selectable). The Node: NID is excluded from the display.

In this case, it does not return the correct autocomplete - you just get [nid:] returned. If you change the Node:NID field in this scenario to not be excluded, the [nid:123] is correctly populated.

What I am getting at is if your view 'Defaults' is set to Grid and your Node:NID is excluded from display, the [nid:123] is not returned correctly to the autocomplete box.

the title field cant be hidden either, some might want this to create another "title" field from several CCK fields, this fixes it

--- nodereference_explorer/includes/nodereference_explorer.views.inc    2009-09-19 14:34:29.000000000 +0000
+++ nodereference_explorer/includes/nodereference_explorer.views.inc       2009-09-19 14:15:18.000000000 +0000
@@ -61,11 +61,12 @@ function nodereference_explorer_preproce
       $vars['row_classes'][$id][] = 'views-item-selectable';
       //get node id for each row, retrieve from result if nid is hidden
       $nid = isset($row['nid']) ? $row['nid'] : $vars['view']->result[$id]->nid;
+      $title = isset($row['title']) ? $row['title'] : $vars['view']->result[$id]->node_title;
       $element = array( //wrap id table fields with radio buttons
         '#type' => 'radio',
         '#id' => 'nodereference-explorer-row-'. $id,
         '#name' => 'nodereference-explorer-row',
-        '#return_value' => $row['title'] . ' [nid:'. $nid .']',
+        '#return_value' => $title . ' [nid:'. $nid .']',
       );
       $row['nid'] .= drupal_render($element);
     }

The $vars['row'] variable in the field (selectable) preprocess function has information about ALL fields, including the excluded from display.

The patch from #9 is here included as well.

Sorry guys patch from #10 is buggy, got the condtionals wrong. Patch #10 should be considered as obsolete.

Here is the right one for solving #10.