Fix app menu calculation for random size of the right header

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

Merge pull request #11404 from flokli/sendmail-path

Mailer: discover sendmail path instead of hardcoding it to /usr/sbin/sendmail

Merge pull request #11875 from nextcloud/add-support-for-sending-the-password-for-a-link-share-by-nextcloud-talk

Add support for sending the password for a link share by Nextcloud Talk

Merge pull request #12200 from nextcloud/tech-debt/noid/cleanup-legacy-sharing

Cleanup some unused sharing methods from the old sharing code

Fallback to default path is sendmail can't be found

If the sendmail binary can't be found at all we fallback to the default
path.
It most likely is not there but then at least a proper error message
pops up.

Updated the tests to also properly pass.

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

Merge pull request #12190 from forsd/AddtestFiles_Trashbin

Add test OCA\Files_Trashbin\Capabilities

Mailer: discover sendmail path instead of hardcoding it to /usr/sbin/sendmail

`sendmail` can very well be in a path different from
`/usr/sbin/sendmail`.

We already search `$PATH` at `lib/private/Settings/Admin/Mail.php` to
detect whether we want to offer sendmail as a mail transfer method, so
let's be consistent and actually initialize `\Swift_SendmailTransport`
with this path to sendmail, instead of just hardcoding
`/usr/sbin/sendmail`.

Signed-off-by: Florian Klink <flokli@flokli.de>

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

suppress wrong audit log messages about failed login attempts

Merge pull request #12198 from nextcloud/fix/noid/during-hard-failure-log-original-exception

Log the previous exception during hard failure

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>

Merge pull request #12182 from nextcloud/3rdparty/sabre-3.2.3

[3rdparty] Bump sabre/dav to 3.2.3

Merge pull request #12207 from nextcloud/item-list-popover-fix

Fix popover menu alignment in list

Cleanup some unused sharing methods from the old sharing code

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

Fix popover menu alignment in list

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

Merge pull request #12177 from nextcloud/fix/apptoken_login_flow_proper_host

Use the proper server for the apptoken flow login

Merge pull request #12195 from nextcloud/fulltextsearch-info-xsd

add the tag fulltextsearch

Merge pull request #12193 from nextcloud/interface-fulltxtsearch-issue-00002

+infoBool / +infoInt

Merge pull request #12194 from nextcloud/fix-path-as-dashboard-is-already-imported

fix path to DashboardManager, as the class is already imported (minor)

[tx-robot] updated from transifex

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

Log the previous exception during hard failure

* otherwise the exception that was caused by the template is logged

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

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

init some var

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

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.

add the tag fulltextsearch

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

fix path

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

+infoBool / +infoInt

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

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

Add test OCA\Files_Trashbin\Capabilities

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

Use the proper server for the apptoken flow login

If a user can't authenticate normally (because they have 2FA that is not
available on their devices for example). The redirect that is generated
should be of the proper format.

This means

1. Include the protocol
2. Include the possible subfolder

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

Fix tests

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

[3rdparty] Bump sabre/dav to 3.2.3

Including deps

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

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>