fix PHP8 array_slice($results, null) which cause diffusion pattern search broken
ClosedPublic
Actions

Authored by gemini133 on Jul 30 2024, 01:53.

Tags

None

Referenced Files

	F2981935: D25756.1739943095.diff
	Tue, Feb 18, 05:31

	F2949766: D25756.1738671580.diff
	Mon, Feb 3, 12:19

	F2932578: D25756.1737740104.diff
	Thu, Jan 23, 17:35

	F2932577: D25756.1737740104.diff
	Thu, Jan 23, 17:35

	F2932518: D25756.1737737838.diff
	Thu, Jan 23, 16:57

	F2904830: D25756.1737334815.diff
	Jan 19 2025, 01:00

	F2894840: D25756.1737232476.diff
	Jan 17 2025, 20:34

	F2871086: D25756.1736838048.diff
	Jan 13 2025, 07:00

Subscribers

Details

Reviewers

valerio.bozzolan

Group Reviewers

O1: Blessed Committers

Commits

rP721d5ed64b70: fix PHP8 array_slice($results, null) which cause diffusion pattern search broken

Test Plan

check diffusion pattern search if working

Diff Detail

Repository

rP Phorge

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

gemini133 created this revision.Jul 30 2024, 01:53

Owners added a reviewer: O1: Blessed Committers.Jul 30 2024, 01:53

Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan, tobiaswiese. · View Herald TranscriptJul 30 2024, 01:53

gemini133 requested review of this revision.Jul 30 2024, 01:53

Harbormaster completed remote builds in B1475: Diff 2297.Jul 30 2024, 01:53

follow @valerio.bozzolan what suggested at https://we.phorge.it/D25750#20386.
we barely used pattern search but it indeed was broken.
with this fix, it works.

will submit another diff resolving the strlen problem accroding to @valerio.bozzolan

sgtm

This revision is now accepted and ready to land.Jul 30 2024, 06:36

Closed by commit rP721d5ed64b70: fix PHP8 array_slice($results, null) which cause diffusion pattern search broken. · Explain WhyJul 31 2024, 01:51

This revision was automatically updated to reflect the committed changes.

gemini133 added a commit: rP721d5ed64b70: fix PHP8 array_slice($results, null) which cause diffusion pattern search broken.

Revision Contents
Changeset List

Path

Size

Packages

src/

applications/

diffusion/

conduit/

DiffusionSearchQueryConduitAPIMethod.php

2 lines

O1: Blessed Committers

Diff 2303

View Options

src/applications/diffusion/conduit/DiffusionSearchQueryConduitAPIMethod.php

fix PHP8 array_slice($results, null) which cause diffusion pattern search brokenClosedPublicActions