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>

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

Only use width and opacity for transition

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

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

Fix wrong share popover opening on share link

Fix wrong share popover opening on share link

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

[tx-robot] updated from transifex

Merge pull request #12436 from nextcloud/version/noid/14.0.4RC1

14.0.4 RC 1

14.0.4 RC 1

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

Update autoloader dump

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

Merge pull request #12459 from nextcloud/bugfix-stable14/dav_shares_hickup

[stable14] fixes dav share issue with owner

fix caldav tests

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

[tx-robot] updated from transifex

fixes dav share issue with owner

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

Merge pull request #12441 from nextcloud/stable14-10888

[stable14] Fix missing quickaccess favorite folder on add

Update PHPDoc to fix typos

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

Merge pull request #12432 from nextcloud/stb14-accessibility-cache-fixes

[stable14] Added cache override to ensure an always up-to-date accessibility css

[tx-robot] updated from transifex

fix for favorites-quickaccess: newly added folders dont show up when added to favorites

Signed-off-by: fnuesse <felix.nuesse@t-online.de>

Merge pull request #12440 from nextcloud/stable14-10890

[stable14] Fix app menu calculation for random size of the right header

Fix app menu calculation for random size of the right header

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

Merge pull request #12372 from nextcloud/suppress-wrong-audit-log-message-stable14

[stable14] suppress wrong audit log messages about failed login attempts

Merge pull request #12375 from nextcloud/backport/12284/stable14

[14] Properly search the root of a shared external storage

Merge pull request #12431 from nextcloud/stb14-ie11-width-fix

Fix IE rule for min width

Merge pull request #12433 from nextcloud/backport/12411-12413/unique-constraint-fix

[stable14] Unique contraint and deadlock fixes for filecache and file_locks

Added cache override to ensure an always up-to-date accessibility css

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

Fix UniqueConstraintViolationException while insert into oc_file_locks

* fixes #9305 by not being prone to the race condition in insertIfNotExists
* fixes #6899 by not using a query that can result in a deadlock
* replaces the insertIfNotExists call with an insert which is wrapped into a try-catch block
* followup to #12371

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

Fix UniqueConstraintViolationException while insert into oc_filecache

* fixes #6160 by not being prone to the race condition in insertIfNotExists
* fixes #12228 by not using a query that can result in a deadlock
* replaces the insertIfNotExists call with an insert which is wrapped into a try-catch block

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

Fix IE rule for min width

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

[tx-robot] updated from transifex

Merge pull request #12416 from nextcloud/stb14-admin-vue-approval-fix

[stable14] Use nextcloud-password-confirmation

Merge pull request #12412 from nextcloud/backport/12358/fix-update-check

[stable14] Fix app update available check

Use nextcloud-password-confirmation

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

Adjust unit tests

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

Improve style of incompatible apps list

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

Use the version number when checking the appstore for compatible updates

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

Fix overlapping update button in notifications

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #12374 from nextcloud/backport/12342/stable14

[14] Add fix for IE11 flexbox height bug

Properly search the root of a shared external storage

Fixes #1020

When an external storage is shared with you in full the root is ''
(empty). This adds an extra check for an empty jail basically.

Because if the jail is on the empty string. It matches all paths.

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

Add fix for IE11 flexbox height bug

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

first check if the user is already logged in and then try to authenticate via apache, this way we suppress wrong audit log messages about failed login attempts

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

[tx-robot] updated from transifex

Merge pull request #12353 from nextcloud/backport/11838/fix-default-types-of-members

[stable14] Fix default types of activity event member variables

Fix default types of activity event member variables

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

Merge pull request #12339 from nextcloud/bugfix/12007

fix - Add to favorites not working in IE11

fix - Add to favorites not working in IE11 - polyfill needed #12007

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

Merge pull request #12341 from nextcloud/stb14-ie11-function-fix

Remove arrow function for ie compatibility