summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Bugfix to support postgres15Vitor Mattos2022-11-112-3/+50
| | | | | | Signed-off-by: Vitor Mattos <vitor@php.rio> Signed-off-by: Simon L <szaimen@e.mail.de> Co-Authored-By: Joas Schilling <213943+nickvergessen@users.noreply.github.com>
* Merge pull request #34994 from Glandos/fileinfo_perfSimon L2022-11-111-3/+3
|\ | | | | [Performance] Test valid path only if file info has to be retrieved
| * Test valid path only if file info has to be retrievedGlandos2022-11-061-3/+3
| | | | | | | | Signed-off-by: Glandos <bugs-github@antipoul.fr>
* | Merge pull request #34485 from ↵Robin Appelman2022-11-111-0/+1
|\ \ | | | | | | | | | | | | michaelkuhn/hasBigIntConversionPendingColumns-files_trash Fix hasBigIntConversionPendingColumns missing files_trash
| * | Fix hasBigIntConversionPendingColumns missing files_trashMichael Kuhn2022-10-091-0/+1
| | | | | | | | | | | | | | | | | | | | | The two tables seem to have gone out of sync, causing the web interface to miss `files_trash.auto_id` not being a bigint. Signed-off-by: Michael Kuhn <github@ikkoku.de>
* | | Merge pull request #35015 from natoponen/patch-1Simon L2022-11-111-2/+2
|\ \ \ | | | | | | | | Appropriate length check in Notification.php
| * | | Appropriate length check in Notification.phpnatoponen2022-11-081-2/+2
| | | | | | | | | | | | | | | | | | | | There is an issue(bug) when using UTF-8 symbols in any method, which checks the length of string as `isset($id[64])`. You can set only 32 UTF-8 symbols because they are 2 byte, and this "array" check seems inapropriate in this case, as it throws unexpected exceptions. Signed-off-by: natoponen <57988162+natoponen@users.noreply.github.com>
* | | | Merge pull request #35105 from nextcloud/fix/noid/imaginary-distorted-previewsSimon L2022-11-111-4/+14
|\ \ \ \ | | | | | | | | | | Fix distorted previews when using imaginary
| * | | | Fix distorted previews when using imaginaryRichard Steinmetz2022-11-111-4/+14
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
* | | | | Merge pull request #35077 from nextcloud/fix/public-key-token-provider-testJulius Härtl2022-11-111-4/+2
|\ \ \ \ \ | |/ / / / |/| | | | Fix PublicKeyTokenProviderTest import and mock
| * | | | Fix PublicKeyTokenProviderTest import and mockChristoph Wurst2022-11-101-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * IDBConnection import missing * Atomic doesn't need a mock Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | | | [tx-robot] updated from transifexNextcloud bot2022-11-1124-4/+38
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* | | | | Merge pull request #35074 from nextcloud/fix/fix-path-normalizingJoas Schilling2022-11-101-7/+8
|\ \ \ \ \ | | | | | | | | | | | | Make sure that path is normalized and then checked,
| * | | | | Make sure that path is normalized and then checked,Côme Chilliet2022-11-101-7/+8
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | and not the other way around Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | | | | Merge pull request #35054 from ↵Pytal2022-11-106-16/+68
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | nextcloud/fix-html-entities-not-decoded-in-comment-just-added
| * | | | | Compile assetsnextcloud-command2022-11-103-3/+25
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
| * | | | | Fix HTML entities not decoded in comment just addedDaniel Calviño Sánchez2022-11-101-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The XML data received from the comments endpoint has an inconsistent encoding; some entities are encoded once and others are encoded twice. When the comment list is loaded the comments are fetched using GetComments, which handles all that, and therefore shows the messages and author names as expected. However, when a new comment is posted the list is not got again; instead the new comment is loaded from the comment data returned after posting it. This is done in NewComment, which did not decode the messages nor the author names, and therefore showed, for example, "&amp;" instead of "&". To solve that now the same decoding logic used in GetComments is applied too in NewComment. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
| * | | | | Extract "decodeHtmlEntities" function to its own fileDaniel Calviño Sánchez2022-11-102-13/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will make possible to use it from files other than "GetComments.js". Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | | | | Merge pull request #35076 from ↵Côme Chilliet2022-11-101-0/+0
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | nextcloud/fix/bump-swiftmailer/swiftmailer-to-6.3.0 Bump swiftmailer/swiftmailer to 6.3.0
| * | | | | Bump swiftmailer/swiftmailer to 6.3.0Côme Chilliet2022-11-101-0/+0
|/ / / / / | | | | | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | | | | [tx-robot] updated from transifexNextcloud bot2022-11-1034-470/+64
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* | | | | Merge pull request #35063 from nextcloud/fix/noid/versions-empty-top-marginPytal2022-11-093-3/+7
|\ \ \ \ \
| * | | | | Fix top margin in empty versions listRichard Steinmetz2022-11-093-3/+7
|/ / / / / | | | | | | | | | | | | | | | Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
* | | | | Merge pull request #34989 from nextcloud/Valdnet-patch-4Simon L2022-11-092-2/+2
|\ \ \ \ \ | | | | | | | | | | | | l10n: Shorten link
| * | | | | Fix testValdnet2022-11-091-1/+1
| | | | | | | | | | | | | | | | | | Signed-off-by: Valdnet <47037905+Valdnet@users.noreply.github.com>
| * | | | | l10n: Shorten linkValdnet2022-11-051-1/+1
| | | | | | | | | | | | | | | | | | Signed-off-by: Valdnet <47037905+Valdnet@users.noreply.github.com>
* | | | | | Merge pull request #35052 from ↵Simon L2022-11-092-3/+3
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | nextcloud/fix-sidebar-not-hidden-when-changing-section-in-files-app Fix sidebar not automatically hidden in Files app
| * | | | | | Fix sidebar not automatically hidden in Files appDaniel Calviño Sánchez2022-11-092-3/+3
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Although the Files app creates the legacy sidebar (details view) it is then replaced with the newer Vue app sidebar. Due to this ".detailsView" no longer finds an element and therefore nothing was hidden when "hideAppSidebar($('.detailsView'))" was called (for example, when changing to another section). However, "OC.Apps.hideAppSidebar()" does not properly work either with the Vue sidebar used in the Files app (once hidden the sidebar is not shown again). For simplicity, and to avoid any possible side effect in other apps from changing "OC.Apps.hideAppSidebar", now "OC.Files.Sidebar.close()" is used instead. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | | | | Merge pull request #35042 from nextcloud/enh/uri-descriptionPytal2022-11-091-0/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | Add description for calendar uri
| * | | | | | Add description for calendar uriChristopher Ng2022-11-091-0/+1
| |/ / / / / | | | | | | | | | | | | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* / / / / / [tx-robot] updated from transifexNextcloud bot2022-11-0922-4/+52
|/ / / / / | | | | | | | | | | | | | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* | | | | Merge pull request #34613 from nextcloud/enh/noid/fix-checkmark-iconSimon L2022-11-083-5/+5
|\ \ \ \ \ | | | | | | | | | | | | fix checkmark for theme settings in dark mode
| * | | | | Compile assetsnextcloud-command2022-11-082-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
| * | | | | fix checkmark for theme settings in dark modeszaimen2022-11-081-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: szaimen <szaimen@e.mail.de>
* | | | | | Merge pull request #32211 from nextcloud/feat/repair_share_ownershipSimon L2022-11-084-0/+199
|\ \ \ \ \ \ | | | | | | | | | | | | | | Add repair command to fix wrong share ownership
| * | | | | | update autoloaderRobin Appelman2022-11-082-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | | | | | fix repairing non user shares when repairing all sharesRobin Appelman2022-11-081-10/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | | | | | remove no longer needed codeRobin Appelman2022-11-081-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | | | | | split repairing into two stages to prevent long open transactionRobin Appelman2022-11-082-33/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | | | | | update shares directly in dbRobin Appelman2022-11-081-13/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | | | | | More verbose output for repair share ownership cmdVincent Petry2022-11-081-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Vincent Petry <vincent@nextcloud.com>
| * | | | | | move share owner repair to occ commandRobin Appelman2022-11-085-95/+139
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | | | | | Add repair step to fix wrong share ownershipLouis Chemineau2022-11-084-0/+98
|/ / / / / / | | | | | | | | | | | | | | | | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* | | | | | Merge pull request #35028 from nextcloud/fix/dns-pin-middleware-throws-public-ipRichard Steinmetz2022-11-081-1/+1
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Fix DNS Pin Middleware throwing for public IPs
| * | | | | Fix DNS Pin Middleware throwing for public IPsChristoph Wurst2022-11-081-1/+1
|/ / / / / | | | | | | | | | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | | | Merge pull request #32838 from markuszeller/bugfix/32837/implicitFloatConversionSimon L2022-11-081-1/+1
|\ \ \ \ \ | |_|/ / / |/| | | | Cast to int
| * | | | Cast to int #32837Markus Zeller2022-11-071-1/+1
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Markus Zeller <git@markuszeller.com>
* | | | | Merge pull request #34985 from ↵Joas Schilling2022-11-084-47/+11
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | nextcloud/dependabot/npm_and_yarn/nextcloud/l10n-1.6.0 Bump @nextcloud/l10n from 1.4.1 to 1.6.0
| * | | | | Bump @nextcloud/l10n from 1.4.1 to 1.6.0dependabot[bot]2022-11-074-47/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [@nextcloud/l10n](https://github.com/nextcloud/nextcloud-l10n) from 1.4.1 to 1.6.0. - [Release notes](https://github.com/nextcloud/nextcloud-l10n/releases) - [Changelog](https://github.com/nextcloud/nextcloud-l10n/blob/master/CHANGELOG.md) - [Commits](https://github.com/nextcloud/nextcloud-l10n/compare/v1.4.1...v1.6.0) --- updated-dependencies: - dependency-name: "@nextcloud/l10n" dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
* | | | | | [tx-robot] updated from transifexNextcloud bot2022-11-088-6/+78
|/ / / / / | | | | | | | | | | | | | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>