View GitHub Repository route

Schedule Inspection
Subscribe
Completed
Pull Request — master (#283)
by Phil
02:56
created

442 +37 Covered Lines

464 +41 Total Lines

95.3 % -0.5% Code Coverage

PHP Classes Changes

 
 ⇢
91.3%
CachedRouter added
 
 ⇢
100%
DefaultHeaderDecorator added
100%
 ↘
0%
 anonymous//src/Strategy/JsonStrategy.php$0 got worse
71.43%
 ↗
91.43%
 MiddlewareAwareTrait improved
100%
 ↘
90.79%
 Dispatcher got worse
93.48%
 ↗
100%
 Route improved
97.22%
 ↘
92.47%
 Router got worse

CRAP Changes

 
 ⇢
7.025
Router::prepareRoutes() added
 
 ⇢
7
Dispatcher::isExtraConditionMatch() added
 
 ⇢
6.288
Router::buildOptionsRoutes() added
 
 ⇢
6.0852
CachedRouter::buildRouter() added
 
 ⇢
4
Route::resolve() added
 
 ⇢
4
JsonStrategy::isJsonSerializable() added
 
 ⇢
3
DefaultHeaderDecorator::__invoke() added
 
 ⇢
2
DefaultHeaderDecorator::addDefaultHeaders() added
 
 ⇢
2
AbstractStrategy::decorateResponse() added
 
 ⇢
2
JsonStrategy::getOptionsCallable() added
⋮ view more  
2.0625
 ↘
2
 Route::setParentGroup() improved
2
 ↗
2.0625
 Dispatcher::ensureHandlerIsRoute() got worse
2
 ↘
1
 MiddlewareAwareTrait::lazyPrependMiddleware() improved
1
 ↗
2
 JsonStrategy.php$0 ➔ __construct() got worse
1
 ↗
2
 JsonStrategy::getNotFoundDecorator() got worse
1
 ↗
2
 JsonStrategy::getMethodNotAllowedDecorator() got worse
1
 ↗
2.0116
 Dispatcher::setNotFoundDecoratorMiddleware() got worse
1
 ↗
2.0116
 Dispatcher::setMethodNotAllowedDecoratorMiddleware... got worse
12.0654
 ↘
11
 Route::getCallable() improved
7
 ↗
8.7021
 Dispatcher::setFoundMiddleware() got worse
⋮ view more  
9
 ↛
 
 Router::prepRoutes() removed
3
 ↛
 
 AbstractStrategy::applyDefaultResponseHeaders() removed
3
 ↛
 
 Route::resolveClass() removed
3
 ↛
 
 JsonStrategy::isJsonEncodable() removed
2
 ↛
 
 AbstractStrategy::addDefaultResponseHeaders() removed
1
 ↛
 
 JsonStrategy::getExceptionHandler() removed
1
 ↛
 
 AbstractStrategy::getDefaultResponseHeaders() removed
1
 ↛
 
 ApplicationStrategy.php$0 ➔ __construct() removed
1
 ↛
 
 ApplicationStrategy::getExceptionHandler() removed
1
 ↛
 
 AbstractStrategy::addDefaultResponseHeader() removed