Fixed namespace issue
ClosedPublic
Actions

Authored by dereckson on Jan 1 2016, 00:04.

Tags

None

Referenced Files

	F10328433: D144.id.diff
	Sun, Jun 29, 00:27

	Unknown Object (File)
	Sat, Jun 28, 12:21

	Unknown Object (File)
	Sat, Jun 28, 08:23

	Unknown Object (File)
	Sat, Jun 28, 04:54

	Unknown Object (File)
	Fri, Jun 27, 11:44

	Unknown Object (File)
	Fri, Jun 27, 08:51

	Unknown Object (File)
	Thu, Jun 12, 20:37

	Unknown Object (File)
	Thu, Jun 12, 20:24

View All Files

Subscribers

None

Details

Reviewers

dereckson

Commits

Unknown Object (Diffusion Commit)
rNOTIFdd075a696bcd: Fixed namespace issue

Summary

PhabricatorAPIException inherits from \RuntimeException,
not from Nasqueron\Notifications\Phabricator\RuntimeException.

This issue has been noticed running phpDox:

[phpdox] [31.12.2015 - 23:48:38] -
Nasqueron\Notifications\Phabricator\PhabricatorAPIException
(missing Nasqueron\Notifications\Phabricator\RuntimeException)

Test Plan

trivial

Diff Detail

Repository

rNOTIF Notifications center

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson updated this revision to Diff 344.Jan 1 2016, 00:04

dereckson retitled this revision from to Fixed namespace issue.

dereckson updated this object.

dereckson edited the test plan for this revision. (Show Details)

dereckson added a reviewer: dereckson.

dereckson accepted this revision.Jan 1 2016, 00:05

dereckson edited edge metadata.

This revision is now accepted and ready to land.Jan 1 2016, 00:05

Closed by commit rNOTIFdd075a696bcd: Fixed namespace issue (authored by dereckson, committed by dereckson). · Explain WhyJan 1 2016, 00:05

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

app/

Phabricator/

PhabricatorAPIException.php

2 lines

Diff 345

View Options

app/Phabricator/PhabricatorAPIException.php

Fixed namespace issueClosedPublicActions