Closed (won't fix)
Project:
Outline
Version:
6.x-0.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
30 Sep 2008 at 07:47 UTC
Updated:
6 Jul 2016 at 03:47 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
beginner commentedThere are two ways to go about it:
1) either we initialize all existing book nodes to work with outline.module when the latter is initialized.
2) or we do nothing but we initialize each node separately as we go along.
solution 1 would make more sense, I think.
Next question: what needs to be initialized?
Comment #2
beginner commentedLooking at the {outline_book} columns, I'm thinking we should update the table definition to create default values for some of them.
Comment #3
beginner commentedHere is a very beginning of a patch. It's not complete, but I don't have time to play more today.
Comment #4
beginner commentedcommitted full fix today.
Comment #5
beginner commentedThe last patch didn't account for the following scenario:
install outline -> disable outline -> use book.module some more -> re-enable outline.
Comment #6
beginner commentedhttp://api.drupal.org/api/function/hook_enable/6
Comment #7
beginner commentedDone:
Import data from book.module where we don't have a corresponding outline entry:
TODO:
Update data where the vid, the place in the outline has changed, where nodes have been removed, etc...
Comment #8
captaindav commented