Tests needed for:
Editing top-level book pages.
Attempting to update the outline of a non-book.
Deleting of book pages.
Deleting of book pages with children.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | 299136-book-test-extended.patch | 5.47 KB | damien tournoud |
Tests needed for:
Editing top-level book pages.
Attempting to update the outline of a non-book.
Deleting of book pages.
Deleting of book pages with children.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | 299136-book-test-extended.patch | 5.47 KB | damien tournoud |
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