fix(UserApiController): Fix warning during test run with PHP 8.2

[juliusknorr]

Fix test failure that only happens on PHP8.2

stripos(): Passing null to parameter #1 ($haystack) of type string is deprecated

[cypress]

14 failed and 25 flaky tests on run #10001 ↗︎

14

86

1

42

25

Details:

fix(UserApiController): Fix warning during test run with PHP 8.2
Project: Text	Commit: 851528c182
Status: Failed	Duration: 18:24 💡
Started: Jun 6, 2023 6:27 AM	Ended: Jun 6, 2023 6:45 AM

  workspace.spec.js • 2 failed tests

View Output Video

Test		Artifacts
Workspace > takes README.md into account		Output Screenshots
Workspace > emoji picker		Output Screenshots

  sync.spec.js • 1 failed test

View Output Video

Test		Artifacts
Sync > saves the actual file and document state		Output Screenshots

  shortcuts.spec.js • 1 failed test

View Output Video

Test		Artifacts
keyboard shortcuts > strikethrough		Output Screenshots

  nodes/Links.spec.js • 1 failed test

View Output Video

Test		Artifacts
test link marks > autolink > with protocol but without space		Output Screenshots

  attachments.spec.js • 1 failed test

View Output Video

Test		Artifacts
Test all attachment insertion methods > Insert an image file from Files		Output Screenshots

The first 5 failed specs are shown, see all 13 specs in Cypress Cloud.

  workspace.spec.js • 7 flaky tests

View Output Video

Test		Artifacts
Workspace > Hides the workspace when switching to another folder		Output Screenshots
Workspace > formats text		Output Screenshots
Workspace > creates lists		Output Screenshots
Workspace > callouts > toggle callouts		Output Screenshots
Workspace > create Readme.md > click		Output Screenshots
Workspace > create Readme.md > enter		Output Screenshots
Workspace > create Readme.md > spacebar		Output Screenshots

  shortcuts.spec.js • 2 flaky tests

View Output Video

Test		Artifacts
keyboard shortcuts > bold		Output Screenshots
keyboard shortcuts > italic		Output Screenshots

  nodes/Links.spec.js • 2 flaky tests

View Output Video

Test		Artifacts
test link marks > link preview > shows a link preview		Output Screenshots
test link marks > link preview > does not show a link preview for links within a paragraph		Output Screenshots

  share.spec.js • 1 flaky test

View Output Video

Test		Artifacts
Open test.md in viewer > Shares the file as a public read only link		Output Screenshots

  sections.spec.js • 1 flaky test

View Output Video

Test		Artifacts
Content Sections > Heading anchors > Can change heading level		Output Screenshots

The first 5 flaky specs are shown, see all 13 specs in Cypress Cloud.

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

[max-nextcloud]

Cypress failures cannot be related - but I wonder where the block merges during freezes is coming from...

And i wonder why i could still merge this.

[juliusknorr]

Merge freeze seems like I overseen a change when updating the actions to adapt the server branch when text is using main:

[juliusknorr]

Cypress failure looks a bit like it could be a upstream issue with loading the files app:

https://cloud.cypress.io/projects/hx9gqy/runs/e149dd86-8742-4946-b5fb-7e5277760429/test-results/45605c10-1ad8-4149-84fa-1fc611744854/screenshots

[juliusknorr]

Pushed a fix for the merge block #4243

[juliusknorr]

/backport to stable27

[juliusknorr]

/backport to stable26