Fix numeric folders throwing on markDirty

TypeError: strpos() expects parameter 1 to be string, int given

The problem is that in cacheNode() we strip of any slashes, so
a folder "0/" will be trimmed to "0" and be used as an array key.
Since PHP automatically casts numeric array keys to integers,
you afterwards get $nodePath as int(0). Since it's now a number,
the strpos() function does not accept it anymore. Simply casting
$nodePath to a string again in the foreach solves the issue

Signed-off-by: Joas Schilling <coding@schilljs.com>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #22926 from nextcloud/backport/22915/stable19

[stable19] improve handling of out of space errors for smb

[tx-robot] updated from transifex

improve handling of out of space errors for smb

Signed-off-by: Robin Appelman <robin@icewind.nl>

Merge pull request #22899 from nextcloud/backport/21894/stable19

[stable19] fix: file quota was not applied in all cases

fix: file quota was not applied in all cases

[tx-robot] updated from transifex

Merge pull request #22893 from nextcloud/backport/22880/stable19

[stable19] Revoke secsignid

Merge pull request #22892 from nextcloud/backport/22744/stable19

[stable19] only get the permissions from the share source if it's not already cached

Revoke secsignid

See nextcloud/app-certificate-requests#343

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

only get the permissions from the share source if it's not already cached

this prevents having to setup the share source when doing metadata stuff on the share root

Signed-off-by: Robin Appelman <robin@icewind.nl>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #22840 from nextcloud/backport/22770/stable19

[stable19] Mitigate encoding issue with user principal uri

Mitigate encoding issue with user principal uri

Signed-off-by: Georg Ehrke <developer@georgehrke.com>

Merge pull request #22775 from nextcloud/fix/noid/stable19-ldap-group-update-events

[stable19] makes LDAP's group updater dispatch type events

Merge pull request #22778 from nextcloud/backport/22739/stable19

[stable19] Don't fail if copying a file of 0 byte size

Merge pull request #22776 from nextcloud/backport/22747/stable19

[stable19] fix s3 doesDirectoryExist check for empty directories

Merge pull request #22591 from nextcloud/backport/22577/stable19

[stable19] Better error message when blocked by access control

makes LDAP's group updater dispatch type events

* fixes group share updates on added users

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

Annotate that writeStream can throw a GenericFileException

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Properly catch exception from writing to stream when copying a file

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Set delimiter so that the CommonPrefixes response can properly be determined

Signed-off-by: Julius Härtl <jus@bitgrid.net>

fix s3 doesDirectoryExist check for empty directories

if a directory is empty, only the 'marker' object `/` exists.

since not all s3 implementations return just the prefix when listing objects by prefix,
when listing objects by the folder prefix, nothing will be returned if the directory is empty.
by not including the trailing slash in the prefix, the folder marked will always be returned if it exists

Signed-off-by: Robin Appelman <robin@icewind.nl>

Merge pull request #22769 from nextcloud/backport/22766/stable19

[stable19] Clarify PHP warning in admin settings

Clarify PHP warning in admin settings

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Merge pull request #22752 from nextcloud/backport/22587/stable19

[stable19] Check if var debugMode exists

Merge pull request #22753 from nextcloud/backport/22651/stable19

[stable19] Fix reading empty files from objectstorage

Merge pull request #22630 from nextcloud/backport/22583/stable19

Fix reading empty files from objectstorage

Since we try to do range requests this will fail hard.
However since empty files are not that interesting to read anyways we
just read from an emptry memory stream.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Check if var debugMode exists

Closes #21150 a second time.
2nd appearance of debugMode may not seen in first fix.

Merge pull request #22742 from nextcloud/backport/22589/stable19

[stable19] Allow additional personal settings via normal registration

Merge pull request #22745 from nextcloud/backport/22646/stable19

[stable19] Fix detecting text/x-php mimetype and secure mimetype mapping

Merge pull request #22652 from nextcloud/backport/22643/stable19

[stable19] Fix installing on Oracle

Merge pull request #22713 from nextcloud/backport/stable19/22557

[stable19] Do not fail if share for mountpoint is no longer available

Merge pull request #22720 from nextcloud/backport/22469/stable19

[stable19] Don't use SELECT DISTINCT when to_char() is used in a WHERE statement

Add text/x-php as an alternative for application/x-php

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Properly add both mimetypes to secure mimetype mapping

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Merge pull request #22725 from nextcloud/version/19.0.3/final

19.0.3 final

Allow additional personal settings via normal registration

Signed-off-by: Joas Schilling <coding@schilljs.com>

19.0.3 final

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Don't use SELECT DISTINCT when to_char() is used in a WHERE statement

Nice bug in Oracle

Signed-off-by: Joas Schilling <coding@schilljs.com>

Do not fail if share for mountpoint is no longer available

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Warn about adding NotNull columns with nullable default

