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

	F3341534: D25756.1743653888.diff
	Wed, Apr 2, 04:18

	F3336967: D25756.1743581934.diff
	Tue, Apr 1, 08:18

	F3335153: D25756.1743558834.diff
	Tue, Apr 1, 01:53

	F3335068: D25756.1743558051.diff
	Tue, Apr 1, 01:40

	F3333529: D25756.1743536547.diff
	Mon, Mar 31, 19:42

	F3321570: D25756.1743356495.diff
	Sat, Mar 29, 17:41

	F3319332: D25756.1743305313.diff
	Sat, Mar 29, 03:28

	F3305179: D25756.1743112055.diff
	Wed, Mar 26, 21:47

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