Page MenuHomePhabricator
Differential D13759

Remove extra `/phurl` in request url
AbandonedPublic
Actions

Authored by ox on Jul 30 2015, 12:15 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 6, 3:22 PM
Unknown Object (File)
Wed, Dec 4, 2:59 AM
Unknown Object (File)
Sun, Dec 1, 8:53 AM
Unknown Object (File)
Thu, Nov 28, 1:03 AM
Unknown Object (File)
Wed, Nov 20, 12:27 PM
Unknown Object (File)
Wed, Nov 20, 12:27 PM
Unknown Object (File)
Nov 16 2024, 1:26 PM
Unknown Object (File)
Nov 12 2024, 4:05 AM
View All Files
Subscribers
chad
epriestley

Details

Reviewers
epriestley
chad
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T9009: Cannot comment on Phurl links
Summary

Should fix T9009

Test Plan

I cannot get Phabricator to run on my dev machine (OS X), but this should be correct.

Diff Detail

Repository
rP Phabricator
Branch
fix/phurl-commenting
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 7448
Build 7936: [Placeholder Plan] Wait for 30 Seconds
Build 7935: arc lint + arc unit

Event Timeline

ox updated this revision to Diff 33232.Jul 30 2015, 12:15 AM
ox retitled this revision from to Remove extra `/phurl` in request url.
ox updated this object.
ox edited the test plan for this revision. (Show Details)
ox added a task: T9009: Cannot comment on Phurl links.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJul 30 2015, 12:15 AM
Herald added a subscriber: epriestley. · View Herald Transcript
chad added a subscriber: chad.Jul 30 2015, 12:26 AM
chad added inline comments.
src/applications/phurl/controller/PhabricatorPhurlURLViewController.php
48

I think you don't need the first / either

ox updated this revision to Diff 33233.Jul 30 2015, 3:16 AM
ox marked an inline comment as done.
ox edited edge metadata.

remove leading / as it was getting trimmed anyway

chad accepted this revision.Jul 30 2015, 3:18 AM
chad added a reviewer: chad.
This revision is now accepted and ready to land.Jul 30 2015, 3:18 AM
chad added a comment.EditedJul 30 2015, 3:21 AM

Although, looking at the code, PhabricatorPhurlCommentController doesn't exist either, nor the route.

But this change is correct!

chad mentioned this in T9009: Cannot comment on Phurl links.Jul 30 2015, 3:40 AM
ox added a comment.Jul 30 2015, 2:07 PM

I don't know enough PHP to really get what this app is doing, but I imagined some smart folks at Phacility had created a generic commenting service and I was just not seeing it. Thanks for the help!

epriestley requested changes to this revision.Jul 30 2015, 2:11 PM
epriestley added a reviewer: epriestley.

I cannot get Phabricator to run on my dev machine (OS X), but this should be correct.

Let's just wait for the normal dev process here.

This revision now requires changes to proceed.Jul 30 2015, 2:11 PM
ox abandoned this revision.Aug 3 2015, 6:28 PM

Revision Contents
Changeset List

PathSize
src/
applications/
phurl/
controller/
2 lines

Diff 33232

View Options

src/applications/phurl/controller/PhabricatorPhurlURLViewController.php

Loading...
Phabricator · Built by Phacility