Drupal 8.7 is released

[Natkeeran]

Drupal 8.7 has been released.

We need to test and update Drupal to 8.7 the drupal project composer.json.

This issue that was throwing exceptions in ansible and that necessitated https://github.com/Islandora-Devops/claw-playbook/blob/master/post-install.yml#L13 has been fixed.

[mjordan]

Can we test #1035 as part of this?

[Natkeeran]

@dannylamb It seems like there is more to that than simply changing the drupal version. Also, wondering if you want to pull in changes from https://github.com/drupal-composer/drupal-project.

[dannylamb]

@Natkeeran Yes, please. I usually pull in the latest from upstream whenever I'm stomping around in there. What are you running into?

[Natkeeran]

@dannylamb
Here is an attempt: https://github.com/Islandora-CLAW/drupal-project/tree/drupal_87

1. how to test this without being able to update composer package
2. how does the composer.lock file gets updated?

[Natkeeran]

@mjordan

This issue (https://www.drupal.org/project/drupal/issues/2350939) is not fixed, thus 1035 is not fixed.

The ansible issue is fixed.

[dannylamb]

Just vagrant upd with the 8.7 release of drupal-project. Red 5, I'm going in.

[dannylamb]

Alright, so, a bit of a red herring. It looked like even with the configuration set to expose /media/* urls, the REST endpoints were /media/*/edit, which wrecked everything. So I went through and refactored and put url generating function is IslandoraUtils and just call those from everywhere. Still no dice. Then I cleared my cache and everything worked just fine.

So the solution to this is to just add a cache clear after setting the variable in claw-playbook.

BUT not all is lost, as that refactor actually fixes #1095 🍾

I'll throw up PRs for things as I clean up. That refactor means I have to go touch up all the tests.

[whikloj]

@dannylamb I thought that as well, but it is not true. If you change the configuration, then you need to clear the cache but the REST endpoints do also change.

[dannylamb]

@whikloj I know right? Threw me for a loop.