Book outline not limited to allowed content types

The Needs Review Queue Bot → tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

This extension is being deprecated, see 🌱 [Meta] Tasks to deprecate Book module Active . It will be removed from core and moved to a contrib project, 📌 [11.x] [Meta] Tasks to remove Book Active .

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.

Reroll of patch 2862291-34.patch

Also, in the book.module file book_form_node_form_alter() function the 'Change book (update list of parents)' submit button is placed in the forms, if user has 'administer book outlines' permission, disregarding the allowed node type.

This is patch #43 with the issues from the automated tests fixed.

There doesn't appear to be any progress towards moving the book module to contrib.

The Needs Review Queue Bot → tested this issue.

While you are making the above changes, we recommend that you convert this patch to a merge request → . Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)

I created an issue fork with patch 45 plus an additional fix. It has not displayed the button to make a merge request.

The Needs Review Queue Bot → tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

This does not mean that the patch needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Opened an MR for the changes made in #47, thanks!

I do not think we could easily change this for reasons mentioned in #35 and #12. Changing this will break existing sites already using this feature. So if we want to do this, we should think about existing sites and BC.

On the other hand, I do not think this issue is critical enough, so I suppose it should remain as Postponed, see #42. I see some progress in child issues here: 🌱 [Meta] Tasks to deprecate Book module Active . Thanks!

Rerolled #45 ✨ Book outline not limited to allowed content types Needs work for the contrib version of the module.

Actually reverted in place of ✨ Allow the book outline functionality on non-book-enabled node types to be hidden from users with the "Administer book outlines" permission Needs work

So maybe we don't actually need.

Curious if this is needed anymore?

Also appears to be fixed in 🐛 Cannot save unpublished versions of published content for users without manage book privileges Needs work

smustgrave → closed merge request !34

This is a duplicate pretty sure. But don't for ease going to mark fixed so everyone gets their credit who helped.

Automatically closed - issue fixed for 2 weeks with no activity.