summaryrefslogtreecommitdiffstats
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #7918 from nextcloud/properly-log-exceptionsRoeland Jago Douma2018-01-2314-59/+141
|\ | | | | Properly log the full exception instead of only the message
| * Properly log the full exception instead of only the messageMorris Jobke2018-01-2314-59/+141
| | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Merge pull request #7858 from nextcloud/fix-local-storageMorris Jobke2018-01-233-7/+15
|\ \ | | | | | | Use fallback path if data dir is not available for Storage/Local.php
| * | Use fallback path if data dir is not available for Storage/Local.phpMorris Jobke2018-01-183-7/+15
| | | | | | | | | | | | | | | | | | | | | Found while testing strict types for PHP7+. Signed-off-by: Morris Jobke <hey@morrisjobke.de> Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | Merge pull request #7989 from nextcloud/cleanup-unused-share-methodsMorris Jobke2018-01-231-107/+0
|\ \ \ | | | | | | | | Remove not used methods of legacy share.php
| * | | Remove not used methods of legacy share.phpMorris Jobke2018-01-221-107/+0
| | | | | | | | | | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | | Merge pull request #7354 from nextcloud/refacor-update-page-printblizzz2018-01-221-32/+19
|\ \ \ \ | | | | | | | | | | Refactor method to check if update is needed
| * | | | Reuse systemconfigMorris Jobke2017-11-301-6/+5
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
| * | | | Refactor method to check if update is neededMorris Jobke2017-11-301-27/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There was only one call, that actually needed the parameter to be set to true. So this change moved the print of the page to that location and replaces all other occurences with a direct call to the underlying OCP API. Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | | | Merge pull request #7988 from nextcloud/cleanup-old-repair-stepsRoeland Jago Douma2018-01-2211-743/+0
|\ \ \ \ \ | |_|_|_|/ |/| | | | Remove old repair steps that aren't executed anymore
| * | | | Remove old repair steps that aren't executed anymoreMorris Jobke2018-01-2211-743/+0
| | |/ / | |/| | | | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* / | | If the preview is size 0 it is invalidRoeland Jago Douma2018-01-221-3/+13
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | * delete it * throw a NotFound Exception - This should a proper 404 to the user - Next time it is then regenerated Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | [tx-robot] updated from transifexNextcloud bot2018-01-2132-30/+32
| | |
* | | Disable vendor detection when upgrading from 11.0.xMorris Jobke2018-01-201-10/+2
| | | | | | | | | | | | | | | | | | Upgrade from 11 to 14 is not supported anyways. Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | Remove not needed 3rdparty app disabling during upgrade for PHP 5.xMorris Jobke2018-01-191-29/+0
| | | | | | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | Do not show "Not installed warning" during occ installMorris Jobke2018-01-181-1/+1
| | | | | | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | Merge pull request #7911 from nextcloud/improve-oc_imageJulius Härtl2018-01-184-40/+38
|\ \ \ | | | | | | | | Improve OC_Image code to not guess the type of input
| * | | Improve OC_Image code to not guess the type of input, but actually request ↵Morris Jobke2018-01-184-40/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the specific methods to be called Followup to #7836 Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | | Remove IAppConfig::setValueRoeland Jago Douma2018-01-174-18/+4
| | | | | | | | | | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | Remove IAppConfig::getValueRoeland Jago Douma2018-01-179-34/+18
| | | | | | | | | | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | Simple IAppConfig deprecationsRoeland Jago Douma2018-01-173-39/+3
| | | | | | | | | | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | Merge pull request #7914 from nextcloud/fix-7254Roeland Jago Douma2018-01-173-4/+20
|\ \ \ \ | | | | | | | | | | format self-mentions, but don't offer them
| * | | | add types to php docArthur Schiwon2018-01-171-4/+4
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * | | | do not offer the handle of the current user for auto completionArthur Schiwon2018-01-172-0/+20
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * | | | comments should compile mentions also if done by authorArthur Schiwon2018-01-171-4/+0
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | it is used by clients for formatting reasons, there is no reason not format the author if her handle is included in the comment body. It is unrelated to sending out notifications. Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | | | Merge pull request #7909 from nextcloud/fix-migration-type-hintsMorris Jobke2018-01-178-35/+124
|\ \ \ \ | | | | | | | | | | Fix the type hints of migrations and correctly inject the wrapped sch…
| * | | | Fix the type hints of migrations and correctly inject the wrapped schema ↵Joas Schilling2018-01-178-35/+124
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | into migrations Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | | Merge pull request #7847 from nextcloud/avatarmanager_strictRoeland Jago Douma2018-01-173-12/+16
|\ \ \ \ \ | | | | | | | | | | | | Fixes for avatarmanager
| * | | | | Use \OC\User\Manager internally in the avatarmanagerRoeland Jago Douma2018-01-142-7/+8
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
| * | | | | Made AvatarManager string and add return typesRoeland Jago Douma2018-01-142-5/+8
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | | | Merge pull request #7912 from ↵Roeland Jago Douma2018-01-171-1/+1
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | | | | | | | | | | | | | nextcloud/bugfix/noid/correctly-drop-owncloud-migrations-table Correctly drop the ownCloud migrations table
| * | | | | Correctly drop the ownCloud migrations tableJoas Schilling2018-01-171-1/+1
| | |/ / / | |/| | | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | | Merge pull request #7813 from nextcloud/getHeader-should-only-return-stringMorris Jobke2018-01-174-7/+9
|\ \ \ \ \ | | | | | | | | | | | | Request->getHeader() should always return a string
| * | | | | Request->getHeader() should always return a stringMorris Jobke2018-01-174-7/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPDoc (of the public API) says that this method returns string but it also returns null, which is not allowed in some method calls. This fixes that behaviour and returns an empty string and fixes all code paths that explicitly checked for null to be still compliant. Found while enabling the strict_typing for lib/private for the PHP7+ migration. Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | | | | Merge pull request #7836 from nextcloud/image-type-hintingMorris Jobke2018-01-174-10/+22
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Fix float/integer handling in image API
| * | | | | Fix float/integer handling in image APIMorris Jobke2018-01-174-10/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * IImage::crop/preciseResize now have type hinting for integers * found while testing strict typing for PHP 7+ Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | | | | Merge pull request #7899 from nextcloud/strict_discserviceRoeland Jago Douma2018-01-172-11/+16
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Strict DiscoveryService
| * | | | | Add extra check so we don't error out on typeRoeland Jago Douma2018-01-161-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | json_decode can return false if we have invalid data. In that case just assume there was nothing cached Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
| * | | | | Strict DiscoveryServiceRoeland Jago Douma2018-01-162-10/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Made strict * Type hints * Return types Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | | | Merge pull request #7897 from nextcloud/strict_lockingprovidersMorris Jobke2018-01-176-38/+44
|\ \ \ \ \ \ | | | | | | | | | | | | | | Made locking providers strict
| * | | | | | Made locking providers strictRoeland Jago Douma2018-01-166-38/+44
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Added typehints * Added return types * Made strict Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | | | Merge pull request #7900 from nextcloud/strict_securityMorris Jobke2018-01-175-16/+21
|\ \ \ \ \ \ | | | | | | | | | | | | | | Make OCP\Security stricter
| * | | | | | Make OCP\Security stricterRoeland Jago Douma2018-01-165-16/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add typehints * Add return types * Opcode opts from phpstorm * Made strict * Fixed tests: No need to test bogus values anymore strict typing fixes this Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | | | | Merge pull request #7901 from nextcloud/strict_cache_factoryRoeland Jago Douma2018-01-172-24/+28
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Made the cache factory strict
| * | | | | | | Made the cache factory strictRoeland Jago Douma2018-01-162-24/+28
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Return types * Typehints * made strict * fix tests Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | | | | Merge pull request #7884 from nextcloud/cleanup-oc_userMorris Jobke2018-01-175-100/+7
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Cleanup OC_User and OCP\User
| * | | | | | | Cleanup OC_User and OCP\UserMorris Jobke2018-01-165-100/+7
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * mainly removes deprecated methods and old static code Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | | | | | Merge pull request #7894 from nextcloud/strict_http_clientMorris Jobke2018-01-166-30/+38
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Make OCP\Http strict
| * | | | | | | Make OCP\Http strictRoeland Jago Douma2018-01-166-30/+38
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Handle private files * Add return types * Add scalar typehints * Made strict * Fixed requiring proper guzzle message interface that is passed around Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | | | | Merge pull request #7892 from nextcloud/cloudid_strictJulius Härtl2018-01-164-17/+21
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | Make OCP\Federation strict