In function theme_views_bonus_lineage_tree_summary, the algorithm for closing opened < li > and < ul > tags does not seem to produce the desired effects. I don't quite have the time to delve deeper into what precisely is wrong, but here is some sample code generated on my site

<div class='view view-biblioview'><div class='view-summary view-summary-biblioview'><ul></li>
<li><a href="/literature/Bipolar+Disorder">Bipolar Disorder</a> (3)
<ul>
<li><a href="/literature/Cultural+analysis">Cultural analysis</a> (3)</li>
<li><a href="/literature/Helpful+skills+for+daily+life">Helpful skills for daily life</a> (1)</li>

</ul>
</li>
</ul>
<li><a href="/literature/Suicide">Suicide</a> (2)</ul></div></div>

"biblioview" is here my custom name for the view. The < /li > in the first line closes a tag that hasn't been opened, the < /ul > in the penultimate line closes the list prematurely (it's the top level and should contain the next item as well), and in the last line, the < li > tag is not closed.

Thanks for the module, and for looking into it.

Comments

dmitrig01’s picture

dmitrig01 commented

would you mind posting an export of your view?

Tobi2000’s picture

Tobi2000 commented

Here comes an export of the view that produces the faulty ul list. You can look at it at www.livecrazy.org/literature :

  $view = new stdClass();
  $view->name = 'biblioview';
  $view->description = 'Tree view of bibliography entries';
  $view->access = array (
  0 => '1',
  1 => '2',
  2 => '3',
);
  $view->view_args_php = '';
  $view->page = TRUE;
  $view->page_title = 'Literature';
  $view->page_header = '';
  $view->page_header_format = '1';
  $view->page_footer = '';
  $view->page_footer_format = '1';
  $view->page_empty = '';
  $view->page_empty_format = '1';
  $view->page_type = 'lineage_tree';
  $view->url = 'literature';
  $view->use_pager = TRUE;
  $view->nodes_per_page = '50';
  $view->sort = array (
  );
  $view->argument = array (
    array (
      'type' => 'lineage_branch',
      'argdefault' => '3',
      'title' => '',
      'options' => '',
      'wildcard' => '',
      'wildcard_substitution' => '',
    ),
    array (
      'type' => 'nodeletter',
      'argdefault' => '2',
      'title' => '',
      'options' => '',
      'wildcard' => '',
      'wildcard_substitution' => '',
    ),
  );
  $view->field = array (
    array (
      'tablename' => 'biblio',
      'field' => 'biblio_authors',
      'label' => 'Author',
      'sortable' => '1',
      'defaultsort' => 'ASC',
    ),
    array (
      'tablename' => 'biblio',
      'field' => 'biblio_year',
      'label' => 'Year',
      'sortable' => '1',
    ),
    array (
      'tablename' => 'node',
      'field' => 'title',
      'label' => 'Title',
      'handler' => 'views_handler_field_nodelink',
      'sortable' => '1',
      'options' => 'link',
    ),
  );
  $view->filter = array (
    array (
      'tablename' => 'node',
      'field' => 'type',
      'operator' => 'OR',
      'options' => '',
      'value' => array (
  0 => 'biblio',
),
    ),
    array (
      'tablename' => 'node',
      'field' => 'status',
      'operator' => '=',
      'options' => '',
      'value' => '1',
    ),
  );
  $view->exposed_filter = array (
  );
  $view->requires = array(biblio, node);
  $views[$view->name] = $view;
pomliane’s picture

pomliane commented
Status: Active » Closed (won't fix)

This version of Views Bonus Pack is not supported anymore. The issue is closed for this reason.
Please upgrade to a supported version and feel free to reopen the issue on the new version if applicable.

This issue has been automagically closed by a script.