If the parent is not cached we should request the file

Else we ignore all incomming shares. Which basically does :boom:

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

Add acceptance tests for comments on shared files and folders

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

[tx-robot] updated from transifex

Merge pull request #12860 from nextcloud/fix/stable14-fix-password-confirmation-ie11

[stable14] Fix password confirmation lib for ie11

Merge pull request #12869 from nextcloud/backport/12857/stable14

[stable14] Emit write update only once on touch

[tx-robot] updated from transifex

Merge pull request #12862 from nextcloud/backport/12855/stable14

[stable14] Only execute query in propagateChange once

Emit write update only once on touch

When a touch is done (i.e. when creating a new file in the node API)
The hooks contain 'write' and 'touch' (and the operation is touch as
well). This would cause a double write update. Which doesn't make sense.

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

Only execute query in propagateChange once

The second execute statement should be inside the if block. Else it gets
executed twice which makes no sense.

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

Fix password confirmation lib for ie11

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

Merge pull request #12850 from nextcloud/stb14-missing-click-apps-update

[stable14] Fix missing click event on update button

Fix missing click event on update button

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

Merge pull request #12834 from nextcloud/backport/12831/stable14

[stable14] Do not update child all child shares on group share update

Merge pull request #12817 from nextcloud/backport/12813/stable14

[stable14] Fix the system address book

[tx-robot] updated from transifex

Do not update child all child shares on group share update

Else we overwrite the whole share for link shares. Basically breaking
them.

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

Merge pull request #12826 from nextcloud/backport/12822/stable14

[stable14] PHP module is named mbstring

PHP module is named mbstring

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

Fix the system address book

c23a66cda46d9a1f0ce8dc60a6c160809c87d9f6 broke the system address book.
We now move the ACL rules for this special case up and all is good in
the world again.

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

Merge pull request #12748 from nextcloud/backport/14/11200/stable14

[stable14] Fix interaction with groups that contain a slash

Merge pull request #12812 from nextcloud/backport/12808/stable14

[stable14] Only run the AnonymousOptionsPlugion on Anonymous requests

Only run the AnonymousOptionsPlugion on Anonymous requests

Fixes #12744

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

