fix: Fallback to old eventdispatcher on older releases

[juliushaertl]

Fix #828

🏁 Checklist

• Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
• Sign-off message is added to all commits
• Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests
• Documentation (README or documentation) has been updated or is not required

[cypress]

4 failed and 1 flaky tests on run #1114 ↗︎

4

75

0

0

1

Details:

fix: Fallback to old eventdispatcher on older releases
Project: Collectives	Commit: d8abab7709
Status: Failed	Duration: 11:23 💡
Started: Sep 8, 2023 1:05 PM	Ended: Sep 8, 2023 1:16 PM

  pages.spec.js • 1 failed test

View Output Video

Test		Artifacts
Page > Using the reference picker > Supports selecting a page from a collective		Test Replay Output Screenshots

  page-list.spec.js • 1 failed test

View Output Video

Test		Artifacts
Page list > Page trash > allows to trash and restore page with subpage and attachment		Test Replay Output Screenshots

  page-details.spec.js • 1 failed test

View Output Video

Test		Artifacts
Page details > Displaying backlinks > Lists backlinks for a page		Test Replay Output Screenshots

  apps.spec.js • 1 failed test

View Output Video

Test		Artifacts
The apps > Disabled circles app does not break files view > Renders the default files list		Test Replay Output Screenshots

  cypress/e2e/pages.spec.js • 1 flaky test

View Output Video

Test		Artifacts
Page > Editing a page > Lists attachments for the page and allows restore		Test Replay Output Screenshots

_{This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.}

[max-nextcloud]

would be nice to have a test for the background job on older versions so we'd see when we break it.

[juliushaertl]

Let me just push a unit test to assert the relevant bits, a full test for the background job is more tricky

[juliushaertl]

Pushed, though the test is not very fancy

[max-nextcloud]

Thanks for the test! 🎉

[theroch]

I've have running this patch for several days now without any issues.

[dorpauli]

I have also tried it. Logs are gone and so far no problems.

[level420]

manually applied the PR and it works as exptected.
please release a new version of collectives compatible with nextcloud 27 (and 26) with this PR!