Add acceptance test steps to be used from Talk acceptance tests

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

Test that "Password protect by Talk" is not shown if Talk is not enabled

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

Add "Password protect by Talk" to the menu of link shares

When Talk is enabled the menu for link shares now shows a checkbox to
protect the password by Talk (that is, to show the "Request password by
Talk" UI in the authentication page for the link share).

Although in e-mail shares protecting the share with a password and
protecting the password by Talk are mutually exclusive actions (as when
the password is set it is sent to the sharee, so it must be set again
when protecting it by Talk to be able to verify the identity of the
sharee), in the case of link shares protecting the password by Talk is
an additional step to protecting the share with a password (as just
setting the password does not disclose it to anyone). As such, the
checkbox is shown only when there is a password set for the link share
(even if the field itself for the password is not shown, like when they
are enforced in the settings).

Note that the icon set for the field, "icon-passwordtalk", does not
currently exist; it is the same used for e-mail shares, and it is needed
simply to get the right padding in the menu.

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

Add sending the password by Talk for a link share to ShareAPIController

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

Store "sendPasswordByTalk" property of link shares in the database

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

Check "note", "label" and "hide download" too in update share tests

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

Fix update share tests

The update share tests only checked that the share returned by
"update()" had the expected values. However, as "update()" returns the
same share that was given as a parameter the tests were not really
verifying that the values were updated in the database.

In a similar way, the test that checked that a password was removed did
not set a password first, so even if the database returned null it could
be simply returning the default value for the share; a password must be
set first to ensure that it is removed.

Besides that, a typo was fixed too that made the checks on the original
share instead of on the one returned by "update()"; right now it is the
same share, so the change makes no difference, but it is how the check
should be done anyway.

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

Merge pull request #11053 from nextcloud/occ-app-update-command

occ app update command

Merge pull request #12192 from nextcloud/development-setup

Development setup in readme, and some adjustments

Merge pull request #11844 from nextcloud/multiple-link-shares

allow to create multiple link shares via share api

Merge pull request #12188 from nextcloud/revert/9900/revert-wait-for-cron

Revert "Wait for cron to finish before running upgrade command"

Merge pull request #12196 from nextcloud/revert-12138-and-12138

Revert important rule for hidden class and inline button style

Fixed jsunit

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

Password policy fix and menu open on first share

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

Fix wording, sort 'Add another link' to bottom so it does not confuse

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

Adjust acceptance tests to changes in link shares

Although now it is possible to create several link shares the acceptance
tests currently handles only the first link share; this first link share
is now created by clicking an "Add new share" button instead of a
checkbox.

Besides that, the "Copy link" button has been moved from the menu to the
row, next to the menu trigger.

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

Use primary color for share link icon background instead of bland grey

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

Fix password enforcement

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

Fix enforce date

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

Fix note display

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

Copy link outside menu

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

Migrate link shares to array

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

Revert "Fix .hidden class specificity, should not be overridable, ref #12138"

This reverts commit 25e70e1eb798d9292b4d5864d89c6c424ff0218c.

Revert "make .button inline-block"

This reverts commit 87c3f79399f4b3e71fa1d0ff6d774c9c9f6ddc18.

Log exception in case an app update fails

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

Add occ app:update command

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

Readme: Simply use emojis instead of images for tools section

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

Merge pull request #12152 from brad2014/bugfix/12151/fix-emailtemplate-list-formatting

Fix issue #12151: fix list formatting by correcting malformed html

Readme: Reduce size of Tools logos, use HTML for part

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

Readme: Add line breaks between tool images

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

Readme: Replace Github emoji codes with real emoji

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

Readme: Add note about signing off commits

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

Add sentence about non-technical contributions

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

Readme: Fix screenshot link

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

Readme: adjust structure, wording, small fixes

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

Revert "Wait for cron to finish before running upgrade command"

This reverts commit 18e9631810ad1d3d72c2b4bbee330169808108ad.

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

Add front-end building documentation to readme

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

Merge pull request #12150 from nextcloud/techdebt/noid/ldap-rm-iusertools

remove app specific IUserTools and consolidate test

Merge pull request #12180 from nextcloud/sidebar-fixes

Sidebar fixes, mostly fixing design of tabs

Merge pull request #12163 from nextcloud/bugfix/11730/file-share-edit-state

Do not set indeterminate state for file shares

Merge pull request #12162 from nextcloud/file-menu-align

Align file menu below '+' button, fix #5595

Merge pull request #12129 from nextcloud/versions-modular-api

modular versions api

Merge pull request #12181 from nextcloud/fix-header-clickability

Fix header app name clickability issue, fix #12165

Merge pull request #12072 from nextcloud/writestream

extend storage api to allow directly writing a stream to storage

Fix header app name clickability issue, fix #12165

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

adjust unit tests

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

Fix design of tabs in sidebar

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

Fix issues with clipboard button for local link in sidebar

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

Add icon to Versions sidebar tab

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

allow to update lables for public link shares

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

allow to add labels to shares

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

[tx-robot] updated from transifex

modular versions api

Allows apps to register version backends for storage types
The existing versions backend is wrapped in a "legacy" backend.

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

fix writeStream for jail wrapper

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

update dav tests

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

allways fall back to fopen for encryption wrapper

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

Update autoloader

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

extend storage api to allow directly writing a stream to storage

this removes the need for temporary storages with some external storage backends.
The new method is added to a separate interface to maintain compatibility with
storage backends implementing the storage interface directly (without inheriting common)

Currently the interface is implemented for objectstorage based storages and local storage
and used by webdav uploads

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

Merge pull request #12167 from nextcloud/hidden-class

Fix .hidden class specificity, should not be overridable, ref #12138

Merge pull request #12166 from nextcloud/feature/limit_preview_sizes

Only generate previews in powers of 4 and set min

Merge pull request #12169 from nextcloud/interface-fulltextsearc-issue-00001

wrong method called

missing type

Signed-off-by: Maxence Lange <maxence@artificial-owl.com>

wrong method called

Signed-off-by: Maxence Lange <maxence@artificial-owl.com>

Only generate previews in powers of 4 and set min

Before we'd round up all preview request to their nearest power of two.
This resulted still in a lot of possible images. Generating a lot of
server load and taking up a lot of space.

This moves it to previews to be powers of 4: 64, 256, 1024 and 4096
Also the first two powers are always skipped (4, 16) as it doesn't make
sense to generate previews for that.

We cache preview pretty agressively and I feel this is a better
tradeoff.

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

Merge pull request #12170 from nextcloud/fix/noid/tests-from-12009

Revert "Populate the mention-notification with the actual message"

Revert "Populate the mention-notification with the actual message"

This reverts commit d295ff53672d542f3b09ff0becd6a4a20ba1d494.

Merge pull request #11647 from nextcloud/make-possible-for-tab-views-to-add-css-classes-to-their-container

Make possible for tab views to add CSS classes to their container

Fix .hidden class specificity, should not be overridable, ref #12138

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

Merge pull request #12146 from nextcloud/fix/noid/verifyuserdata-on-null

fixes an Exception on the background job because the config was not set

Add tests for edit permission state on file shares

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

Merge pull request #12009 from nextcloud/feature/noid/populate-notification-message-with-the-comment

Populate the mention-notification with the actual message

Do not set indeterminate state for file shares

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

Align file menu below '+' button, fix #5595

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

Merge pull request #12095 from MasterOfDeath/OCA-ShareByMail-Capabilities-test-unit

OCA\ShareByMail\Capabilities add unit test

Merge pull request #12147 from MasterOfDeath/OCA-SystemTags-Activity-Setting-add-unit-test

OCA\SystemTags\Activity\Setting unit test

Merge pull request #12153 from nextcloud/appnames

Show all app titles on hovering app menu area, fix #10952, fix #4619

Merge pull request #12089 from Rayn0r/master

added possibility to disable autocomplete in login form

Merge pull request #12154 from nextcloud/files-details

Design details in file picker

Merge pull request #12125 from MasterOfDeath/expire-share-days-validate

Validate expire share days in settings (issue: #11930)

allow to create multiple link shares via share api

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

Added round border for thumbnail

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

[tx-robot] updated from transifex

Filepicker: center text in grid view, improve spacing

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

slightly round off thumbnails in list view too

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

Fix bug #12151: fix list formatting by correcting malformed html
Signed-off-by: Brad Rubenstein <brad@wbr.tech>

Show all app titles on hovering app menu area, fix #10952, fix #4619

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

remove app specific IUserTools and consolidate test

Just some house keeping. IUserTools with used in even older days for
easier creation of Access instances…

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

Merge pull request #12020 from nextcloud/bugfix/11795/calendar_invitation_fallback_lang

Calendar invitation: use system default instead of sender's language as fallback

OCA\SystemTags\Activity\Setting unit test

Signed-off-by: Rinat Gumirov <rinat.gumirov@mail.ru>

Merge pull request #12047 from nextcloud/interfaces-files-fulltextsearch

Public interfaces Files_FullTextSearch

added sample config for turning off form autocompletion on login page

Signed-off-by: Rayn0r <Andre.Weidemann@web.de>

Merge pull request #12140 from nextcloud/fix/expired_token_throttler

Expired tokens should not trigger bruteforce protection

expire share days in settings validate

Signed-off-by: Rinat Gumirov <rinat.gumirov@mail.ru>

OCA\ShareByMail\Capabilities unit test

OCA\ShareByMail\Capabilities unit test

Signed-off-by: Rinat Gumirov <rinat.gumirov@mail.ru>

+files_fulltextsearch in autoloader

Signed-off-by: Maxence Lange <maxence@artificial-owl.com>

Merge remote-tracking branch 'origin/interfaces-files-fulltextsearch' into interfaces-files-fulltextsearch

Signed-off-by: Maxence Lange <maxence@artificial-owl.com>

Merge pull request #12056 from nextcloud/interfaces-fulltextsearch-2

Public interfaces FullTextSearch

Merge pull request #12145 from olivermg/master

Extending documentation in config.sample.php for `trusted_proxies`

Error out early on an expired token

Fixes #12131

If we hit an expired token there is no need to continue checking. Since
we know it is a token.

We also should not register this with the bruteforce throttler as it is
actually a valid token. Just expired. Instead the authentication should
fail. And buisness continues as usual.

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

Move ExpiredTokenException to the correct namespace

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