aboutsummaryrefslogtreecommitdiffstats
path: root/lib/private/App/CodeChecker
Commit message (Collapse)AuthorAgeFilesLines
* Make the app code checker a NOOPChristoph Wurst2021-02-1112-1559/+0
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Update all license headers for Nextcloud 21Christoph Wurst2020-12-161-0/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Format code to a single space around binary operatorsChristoph Wurst2020-10-052-10/+10
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Update license headers for 19Christoph Wurst2020-04-292-1/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Add visibility to all constantsChristoph Wurst2020-04-101-8/+8
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Format control structures, classes, methods and functionChristoph Wurst2020-04-102-11/+7
| | | | | | | | | | | | | | | To continue this formatting madness, here's a tiny patch that adds unified formatting for control structures like if and loops as well as classes, their methods and anonymous functions. This basically forces the constructs to start on the same line. This is not exactly what PSR2 wants, but I think we can have a few exceptions with "our" style. The starting of braces on the same line is pracrically standard for our code. This also removes and empty lines from method/function bodies at the beginning and end. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Use elseif instead of else ifChristoph Wurst2020-04-101-7/+7
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Unify function spacing to PSR2 recommendationChristoph Wurst2020-04-091-2/+2
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Remove unused importsChristoph Wurst2020-03-251-2/+0
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Only parse php7 code in app code checkerChristoph Wurst2019-12-181-1/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Update license headersChristoph Wurst2019-12-0512-12/+19
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Some php-cs fixesRoeland Jago Douma2019-11-222-0/+2
| | | | | | | | | | | * Order the imports * No leading slash on imports * Empty line before namespace * One line per import * Empty after imports * Emmpty line at bottom of file Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Make sure they keys are uniqueRoeland Jago Douma2019-08-281-1/+0
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Codechecker: removed unused useRoeland Jago Douma2019-08-191-1/+1
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Adjust deprecation testsMorris Jobke2019-07-221-0/+10
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Skip check if CONSTANT on real object is usedDaniel Kesselberg2019-04-011-1/+1
| | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* Update PHP-Parser for 7.1 compatibilityJoas Schilling2019-02-141-2/+2
| | | | | Signed-off-by: Joas Schilling <coding@schilljs.com> Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Fix app code checker to ignore `build` directoryChristoph Wurst2018-08-091-1/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Deprecate OCP\FilesMorris Jobke2018-03-211-0/+8
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Remove unused and deprecated class and interface AppHelperMorris Jobke2018-03-121-1/+0
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Use normal header() calls instead of private method callsMorris Jobke2018-02-191-0/+2
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Validate the info.xml against the appstore schema fileJoas Schilling2018-02-161-91/+58
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Add deprecation checksMorris Jobke2018-01-311-0/+8
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Merge pull request #8064 from nextcloud/background-jobsMorris Jobke2018-01-261-0/+3
|\ | | | | Deprecated OCP interface to fetch background job type
| * Deprecated OCP interface to fetch background job typeMorris Jobke2018-01-261-0/+3
| | | | | | | | | | | | | | * was not used by apps and also is not needed * migrated the documentation to IJobList Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Implements are not needed because they are already done by base classMorris Jobke2018-01-252-2/+2
|/ | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Add OCP\User deprecations to app code checkerMorris Jobke2018-01-161-0/+4
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Merge pull request #7833 from nextcloud/fix-code-checkerChristoph Wurst2018-01-141-4/+5
|\ | | | | Fix casting in app code checker
| * Fix casting in app code checkerMorris Jobke2018-01-121-4/+5
| | | | | | | | | | | | Found while adding strict typing for PHP7+. This adds type hinting to all methods in CodeChecker.php and fixes one implicit cast. Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Merge pull request #7840 from nextcloud/dep_configRoeland Jago Douma2018-01-141-1/+0
|\ \ | | | | | | Remove deprecated \OCP\Config
| * | Remove deprecated \OCP\ConfigRoeland Jago Douma2018-01-131-1/+0
| |/ | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* / Add casts to XML parser in DatabaseSchemaCheckerMorris Jobke2018-01-121-15/+15
|/ | | | | | Preparation for strict_typing for PHP7+. Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Update license headersMorris Jobke2017-11-062-0/+2
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Fix MigrationSchemaChecker and CryptoWrapperLukas Reschke2017-08-011-0/+6
| | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Check the migration files for table, column and index length errorsJoas Schilling2017-07-052-2/+212
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Deprecate static writeLog() methodMorris Jobke2017-06-231-0/+1
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Version and dependency are now requiredJoas Schilling2017-05-221-31/+7
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* We don't use the prefix on index namesJoas Schilling2017-05-161-6/+4
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Fix apps without translationsJoas Schilling2017-05-161-0/+4
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Check the name length of database itemsJoas Schilling2017-05-161-0/+107
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Check the syntax of the language fileJoas Schilling2017-05-161-0/+56
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Remove unused use statementsMorris Jobke2017-04-221-2/+0
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Check for nextcloud version as wellLukas Reschke2016-12-051-2/+8
| | | | | | The code checker didn't consider the "nextcloud" version before, resulting in fails such as https://travis-ci.org/nextcloud/richdocuments/jobs/181470760#L415-L416 Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Cleanup usagesRoeland Jago Douma2016-10-291-7/+0
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Allow to validate the password_policy appJoas Schilling2016-09-011-3/+9
|
* Fix "Undefined index" when the values do not existJoas Schilling2016-08-181-2/+2
|
* Fix othersJoas Schilling2016-07-219-18/+27
|
* Move \OC\App to PSR-4Roeland Jago Douma2016-04-139-0/+1183