[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 #12745 from nextcloud/backport/12647/stable14

[stable14] update compatible versions for cloud federation api

Fix interaction with groups that contain a slash

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

update compatible versions for cloud federation api

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

Merge pull request #12738 from nextcloud/backport/12690/stable14

[stable14] Set Referrer-Policy also in addSecurityHeaders()

Set Referrer-Policy also in addSecurityHeaders()

Fix: #12689
Signed-off-by: Peter Kraume <peter.kraume@gmx.de>

[tx-robot] updated from transifex

Merge pull request #12715 from nextcloud/backport/12708/stable14

[stable14] Fix folder path containing leading slash when getting mount root by id

Fix folder path containing leading slash when getting mount root by id

This fixes collabora on public link shared groupfolders

Fixes https://github.com/nextcloud/groupfolders/issues/225

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

Merge pull request #12701 from nextcloud/stable14-12632-add-acceptance-tests-for-sharing-files-and-folders-with-another-user

[stable14] Add acceptance tests for sharing files and folders with another user

[tx-robot] updated from transifex

Merge pull request #12697 from nextcloud/backport/12649/stable14

[stable14] Make acceptance tests for comments more robust

Wait for the comment to be added before changing to another file

If the server is too slow, changing to a different file immediately
after sending a new comment but without waiting for the comment to be
shown for the original file could cause the new comment to be shown for
the current file instead.

This is, indeed, a bug in the comments. However, it is not possible to
test it reliably in the acceptance tests, as it depends on how fast the
server adds the message and how fast the client changes to a different
file; sometimes the test would fail and sometimes it would not.

Therefore, now it is waited for the comment to be added before changing
to another file, as in this case it can be reliably tested that changing
to a different file does not cause the comments from the previous file
to be shown in the current file (this was a different bug already fixed
and due to which this test was added in the first place).

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

Wait for the empty content element to be shown

When the "Comments" tab is open the empty content element is always in
the DOM, although it is only shown once the message collection was
fetched and there were no messages. Due to this it is necessary to
explicitly wait for it to be shown instead of relying on the implicit
wait made to find the element; otherwise it would be found immediately
and if the collection was not fetched yet it would not be visible,
causing the test to fail.

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

Merge pull request #12684 from nextcloud/backport/12650/stable14

[stable14] Fix count on string

Add return type

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

Fix count on string

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

[tx-robot] updated from transifex

Merge pull request #12639 from nextcloud/backport/12556/stable14

[stable14] extend anonymous options to work on every dav url

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

extend anonymous options to work on every dav url

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

Rename "FilesSharingAppContext" to "PublicShareContext"

Having both "FilesAppSharingContext" and "FilesSharingAppContext" was
confusing, so "FilesSharingAppContext" was renamed to a more descriptive
name.

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

Extract locators and steps for sharing in Files app to their own context

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

Extract sharing related acceptance tests to their own feature files

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

Add acceptance tests for folders that can not be reshared

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

Add acceptance tests for resharing a folder with another user

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

Add acceptance test for checking that the owner can see reshares

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

Add acceptance test for resharing a file with another user

To reshare a file there must be at least three enabled users in the
system; although it would be possible to run the steps to create a third
user in the scenarios that need it for convenience a third enabled user
besides "admin" and "user0" was added to the default setup.

In a similar way, a new step was added too to login as a given user
name, similar to the steps to log in as "user0" and as "admin".

Finally, another actor, "Jim", was introduced for those scenarios which
should be played by three standard actors (that is, without a special
configuration like "Rubeus").

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

Add acceptance tests for sharing a folder with another user

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

Add acceptance tests for sharing a file with another user

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

Merge pull request #12598 from nextcloud/stable14-12577-fix-rendering-of-the-sidebar-in-files-app

[stable14] Fix rendering of the sidebar in Files app

Merge pull request #12560 from nextcloud/stb14-share-menu-click-fix

[stable14] Fix share link password input

Merge pull request #12593 from nextcloud/backport/12554/stable14

[stable14] Do not switch to root folder if filelist is already shown

[tx-robot] updated from transifex

Merge pull request #12596 from nextcloud/backport/12448/stable14

[stable14] Load apps that have a dav type set before the dav server plugins

Fix share link password input

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

Merge pull request #12600 from nextcloud/backport/12592/stable14

[stable14] Fix Typo in Original English String and All Translations

fix typo in original english string and all translations

Signed-off-by: Patrick Baber <patrick@p-squared.de>

Fix rendering of the sidebar in Files app

When a view is rendered it should not be concerned with where it is
going to be placed in the document; in general this should be a
responsibility of the object using the view.

Moreover, when the details view is rendered it should simply prepare a
skeleton that includes the root elements provided by the plugins; those
elements will be updated by the plugins as needed when a file or a tab
is selected.

Finally, the details view should not be explicitly rendered. The
rendering removes the previous elements, but that is needed only when
the details view is in a dirty state, that is, when new plugins were
added since the last time that it was rendered. However, that dirty
state is internally handled, and the view is automatically rendered
again if needed when a file info is set.

Due to all that the details view is no longer explicitly rendered when
updating it with a different file. Also, as each file list has its own
details view, and each details view has its own element, but there can
be only one details view/sidebar element in the document, when the file
list updates the details view it also replaces the current one in the
document with its own details view if needed (that is, if it is not the
current one already).

Besides that, when the element of a details view is replaced with the
element of a different details view the old one should be detached from
the document, but never removed. Otherwise the event handlers would not
work when that element is attached again later (when changing to a
different section in the Files app and then going back to the previous
one).

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

Add acceptance tests for showing again the input field for tags

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

Add acceptance tests for opening and closing again the details view

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

Add acceptance tests for marking a file as favorite in the details view

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

Add acceptance tests for switching to the comments of another file

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

Load apps that have a dav type set before the dav server plugins

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

Do not switch to root folder if filelist is already shown

Navigating to the root folder is already handled by
OCA.Files.Navigation.setActiveItem in case the view doesn't change.

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

Merge pull request #12540 from nextcloud/backport/12450/stable14

[stable14] Throw exception if decryption fails

Merge pull request #12555 from nextcloud/backport/12552/stable14

[stable14] Respect the disabled setting for lost_password_link

Merge pull request #12586 from nextcloud/version/noid/14.0.4

14.0.4

14.0.4

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

[tx-robot] updated from transifex

Merge pull request #12561 from nextcloud/backport/12544/stable14

[14] Handle permission in update of share better

[tx-robot] updated from transifex

Merge pull request #12545 from nextcloud/backport/12543/stable14

[stable14] Bearer tokens are app token

Handle permission in update of share better

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

Respect the disabled setting for lost_password_link

Fixes #11146
As documented when it is set to disabled the user can't request a lost
password.

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

Bearer tokens are app token

Fixes #12498

This means that we set that it is a proper app token once it is
validated. This will allow the 2FA middleware to just run the same
check.

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

Merge pull request #12533 from nextcloud/version/noid/14.0.4rc2

14.0.4 RC 2

14.0.4 RC 2

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

Merge pull request #12542 from nextcloud/stb14-settings-svg-icons-scss-fix

Remove unused svg api route

Remove unused sg api route

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

[tx-robot] updated from transifex

Throw exception if decryption fails

For #11868

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

Merge pull request #12529 from nextcloud/backport/12401/php-warning

[stable14] Fix the warning appearing in the admin section when mail_smtpmode is not configured

Use the proper default values

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

Verify that isPhpMailerUsed uses config as expected

Signed-off-by: Kyle Fazzari <kyrofa@ubuntu.com>

Fix the warning appearing check setup when mail_smtpmode is not configured.
Closes #11107

Merge pull request #12502 from nextcloud/swift-object-not-found-14

[14] forward object not found error in swift as dav 404

forward object not found error in swift as dav 404

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

[tx-robot] updated from transifex

Merge pull request #12492 from nextcloud/stable14-12489

[stable14] Only use width and opacity for transition

[tx-robot] updated from transifex

[tx-robot] updated from transifex