There's problem with listing all the outlines.

The following should fix the problem (line 1030 to 1034) function outline_list():

  foreach ($outlines as $volume_id => $title) {
  	$result = db_query("SELECT nid FROM {outline_nodes} WHERE volume_id='%s' AND parent=%d LIMIT 0,1",$volume_id,0);
  	$row = db_fetch_object($result);
    $items[] = l($title, 'node/'. $row->nid);
  }

hope this helps.

CommentFileSizeAuthor
#3 outline_patch_192875.patch1.38 KBBlakeLucchesi

Comments

rapsli’s picture

rapsli commented
Status: Active » Needs review
budda’s picture

budda
Primary language English
Location Cheshire
commented

What url path page does this fix an error on?

BlakeLucchesi’s picture

BlakeLucchesi commented
StatusFileSize
new outline_patch_192875.patch1.38 KB

This fixes the listing of outlines located at admin/content/outline. However, I'm not really sure this is the approach that should be taken, you are talking about hitting the database once per outline just to grab the nid of the parent node. Someone that has 300 outlines will have their database queried 300 times just to list this page...

This patch takes a different approach by creating a new function _list_outline_parents to be used instead of _list_outline (which returns only the volume id and title NOT the node id).

captaindav’s picture

captaindav commented
Issue summary: View changes
Status: Needs review » Closed (won't fix)