Improve error handling/messages when exporting books

This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.

Verified this issue exists on D10

On Drupal 10.1.x with a standard install
Follow the steps in the IS and confirm I get the fatal error
Apply fix and follow steps again and I get a 404

Code looks fine. Triggered for D10 tests.

Patch #21 passed 10.1 tests and review was done in #27

Good job!

This Issue Summary does not have a proposed resolution. How is the error handling being improved? It really helps the reviewer/committer to know what it expected.

I see that patch is being re-tested on a unsupported version of Drupal. Let's get that updated.

Finally, I looked at the patch I don't think that #17.2 has been addressed. We are testing twice that a non book node results in a 404. It would also be easier to read if the arrays were on multiple lines.

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.

Patch #21 works well for me on Drupal 10.3, thanks!

Adding the defensive code I'm okay with.

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