style fixes in config.sample.php

Fixes in config.sample.php

* owncloud -> ownCloud
* add default value
* proper comment style
* fix line wrapping and minor typo
* remove duplication
* add Vincents proposals
* drop master config switches while backporting

re-arrange the config options and sort them in a semantical way

Conflicts:
config/config.sample.php

unify style of config.sample.php

* use ' instead of " for config option
* place default parameters where useful into sample
* use proper comment block
* limit line size to 80 characters

Fix file size comparator return value

Merge pull request #11375 from owncloud/fix_files_external_amazon_s3_stat_call

Fix files external amazon s3

improved visual feedback if user enabled recovery key

improved visual feedback if recovery key password gets changed

improved visual feedback if recovery key gets enabled/disabled

make sure that the notification gets shown again after a second try

restrict $div to local scope

variable for found avatar div

fix for issue #10483

Merge pull request #11457 from owncloud/stable7-fix-rename-position

fix position of rename field, fix #10867

fix position of rename field, fix #10867

reduce bottom padding in filelist, fix perceived glitching, fix #11213

Add CssImportFilter. More elegant fix for documents#348

fix position and style of sort indicator

we also encrypt/decrypt files in the versions folder for previews and if encryption is enabled/disabled

extract batchDelete(), better comments

files_extern: remove empty Body and ContentLength in Amazon S3 mount

fixes #10501

Conflicts:
apps/files_external/lib/amazons3.php

Merge pull request #11299 from owncloud/barkport-10934

[stable7] Don't complain about non-writable datadirs before we're installed

fix amazon s3 issues

folder size and mtime is always unknown in s3

more s3 fixes

make rescanDelay of root dir configurable, add on the fly update of legacy storage ids, !isset -> empty when checking strings

reduce number of http calls on remove and rmdir

fix typo

make some storage tests explain what went wrong

strict comparison

log exceptions when listing files

Skip test for unsuported feature for amazon s3

Fix copy overwrite for amazon s3

Fix amazon s3 rename overwrite

Fix AmazonS3 rmdir on the root

Redirect only to absolute URL

We do not want to redirect to other domains using the "?redirect_url=" feature. Please notice, that the ownCloud project does not consider open redirectors as security issue.

Use SVG previews for public sharing

Fixes https://github.com/owncloud/core/pull/11367#issuecomment-57306037 and https://github.com/owncloud/core/issues/9218

Add flock to config

This adds a file lock to the config in hope that this prevents race conditions as reported in https://github.com/owncloud/core/issues/11070

Testplan:

- [ ] Delete config.php and make it read-only => Error is thrown that it is not writeable
- [ ] Installation still works
- [ ] Changing config settings works (i.e. using the SMTP config switches in the administration menu)
- [ ] Your PC didn't blow up
- [ ] Installing the news app and the "Disable AppCode checker" app did not destroy your installation

Only skip the main config

Otherwise read only additional configs might not be processed

Test on tmpdir

don't clean up the test environment if a data provider was finished

Try to create datadirectory to test whether the .htaccess works

Fixes https://github.com/owncloud/core/pull/11299#issuecomment-56968588 and also https://github.com/owncloud/core/issues/10628 together with https://github.com/owncloud/core/pull/11299#issuecomment-56968588 when backported.

Don't test for htaccess if we cant write into the datadir anyway

Check for writable datadir during setup

Don't complain about non-writable datadirs before we're installed

Conflicts:
lib/private/util.php

Inject config into checkserver and cleanup tests

Conflicts:
lib/private/util.php

Close the session for preview generation

Without closing the session every preview image generation is locking the session which makes the webinterface unresponsive.

dn needs to be fetched to be able to detect memberOf support

Clear up confusion of log_rotate_size config value

Remove the second comment which could be confused with suggesting an invalid value: `// 100 MiB`.

To keep the easily readable example value, it has been moved into the comment header for the config item.

increase version

Merge pull request #11363 from owncloud/sharing_group_shares_stable7

Sharing group shares stable7

catch and log exception in OC_Helper::cleanTmp()

some small fixed, suggested by scrutinizer

throw a exception if we can't handle the provided path

on unshare only unshare childrens if there is no other parent available

only create a new share entry, if the user needs a different target name than the group share

for group shares we don't need a extra db entry of groupTarget equals itemTarget

for group shares only the parent has the up-to-date permission. Make sure that we always use this permission, except if the user permission is '0' because in this case the user unshared the group share from self

call \OCP\Share::getItemsSharedWithUser() to get exclude list, this way all checks are executed, e.g. to check if the share is really visible

always select permissions, used in getItems()

some small fixes

clear share table after each test run

only add a new row if it isn't a unique share, otherwise update the existing row

fix unit tests

mark exclude list as deprecated. It neither used by the files app nor by the
calendar or contacts app. It doesn't make sense to build a exclude list by the
share API, the apps knows best which are valid targets.

