Skip to content

Commit

Permalink
Support sentry/sentry instead of raven/raven
Browse files Browse the repository at this point in the history
  • Loading branch information
gerryd committed Jul 7, 2016
1 parent 873b724 commit b971933
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@

This library takes care of the error reporting.

It will use Whoops and Sentry_Raven if their presence can be detected and the
It will use Whoops and Sentry if their presence can be detected and the
required configuration exists, otherwise it will fall back to its own basic
error handler.

Expand All @@ -29,5 +29,5 @@ Now make it the default error handler

\Skeleton\Error\Handler::enable();

If you install the composer packages `filp/whoops` and/or `raven/raven`, the
If you install the composer packages `filp/whoops` and/or `sentry/sentry`, the
handler wil use them.
2 changes: 1 addition & 1 deletion lib/Skeleton/Error/Config.php
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ class Config {
/**
* sentry_dsn
*
* If you have Sentry Raven installed, you can supply the Sentry DSN.
* If you have sentry/sentry installed, you can supply the Sentry DSN.
*
* @access public
* @var string $sentry_dsn;
Expand Down
6 changes: 3 additions & 3 deletions lib/Skeleton/Error/Handler.php
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ public static function enable() {
public function register() {
if (!$this->is_registered) {
// Automatically use Sentry if detected
if ($this->detected_sentry_raven() and Config::$sentry_dsn !== null) {
if ($this->detected_sentry() and Config::$sentry_dsn !== null) {
$this->add_handler(new Handler\Sentry());
}

Expand Down Expand Up @@ -221,11 +221,11 @@ private function detected_whoops() {
}

/**
* Check if we have detected a Sentry Raven installation
* Check if we have detected a sentry/sentry package
*
* @return bool
*/
private function detected_sentry_raven() {
private function detected_sentry() {
if (class_exists('Raven_Client')) {
return true;
}
Expand Down

0 comments on commit b971933

Please sign in to comment.