Page MenuHomePhabricator
Differential D21719

Fix searching legalpad documents by contributors
ClosedPublic
Actions

Authored by cspeckmim on Sep 4 2021, 5:48 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 29, 5:22 PM
Unknown Object (File)
Apr 14 2025, 11:44 PM
Unknown Object (File)
Apr 11 2025, 8:06 AM
Unknown Object (File)
Apr 7 2025, 9:15 AM
Unknown Object (File)
Apr 4 2025, 9:01 AM
Unknown Object (File)
Apr 3 2025, 4:01 AM
Unknown Object (File)
Mar 30 2025, 4:12 AM
Unknown Object (File)
Mar 23 2025, 11:46 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
rP3b2868e15553: Fix searching legalpad documents by contributors
Summary

Found in a change submitted to Phorge (https://we.phorge.it/D25018#inline-231), this fixes a typo in populating search the contributorPHIDs constraint in the Legalpad search engine.

Currently when trying to search legalpad documents by contributor an error is encountered:

Array for %Ls conversion is empty. Query: contributor.dst IN (%Ls)
Test Plan

I searched for legalpad documents based on a contributor and got back correct results.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

cspeckmim created this revision.Sep 4 2021, 5:48 PM
cspeckmim held this revision as a draft.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptSep 4 2021, 5:48 PM
Herald added a subscriber: epriestley. · View Herald Transcript
cspeckmim edited the summary of this revision. (Show Details)Sep 4 2021, 5:49 PM
Harbormaster completed remote builds in B25541: Diff 51743.Sep 4 2021, 5:49 PM
cspeckmim published this revision for review.Sep 4 2021, 11:10 PM
cspeckmim edited the summary of this revision. (Show Details)
cspeckmim edited the test plan for this revision. (Show Details)

I didn't make a task for this as it's a fairly small change and the reproduction/test case is also fairly simple.

cspeckmim retitled this revision from Fix typo in legalpad search engine to Fix searching legalpad documents by contributors.Sep 4 2021, 11:51 PM
epriestley accepted this revision.Sep 5 2021, 12:17 AM

Thanks!

This revision is now accepted and ready to land.Sep 5 2021, 12:17 AM
Herald added a subscriber: Korvin. · View Herald TranscriptSep 5 2021, 12:17 AM
Closed by commit rP3b2868e15553: Fix searching legalpad documents by contributors (authored by cspeckmim). · Explain WhySep 5 2021, 12:37 AM
This revision was automatically updated to reflect the committed changes.
cspeckmim added a commit: rP3b2868e15553: Fix searching legalpad documents by contributors.

Revision Contents
Changeset List

PathSize
src/
applications/
legalpad/
query/
2 lines

Diff 51756

View Options

src/applications/legalpad/query/LegalpadDocumentSearchEngine.php

Loading...
Phabricator · Built by Phacility