Browse Source

addtemplate patch

http://drupal.org/node/1313676#comment-6169000
Signed-off-by: bachy <git@g-u-i.net>
bachy 12 years ago
parent
commit
68126583b1

+ 457 - 0
addtemplates-1313676-16.patch

@@ -0,0 +1,457 @@
+diff --git a/search-api-page-result.tpl.php b/search-api-page-result.tpl.php
+new file mode 100644
+index 0000000..bf56e48
+--- /dev/null
++++ b/search-api-page-result.tpl.php
+@@ -0,0 +1,56 @@
++<?php
++/**
++ * @file
++ * Default theme implementation for displaying a single search result.
++ *
++ * This template renders a single search result and is collected into
++ * search_api_page-results.tpl.php. This and the parent template are
++ * dependent to one another sharing the markup for definition lists.
++ *
++ * View Mode is set in the Search page settings. If you select
++ * "Themed as search results", then the child template will be used for
++ * theming the individual result. Any other view mode will bypass the
++ * child template.
++ *
++ * Available variables:
++ * - $url: URL of the result.
++ * - $title: Title of the result.
++ * - $snippet: A small preview of the result. Does not apply to user searches.
++ * - $info: String of all the meta information ready for print. Does not apply
++ *   to user searches.
++ * - $info_split: Contains same data as $info, split into a keyed array.
++ * - $list_classes: CSS classes for this list element.
++ *
++ * Default keys within $info_split:
++ * - $info_split['user']: Author of the node linked to users profile. Depends
++ *   on permission.
++ * - $info_split['date']: Last update of the node. Short formatted.
++ *
++ * Since $info_split is keyed, a direct print of the item is possible.
++ * This array does not apply to user searches so it is recommended to check
++ * for its existence before printing. The default keys of 'user' and
++ * 'date' always exist for node searches.
++ *
++ * To check for all available data within $info_split, use the code below.
++ * @code
++ *   <?php print '<pre>'. check_plain(print_r($info_split, 1)) .'</pre>'; ?>
++ * @endcode
++ *
++ * @see template_preprocess()
++ * @see template_preprocess_search_result()
++ * @see template_process()
++ */
++?>
++<li<?php print $list_classes; ?>>
++  <h3 class="title">
++    <?php print $url ? l($title, $url['path'], $url['options']) : check_plain($title); ?>
++  </h3>
++  <div class="search-snippet-info">
++    <?php if ($snippet) : ?>
++      <p class="search-snippet"><?php print $snippet; ?></p>
++    <?php endif; ?>
++    <?php if ($info) : ?>
++      <p class="search-info"><?php print $info; ?></p>
++    <?php endif; ?>
++  </div>
++</li>
+diff --git a/search-api-page-results.tpl.php b/search-api-page-results.tpl.php
+new file mode 100644
+index 0000000..9bbf343
+--- /dev/null
++++ b/search-api-page-results.tpl.php
+@@ -0,0 +1,53 @@
++<?php
++/**
++ * @file
++ * Default theme implementation for displaying search results.
++ *
++ * This template collects each invocation of theme_search_result(). This and
++ * the child template are dependent to one another sharing the markup for
++ * definition lists.
++ *
++ * Note that modules and themes may implement their own search type and theme
++ * function completely bypassing this template.
++ *
++ * Available variables:
++ * - $result_count: Number of results.
++ * - $spellcheck: Possible spelling suggestions from Search spellcheck module.
++ * - $search_results: All results rendered as list items in a single HTML string.
++ * - $items: All results as it is rendered through search-result.tpl.php.
++ * - $search_performance: The number of results and how long the query took.
++ * - $sec: The number of seconds it took to run the query.
++ * - $pager: Row of control buttons for navigating between pages of results.
++ * - $index:
++ * - $keys: The keywords of the executed search.
++ * - $page_machine_name: Machine name of the current Search API Page.
++ *
++ * View Mode is set in the Search page settings. If you select
++ * "Themed as search results", then the child template will be used for
++ * theming the individual result. Any other view mode will bypass the
++ * child template.
++ *
++ * @see template_preprocess_search_api_page_results()
++ */
++
++?>
++<?php if (!empty($result_count)) : ?>
++  <div class="search-api-page <?php print 'search-api-page-' . $page_machine_name . ' view-mode-' . $variables['view_mode'];?>">
++    <?php if ($result_count) : ?>
++      <?php print render($search_performance); ?>
++      <?php print render($spellcheck); ?>
++      <h2><?php print t('Search results');?></h2>
++
++      <?php if ($variables['view_mode'] == 'search_api_page_result') : // Uses child template. ?>
++        <?php print render($search_results); ?>
++      <? else : // All other view modes (Teaser, Full content, RSS and so forth). ?>
++        <div class="search-results">
++          <?php print render(entity_view($index->item_type, $items, $variables['view_mode'])); ?>
++        </div>
++      <?php endif; ?>
++      <?php print $pager; ?>
++    <?php else : ?>
++      <h2><?php print t('Your search yielded no results.');?></h2>
++    <?php endif; ?>
++  </div>
++<?php endif; ?>
+diff --git a/search_api_page.info b/search_api_page.info
+index 148c481..263ce24 100644
+--- a/search_api_page.info
++++ b/search_api_page.info
+@@ -4,5 +4,6 @@ description = "Create search pages using Search API indexes."
+ dependencies[] = search_api
+ core = 7.x
+ package = Search
++stylesheets[all][] = search_api_page.css
+ 
+ configure = admin/config/search/search_api/page
+diff --git a/search_api_page.module b/search_api_page.module
+index a9a00cb..6e15c45 100755
+--- a/search_api_page.module
++++ b/search_api_page.module
+@@ -60,8 +60,12 @@ function search_api_page_theme() {
+       'items' => array(),
+       'view_mode' => 'search_api_page_result',
+       'keys' => '',
++      'page_machine_name' => NULL,
++      'spellcheck' => NULL,
++      'pager' => NULL,
+     ),
+     'file' => 'search_api_page.pages.inc',
++    'template' => 'search-api-page-results',
+   );
+   $themes['search_api_page_result'] = array(
+     'variables' => array(
+@@ -69,14 +73,99 @@ function search_api_page_theme() {
+       'result' => NULL,
+       'item' => NULL,
+       'keys' => '',
++      'list_classes' => '',
+     ),
+     'file' => 'search_api_page.pages.inc',
++    'template' => 'search-api-page-result',
++  );
++  $themes['search_performance'] = array(
++    'render element' => 'element',
++  );
++  $themes['search_results_list'] = array(
++    'render element' => 'element',
+   );
+ 
+   return $themes;
+ }
+ 
+ /**
++ * Implements theme for rendering search-performance
++ */
++function theme_search_performance($variables) {
++  $element = array_shift($variables);
++
++  return $element['#markup'];
++}
++
++/**
++ * Returns HTML for a list of search results.
++ * Taken from theme_item_list().
++ *
++ * @param $variables
++ *   An associative array containing:
++ *   - items: An array of items to be displayed in the list. If an item is a
++ *     string, then it is used as is. If an item is an array, then the "data"
++ *     element of the array is used as the contents of the list item. If an item
++ *     is an array with a "children" element, those children are displayed in a
++ *     nested list. All other elements are treated as attributes of the list
++ *     item element.
++ *   - type: The type of list to return (e.g. "ul", "ol").
++ *   - attributes: The attributes applied to the list element.
++ */
++function theme_search_results_list($variables) {
++  // Pull Element array from the $variables array.
++  $variables = $variables['element'];
++
++  $items = $variables['items']; // Full data
++  $type = $variables['type'];
++
++  // CSS classes for ul
++  $attributes = (!empty($variables['attributes'])) ? $variables['attributes'] : array();
++  $attributes['class'] = array_merge(
++    array('item-list', 'search-results-list'),
++    (isset($attributes['class'])) ?  $attributes['class'] : array()
++  );
++
++  // Render items within a list
++  if (!empty($items)) {
++    $output = "<$type" . drupal_attributes($attributes) . '>';
++    $num_items = count($items);
++
++    // Parse search results as tokens to access items with full data.
++    $i = 0;
++    foreach ($variables['results'] as $result) {
++      // Set css classes.
++      $item_attributes = array();
++      if ($i == 0) {
++        $item_attributes['class'][] = 'first';
++      }
++      if ($i == $num_items - 1) {
++        $item_attributes['class'][] = 'last';
++      }
++      (($i+1)%2) ? $item_attributes['class'][] = 'odd': $item_attributes['class'][] = 'even';
++
++      // Define render array.
++      $data = theme(
++        'search_api_page_result', array(
++          'index' => $variables['index'], // Use full results index.
++          'result' => $result,
++          'item' => isset($items[$result['id']]) ?
++            $items[$result['id']] :
++            NULL,
++          'keys' => $variables['keys'],
++          'list_classes' => drupal_attributes($item_attributes),
++        )
++      );
++      $output .=  $data . "\n";
++      $i++;
++    }
++    $output .= "</$type>";
++
++    return $output;
++  }
++}
++
++/**
+  * Implements hook_permission().
+  */
+ function search_api_page_permission() {
+diff --git a/search_api_page.pages.inc b/search_api_page.pages.inc
+index 7b72a0d..c514909 100644
+--- a/search_api_page.pages.inc
++++ b/search_api_page.pages.inc
+@@ -33,14 +33,18 @@ function search_api_page_view($id, $keys = NULL) {
+       watchdog('search_api_page', 'An error occurred while executing a search: !msg.', array('!msg' => $e->getMessage()), WATCHDOG_ERROR, l(t('search page'), $_GET['q']));
+     }
+ 
+-    // If spellcheck results are returned then add them to the render array.
++    // Load spellcheck.
+     if (isset($results['search_api_spellcheck'])) {
+-      $ret['search_api_spellcheck']['#theme'] = 'search_api_spellcheck';
+-      $ret['search_api_spellcheck']['#spellcheck'] = $results['search_api_spellcheck'];
++      $ret['results']['#spellcheck'] = array(
++        '#theme' => 'search_api_spellcheck',
++        '#spellcheck' => $results['search_api_spellcheck'],
+       // Let the theme function know where the key is stored by passing its arg
+       // number. We can work this out from the number of args in the page path.
+-      $ret['search_api_spellcheck']['#options'] = array(
+-        'arg' => array(count(arg(NULL, $page->path))),
++        '#options' => array(
++          'arg' => array(count(arg(NULL, $page->path))),
++        ),
++        '#prefix' => '<p class="search-api-spellcheck suggestion">',
++        '#suffix' => '</p>',
+       );
+     }
+ 
+@@ -49,11 +53,12 @@ function search_api_page_view($id, $keys = NULL) {
+     $ret['results']['#results'] = $results;
+     $ret['results']['#view_mode'] = isset($page->options['view_mode']) ? $page->options['view_mode'] : 'search_api_page_result';
+     $ret['results']['#keys'] = $keys;
++    $ret['results']['#page_machine_name'] = $page->machine_name;
+ 
++    // Load pager.
+     if ($results['result count'] > $page->options['per_page']) {
+       pager_default_initialize($results['result count'], $page->options['per_page']);
+-      $ret['pager']['#theme'] = 'pager';
+-      $ret['pager']['#quantity'] = 9;
++      $ret['results']['#pager'] = theme('pager');
+     }
+ 
+     if (!empty($results['ignored'])) {
+@@ -108,96 +113,66 @@ function search_api_page_search_execute(Entity $page, $keys) {
+ 
+ /**
+  * Function for preprocessing the variables for the search_api_page_results
+- * theme.
++ * template.
+  *
+  * @param array $variables
+  *   An associative array containing:
+- *   - index: The index this search was executed on.
+- *   - results: An array of search results, as returned by
++ *   - $index: The index this search was executed on.
++ *   - $results: An array of search results, as returned by
+  *     SearchApiQueryInterface::execute().
+- *   - keys: The keywords of the executed search.
+- */
+-function template_preprocess_search_api_page_results(array &$variables) {
+-  if (!empty($variables['results']['results'])) {
+-    $variables['items'] = $variables['index']->loadItems(array_keys($variables['results']['results']));
+-  }
+-}
+-
+-/**
+- * Theme function for displaying search results.
++ *   - $keys: The keywords of the executed search.
+  *
+- * @param array $variables
+- *   An associative array containing:
+- *   - index: The index this search was executed on.
+- *   - results: An array of search results, as returned by
+- *     SearchApiQueryInterface::execute().
+- *   - items: The loaded items for all results, in an array keyed by ID.
+- *   - view_mode: The view mode to use for displaying the individual results,
+- *     or the special mode "search_api_page_result" to use the theme function
+- *     of the same name.
+- *   - keys: The keywords of the executed search.
++ * @see search_api_page-results.tpl.php
+  */
+-function theme_search_api_page_results(array $variables) {
+-  drupal_add_css(drupal_get_path('module', 'search_api_page') . '/search_api_page.css');
+-
+-  $index = $variables['index'];
++function template_preprocess_search_api_page_results(array &$variables) {
+   $results = $variables['results'];
+-  $items = $variables['items'];
+   $keys = $variables['keys'];
+ 
+-  $output = '<p class="search-performance">' . format_plural($results['result count'],
+-      'The search found 1 result in @sec seconds.',
+-      'The search found @count results in @sec seconds.',
+-      array('@sec' => round($results['performance']['complete'], 3))) . '</p>';
+-
+-  if (!$results['result count']) {
+-    $output .= "\n<h2>" . t('Your search yielded no results') . "</h2>\n";
+-    return $output;
+-  }
+-
+-  $output .= "\n<h2>" . t('Search results') . "</h2>\n";
+-
+-  if ($variables['view_mode'] == 'search_api_page_result') {
+-    $output .= '<ol class="search-results">';
+-    foreach ($results['results'] as $item) {
+-      $output .= '<li class="search-result">' . theme('search_api_page_result', array('index' => $index, 'result' => $item, 'item' => isset($items[$item['id']]) ? $items[$item['id']] : NULL, 'keys' => $keys)) . '</li>';
+-    }
+-    $output .= '</ol>';
+-  }
+-  else {
+-    // This option can only be set when the items are entities.
+-    $output .= '<div class="search-results">';
+-    $render = entity_view($index->item_type, $items, $variables['view_mode']);
+-    $output .= render($render);
+-    $output .= '</div>';
+-  }
++  $variables['items'] = $variables['index']->loadItems(array_keys($variables['results']['results']));
++  $variables['result_count'] = $results['result count'];
++  $variables['sec'] = round($results['performance']['complete'], 3);
++  $variables['search_performance'] = array(
++      '#theme' => 'search_performance',
++      '#markup' => format_plural(
++        $results['result count'],
++        'The search found 1 result in @sec seconds.',
++        'The search found @count results in @sec seconds.',
++        array('@sec' => $variables['sec'])
++      ),
++      '#prefix' => '<p class="search-performance">',
++      '#suffix' => '</p>',
++  );
+ 
+-  return $output;
++  $variables['search_results'] = array(
++    '#theme' => 'search_results_list',
++    'results' => $variables['results']['results'],
++    'items' => $variables['items'],
++    'index' => $variables['index'],
++    'type' => 'ul',
++  );
+ }
+ 
+ /**
+- * Theme function for displaying search results.
++ * Process variables for search-result.tpl.php.
+  *
+- * @param array $variables
+- *   An associative array containing:
+- *   - index: The index this search was executed on.
+- *   - result: One item of the search results, an array containing the keys
+- *     'id' and 'score'.
+- *   - item: The loaded item corresponding to the result.
+- *   - keys: The keywords of the executed search.
++ * The $variables array contains the following arguments:
++ * - $result
++ *
++ * @see search_api_page-result.tpl.php
+  */
+-function theme_search_api_page_result(array $variables) {
++function template_preprocess_search_api_page_result(&$variables) {
+   $index = $variables['index'];
+-  $id = $variables['result']['id'];
++  $variables['id'] = $variables['result']['id'];
++  $variables['excerpt'] = $variables['result']['excerpt'];
+   $item = $variables['item'];
+ 
+   $wrapper = $index->entityWrapper($item, FALSE);
+ 
+-  $url = $index->datasource()->getItemUrl($item);
+-  $name = $index->datasource()->getItemLabel($item);
++  $variables['url'] = $index->datasource()->getItemUrl($item);
++  $variables['title'] = $index->datasource()->getItemLabel($item);
+ 
+-  if (!empty($variables['result']['excerpt'])) {
+-    $text = $variables['result']['excerpt'];
++  if (!empty($variables['excerpt'])) {
++    $text = $variables['excerpt'];
+   }
+   else {
+     $fields = $index->options['fields'];
+@@ -239,10 +214,24 @@ function theme_search_api_page_result(array $variables) {
+     }
+   }
+ 
+-  $output = '<h3>' . ($url ? l($name, $url['path'], $url['options']) : check_plain($name)) . "</h3>\n";
+-  if ($text) {
+-    $output .= $text;
++  // Check for existence. User search does not include snippets.
++  $variables['snippet'] = isset($text) ? $text : '';
++
++  // Meta information
++  global $language;
++  if (isset($item->language) && $item->language != $language->language && $item->language != LANGUAGE_NONE) {
++    $variables['title_attributes_array']['xml:lang'] = $item->language;
++    $variables['content_attributes_array']['xml:lang'] = $item->language;
+   }
+ 
+-  return $output;
++  $info = array();
++  if (!empty($item->name)) {
++    $info['user'] = $item->name;
++  }
++  if (!empty($item->created)) {
++    $info['date'] = format_date($item->created, 'short');
++  }
++  // Provide separated and grouped meta information..
++  $variables['info_split'] = $info;
++  $variables['info'] = implode(' - ', $info);
+ }

+ 56 - 0
search-api-page-result.tpl.php

@@ -0,0 +1,56 @@
+<?php
+/**
+ * @file
+ * Default theme implementation for displaying a single search result.
+ *
+ * This template renders a single search result and is collected into
+ * search_api_page-results.tpl.php. This and the parent template are
+ * dependent to one another sharing the markup for definition lists.
+ *
+ * View Mode is set in the Search page settings. If you select
+ * "Themed as search results", then the child template will be used for
+ * theming the individual result. Any other view mode will bypass the
+ * child template.
+ *
+ * Available variables:
+ * - $url: URL of the result.
+ * - $title: Title of the result.
+ * - $snippet: A small preview of the result. Does not apply to user searches.
+ * - $info: String of all the meta information ready for print. Does not apply
+ *   to user searches.
+ * - $info_split: Contains same data as $info, split into a keyed array.
+ * - $list_classes: CSS classes for this list element.
+ *
+ * Default keys within $info_split:
+ * - $info_split['user']: Author of the node linked to users profile. Depends
+ *   on permission.
+ * - $info_split['date']: Last update of the node. Short formatted.
+ *
+ * Since $info_split is keyed, a direct print of the item is possible.
+ * This array does not apply to user searches so it is recommended to check
+ * for its existence before printing. The default keys of 'user' and
+ * 'date' always exist for node searches.
+ *
+ * To check for all available data within $info_split, use the code below.
+ * @code
+ *   <?php print '<pre>'. check_plain(print_r($info_split, 1)) .'</pre>'; ?>
+ * @endcode
+ *
+ * @see template_preprocess()
+ * @see template_preprocess_search_result()
+ * @see template_process()
+ */
+?>
+<li<?php print $list_classes; ?>>
+  <h3 class="title">
+    <?php print $url ? l($title, $url['path'], $url['options']) : check_plain($title); ?>
+  </h3>
+  <div class="search-snippet-info">
+    <?php if ($snippet) : ?>
+      <p class="search-snippet"><?php print $snippet; ?></p>
+    <?php endif; ?>
+    <?php if ($info) : ?>
+      <p class="search-info"><?php print $info; ?></p>
+    <?php endif; ?>
+  </div>
+</li>

+ 53 - 0
search-api-page-results.tpl.php

@@ -0,0 +1,53 @@
+<?php
+/**
+ * @file
+ * Default theme implementation for displaying search results.
+ *
+ * This template collects each invocation of theme_search_result(). This and
+ * the child template are dependent to one another sharing the markup for
+ * definition lists.
+ *
+ * Note that modules and themes may implement their own search type and theme
+ * function completely bypassing this template.
+ *
+ * Available variables:
+ * - $result_count: Number of results.
+ * - $spellcheck: Possible spelling suggestions from Search spellcheck module.
+ * - $search_results: All results rendered as list items in a single HTML string.
+ * - $items: All results as it is rendered through search-result.tpl.php.
+ * - $search_performance: The number of results and how long the query took.
+ * - $sec: The number of seconds it took to run the query.
+ * - $pager: Row of control buttons for navigating between pages of results.
+ * - $index:
+ * - $keys: The keywords of the executed search.
+ * - $page_machine_name: Machine name of the current Search API Page.
+ *
+ * View Mode is set in the Search page settings. If you select
+ * "Themed as search results", then the child template will be used for
+ * theming the individual result. Any other view mode will bypass the
+ * child template.
+ *
+ * @see template_preprocess_search_api_page_results()
+ */
+
+?>
+<?php if (!empty($result_count)) : ?>
+  <div class="search-api-page <?php print 'search-api-page-' . $page_machine_name . ' view-mode-' . $variables['view_mode'];?>">
+    <?php if ($result_count) : ?>
+      <?php print render($search_performance); ?>
+      <?php print render($spellcheck); ?>
+      <h2><?php print t('Search results');?></h2>
+
+      <?php if ($variables['view_mode'] == 'search_api_page_result') : // Uses child template. ?>
+        <?php print render($search_results); ?>
+      <? else : // All other view modes (Teaser, Full content, RSS and so forth). ?>
+        <div class="search-results">
+          <?php print render(entity_view($index->item_type, $items, $variables['view_mode'])); ?>
+        </div>
+      <?php endif; ?>
+      <?php print $pager; ?>
+    <?php else : ?>
+      <h2><?php print t('Your search yielded no results.');?></h2>
+    <?php endif; ?>
+  </div>
+<?php endif; ?>

+ 1 - 0
search_api_page.info

@@ -4,6 +4,7 @@ description = "Create search pages using Search API indexes."
 dependencies[] = search_api
 core = 7.x
 package = Search
+stylesheets[all][] = search_api_page.css
 
 configure = admin/config/search/search_api/page
 

+ 15 - 0
search_api_page.info.orig

@@ -0,0 +1,15 @@
+
+name = Search pages
+description = "Create search pages using Search API indexes."
+dependencies[] = search_api
+core = 7.x
+package = Search
+
+configure = admin/config/search/search_api/page
+
+; Information added by drupal.org packaging script on 2012-06-26
+version = "7.x-1.0-beta2+5-dev"
+core = "7.x"
+project = "search_api_page"
+datestamp = "1340671392"
+

+ 89 - 0
search_api_page.module

@@ -60,8 +60,12 @@ function search_api_page_theme() {
       'items' => array(),
       'view_mode' => 'search_api_page_result',
       'keys' => '',
+      'page_machine_name' => NULL,
+      'spellcheck' => NULL,
+      'pager' => NULL,
     ),
     'file' => 'search_api_page.pages.inc',
+    'template' => 'search-api-page-results',
   );
   $themes['search_api_page_result'] = array(
     'variables' => array(
@@ -69,13 +73,98 @@ function search_api_page_theme() {
       'result' => NULL,
       'item' => NULL,
       'keys' => '',
+      'list_classes' => '',
     ),
     'file' => 'search_api_page.pages.inc',
+    'template' => 'search-api-page-result',
+  );
+  $themes['search_performance'] = array(
+    'render element' => 'element',
+  );
+  $themes['search_results_list'] = array(
+    'render element' => 'element',
   );
 
   return $themes;
 }
 
+/**
+ * Implements theme for rendering search-performance
+ */
+function theme_search_performance($variables) {
+  $element = array_shift($variables);
+
+  return $element['#markup'];
+}
+
+/**
+ * Returns HTML for a list of search results.
+ * Taken from theme_item_list().
+ *
+ * @param $variables
+ *   An associative array containing:
+ *   - items: An array of items to be displayed in the list. If an item is a
+ *     string, then it is used as is. If an item is an array, then the "data"
+ *     element of the array is used as the contents of the list item. If an item
+ *     is an array with a "children" element, those children are displayed in a
+ *     nested list. All other elements are treated as attributes of the list
+ *     item element.
+ *   - type: The type of list to return (e.g. "ul", "ol").
+ *   - attributes: The attributes applied to the list element.
+ */
+function theme_search_results_list($variables) {
+  // Pull Element array from the $variables array.
+  $variables = $variables['element'];
+
+  $items = $variables['items']; // Full data
+  $type = $variables['type'];
+
+  // CSS classes for ul
+  $attributes = (!empty($variables['attributes'])) ? $variables['attributes'] : array();
+  $attributes['class'] = array_merge(
+    array('item-list', 'search-results-list'),
+    (isset($attributes['class'])) ?  $attributes['class'] : array()
+  );
+
+  // Render items within a list
+  if (!empty($items)) {
+    $output = "<$type" . drupal_attributes($attributes) . '>';
+    $num_items = count($items);
+
+    // Parse search results as tokens to access items with full data.
+    $i = 0;
+    foreach ($variables['results'] as $result) {
+      // Set css classes.
+      $item_attributes = array();
+      if ($i == 0) {
+        $item_attributes['class'][] = 'first';
+      }
+      if ($i == $num_items - 1) {
+        $item_attributes['class'][] = 'last';
+      }
+      (($i+1)%2) ? $item_attributes['class'][] = 'odd': $item_attributes['class'][] = 'even';
+
+      // Define render array.
+      $data = theme(
+        'search_api_page_result', array(
+          'index' => $variables['index'], // Use full results index.
+          'result' => $result,
+          'item' => isset($items[$result['id']]) ?
+            $items[$result['id']] :
+            NULL,
+          'keys' => $variables['keys'],
+          'list_classes' => drupal_attributes($item_attributes),
+        )
+      );
+      $output .=  $data . "\n";
+      $i++;
+    }
+    $output .= "</$type>";
+
+    return $output;
+  }
+}
+
 /**
  * Implements hook_permission().
  */

+ 70 - 81
search_api_page.pages.inc

@@ -22,7 +22,7 @@ function search_api_page_view($id, $keys = NULL) {
     $page->options['per_page'] = (int) $_GET['per_page'];
   }
 
-  if ($page->options['result_page_search_form']) {
+  if (isset($page->options['result_page_search_form']) && $page->options['result_page_search_form']) {
     $ret['form'] = drupal_get_form('search_api_page_search_form', $page, $keys);
   }
 
@@ -35,14 +35,18 @@ function search_api_page_view($id, $keys = NULL) {
       watchdog('search_api_page', 'An error occurred while executing a search: !msg.', array('!msg' => $e->getMessage()), WATCHDOG_ERROR, l(t('search page'), $_GET['q']));
     }
 
-    // If spellcheck results are returned then add them to the render array.
+    // Load spellcheck.
     if (isset($results['search_api_spellcheck'])) {
-      $ret['search_api_spellcheck']['#theme'] = 'search_api_spellcheck';
-      $ret['search_api_spellcheck']['#spellcheck'] = $results['search_api_spellcheck'];
+      $ret['results']['#spellcheck'] = array(
+        '#theme' => 'search_api_spellcheck',
+        '#spellcheck' => $results['search_api_spellcheck'],
       // Let the theme function know where the key is stored by passing its arg
       // number. We can work this out from the number of args in the page path.
-      $ret['search_api_spellcheck']['#options'] = array(
-        'arg' => array(count(arg(NULL, $page->path))),
+        '#options' => array(
+          'arg' => array(count(arg(NULL, $page->path))),
+        ),
+        '#prefix' => '<p class="search-api-spellcheck suggestion">',
+        '#suffix' => '</p>',
       );
     }
 
@@ -51,11 +55,12 @@ function search_api_page_view($id, $keys = NULL) {
     $ret['results']['#results'] = $results;
     $ret['results']['#view_mode'] = isset($page->options['view_mode']) ? $page->options['view_mode'] : 'search_api_page_result';
     $ret['results']['#keys'] = $keys;
+    $ret['results']['#page_machine_name'] = $page->machine_name;
 
+    // Load pager.
     if ($results['result count'] > $page->options['per_page']) {
       pager_default_initialize($results['result count'], $page->options['per_page']);
-      $ret['pager']['#theme'] = 'pager';
-      $ret['pager']['#quantity'] = 9;
+      $ret['results']['#pager'] = theme('pager');
     }
 
     if (!empty($results['ignored'])) {
@@ -110,96 +115,66 @@ function search_api_page_search_execute(Entity $page, $keys) {
 
 /**
  * Function for preprocessing the variables for the search_api_page_results
- * theme.
+ * template.
  *
  * @param array $variables
  *   An associative array containing:
- *   - index: The index this search was executed on.
- *   - results: An array of search results, as returned by
+ *   - $index: The index this search was executed on.
+ *   - $results: An array of search results, as returned by
  *     SearchApiQueryInterface::execute().
- *   - keys: The keywords of the executed search.
- */
-function template_preprocess_search_api_page_results(array &$variables) {
-  if (!empty($variables['results']['results'])) {
-    $variables['items'] = $variables['index']->loadItems(array_keys($variables['results']['results']));
-  }
-}
-
-/**
- * Theme function for displaying search results.
+ *   - $keys: The keywords of the executed search.
  *
- * @param array $variables
- *   An associative array containing:
- *   - index: The index this search was executed on.
- *   - results: An array of search results, as returned by
- *     SearchApiQueryInterface::execute().
- *   - items: The loaded items for all results, in an array keyed by ID.
- *   - view_mode: The view mode to use for displaying the individual results,
- *     or the special mode "search_api_page_result" to use the theme function
- *     of the same name.
- *   - keys: The keywords of the executed search.
+ * @see search_api_page-results.tpl.php
  */
-function theme_search_api_page_results(array $variables) {
-  drupal_add_css(drupal_get_path('module', 'search_api_page') . '/search_api_page.css');
-
-  $index = $variables['index'];
+function template_preprocess_search_api_page_results(array &$variables) {
   $results = $variables['results'];
-  $items = $variables['items'];
   $keys = $variables['keys'];
 
-  $output = '<p class="search-performance">' . format_plural($results['result count'],
-      'The search found 1 result in @sec seconds.',
-      'The search found @count results in @sec seconds.',
-      array('@sec' => round($results['performance']['complete'], 3))) . '</p>';
-
-  if (!$results['result count']) {
-    $output .= "\n<h2>" . t('Your search yielded no results') . "</h2>\n";
-    return $output;
-  }
-
-  $output .= "\n<h2>" . t('Search results') . "</h2>\n";
-
-  if ($variables['view_mode'] == 'search_api_page_result') {
-    $output .= '<ol class="search-results">';
-    foreach ($results['results'] as $item) {
-      $output .= '<li class="search-result">' . theme('search_api_page_result', array('index' => $index, 'result' => $item, 'item' => isset($items[$item['id']]) ? $items[$item['id']] : NULL, 'keys' => $keys)) . '</li>';
-    }
-    $output .= '</ol>';
-  }
-  else {
-    // This option can only be set when the items are entities.
-    $output .= '<div class="search-results">';
-    $render = entity_view($index->item_type, $items, $variables['view_mode']);
-    $output .= render($render);
-    $output .= '</div>';
-  }
+  $variables['items'] = $variables['index']->loadItems(array_keys($variables['results']['results']));
+  $variables['result_count'] = $results['result count'];
+  $variables['sec'] = round($results['performance']['complete'], 3);
+  $variables['search_performance'] = array(
+      '#theme' => 'search_performance',
+      '#markup' => format_plural(
+        $results['result count'],
+        'The search found 1 result in @sec seconds.',
+        'The search found @count results in @sec seconds.',
+        array('@sec' => $variables['sec'])
+      ),
+      '#prefix' => '<p class="search-performance">',
+      '#suffix' => '</p>',
+  );
 
-  return $output;
+  $variables['search_results'] = array(
+    '#theme' => 'search_results_list',
+    'results' => $variables['results']['results'],
+    'items' => $variables['items'],
+    'index' => $variables['index'],
+    'type' => 'ul',
+  );
 }
 
 /**
- * Theme function for displaying search results.
+ * Process variables for search-result.tpl.php.
  *
- * @param array $variables
- *   An associative array containing:
- *   - index: The index this search was executed on.
- *   - result: One item of the search results, an array containing the keys
- *     'id' and 'score'.
- *   - item: The loaded item corresponding to the result.
- *   - keys: The keywords of the executed search.
+ * The $variables array contains the following arguments:
+ * - $result
+ *
+ * @see search_api_page-result.tpl.php
  */
-function theme_search_api_page_result(array $variables) {
+function template_preprocess_search_api_page_result(&$variables) {
   $index = $variables['index'];
-  $id = $variables['result']['id'];
+  $variables['id'] = $variables['result']['id'];
+  $variables['excerpt'] = $variables['result']['excerpt'];
   $item = $variables['item'];
 
   $wrapper = $index->entityWrapper($item, FALSE);
 
-  $url = $index->datasource()->getItemUrl($item);
-  $name = $index->datasource()->getItemLabel($item);
+  $variables['url'] = $index->datasource()->getItemUrl($item);
+  $variables['title'] = $index->datasource()->getItemLabel($item);
 
-  if (!empty($variables['result']['excerpt'])) {
-    $text = $variables['result']['excerpt'];
+  if (!empty($variables['excerpt'])) {
+    $text = $variables['excerpt'];
   }
   else {
     $fields = $index->options['fields'];
@@ -241,10 +216,24 @@ function theme_search_api_page_result(array $variables) {
     }
   }
 
-  $output = '<h3>' . ($url ? l($name, $url['path'], $url['options']) : check_plain($name)) . "</h3>\n";
-  if ($text) {
-    $output .= $text;
+  // Check for existence. User search does not include snippets.
+  $variables['snippet'] = isset($text) ? $text : '';
+
+  // Meta information
+  global $language;
+  if (isset($item->language) && $item->language != $language->language && $item->language != LANGUAGE_NONE) {
+    $variables['title_attributes_array']['xml:lang'] = $item->language;
+    $variables['content_attributes_array']['xml:lang'] = $item->language;
   }
 
-  return $output;
+  $info = array();
+  if (!empty($item->name)) {
+    $info['user'] = $item->name;
+  }
+  if (!empty($item->created)) {
+    $info['date'] = format_date($item->created, 'short');
+  }
+  // Provide separated and grouped meta information..
+  $variables['info_split'] = $info;
+  $variables['info'] = implode(' - ', $info);
 }

+ 3 - 1
search_api_page.pages.inc.orig

@@ -22,7 +22,9 @@ function search_api_page_view($id, $keys = NULL) {
     $page->options['per_page'] = (int) $_GET['per_page'];
   }
 
-  $ret['form'] = drupal_get_form('search_api_page_search_form', $page, $keys);
+  if (isset($page->options['result_page_search_form']) && $page->options['result_page_search_form']) {
+    $ret['form'] = drupal_get_form('search_api_page_search_form', $page, $keys);
+  }
 
   if ($keys) {
     try {