Closed (fixed)
Project:
Drupal core
Version:
6.x-dev
Component:
book.module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
13 Aug 2007 at 15:07 UTC
Updated:
4 Sep 2007 at 13:01 UTC
Jump to comment: Most recent file
Comments
Comment #1
pwolanin commentedI see the bug also, and this patch fixes it. However, warning text like that used in function book_form_node_delete_confirm_alter() should be added as well if the page has children.
Comment #2
webernet commentedNow notifies you that "Any child pages will be relocated automatically."
Comment #3
pwolanin commentedI was thinking something more like this in terms of varying the description depending on whether the node has children.
Comment #4
webernet commentedSome additional fine tunning of the wording.
Comment #5
pwolanin commentedLooks fine, applies cleanly, and works. Much more informative than before.
Comment #6
gábor hojtsyI would not say "This may make it difficult to re-create the existing hierarchy." is anywhere close to informative. This suggests an uncertain outcome of the action, which is not something one can decide to agree or not. Can we be more specific, ie. not uncertain?
Comment #7
pwolanin commentedhow about something like:
It's wordy, but perhaps more accurate.
Comment #8
gábor hojtsyIt is more accurate, but I needed to stop on the "in order to" part and reread multiple times to get it. I think you mean:
'%title has associated child pages, which will be relocated automatically to maintain their connection to the book. If you'd like to recreate the hierarchy used before, %title may be added again using the Outline tab, and each of its former child pages relocated manually.'
The "existing" word without (to me) proper time information mislead me actually. I am not a native English speaker, so the above might not be correct. Feel free to fix or come up with better ideas.
Comment #9
pwolanin commentedthat maybe?
Comment #10
gábor hojtsyYes, this sounds much better.
Comment #11
pwolanin commentedpatch with that wording attached.
Comment #12
webernet commentedSlightly clearer in my opinion.
Comment #13
pwolanin commentedpatch applies - looks fine and works.
Comment #14
gábor hojtsyThanks, committed!
Comment #15
(not verified) commented