Paths

Table of Contentst

Make file handles have "/Fxxx" as the URI
ClosedPublic
Actions

Authored by epriestley on Aug 21 2014, 7:16 PM.

Tags

None

Referenced Files

	F13280781: D10330.diff
	Sun, Jun 2, 9:18 AM

	F13267469: D10330.diff
	Wed, May 29, 3:26 AM

	F13266106: D10330.id24870.diff
	Tue, May 28, 9:13 AM

	F13252911: D10330.diff
	Sat, May 25, 2:12 AM

	F13231225: D10330.diff
	Tue, May 21, 12:17 AM

	F13229724: D10330.id.diff
	Mon, May 20, 6:45 PM

	F13219727: D10330.id24871.diff
	Sat, May 18, 8:20 PM

	F13219172: D10330.diff
	Sat, May 18, 4:04 PM

View All Files

Subscribers

epriestley

Details

Reviewers

btrahan

Commits

Restricted Diffusion Commit
rP79c837d505c3: Make file handles have "/Fxxx" as the URI

Summary

Primarily, this fixes searching for F123 in global search.

The info URI is now a better URI than the "best" URI for files, and doesn't have redirect issues.

Test Plan

Searched for F123 in global search.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 24870.Aug 21 2014, 7:16 PM

epriestley retitled this revision from to Make file handles have "/Fxxx" as the URI.

epriestley updated this object.

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

epriestley added a reviewer: btrahan.

Herald added a subscriber: epriestley. · View Herald TranscriptAug 21 2014, 7:16 PM

Harbormaster completed remote builds in B2321: Diff 24870.Aug 21 2014, 7:17 PM

Thanks!

This revision is now accepted and ready to land.Aug 21 2014, 7:18 PM

Closed by commit rP79c837d505c3 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

files/

phid/

PhabricatorFileFilePHIDType.php

2 lines

Diff 24871

View Options

src/applications/files/phid/PhabricatorFileFilePHIDType.php

Make file handles have "/Fxxx" as the URIClosedPublicActions