Page MenuHomePhabricator
Differential D11295

T6594, Logged out users should see "View Raw" in dropdown of timeline items
ClosedPublic
Actions

Authored by lpriestley on Jan 9 2015, 1:26 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 25, 2:55 PM
Unknown Object (File)
Sat, Dec 21, 6:16 AM
Unknown Object (File)
Sun, Dec 15, 5:16 PM
Unknown Object (File)
Sat, Dec 7, 11:40 AM
Unknown Object (File)
Fri, Dec 6, 1:57 PM
Unknown Object (File)
Wed, Dec 4, 1:14 AM
Unknown Object (File)
Tue, Dec 3, 4:34 PM
Unknown Object (File)
Tue, Dec 3, 2:17 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T6594: Logged-out users can not access "View Raw" in comment dropdown menus
Commits
Restricted Diffusion Commit
rP7a6f4ab75a1f: T6594, Logged out users should see "View Raw" in dropdown of timeline items
Summary

Fixes T6594, Logged out users should be able to "View Raw" comments in public objects.

Test Plan

Log out, open maniphest task with comments, open dropdown associated with comment, click "View Raw", should be able to see raw comment.

Diff Detail

Repository
rP Phabricator
Branch
loggedoutviewrawtimelineitem
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 3709
Build 3720: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

lpriestley updated this revision to Diff 27137.Jan 9 2015, 1:26 AM
lpriestley retitled this revision from to T6594, Logged out users should see "View Raw" in dropdown of timeline items.
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.
lpriestley added a task: T6594: Logged-out users can not access "View Raw" in comment dropdown menus.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJan 9 2015, 1:26 AM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
epriestley accepted this revision.Jan 9 2015, 2:47 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jan 9 2015, 2:47 PM
Closed by commit rP7a6f4ab75a1f: T6594, Logged out users should see "View Raw" in dropdown of timeline items (authored by lpriestley, committed by epriestley). · Explain WhyJan 9 2015, 2:47 PM
This revision was automatically updated to reflect the committed changes.
btrahan mentioned this in T7232: View Raw on removed comments makes 400 bad request.Feb 11 2015, 10:14 PM

Revision Contents
Changeset List

PathSize
src/
applications/
transactions/
controller/
4 lines
view/
2 lines
view/
phui/
13 lines

Diff 27137

View Options

src/applications/transactions/controller/PhabricatorApplicationTransactionCommentRawController.php

Loading...
View Options

src/applications/transactions/view/PhabricatorApplicationTransactionView.php

Loading...
View Options

src/view/phui/PHUITimelineEventView.php

Loading...
Phabricator · Built by Phacility