Page MenuHomePhabricator
Differential D10254

Probably fix bad method call in Diffusion
ClosedPublic
Actions

Authored by epriestley on Aug 13 2014, 3:26 PM.
Tags
None
Referenced Files
F19304050: D10254.diff
Wed, Dec 24, 6:54 AM
F19149320: D10254.diff
Wed, Dec 10, 7:42 PM
F19149317: D10254.diff
Wed, Dec 10, 7:42 PM
F19050092: D10254.id24685.diff
Thu, Nov 27, 7:13 PM
F19049717: D10254.diff
Thu, Nov 27, 6:02 PM
F18850363: D10254.id.diff
Oct 30 2025, 4:08 PM
F18844204: D10254.diff
Oct 29 2025, 3:57 AM
F18703391: D10254.diff
Sep 28 2025, 3:54 AM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
T4896: Move Audit to ApplicationTransactions
T5869: Crash when viewing lint messages in Diffusion
Commits
Restricted Diffusion Commit
rPa5d24609748c: Probably fix bad method call in Diffusion
Summary

Fixes T5869. Ref T4896. This setID() method no longer exists.

Test Plan
This is a pain to reproduce locally so I'm just winging it. I'm 99% sure this ID is only used to generate an anchor link. This is a hack to start with, and T4896 will eventualy clean it up properly.

Diff Detail

Repository
rP Phabricator
Branch
setid
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 2208
Build 2212: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 24685.Aug 13 2014, 3:26 PM
epriestley retitled this revision from to Probably fix bad method call in Diffusion.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added tasks: T5869: Crash when viewing lint messages in Diffusion, T4896: Move Audit to ApplicationTransactions.
Herald added a subscriber: epriestley. · View Herald TranscriptAug 13 2014, 3:26 PM
Harbormaster completed remote builds in B2208: Diff 24685.Aug 13 2014, 3:27 PM
btrahan accepted this revision.Aug 13 2014, 4:24 PM
btrahan edited edge metadata.

whatcouldgowrong

This revision is now accepted and ready to land.Aug 13 2014, 4:24 PM
epriestley closed this revision.Aug 13 2014, 5:06 PM
epriestley updated this revision to Diff 24686.

Closed by commit rPa5d24609748c (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
controller/
1 line

Diff 24685

View Options

src/applications/diffusion/controller/DiffusionBrowseFileController.php

Loading...
Phabricator · Built by Phacility