View GitHub Repository graphql-php

Schedule Inspection
Subscribe
Failed Conditions
Pull Request — master (#321)
by Šimon
04:18
created

7,634 +104 Covered Lines

8,303 +111 Total Lines

91.9 % +0.024% Code Coverage

PHP Classes Changes

 
 ⇢
0%
ValidationRule added
 
 ⇢
100%
QuerySecurity added
70.59%
 ↗
76.19%
 VariablesInAllowedPosition improved
81.93%
 ↘
77.01%
 DocumentValidator got worse
92.74%
 ↘
90.15%
 QueryComplexity got worse
90.14%
 ↘
88%
 KnownDirectives got worse
88.68%
 ↗
90.16%
 ValuesOfCorrectType improved
96.05%
 ↗
96.94%
 OverlappingFieldsCanBeMerged improved
97.67%
 ↗
97.92%
 NoFragmentCycles improved
0%
 ↛
 
 AbstractValidationRule removed
100%
 ↛
 
 AbstractQuerySecurity removed

CRAP Changes

 
 ⇢
18
QuerySecurity::collectFieldASTsAndDefs() added
 
 ⇢
6
ValidationRule::getName() added
 
 ⇢
3
QuerySecurity::gatherFragmentDefinition() added
 
 ⇢
2
ValidationRule::__invoke() added
 
 ⇢
2
QuerySecurity::invokeIfNeeded() added
 
 ⇢
2
QuerySecurity::checkIfGreaterOrEqualToZero() added
 
 ⇢
2
QuerySecurity::getFieldName() added
 
 ⇢
1
QuerySecurity::getFragments() added
 
 ⇢
1
QuerySecurity::getFragment() added
11.0036
 ↗
11.004
 OverlappingFieldsCanBeMerged::internalCollectField... got worse
7.0046
 ↗
7.0155
 OverlappingFieldsCanBeMerged::collectConflictsBetw... got worse
5.0342
 ↗
5.0812
 OverlappingFieldsCanBeMerged::collectConflictsBetw... got worse
7.059
 ↘
7
 OverlappingFieldsCanBeMerged::findConflictsBetween... improved
3.0987
 ↗
3.243
 QueryComplexity::buildFieldArguments() got worse
26.4451
 ↗
26.7173
 KnownDirectives::getDirectiveLocationForASTPath() got worse
5.3374
 ↗
5.7044
 QueryComplexity::directiveExcludesField() got worse
8.0023
 ↘
7.0014
 NoFragmentCycles::detectCycleRecursive() improved
18
 ↛
 
 AbstractQuerySecurity::collectFieldASTsAndDefs() removed
6
 ↛
 
 AbstractValidationRule::getName() removed
3
 ↛
 
 AbstractQuerySecurity::gatherFragmentDefinition() removed
2
 ↛
 
 AbstractQuerySecurity::checkIfGreaterOrEqualToZero... removed
2
 ↛
 
 AbstractValidationRule::__invoke() removed
2
 ↛
 
 AbstractQuerySecurity::getFieldName() removed
2
 ↛
 
 AbstractQuerySecurity::invokeIfNeeded() removed
2
 ↛
 
 AbstractQuerySecurity::getFragment() removed
1
 ↛
 
 AbstractQuerySecurity::getFragments() removed