Livre - evaluate Book 2.0.x with a clean full backport of 3.0.x

Ok I brought everything from book 3.0.x over to book 2.0.x

and then I backed off the changes that make 3.0.x incompatible with Drupal 11.1.8.

I changed all the version pinning from ^11.2 to ^11.1

made another change to book.post_update.php as it crashed my environment, the changes are in the merge request.

"drupal/book": "2.0.x-dev#b966267a as 2.0.2",

this patch:

"patches": {
     "drupal/book": {
          "3554905 - Port book allow other types to create books to 2.0.x":
           "https://git.drupalcode.org/project/book/-/merge_requests/127.diff"
      }
 },

Need to make a new MR based off of 3555112 and set it to merge to 2.0.x

#3555112: Make 1.0.x Livre a version of Book 3.0.x that is compatible with Drupal 11.1.8+

We need this desperately before a prod release. Cannot upgrade to D11.2.x so we need this functionality in D11.1.8.

All tests are passing 100% in Drupal 11.2.x while there's now only two fails , possibly insignificant fails for D11.1.8.

So, we had the desired functionality working with Drupal 10 using a core patch.

When we upgraded to Drupal 11, we dropped the core patch obviously, since it no longer applied, forgot about it, and then 8 months after upgrading we noticed a bunch of broken functionality that previously was not broken.

The fix/features were applied to 3.0.x however 3.0.x was made incompatible with Drupal 11.1.8.

So, I've made it compatible with Drupal 11.1.8 and only seeing two errors in the tests that likely won't affect us, and to avoid problems with composer I'm creating a merge patch to bring this back to 2.0.x.

Would like to fix the two remaining issues but probably don't need to.

*** EDIT ***

Noise

*** END EDIT ***

Almost perfect now.

"drupal/book": "2.0.x-dev#b966267a as 2.0.2",

this patch:

"patches": {
     "drupal/book": {
          "3554905 - Port book allow other types to create books to 2.0.x":
           "https://git.drupalcode.org/project/book/-/merge_requests/131.diff"
      }
 },

Next commit will add test coverage for 10.5.x

Basically not needed now, Livre 1.0.x is compatible with Drupal 10.5.x, Drupal 11.1.x, Drupal 11.2.x and Drupal 11.3.x and is based off of the 3.0.x branch of the Book module.

Keeping an eye on the upstream for changes, will ensure git aligns when possible.

updating the merge request.

All tests passing, 100% green!

Reviewing this

Patch #44 essentially makes book 2.0.x equivalent to livre 1.0.2 , which it'self is essentially book 3.0.x except backwards compatible with all supported releases of Drupal core.

With that said, probably easier to move forward with livre until we're ready for Drupal 11.2/11.3.

Keeping both options on the table for the moment:

Option #1 ) Patch the book module (276 kilobytes of patching)
Option #2 ) Simply require drupal/livre , no need to patch, replaces book without fuss.

Rolled again for 2.0.3

Ok patch 47 won't roll, fatal merge conflict, I won't be able to maintain book 2.0.x this way, recommending everyone use livre instead which aligns well with book 3.0.x.

hmm , might have been a composer glitch

Ok for book 2.0.3:

apply this patch which reverts the conflicting bit
Revert conflict

then this patch
backport 3x

then this patch
what was removed from conflict, apply forward fix from 3x instead

This patch is for 2.0.3 specifically

Ok comment #50 is for 2.0.x dev only.

I've made a new one, one patch for book 2.0.3 which brings all of the 3.0.x functionality to 2.0.x , this should work.

The conflict with patch 47 and 2.0.3 was related to the stupid build info yml with the release date

New patch now

Unfortunately we already upgraded to 2.0.x with wxt 5.4.x and 6.1.x so we're committed now.

Also, I've aligned the configuration to the new array format in the bundle settings for book 3.0.x / livre 1.0.x

I recomment livre 1.0.7+

Reviewing this again.

Not needed, livre 1.0.5 is based on book 3.0.x, has nothing from 2.0.x.