From 7bfbc46a635ca0872464c4aefd6093bebe89aac1 Mon Sep 17 00:00:00 2001 From: Bob Trahan Date: Tue, 12 May 2015 14:44:47 -0700 Subject: [PATCH] Quicksand - add phame live URLs to quicksand blacklist Summary: Fixes T8160. AFAIK this is the only route pattern that needs blacklisting. Double checked that the resource controller is good to go; it is because its a celerity resource controller descendant and returns data differently than normal controllers Test Plan: Clicked "view live" on a block. Read a few posts. Clicked into a post and read it. Clicked an image and it linked to the image. Reviewers: epriestley Reviewed By: epriestley Subscribers: Korvin, epriestley Maniphest Tasks: T8160 Differential Revision: https://secure.phabricator.com/D12817 --- .../phame/application/PhabricatorPhameApplication.php | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/applications/phame/application/PhabricatorPhameApplication.php b/src/applications/phame/application/PhabricatorPhameApplication.php index b466ad5191..4d8cebffa0 100644 --- a/src/applications/phame/application/PhabricatorPhameApplication.php +++ b/src/applications/phame/application/PhabricatorPhameApplication.php @@ -69,4 +69,10 @@ final class PhabricatorPhameApplication extends PhabricatorApplication { ); } + public function getQuicksandURIPatternBlacklist() { + return array( + '/phame/live/.*', + ); + } + }