Signed-off-by: Joas Schilling <coding@schilljs.com>

Fix installing on Oracle

Empty strings are stored as null in Oracle,
so a column with NotNull can not have an empty string as default

Signed-off-by: Joas Schilling <coding@schilljs.com>

add repair step to clean up DB off lastFeatureRefresh entries in user prefs

- also removes related app setting "updateAttributesInterval"

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

[tx-robot] updated from transifex

these code bits were part of old logic that was already refactored out

- only references were in unit tests

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #22574 from nextcloud/version/19.0.3/rc1

19.0.3 RC1

Better error message when blocked by access control

Signed-off-by: Joas Schilling <coding@schilljs.com>

Merge pull request #22515 from nextcloud/backport/22472/stable19

[stable19] Fix writing BLOBs to postgres with recent contacts interaction

Merge pull request #22579 from nextcloud/backport/22550/stable19

[stable19] Use the correct root to determinate the webroot for the resource

Merge pull request #22442 from nextcloud/backport/22421/stable19

[stable19] show better quota warning for group folders and external storage

Merge pull request #22581 from nextcloud/backport/22578/stable19

[stable19] Upgrade icewind/smb to 3.2.7

Upgrade icewind/smb to 3.2.7

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Merge pull request #22569 from nextcloud/backport/22524/stable19

[stable19] Fix app password updating out of bounds

Use the correct root to determinate the webroot for the resource

Since all the compiled routes are based on the server webroot,
we have to use this, independent from which app this belongs to.

Signed-off-by: Joas Schilling <coding@schilljs.com>

Merge pull request #22553 from nextcloud/backport/22528/stable19

[stable19] Change free space calculation

19.0.3 RC1

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

[tx-robot] updated from transifex

Fix app password updating out of bounds

When your password changes out of bounds your Nextcloud tokens will
become invalid. There is no real way around that. However we should make
sure that if you successfully log in again your passwords are all
updates

* Added event listener to the PostLoggedInEvent so that we can act on it
  - Only if it is not a token login
* Make sure that we actually reset the invalid state when we update a
  token. Else it keeps being marked invalid and thus not used.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Merge pull request #22560 from nextcloud/backport/22556/stable19

[stable19] Do not keep the part file if the forbidden exception has no retry set

Do not keep the part file if the forbidden exception has retry set to false

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Tests only: Forward calls to getUserValue and getAppValue

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>

Change free space calculation

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>

Merge pull request #22537 from nextcloud/backport/22535/stable19

[stable19] Only disable zip64 if the size is known

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Only disable zip64 if the size is known

Else we might run into the issue that for an external storage where the
size is not known yet we do not use zip64. Which then of course fails on
large zip files.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

[tx-robot] updated from transifex

Merge pull request #22519 from nextcloud/backport/22517/stable19

[stable19] set the mount id before calling storage wrapper

Merge pull request #22521 from nextcloud/backport/22514/stable19

[stable19] Fix S3 error handling

Only update the filecache entry once the file has been written to S3

If we already update before we have no way to revert if the upload
fails.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Don't lose filecache entry on s3 overwrite error

If the object store errors we should not always delete the filecache
entry. As this might lead to people losing access to their files.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

set the mount id before calling storage wrapper

this allows the storage wrapper to use the mount id for it's own logic

Signed-off-by: Robin Appelman <robin@icewind.nl>

Fix writing BLOBs to postgres with recent contacts interaction

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>

Merge pull request #22503 from nextcloud/backport/22481/stable19

[stable19] Fix clicks on actions menu of non opaque file rows in acceptance tests

[tx-robot] updated from transifex

Fix clicks on actions menu of non opaque file rows in acceptance tests

When a row is added to the file list the opacity of the file row is
animated from transparent to fully opaque. As the file actions menu is a
descendant of the row but overflows it when the row is not fully opaque
clicks on the menu entries "fall-through" and are received instead by
the rows behind.

The opacity animation is a CSS animation, and it is not possible to know
if a row is appearing or not except from its opacity (the row will have
the "appear" CSS class even after the animation ended). Therefore it
should be waited until the row of the file is fully opaque before using
the menu.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #22448 from nextcloud/stable19-phpdoc

[stable19]  Add php docs build script

Use proper branch name in phpdoc builds

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Remove @package annotations from public namespace

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Remove wrongly annotated package names

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Add php docs build script

Signed-off-by: Julius Härtl <jus@bitgrid.net>

[tx-robot] updated from transifex

fix tests

Signed-off-by: Robin Appelman <robin@icewind.nl>

show better quota warning for group folders and external storage

instead of showing the generic 'Your storage is full' message, better explain that it's the group folder/external storage that is full

Signed-off-by: Robin Appelman <robin@icewind.nl>

Merge pull request #22410 from nextcloud/backport/22359/stable19

[stable19] fix possible leaking scope in Flow