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
F13278041: D21719.id51743.diff
Fri, May 31, 6:09 PM
F13278040: D21719.id51756.diff
Fri, May 31, 6:09 PM
F13270141: D21719.diff
Wed, May 29, 9:05 AM
F13245379: D21719.diff
Thu, May 23, 5:58 AM
F13242026: D21719.id.diff
Thu, May 23, 1:32 AM
F13241984: D21719.diff
Thu, May 23, 1:18 AM
F13223420: D21719.diff
Sun, May 19, 4:35 AM
F13205584: D21719.diff
Wed, May 15, 2:10 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