summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Micro-optimize validation of empty email addressesChristoph Wurst2021-02-112-0/+5
| | | | | | Then we don't have to construct any validators. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Merge pull request #25378 from nextcloud/bugfix/noid/mention-match-improvementsRoeland Jago Douma2021-02-114-13/+14
|\ | | | | Improve mention matches
| * Fix linterJoas Schilling2021-02-061-1/+1
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
| * Fix unit testsJoas Schilling2021-02-061-3/+3
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
| * Make Psalm happyJoas Schilling2021-02-061-1/+3
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
| * Improve mention matchesJoas Schilling2021-02-063-10/+9
| | | | | | | | | | | | The previous regex didn't correctly match users with . ' or - Signed-off-by: Joas Schilling <coding@schilljs.com>
* | [tx-robot] updated from transifexNextcloud bot2021-02-116-18/+28
| |
* | Merge pull request #25570 from nextcloud/enh/dashboard/psalm_fixesChristoph Wurst2021-02-101-14/+14
|\ \ | | | | | | Some psalm fixes for the DashboardController
| * | Some psalm fixes for the DashboardControllerRoeland Jago Douma2021-02-101-14/+14
| | | | | | | | | | | | | | | | | | | | | * Use the AppFramework IInitialState * Set return type Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | Merge pull request #25460 from nextcloud/fix/apppassword/always_renewRoeland Jago Douma2021-02-102-22/+1
|\ \ \ | | | | | | | | Always renew apppasswords on login
| * | | Always renew apppasswords on loginRoeland Jago Douma2021-02-102-22/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Else you can end up that you renewed your password (LDAP for example). But they still don't work because you did not use them before you logged in. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | Merge pull request #24904 from nextcloud/enh/github-psalmRoeland Jago Douma2021-02-101-0/+26
|\ \ \ \ | | | | | | | | | | Have psalm analysis directly on github
| * | | | Have psalm analysis directly on githubRoeland Jago Douma2021-02-101-0/+26
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | | Merge pull request #25558 from nextcloud/deprecate/generic-event-wrapperRoeland Jago Douma2021-02-101-0/+17
|\ \ \ \ \ | | | | | | | | | | | | Deprecate the GenericEvent wrapper
| * | | | | Deprecate the GenericEvent wrapperChristoph Wurst2021-02-101-0/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This class is a convenience type to replace the old Symfony GenericEvent with our own type, to ease the transition from Symfony events to our (PSR) events. Hence we can deprecate the class as precaution to smoothen the next transition step in the future. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | | | | Merge pull request #25566 from nextcloud/fix/25359/followupRoeland Jago Douma2021-02-102-9/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix remaining #25359
| * | | | | | Fix remaining #25359Roeland Jago Douma2021-02-102-9/+6
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As a wise man once said: "I like PRs that pass tests before merging" C. Wurst, Feb 9th 2021 Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | | | Merge pull request #25560 from nextcloud/enhancement/pure-psr-eventChristoph Wurst2021-02-102-2/+35
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Make our event base class independent of Symfony and follow PSR
| * | | | | Make our event base class independent of Symfony and follow PSRChristoph Wurst2021-02-102-2/+35
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Symfony is migrating towards a pure PSR event dispatcher, hence their event class is POPO that implements the PSR stoppable event interface. Since we can do that ourself and this change doesn't come with any API changes (breakage), it's easy for us to become independent of Symfony but also stay PSR-compliant at the same step. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | | | Merge pull request #25359 from nextcloud/enhancement/typed-service-registrationChristoph Wurst2021-02-1015-117/+468
|\ \ \ \ \ | | | | | | | | | | | | Type the service registration
| * | | | | Type the service registrationChristoph Wurst2021-02-1015-117/+468
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | | | | Merge pull request #25435 from nextcloud/fix/favorite-iconRoeland Jago Douma2021-02-101-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix favorite dark icon
| * | | | | | Fix favorite dark iconGretaD2021-02-021-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: GretaD <gretadoci@gmail.com>
* | | | | | | Merge pull request #25557 from ↵Roeland Jago Douma2021-02-101-8/+0
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | nextcloud-pr-bot/automated/noid/psalm-baseline-update [Automated] Update psalm-baseline.xml
| * | | | | | Update psalm baselineNextcloud-PR-Bot2021-02-101-8/+0
| | |_|/ / / | |/| | | | | | | | | | | | | | | | Signed-off-by: GitHub <noreply@github.com>
* | | | | | Merge pull request #25554 from nextcloud/fix/pslam/tainted_cookieLukas Reschke2021-02-101-4/+18
|\ \ \ \ \ \ | |/ / / / / |/| | | | | tain-escape the cookie input
| * | | | | tain-escape the cookie inputRoeland Jago Douma2021-02-091-4/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | we only set the cookie if it is a proper <=32 char alphanum string. Otherwise we just ignore the input. Makes psalm also happier so that we can focus on other errors. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | | | [tx-robot] updated from transifexNextcloud bot2021-02-108-2/+74
|/ / / / /
* | | | | Merge pull request #25548 from k00ni/pr/add-tests-for-LoginFlowV2CredentialsChristoph Wurst2021-02-091-0/+61
|\ \ \ \ \ | | | | | | | | | | | | Added tests for OC\Core\Data\LoginFlowV2Credentials
| * | | | | simplified testsKonrad Abicht2021-02-091-13/+12
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Konrad Abicht <hi@inspirito.de>
| * | | | | fixed coding style issueKonrad Abicht2021-02-091-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Konrad Abicht <hi@inspirito.de>
| * | | | | added tests for OC\Core\Data\LoginFlowV2CredentialsKonrad Abicht2021-02-091-0/+63
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Konrad Abicht <hi@inspirito.de>
* | | | | | Merge pull request #25545 from ↵Lukas Reschke2021-02-091-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | k00ni/fix/Dockerfile-wrong-package-name-php7.4-acpu Corrected package name used in .devcontainer/Dockerfile (php7.4-acpu => php7.4-apcu)
| * | | | | | corrected package name used in .devcontainer/Dockerfile (php7.4-acpu => ↵Konrad Abicht2021-02-091-1/+1
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | php7.4-apcu) Signed-off-by: Konrad Abicht <hi@inspirito.de>
* | | | | | Merge pull request #25546 from nextcloud/techdept/deprecate-event-constantsChristoph Wurst2021-02-0911-1/+81
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | Deprecate all remaining event constants
| * | | | | Deprecate all remaining event constantsChristoph Wurst2021-02-0911-1/+81
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | | | | Merge pull request #25543 from nextcloud/deprecate/idb-eventsChristoph Wurst2021-02-091-0/+23
|\ \ \ \ \ \ | | | | | | | | | | | | | | Deprecate the internal IDBConnection event name constants
| * | | | | | Deprecate the internal IDBConnection event name constantsChristoph Wurst2021-02-091-0/+23
| | |/ / / / | |/| | | | | | | | | | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | | | | Merge pull request #25549 from nextcloud/remove-reconnect-wrapperChristoph Wurst2021-02-094-58/+4
|\ \ \ \ \ \ | | | | | | | | | | | | | | remove ReconnectWrapper
| * | | | | | remove ReconnectWrapperRobin Appelman2021-02-094-58/+4
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | dbal now handles it's own reconnections: https://github.com/doctrine/dbal/blob/3.0.x/UPGRADE.md#the-pingableconnection-interface-is-removed Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | | | | Merge pull request #25531 from nextcloud/fix/dispatcher-argument-orderRoeland Jago Douma2021-02-092-0/+66
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix the legacy dispatcher argument order
| * | | | | | Fix the legacy dispatcher argument orderChristoph Wurst2021-02-092-0/+66
| |/ / / / / | | | | | | | | | | | | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | | | | Merge pull request #25535 from nextcloud/query-exception-forewardChristoph Wurst2021-02-092-2/+11
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Improve exception when auto-wiring fails
| * | | | | forward query exception when fallback construction failsRobin Appelman2021-02-092-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | | | | forward original query exception when querying by name failsRobin Appelman2021-02-081-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | | | | Merge pull request #25542 from nextcloud/fix-file-checkerRoeland Jago Douma2021-02-091-0/+1
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Add .devcontainer to file-checker.php
| * | | | | Add .devcontainer to file-checker.phpLukas Reschke2021-02-091-0/+1
|/ / / / / | | | | | | | | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* | | | | [tx-robot] updated from transifexNextcloud bot2021-02-098-4/+8
| | | | |
* | | | | Merge pull request #25493 from Worteks/fix-filessharing-debugRoeland Jago Douma2021-02-083-8/+7
|\ \ \ \ \ | | | | | | | | | | | | fix(log): remove debug log in _formatRemoteShare, suggested by @maxbes
| * | | | | fix(ci): build additionalScriptsSamuel2021-02-072-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Samuel <faust64@gmail.com>