Summary - Inspection of "Add Role-Based Access Control to the tool" - enwikipedia-acc/waca - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

by Simon

created 2017-06-10 22:31 UTC

1 failure condition met `issues.severity(>= MAJOR).new.exists` File Path Number of issues includes/Pages/PageUserManagement.php 2
	33 issues (2 major, 8 minor, 23 informational) were introduced. 21 issues were fixed.
	76 new patches (Unused Use Statements, Doc Comments, Indentation, Spacing, Braces) are available.
	58 added classes/operations
		D	PageUserManagement::editRoles() added
		D	SecurityManager::findResult() added
		C	SecurityManager::flattenRoles() added
		C	MigrateToRoles::execute() added
		C	SecurityManager::getActiveRoles() added
		A	UserRole::save() added
		B	NavigationMenuAccessControl::setupNavMenuAccess() added
		A	UserRole::getUser() added
		A	UserRole::setUser() added
		A	UserRole::getRole() added
C	↗	B	SecurityManager::allows() improved
B	↘	C	RequestData::isAllowedPrivateData() got worse
B	↘	C	LogHelper::prepareLogsForTemplate() got worse
A	↘	B	PageOAuth::detach() got worse
B	↗	A	PageCustomClose::showCustomCloseForm() improved
A	↘	B	StatsInactiveUsers::main() got worse
A	↘	B	StatsUsers::main() got worse
B	↗	A	RequestData::setupPrivateData() improved
A	↘	B	TemplateOutput::setUpSmarty() got worse
B	↗	A	PageUserManagement::main() improved
view all

8.92

▼ -0.03

Test Coverage

▶ ±0%

Head	Simon Walker 2017-06-10 22:31 GMT+0000 Merge a48b68f72ed13221d3266728b7c7ca1487c91743 into c888b45782b8f6cd09c5a3bd18e7751f421d7988
⋮	12 more commits
Base	DatGuy1 2017-06-03 20:55 UTC Switch username and IP sections

enwikipedia-acc / waca