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

	F3748681: D25756.1745781101.diff
	Sat, Apr 26, 19:11

	F3731218: D25756.1745737517.diff
	Sat, Apr 26, 07:05

	F3717059: D25756.1745692381.diff
	Fri, Apr 25, 18:33

	F3701739: D25756.1745605578.diff
	Thu, Apr 24, 18:26

	F3675498: D25756.1745512441.diff
	Wed, Apr 23, 16:34

	F3604420: D25756.1745194874.diff
	Sun, Apr 20, 00:21

	F3526566: D25756.1744828964.diff
	Tue, Apr 15, 18:42

	F3387497: D25756.1744437298.diff
	Fri, Apr 11, 05:54

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

Branch

fix-pattern-search (branched from master)

Lint

Lint Passed

Unit

Tests Skipped

Build Status

Buildable 1475
Build 1475: arc lint + arc unit

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 2297

View Options

src/applications/diffusion/conduit/DiffusionSearchQueryConduitAPIMethod.php

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