revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2025-01-01 18:30:59 +01:00
Code Issues Releases Wiki Activity

Fix chatlog application query integration

Browse source 
Summary: `class_exists()` is case-insensitive, but `PhabricatorApplication::getByClass()` is not.

Test Plan: Fixed unit test to fail, then fixed code to pass unit test.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Differential Revision: https://secure.phabricator.com/D7379
This commit is contained in:
epriestley 2013-10-22 13:47:47 -07:00
parent e14ba56394
commit 7dd31a16d9
3 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/applications/chatlog/PhabricatorChatLogChannelQuery.php
View file

@ -57,7 +57,7 @@ final class PhabricatorChatLogChannelQuery
} }
public function getQueryApplicationClass() { public function getQueryApplicationClass() {
return 'PhabricatorApplicationChatlog'; return 'PhabricatorApplicationChatLog';
} }
} }

2
src/applications/chatlog/PhabricatorChatLogQuery.php
View file

@ -56,7 +56,7 @@ final class PhabricatorChatLogQuery
} }
public function getQueryApplicationClass() { public function getQueryApplicationClass() {
return 'PhabricatorApplicationChatlog'; return 'PhabricatorApplicationChatLog';
} }
} }

2
src/applications/policy/__tests__/PhabricatorPolicyTestCase.php
View file

@ -222,7 +222,7 @@ final class PhabricatorPolicyTestCase extends PhabricatorTestCase {
} }
$this->assertEqual( $this->assertEqual(
true, true,
class_exists($class), (bool)PhabricatorApplication::getByClass($class),
"Application class '{$class}' for query '{$qclass}'"); "Application class '{$class}' for query '{$qclass}'");
} }
} }