Problem/Motivation

Remove Book module.

Steps to reproduce

Proposed resolution

Remaining tasks

  • The change record for this issue should include a link to recommendations page, https://www.drupal.org/node/3223395#s-Book. (For example, the CR for removing HAL)
  • Tag this issue 'Needs release note.'
  • Remove the extension ;-).
  • Remove references from core/phpstan-baseline.neon.
  • Remove any spelling words specific to the extension from the dictionary
  • Check for references in @todo.
  • Handle migration tests.
    • In all the functional tests in migrate_drupal_ui make sure that Book is not installed. Book should also be removed from the methods getAvailablePaths() and moved to getMissingPaths() in the tests using those methods.

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3423975

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

quietone created an issue. See original summary.

quietone’s picture

quietone commented
Issue summary: View changes

quietone opened merge request !6783

quietone’s picture

quietone commented
Status: Active » Postponed

quietone credited catch.

quietone credited kostyashupenko.

quietone credited lauriii.

quietone credited smustgrave.

quietone’s picture

quietone commented

Closed #3410220: Remove book module from all core themes as a duplicate, adding credit.

smustgrave’s picture

smustgrave commented
Status: Postponed » Needs work

Book is officially deprecated.

Rebased this MR fingers crossed.

smustgrave’s picture

smustgrave commented
Status: Needs work » Postponed

I was hoping book wasn't part of any fixture but guess I was wrong

catch’s picture

catch
he/him
Primary language English
commented
Status: Postponed » Needs work

Unblocked now.

smustgrave changed the visibility of the branch 3423975-remove-book-module to hidden.

smustgrave opened merge request !7613

smustgrave’s picture

smustgrave commented
Status: Needs work » Needs review
andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs review » Reviewed & tested by the community

Looks ready

catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Needs work

Needs a rebase.

smustgrave’s picture

smustgrave commented
Status: Needs work » Reviewed & tested by the community

Rebased

  • catch committed 85066760 on 11.x 
    Issue #3423975 by smustgrave, quietone, catch, kostyashupenko, lauriii:...
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Still a fan of book module but it doesn't need to be in core any more.

Committed/pushed to 11.x, thanks!

smustgrave closed merge request !7613

Status: Fixed » Closed (fixed)

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

quietone closed merge request !6783