Paths

Table of Contentst

Differential D778

Explicitely log exception message
ClosedPublic
Actions

Authored by dereckson on Dec 30 2016, 02:11.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Mar 3, 23:24

	Unknown Object (File)
	Wed, Feb 26, 07:51

	Unknown Object (File)
	Tue, Feb 25, 11:36

	Unknown Object (File)
	Sat, Feb 22, 12:56

	Unknown Object (File)
	Sun, Feb 16, 17:39

	Unknown Object (File)
	Sat, Feb 15, 14:28

	Unknown Object (File)
	Thu, Feb 13, 05:04

	Unknown Object (File)
	Tue, Feb 11, 09:15

Subscribers

Details

Reviewers

Commits

rNOTIF905274bef2a3: Explicitely log exception message

Summary

The notifications center logs exceptions thrown's messages,
and to do so pass the exception to the log method.

As this method wants a string as parameter, it was converted as string.
But as we plan to use declare(strict_types=1), such implicit conver-
sion isn't welcome.

Fixes the following issue detected by phan:

app/Exceptions/Handler.php:49
PhanTypeMismatchArgument
Argument 1 (message) is \Exception|\Throwable but
\Psr\Log\LoggerInterface::error() takes string defined at
vendor/psr/log/Psr/Log/LoggerInterface.php:66

Test Plan

phan | grep PhanTypeMismatchArgument

Diff Detail

Repository

rNOTIF Notifications center

Lint

Lint Passed

Unit

Tests Passed

Branch

explicit-string-type (branched from master)

Event Timeline

dereckson created this revision.Dec 30 2016, 02:11

Build is green https://ci.nasqueron.org/job/test-notifications-phab/241/ for more details.

Call magic method instead of only the message.

dereckson planned changes to this revision.Dec 30 2016, 02:19

dereckson added inline comments.

app/Exceptions/Handler.php
49	http://php.net/manual/en/exception.tostring.php We're losing information here. `(string)$e` or `$e->__toString()` would allow to keep it.

dereckson accepted this revision.Dec 30 2016, 02:19

dereckson requested a review of this revision.

dereckson edited edge metadata.

dereckson marked an inline comment as done.

This revision is now accepted and ready to land.Dec 30 2016, 02:19

Closed by commit rNOTIF905274bef2a3: Explicitely log exception message (authored by dereckson, committed by dereckson). · Explain WhyDec 30 2016, 02:21

This revision was automatically updated to reflect the committed changes.

Build is green https://ci.nasqueron.org/job/test-notifications-phab/242/ for more details.

Revision Contents
Changeset List

Path

Size

app/

Exceptions/

2 lines

Diff 1969

app/Exceptions/Handler.php

Loading...