diff --git node_reference/node_reference.module node_reference/node_reference.module index 64f19cb..0328451 100644 --- node_reference/node_reference.module +++ node_reference/node_reference.module @@ -29,7 +29,13 @@ function node_reference_field_info() { 'node_reference' => array( 'label' => t('Node reference'), 'description' => t('This field stores the ID of a related node as an integer value.'), - 'settings' => array('referenceable_types' => array()), + 'settings' => array( + 'referenceable_types' => array(), + 'views' => array( + 'view' => '', + 'view_args' => '', + ), + ), // It probably make more sense to have the referenceable types be per-field than per-instance // 'instance settings' => array('referenceable_types' => array()), 'default_widget' => 'options_select', // node_reference_autocomplete', @@ -79,6 +85,41 @@ function node_reference_field_settings_form($field, $instance, $has_data) { : array(), '#options' => array_map('check_plain', node_type_get_names()), ); + + if (module_exists('views')) { + $form['views'] = array( + '#type' => 'fieldset', + '#title' => t('Views - Nodes that can be referenced'), + '#collapsible' => TRUE, + '#collapsed' => !isset($settings['views']['view']) || $settings['views']['view'] == '', + ); + + $views_options = references_get_views('node'); + if ($views_options) { + $views_options = array('' => '<' . t('none') . '>') + $views_options; + $form['views']['view'] = array( + '#type' => 'select', + '#title' => t('View used to select the nodes'), + '#options' => $views_options, + '#default_value' => isset($settings['views']['view']) ? $settings['views']['view'] : '', + '#description' => t('

Choose the "Views module" view that selects the nodes that can be referenced.
Note:

') . + t(''), + ); + $form['views']['view_args'] = array( + '#type' => 'textfield', + '#title' => t('View arguments'), + '#default_value' => isset($settings['views']['view_args']) ? $settings['views']['view_args'] : '', + '#required' => FALSE, + '#description' => t('Provide a comma separated list of arguments to pass to the view.'), + ); + } + else { + $form['views']['no_view_help'] = array( + '#markup' => t("The list of nodes that can be referenced can provided by a view (Views module) using the 'References' display type. but no appropriate views were found."), + ); + } + } + return $form; } @@ -109,7 +150,10 @@ function node_reference_field_validate($entity_type, $entity, $field, $instance, } // Prevent performance hog if there are no ids to check. if ($ids) { - $refs = _node_reference_potential_references($field, '', NULL, $ids); + $options = array( + 'ids' => $ids, + ); + $refs = _node_reference_potential_references($field, $options); foreach ($items as $delta => $item) { if (is_array($item)) { if (!empty($item['nid']) && !isset($refs[$item['nid']])) { @@ -517,13 +561,18 @@ function node_reference_autocomplete_validate($element, &$form_state, $form) { else { // No explicit nid (the submitted value was not populated by autocomplete // selection). Get the nid of a referencable node from the entered title. - $reference = _node_reference_potential_references($field, $value, 'equals', NULL, 1); - if ($reference) { + $options = array( + 'string' => $value, + 'match' => 'equals', + 'limit' => 1, + ); + $references = _node_reference_potential_references($field, $options); + if ($references) { // @todo The best thing would be to present the user with an // additional form, allowing the user to choose between valid // candidates with the same title. ATM, we pick the first // matching candidate... - $nid = key($reference); + $nid = key($references); } else { form_error($element, t('%name: found no valid post with that title.', array('%name' => $instance['label']))); @@ -561,7 +610,7 @@ function _node_reference_options($field) { $options = array(); foreach ($references as $key => $value) { - $options[$key] = $value['title']; + $options[$key] = $value['rendered']; } return $options; @@ -579,7 +628,7 @@ function _node_reference_options($field) { * and lets consumers use the parts they actually need. * * @param $field - * The field description. + * The field definition. * @param $string * Optional string to filter titles on (used by autocomplete). * @param $match @@ -600,19 +649,31 @@ function _node_reference_options($field) { * ), * ... * ) - * @todo Check whether we still need the 'rendered' value (hook_options_list() - * does not need it anymore). Should probably be clearer after the 'Views' - * mode is ported. */ -function _node_reference_potential_references($field, $string = '', $match = 'contains', $ids = array(), $limit = NULL) { +function _node_reference_potential_references($field, $options = array()) { + // Fill in default options. + $options += array( + 'string' => '', + 'match' => 'contains', + 'ids' => array(), + 'limit' => 0, + ); + $results = &drupal_static(__FUNCTION__, array()); // Create unique id for static cache. - $cid = $field['field_name'] . ':' . $match . ':' - . ($string !== '' ? $string : implode('-', $ids)) - . ':' . $limit; + $cid = $field['field_name'] . ':' . $options['match'] . ':' + . ($options['string'] !== '' ? $options['string'] : implode('-', $options['ids'])) + . ':' . $options['limit']; if (!isset($results[$cid])) { - $references = _node_reference_potential_references_standard($field, $string, $match, $ids, $limit); + $references = FALSE; + if (module_exists('views') && !empty($field['settings']['views']['view'])) { + $references = _node_reference_potential_references_views($field, $options); + } + + if ($references === FALSE) { + $references = _node_reference_potential_references_standard($field, $options); + } // Store the results. $results[$cid] = !empty($references) ? $references : array(); @@ -624,9 +685,29 @@ function _node_reference_potential_references($field, $string = '', $match = 'co /** * Helper function for _node_reference_potential_references(). * + * Case of Views-defined referenceable nodes. + */ +function _node_reference_potential_references_views($field, $options) { + // Extract view and display. + list($view_name, $display_name) = explode(':', $field['settings']['views']['view']); + + // Collect arguments if any. + $args = array(); + if (!empty($field['settings']['views']['view_args'])) { + $args = array_map('trim', explode(',', $field['settings']['views']['view_args'])); + } + + $options['title_field'] = 'title'; + + return references_potential_references_view('node', $view_name, $display_name, $args, $options); +} + +/** + * Helper function for _node_reference_potential_references(). + * * List of referenceable nodes defined by content types. */ -function _node_reference_potential_references_standard($field, $string = '', $match = 'contains', $ids = array(), $limit = NULL) { +function _node_reference_potential_references_standard($field, $options) { // Avoid useless work if (!count($field['settings']['referenceable_types'])) { return array(); @@ -643,37 +724,38 @@ function _node_reference_potential_references_standard($field, $string = '', $ma $query->condition('n.type', $field['settings']['referenceable_types'], 'IN'); } - if ($string !== '') { - switch ($match) { + if ($options['string'] !== '') { + switch ($options['match']) { case 'contains': - $query->condition('n.title', '%' . $string . '%', 'LIKE'); + $query->condition('n.title', '%' . $options['string'] . '%', 'LIKE'); break; case 'starts_with': - $query->condition('n.title', $string . '%', 'LIKE'); + $query->condition('n.title', $options['string'] . '%', 'LIKE'); break; case 'equals': default: // no match type or incorrect match type: use "=" - $query->condition('n.title', $string); + $query->condition('n.title', $options['string']); break; } } - elseif ($ids) { - $query->condition('n.nid', $ids, 'IN', $ids); + + if ($options['ids']) { + $query->condition('n.nid', $options['ids'], 'IN'); + } + + if ($options['limit']) { + $query->range(0, $options['limit']); } $query ->orderBy($node_title_alias) ->orderBy($node_type_alias); - if ($limit) { - $query->range(0, $limit); - } - - $result = $query->execute(); + $result = $query->execute()->fetchAll(); $references = array(); - foreach ($result->fetchAll() as $node) { + foreach ($result as $node) { $references[$node->nid] = array( 'title' => $node->node_title, 'rendered' => check_plain($node->node_title), @@ -698,15 +780,23 @@ function node_reference_autocomplete_access($entity_type, $field_name) { function node_reference_autocomplete($entity_type, $field_name, $string = '') { $field = field_info_field($field_name); - // @todo broken - this is in $field['widget']['settings']['autocomplete_match'] - we'd need the $instance. - $match = isset($field['widget']['autocomplete_match']) ? $field['widget']['autocomplete_match'] : 'contains'; - $matches = array(); + $options = array( + 'string' => $string, + // @todo broken - this is in $field['widget']['settings']['autocomplete_match'] - we'd need the $instance. + 'match' => isset($field['widget']['autocomplete_match']) ? $field['widget']['autocomplete_match'] : 'contains', + 'limit' => 10, + ); + $references = _node_reference_potential_references($field, $options); - $references = _node_reference_potential_references($field, $string, $match, array(), 10); + $matches = array(); foreach ($references as $id => $row) { + // Markup is fine in autocompletion results (might happen when rendered + // through Views) but we want to remove hyperlinks. + $suggestion = preg_replace('/([^<]*)<\/a>/', '$2', $row['rendered']); // Add a class wrapper for a few required CSS overrides. - $matches[$row['title'] . " [nid:$id]"] = '
' . $row['rendered'] . '
'; + $matches[$row['title'] . " [nid:$id]"] = '
' . $suggestion . '
'; } + drupal_json_output($matches); } diff --git references.info references.info index c5a0dfe..b6cc1c5 100644 --- references.info +++ references.info @@ -6,3 +6,5 @@ dependencies[] = field dependencies[] = options files[] = views/references_handler_relationship.inc files[] = views/references_handler_argument.inc +files[] = views/references_plugin_display.inc +files[] = views/references_plugin_style.inc diff --git references.module references.module index 550b9a6..c7c9dad 100644 --- references.module +++ references.module @@ -10,6 +10,156 @@ */ function references_views_api() { return array( - 'api' => '3.0', + 'api' => 3, + 'path' => drupal_get_path('module', 'references') . '/views', ); } + +/** + * Implements hook_views_plugins(). + * + * Defines some plugins used by the Views modes for + * user_reference. + */ +function references_views_plugins() { + $plugins = array( + 'display' => array( + 'references' => array( + 'title' => t('References'), + 'admin' => t('References'), + 'help' => 'Selects referenceable entities for a reference field (node_reference, user_reference...)', + 'handler' => 'references_plugin_display', + 'uses hook menu' => FALSE, + 'use ajax' => FALSE, + 'use pager' => FALSE, + 'accept attachments' => FALSE, + // Custom property, used with views_get_applicable_views() to retrieve + // all views with a 'References' display. + 'references display' => TRUE, + ), + ), + 'style' => array( + 'references_style' => array( + 'title' => t('References list'), + 'help' => '@todo Returns the view as a PHP array of names + rendered rows.', + 'handler' => 'references_plugin_style', + 'theme' => 'views_view_unformatted', + 'uses row plugin' => TRUE, + 'uses fields' => TRUE, + // @todo : 'Feed' displays get automatically created with the correct + // overriden style ('Format'). + 'type' => 'references', + 'even empty' => TRUE, + ), + ), + ); + return $plugins; +} + +/** + * @todo Retrieves the list of views + * + * @param $entity_type + * The entity type. + * + * @return + * @todo return format is not great + */ +function references_get_views($entity_type) { + // Filter views that contain a 'references' display. + $views = views_get_applicable_views('references display'); + + // Filter views that list the entity type we want. + $entity_info = entity_get_info($entity_type); + $results = array(); + foreach ($views as $data) { + list($view, $display_id) = $data; + if ($view->base_table == $entity_info['base table']) { + $id = $view->name . ':' . $display_id; + $display_title = $view->name ." ". strtolower($view->display[$display_id]->display_title) . ' ['. $id . ']'; + $results[$id] = $display_title; + } + } + + return $results; +} + +/** + * @todo + * + * @param $entity_type + * @param $view_name + * @param $display_name + * @param $args + * @param $options + * + * @return + * + */ +function references_potential_references_view($entity_type, $view_name, $display_name, $args, $options) { + $entity_info = entity_get_info($entity_type); + + // Check that the view is valid and the display still exists. + $view = views_get_view($view_name); + if (!$view || $view->base_table != $entity_info['base table'] || !isset($view->display[$display_name])) { + return FALSE; + } + + // Temporary backwards compatibility for fields migrated from CCK D6: accept + // 'default' display. + // @todo Test + // @todo Add the corresponding migration code in hook_content_migrate_instance_alter(). + if ($display_name == 'default') { + $display_name = $view->add_display('references'); + $view->display[$display_name]->set_option('style_plugin', 'references_plugin_style'); + } + + $view->set_display($display_name); + + // @todo From merlinofchaos on IRC : arguments using summary view can defeat + // the style setting. + // We might also need to check if there's an argument, and set its + // style_plugin as well. + + // Set additional options to let references_plugin_display::query() narrow + // the results. + $references_options = array( + 'ids' => $options['ids'], + 'title_field' => $options['title_field'], + 'string' => $options['string'], + 'match' => $options['string'], + ); + $view->display_handler->set_option('references_options', $references_options); + + // We do need the title field, so add it if not present (unlikely, but...) + $fields = $view->get_items('field', $display_name); + if (!isset($fields[$options['title_field']])) { + $label_options = array( + 'label' => '', + 'link_to_node' => 0, + ); + $view->add_item($display_name, 'field', 'node', $options['title_field'], $label_options); + } + + // Limit result set size. + $limit = !empty($options['limit']) ? $options['limit'] : 0; + $view->display_handler->set_option('pager', array('type' => 'some', 'options' => array('items_per_page' => $limit))); + + // If not set, make all fields inline and define a separator. + $row_options = $view->display_handler->get_option('row_options'); + if (empty($row_options['inline'])) { + $row_options['inline'] = drupal_map_assoc(array_keys($view->get_items('field', $display_name))); + } + if (empty($row_options['separator'])) { + $row_options['separator'] = '-'; + } + $view->display_handler->set_option('row_options', $row_options); + + // Make sure the query is not cached + $view->is_cacheable = FALSE; + + // Get the results. + $results = $view->execute_display($display_name, $args); + + return $results; +} \ No newline at end of file diff --git user_reference/user_reference.module user_reference/user_reference.module index 41ff2e6..7551536 100644 --- user_reference/user_reference.module +++ user_reference/user_reference.module @@ -27,7 +27,14 @@ function user_reference_field_info() { 'user_reference' => array( 'label' => t('User reference'), 'description' => t('This field stores the ID of a related user as an integer value.'), - 'settings' => array('referenceable_roles' => array(), 'referenceable_status' => array()), + 'settings' => array( + 'referenceable_roles' => array(), + 'referenceable_status' => array(), + 'views' => array( + 'view' => '', + 'view_args' => '', + ), + ), 'default_widget' => 'user_reference_autocomplete', 'default_formatter' => 'user_reference_default', // Support hook_entity_property_info() from contrib "Entity API". @@ -82,6 +89,42 @@ function user_reference_field_settings_form($field, $instance, $has_data) { : array(1), '#options' => array(1 => t('Active'), 0 => t('Blocked')), ); + + if (module_exists('views')) { + $form['views'] = array( + '#type' => 'fieldset', + '#title' => t('Views - Users that can be referenced'), + '#collapsible' => TRUE, + '#collapsed' => !isset($settings['views']['view']) || $settings['views']['view'] == '', + ); + + $views_options = references_get_views('user'); + dsm($views_options); + if ($views_options) { + $views_options = array('' => '<' . t('none') . '>') + $views_options; + $form['views']['view'] = array( + '#type' => 'select', + '#title' => t('View used to select the users'), + '#options' => $views_options, + '#default_value' => isset($settings['views']['view']) ? $settings['views']['view'] : '', + '#description' => t('

Choose the "Views module" view that selects the users that can be referenced.
Note:

') . + t(''), + ); + $form['views']['view_args'] = array( + '#type' => 'textfield', + '#title' => t('View arguments'), + '#default_value' => isset($settings['views']['view_args']) ? $settings['views']['view_args'] : '', + '#required' => FALSE, + '#description' => t('Provide a comma separated list of arguments to pass to the view.'), + ); + } + else { + $form['views']['no_view_help'] = array( + '#markup' => t("The list of users that can be referenced can provided by a view (Views module) using the 'References' display type. but no appropriate views were found."), + ); + } + } + return $form; } @@ -112,7 +155,10 @@ function user_reference_field_validate($entity_type, $entity, $field, $instance, } // Prevent performance hog if there are no ids to check. if ($ids) { - $refs = _user_reference_potential_references($field, '', NULL, $ids); + $options = array( + 'ids' => $ids, + ); + $refs = _user_reference_potential_references($field, $options); foreach ($items as $delta => $item) { if (is_array($item)) { if (!empty($item['uid']) && !isset($refs[$item['uid']])) { @@ -353,13 +399,18 @@ function user_reference_autocomplete_validate($element, &$form_state, $form) { else { // No explicit uid (the submitted value was not populated by autocomplete // selection). Get the uid of a referencable user from the entered name. - $reference = _user_reference_potential_references($field, $value, 'equals', NULL, 1); - if ($reference) { + $options = array( + 'string' => $value, + 'match' => 'equals', + 'limit' => 1, + ); + $references = _user_reference_potential_references($field, $options); + if ($references) { // @todo The best thing would be to present the user with an // additional form, allowing the user to choose between valid // candidates with the same name. ATM, we pick the first // matching candidate... - $uid = key($reference); + $uid = key($references); } else { form_error($element, t('%name: found no valid user with that name.', array('%name' => $instance['label']))); @@ -397,7 +448,7 @@ function _user_reference_options($field) { $options = array(); foreach ($references as $key => $value) { - $options[$key] = $value['title']; + $options[$key] = $value['rendered']; } return $options; @@ -413,7 +464,7 @@ function _user_reference_options($field) { * consumers use the parts they actually need. * * @param $field - * The field description. + * The field definition. * @param $string * Optional string to filter usernames on (used by autocomplete) * @param $match @@ -435,15 +486,29 @@ function _user_reference_options($field) { * ... * ) */ -function _user_reference_potential_references($field, $string = '', $match = 'contains', $ids = array(), $limit = NULL) { +function _user_reference_potential_references($field, $options = array()) { + // Fill in default options. + $options += array( + 'string' => '', + 'match' => 'contains', + 'ids' => array(), + 'limit' => 0, + ); $results = &drupal_static(__FUNCTION__, array()); // Create unique id for static cache. - $cid = $field['field_name'] . ':' . $match . ':' - . ($string !== '' ? $string : implode('-', $ids)) - . ':' . $limit; + $cid = $field['field_name'] . ':' . $options['match'] . ':' + . ($options['string'] !== '' ? $options['string'] : implode('-', $options['ids'])) + . ':' . $options['limit']; if (!isset($results[$cid])) { - $references = _user_reference_potential_references_standard($field, $string, $match, $ids, $limit); + $references = FALSE; + if (module_exists('views') && !empty($field['settings']['views']['view'])) { + $references = _user_reference_potential_references_views($field, $options); + } + + if ($references === FALSE) { + $references = _user_reference_potential_references_standard($field, $options); + } // Store the results. $results[$cid] = !empty($references) ? $references : array(); @@ -455,9 +520,29 @@ function _user_reference_potential_references($field, $string = '', $match = 'co /** * Helper function for _user_reference_potential_references(). * + * Case of Views-defined referenceable users. + */ +function _user_reference_potential_references_views($field, $options) { + // Extract view and display. + list($view_name, $display_name) = explode(':', $field['settings']['views']['view']); + + // Collect arguments if any. + $args = array(); + if (!empty($field['settings']['views']['view_args'])) { + $args = array_map('trim', explode(',', $field['settings']['views']['view_args'])); + } + + $options['title_field'] = 'name'; + + return references_potential_references_view('user', $view_name, $display_name, $args, $options); +} + +/** + * Helper function for _user_reference_potential_references(). + * * List of referenceable users defined by user role and status. */ -function _user_reference_potential_references_standard($field, $string = '', $match = 'contains', $ids = array(), $limit = NULL) { +function _user_reference_potential_references_standard($field, $options) { // Avoid useless work. $filter_roles = array_filter($field['settings']['referenceable_roles']); $filter_status = array_filter($field['settings']['referenceable_status']); @@ -482,32 +567,32 @@ function _user_reference_potential_references_standard($field, $string = '', $ma $query->condition('r.rid', array_keys($filter_roles), 'IN'); } - if ($string !== '') { - $args = array(); - switch ($match) { + if ($options['string'] !== '') { + switch ($options['match']) { case 'contains': - $query->condition('u.name', '%' . $string . '%', 'LIKE'); + $query->condition('u.name', '%' . $options['string'] . '%', 'LIKE'); break; case 'starts_with': - $query->condition('u.name', $string . '%', 'LIKE'); + $query->condition('u.name', $options['string'] . '%', 'LIKE'); break; case 'equals': default: // no match type or incorrect match type: use "=" - $query->condition('u.name', $string, '='); + $query->condition('u.name', $options['string'], '='); break; } } - elseif ($ids) { - $query->condition('u.uid', $ids, 'IN'); + + if ($options['ids']) { + $query->condition('u.uid', $options['ids'], 'IN'); } // Explicitly exclude the anonymous user. $query->condition('u.uid', 0, '<>'); - if ($limit) { - $query->range(0, $limit); + if ($options['limit']) { + $query->range(0, $options['limit']); } $query->orderBy('u.name'); @@ -528,13 +613,21 @@ function _user_reference_potential_references_standard($field, $string = '', $ma function user_reference_autocomplete($field_name, $string = '') { $field = field_info_field($field_name); - $match = isset($field['widget']['autocomplete_match']) ? $field['widget']['autocomplete_match'] : 'contains'; - $matches = array(); + $options = array( + 'string' => $string, + // @todo broken - this is in $field['widget']['settings']['autocomplete_match'] - we'd need the $instance. + 'match' => isset($field['widget']['autocomplete_match']) ? $field['widget']['autocomplete_match'] : 'contains', + 'limit' => 10, + ); + $references = _user_reference_potential_references($field, $options); - $references = _user_reference_potential_references($field, $string, $match, array(), 10); + $matches = array(); foreach ($references as $id => $row) { - // Add a class wrapper for a few required CSS overrides. - $matches[$row['title'] . " [uid:$id]"] = '
' . $row['rendered'] . '
'; + // Markup is fine in autocompletion results (might happen when rendered + // through Views) but we want to remove hyperlinks. + $suggestion = preg_replace('/([^<]*)<\/a>/', '$2', $row['rendered']); + // Remove link tags Add a class wrapper for a few required CSS overrides. + $matches[$row['title'] . " [uid:$id]"] = '
' . $suggestion . '
'; } drupal_json_output($matches); } diff --git views/references_plugin_display.inc views/references_plugin_display.inc new file mode 100644 index 0000000..dd50eee --- /dev/null +++ views/references_plugin_display.inc @@ -0,0 +1,88 @@ +view->render($this->display->id); + } + + function render() { + if (!empty($this->view->result) || !empty($this->view->style_plugin->definition['even empty'])) { + return $this->view->style_plugin->render($this->view->result); + } + return ''; + } + + function uses_exposed() { + return FALSE; + } + + function query() { + $options = $this->get_option('references_options'); + + // Play nice with View UI 'preview' : if the view is not executed through + // _*_reference_potential_references_views(), don't alter the query. + if (empty($options)) { + return; + } + + // Make sure the id field is included in the results, and save its alias + // so that references_plugin_style can retrieve it. + $this->id_field_alias = $this->view->query->add_field($this->view->base_table, $this->view->base_field); + + // Restrict on the incoming string, or incoming ids. + if ($options['string'] !== '') { + switch ($options['match']) { + case 'equals': + $operator = '='; + $value = $options['string']; + break; + + case 'starts_with': + $operator = 'LIKE'; + $value = db_like($options['string']) . '%'; + break; + + case 'contains': + default: + $operator = 'LIKE'; + $value = '%' . db_like($options['string']) . '%'; + break; + } + + $table_alias = $this->view->query->ensure_table($this->view->base_table); + $this->view->query->add_where(NULL, $table_alias . '.' . $options['title_field'], $value, $operator); + } + elseif ($options['ids']) { + $table_alias = $this->view->query->ensure_table($this->view->base_table); + $this->view->query->add_where(NULL, $table_alias . '.' . $this->view->base_field, $options['ids'], 'IN'); + } + } +} diff --git views/references_plugin_style.inc views/references_plugin_style.inc new file mode 100644 index 0000000..06b3ee3 --- /dev/null +++ views/references_plugin_style.inc @@ -0,0 +1,49 @@ +display->handler->get_option('references_options'); + + // Play nice with View UI 'preview' : if the view is not executed through + // _*_reference_potential_references_views(), just display the HTML. + if (empty($options)) { + return parent::render(); + } + + $title_field = $options['title_field']; + + // Group the rows according to the grouping field, if specified. + $sets = $this->render_grouping($this->view->result, $this->options['grouping']); + + // Grab the alias of the 'id' field added by references_plugin_display. + $id_field_alias = $this->display->handler->id_field_alias; + + // @todo We don't display grouping info for now. Could be useful for select + // widget, though. + $results = array(); + $this->view->row_index = 0; + foreach ($sets as $title => $records) { + foreach ($records as $label => $values) { + // Render the row. + $rendered = $this->row_plugin->render($values); + // Remove linebreaks and extra spaces introduced by templates. + $rendered = preg_replace('/\s+/', ' ', trim($rendered)); + + // Collect the rendered row, and the raw title value. + $results[$values->{$id_field_alias}] = array( + 'rendered' => $rendered, + 'title' => $this->view->field[$title_field]->get_value($values), + ); + + $this->view->row_index++; + } + } + unset($this->view->row_index); + + return $results; + } +}