Include 'is_active' value in book tree items

10.1.x update

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.

Needs reroll for contrib only.

Rerolled the patch for contrib module only.

Should be an MR

Created an MR, ready for review.

@kalpanajaiswal I've left some comments and Tests are also failing please check.

Hi, I've addressed the feedback which is mentioned in MR !10.

1. Refactored the $element['is_active'] assignment to use a ternary operator for simplicity.
2. Replaced the direct \Drupal::routeMatch() call with RouteMatchInterface by injecting it into the BookManager constructor.
3. Updated the service definition to include @current_route_match.

Please review the chnages and let me know if any further changes are required.

MR is unmergable. Probably could add a simple assertion somewhere too.

smustgrave → closed merge request !10

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