Differential D10818

Phriction - sever remaining project ties
ClosedPublic
Actions

Authored by btrahan on Nov 8 2014, 2:11 AM.

Tags

None

Referenced Files

	F15410085: D10818.diff
	Wed, Mar 19, 6:45 AM

	F15402035: D10818.id25969.diff
	Mon, Mar 17, 8:14 PM

	F15383563: D10818.diff
	Fri, Mar 14, 5:10 PM

	Unknown Object (File)
	Tue, Feb 18, 8:35 AM

	Unknown Object (File)
	Feb 8 2025, 12:02 AM

	Unknown Object (File)
	Feb 8 2025, 12:02 AM

	Unknown Object (File)
	Feb 8 2025, 12:02 AM

	Unknown Object (File)
	Feb 8 2025, 12:02 AM

Subscribers

Details

Reviewers

Maniphest Tasks

T6497: PhrictionDocumentQuery still has a project-respecting hunk of code

Commits

Restricted Diffusion Commit
rPb655699a6cfb: Phriction - sever remaining project ties

Summary

'cuz the wiki don't play that no more. Fixes T6497. This is mainly important to fix an incorrect policy filtering issue where a project policy can incorrectly override a document policy. Otherwise, it makes things nice and clean.

Test Plan

viewed the wiki - success.
viewed wiki document list under "index" and tried a few different queries
grep'd for things like "hasProject" and "getProject" and saw no phriction-related results

Diff Detail

Repository

Branch

T6497

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 2996
Build 3000: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

btrahan updated this revision to Diff 25967.Nov 8 2014, 2:11 AM

btrahan added a task: T6497: PhrictionDocumentQuery still has a project-respecting hunk of code .

btrahan retitled this revision from to Phriction - sever remaining project ties.

btrahan updated this object.

btrahan edited the test plan for this revision. (Show Details)

btrahan added a reviewer: epriestley.

Herald added subscribers: epriestley, Korvin. · View Herald TranscriptNov 8 2014, 2:11 AM

Harbormaster completed remote builds in B2996: Diff 25967.Nov 8 2014, 2:12 AM

btrahan edited the test plan for this revision. (Show Details)Nov 8 2014, 2:12 AM

Sweet.

This revision is now accepted and ready to land.Nov 8 2014, 3:02 AM

Closed by commit rPb655699a6cfb: Phriction - sever remaining project ties (authored by btrahan, committed by btrahan). · Explain WhyNov 8 2014, 6:34 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

phriction/

query/

PhrictionDocumentQuery.php

32 lines

PhrictionSearchEngine.php

8 lines

storage/

PhrictionDocument.php

14 lines

Diff 25967

src/applications/phriction/query/PhrictionDocumentQuery.php

Loading...

src/applications/phriction/query/PhrictionSearchEngine.php

Loading...

src/applications/phriction/storage/PhrictionDocument.php

Loading...