Skip to content
This repository has been archived by the owner on May 14, 2018. It is now read-only.

Show exception details in console #163

Merged
merged 3 commits into from
Jul 5, 2013
Merged

Conversation

ftdebugger
Copy link
Contributor

When BjyAuthorize rejected request in console, zend output something like this

======================================================================
   The application has thrown an exception!
======================================================================


----------------------------------------------------------------------
:

======================================================================
   Previous Exception(s):

This PR create exception with more information. So zend output

======================================================================
   The application has thrown an exception!
======================================================================
 BjyAuthorize\Exception\UnAuthorizedException
 You are not authorized to access assetic-build
----------------------------------------------------------------------
/var/www/work/logic-by/vendor/bjyoungblood/bjy-authorize/src/BjyAuthorize/Guard/Route.php:133
#0 [internal function]: BjyAuthorize\Guard\Route->onRoute(Object(Zend\Mvc\MvcEvent))
#1 /var/www/work/logic-by/vendor/zendframework/zendframework/library/Zend/EventManager/EventManager.php(468): call_user_func(Array, Object(Zend\Mvc\MvcEvent))
#2 /var/www/work/logic-by/vendor/zendframework/zendframework/library/Zend/EventManager/EventManager.php(207): Zend\EventManager\EventManager->triggerListeners('route', Object(Zend\Mvc\MvcEvent), Object(Closure))
#3 /var/www/work/logic-by/vendor/zendframework/zendframework/library/Zend/Mvc/Application.php(290): Zend\EventManager\EventManager->trigger('route', Object(Zend\Mvc\MvcEvent), Object(Closure))
#4 /var/www/work/logic-by/public/index.php(16): Zend\Mvc\Application->run()
#5 {main}
======================================================================
   Previous Exception(s):

@Ocramius
Copy link
Contributor

@ftdebugger can you fix the failures? Also consider implementing #95 instead of this eventually (I'll allow the BC break there)

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling deebca8 on ftdebugger:master into a30b4b3 on bjyoungblood:master.

@Ocramius
Copy link
Contributor

@ftdebugger is this good to merge? If so, I'll do and tag 1.4.0-beta2

@ftdebugger
Copy link
Contributor Author

Yes, it good to merge.

Ocramius added a commit that referenced this pull request Jul 5, 2013
Show exception details in console
@Ocramius Ocramius merged commit 02723bd into bjyoungblood:master Jul 5, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants