Closed (outdated)
Project:
Drupal core
Version:
11.x-dev
Component:
book.module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
23 Aug 2008 at 20:38 UTC
Updated:
24 Apr 2024 at 06:59 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
damien tournoud commentedHere is the extension of a patch I had in my sandbox for weeks.
Rerolled and extended to demonstrate the bug from #261258: Fix node_save() insertion logic.
Comment #2
catchEverything looks good here. Marking RTBC - but note it's dependent on #261258: Fix node_save() insertion logic (also RTBC) since there's 19 fails without that patch.
Comment #3
catchThe node_save() patch is back to needs work, so marking this postponed.
Comment #4
marcvangendReviving this issue now that #261258: Fix node_save() insertion logic has been committed.
Comment #5
marcvangend(Component automatically changed because 'tests' no longer exists, but this is more appropriate.)
Comment #7
catchComment #8
aspilicious commentedThese tests are not correct anymore. MoveNode stuff doesn't exist (anymore)
Comment #17
pameeela commentedFrom what I can tell this is still valid, there are no tests for the items listed.
Comment #18
catchMoving to a task though.
Comment #23
quietone commentedThis extension is being deprecated, see #3376070: [Meta] Tasks to deprecate Book module. It will be removed from core and moved to a contrib project, #3376101: [11.x] [Meta] Tasks to remove Book.
This is now Postponed. The status is set according to two policies. The Remove a core extension and move it to a contributed project and the Extensions approved for removal policies.
This issue may be re-opened if it can be considered critical, If unsure, re-open the issue and ask in a comment.
Comment #24
quietone commentedI am not sure when they were added but I see tests for deleting various combinations of parent and child pages. This is one issue where testing was added, #2482857: Cannot delete a book parent