Paths

Table of Contentst

Fix chatlog application query integration
ClosedPublic
Actions

Authored by epriestley on Oct 22 2013, 6:41 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Sep 5, 7:38 AM

	Unknown Object (File)
	Wed, Sep 4, 9:27 AM

	Unknown Object (File)
	Tue, Aug 27, 12:56 PM

	Unknown Object (File)
	Mon, Aug 26, 2:33 AM

	Unknown Object (File)
	Sat, Aug 24, 12:56 PM

	Unknown Object (File)
	Sat, Aug 24, 6:40 AM

	Unknown Object (File)
	Fri, Aug 23, 3:05 AM

	Unknown Object (File)
	Aug 19 2024, 10:49 AM

View All Files

Subscribers

aran

Details

Reviewers

btrahan

Commits

Restricted Diffusion Commit
rP7dd31a16d973: Fix chatlog application query integration

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.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

btrahan accepted this revision.Oct 22 2013, 8:39 PM

Closed by commit rP7dd31a16d973 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

chatlog/

PhabricatorChatLogChannelQuery.php

2 lines

PhabricatorChatLogQuery.php

2 lines

policy/

__tests__/

PhabricatorPolicyTestCase.php

2 lines

Diff 16621

View Options

src/applications/chatlog/PhabricatorChatLogChannelQuery.php

View Options

src/applications/chatlog/PhabricatorChatLogQuery.php

View Options

src/applications/policy/tests/PhabricatorPolicyTestCase.php

Fix chatlog application query integrationClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 16621

src/applications/chatlog/PhabricatorChatLogChannelQuery.php

src/applications/chatlog/PhabricatorChatLogQuery.php

src/applications/policy/__tests__/PhabricatorPolicyTestCase.php

Fix chatlog application query integration
ClosedPublic
Actions

Revision Contents
Changeset List

src/applications/policy/tests/PhabricatorPolicyTestCase.php