fix add user to group to work with grouped shares

generateTarget() will always find a unique target

unit tests for grouping of shares pointing to the same source

group shares and combine permissions

Merge pull request #11364 from owncloud/fix_unit_test_oc7

make sure that the users file system is initialized before we emit the post hook

make sure that we re-load the file system after the quota was set

make sure that the users file system is initialized before we emit the post hook

Prevent monkey clicking on labels

Clicking on labels while the spinner is there will not trigger the
hidden checkbox any more.

Add ID to markup to have the checkboxes in the right position

Added spinners in share dropdown

Added spinners for the following actions:
- adding user
- removing user
- changing password
- toggling allowing upload
- setting a password

for whitelabeled edition, show branding name (e.g. 'ownCloud') instead of appname in header bar

Fix dav permissions for folders without create permissions

Expose creatable permissions trough fileinfo

Don't keep the full info of all children in memory

Clarify possible preview providers for type Office

a

Conflicts:
lib/private/preview.php

Merge pull request #11245 from owncloud/backport-11211

Add a configuration switch for enabled preview mimetypes

Show a proper error message when trying to scan the filesystem for a non existing user

Updating 3rdparty - contains updated tar archive and getid3

fix filepicker home icon being partly hidden, fix #10169

Check for blacklisted characters

Fixes https://github.com/owncloud/core/issues/11264

(This should in future get moved to the mountpoint class - but that is something for @icewind1991 ;-))

Moved WebDAV and internet checks to client side JS

- Added setup checks in JavaScript
- Moved isWebDAVWorking to JS using SetupChecks
- Moved internet connection checks to an ajax call that goes through the
  server

Backport of e65ceb08fc4a56d0fb9e4be5d51ba04168cbb59a from master

Merge branch 'stable7' into backport-11211

Conflicts:
config/config.sample.php

Previous commit was not based on master, retry. Removed broken tar cutter, double extensions are not possible in temp files. Added tar support. Fixed extension switch.

using flush() here is pointless as we render the layout into a memory buffer and actually transmit the data later

remove post setup check

Backport of 21412559df9461ae7395d8abe1a4c6850dadfad3 from master

Fix share key finding algorithm in various cases

Instead of inaccurate pattern matching, use the list of users who we
know have access to the file to build the list of share keys.

This covers the following cases:

- Move/copy files into a subfolder within a share
- Unsharing from a user
- Deleting files directlry / moving share keys to trashbin

Backport of 1e631754d78e98d74ba0d3fb477d5eb815e9dfb3 from master

Use non-empty defaults

Do only follow HTTP and HTTPS redirects

We do not want to follow redirects to other protocols since they might allow an adversary to bypass network restrictions. (i.e. a redirect to ftp:// might be used to access files of a FTP server which might be in a secure zone and not be reachable from the net but from the ownCloud server)

Get final redirect manually using get_headers()

Migrate to HTTPHelper class and add unit tests

Conflicts:
apps/files/ajax/newfile.php
lib/private/files/storage/dav.php
lib/private/server.php
lib/private/util.php
lib/public/iservercontainer.php

Append port to trusted domain in case it's not 80 or 443

Ref https://github.com/owncloud/core/pull/10584#issuecomment-54677059

Backport of 2590a4dc85cb23311a9000770bf506c3953d0b04 from master

first check if a private key exists, if not it is always a recovery szenario

Add a configuration switch for enabled preview mimetypes

Backport of https://github.com/owncloud/core/pull/11211 to stable7

WebDAV now throws 403 when deletion did not work

Assume a permission issue whenever a file could not be deleted.

This is because some storages are not able to return permissions, so a
permission denied situation can only be triggered during direct
deletion.

Merge pull request #11230 from owncloud/fix-failing-mountconfig-tests-stable7

Load apps in test situations

Log unsuccessful temp file creation and return false

Added extra check to avoid deleting key folders

Whenever a delete operation is called twice in a row, it could happen
that the first call already deleted the file.

The second call would return an empty $ownerPath because the file does
not exist. That empty $ownerPath would run the key deletion operation on
the wrong path.

This fix adds checks in many places to make sure we don't use $ownerPath
when it's empty or null.

Backport of 8aca127e52f965871bea82cb27c6ea103baf34bb from master

date printed in the admin section regarding last execution time is already displayed in the users timezone - no need to append UTC

Reenable file proxy when renaming between mount points

When moving a folder into another mount point, $renamedFiles is empty
because that goes over a different mechanism.

In such case, this fix makes sure that the file proxy is reenable to
avoid breaking the subsequent files that are being moved.

in quota wrapper use === instead of ! for better readability and as in other wrappers

return boolean in Ciose::stream_seek

return boolean in OC::stream_seek

add seek and tell to streamwrapper test

Load apps in test situations