aboutsummaryrefslogtreecommitdiffstats
path: root/apps/files_sharing/lib/controllers/sharecontroller.php
Commit message (Collapse)AuthorAgeFilesLines
* Move lib/ to PSR-4Joas Schilling2016-05-251-511/+0
|
* Merge pull request #24346 from lenz1111/fix_24331Vincent Petry2016-05-241-0/+2
|\ | | | | Fix: Bugs in shared link with video #24331
| * video posterPiotr Filiciak2016-05-241-0/+2
| |
* | Http Range requests support in downloadsPiotr Filiciak2016-05-201-2/+11
|/ | | | Http range requests support is required for video preview
* move federated sharing settings to the federatedfilesharing appBjörn Schießle2016-04-221-2/+8
|
* Fix archive file name when downloading public shareVincent Petry2016-03-071-1/+1
| | | | | | | | When download a public link share folder using the button on the top right, it doesn't provide a list of files. This fix makes sure to trigger the correct logic when no file list was given.
* Remove Scrutinizer Auto FixerLukas Reschke2016-03-011-1/+0
|
* Update author informationLukas Reschke2016-03-011-0/+2
| | | | Probably nice for the people that contributed to 9.0 to see themselves in the AUTHORS file :)
* check share permissions in share controllerRobin Appelman2016-02-091-0/+17
|
* Example hook checkRoeland Jago Douma2016-02-091-9/+7
|
* throw hooks when accessing a link shareArthur Schiwon2016-02-091-2/+55
|
* [Share 2.0] Fix public link sharingRoeland Jago Douma2016-02-031-8/+8
|
* [Share 2.0] Add exceptions to OCPRoeland Jago Douma2016-02-021-2/+3
|
* [Share 2.0] Fix interfaces and commentsRoeland Jago Douma2016-01-281-15/+15
| | | | | | | | * Made comments more clear * Removed unneeded methods * IShares shareTime is now a proper DateTime object * IShares getPath -> getNode & setPath -> setNode * Fix unit tests
* [Share 2.0] Move IShare to OCPRoeland Jago Douma2016-01-271-3/+2
|
* Directly get from the server containerRoeland Jago Douma2016-01-201-0/+1
| | | | * Updated unit tests
* Do not use deprected activities APIRoeland Jago Douma2016-01-201-21/+33
|
* Get correct path for activity downloadsRoeland Jago Douma2016-01-201-1/+1
|
* [Share 2.0] Make public link work without viewRoeland Jago Douma2016-01-201-2/+13
|
* [Share 2.0] Make link share download use share managerRoeland Jago Douma2016-01-201-53/+63
|
* [Share 2.0] Move tests over the sharemanagerRoeland Jago Douma2016-01-201-29/+22
| | | | Nice side effect... pure unit tests!
* [Share 2.0] Move showing link share to sharing 2.0Roeland Jago Douma2016-01-201-35/+86
|
* Happy new year!Thomas Müller2016-01-121-2/+3
|
* show display name but internally use the user nameBjörn Schießle2015-12-141-0/+1
|
* Scrutinizer Auto-FixesScrutinizer Auto-Fixer2015-12-071-1/+1
| | | | This commit consists of patches automatically generated for this project on https://scrutinizer-ci.com
* Update license headersLukas Reschke2015-10-261-0/+1
|
* Throw exception on `getPath` if file does not existLukas Reschke2015-10-251-2/+1
| | | | | | | | | | Currently the `getPath` methods returned `NULL` in case when a file with the specified ID does not exist. This however mandates that developers are checking for the `NULL` case and if they do not the door for bugs with all kind of impact is widely opened. This is especially harmful if used in context with Views where the final result is limited based on the result of `getPath`, if `getPath` returns `NULL` PHP type juggles this to an empty string resulting in all possible kind of bugs. While one could argue that this is a misusage of the API the fact is that it is very often misused and an exception will trigger an immediate stop of execution as well as log this behaviour and show a pretty error page. I also adjusted some usages where I believe that we need to catch these errors, in most cases this is though simply an error that should hard-fail.
* Set download cookie on public downloadVincent Petry2015-10-081-1/+15
|
* Only intercept exceptions of type "NotFoundException" instead of any ExceptionLukas Reschke2015-09-301-4/+6
| | | | | | The sharing backend may throw another exception for example when the activity app encounters a problem. Previously this just triggered a 404 error page and the exception got not logged at all. With this change such exceptions get not intercepted and regularly handled as exceptions so that we have meaningful log data. Also the user will be shown a window informing him that an error happened. Helps to debug cases such as https://github.com/owncloud/core/issues/19465
* Disable video preview if previews are disabledMorris Jobke2015-06-091-0/+1
|
* Scrutinizer Auto-FixesScrutinizer Auto-Fixer2015-05-191-1/+0
| | | | This commit consists of patches automatically generated for this project on https://scrutinizer-ci.com
* Remove IDE autocomplete failLukas Reschke2015-03-261-1/+0
| | | | :see_no_evil: *cough* *cough*
* Update license headersJenkins for ownCloud2015-03-261-5/+21
|
* Add wrapper for GuzzleLukas Reschke2015-03-251-0/+2
|
* when the path doesnt exist show a proper error pageRobin Appelman2015-03-241-0/+2
|
* Properly catch whether a share is `null`Lukas Reschke2015-03-241-19/+19
| | | | | | | | | | | | | | | | | | | | | Despite it's PHPDoc the function might return `null` which was not properly catched and thus in some situations the share was resolved to the sharing users root directory. To test this perform the following steps: * Share file in owncloud 7 (7.0.4.2) * Delete the parent folder of the shared file * The share stays is in the DB and the share via the sharelink is inaccessible. (which is good) * Upgrade to owncloud 8 (8.0.2) (This step is crucial. The bug is not reproduceable without upgrading from 7 to 8. It seems like the old tokens are handled different than the newer ones) * Optional Step: Logout, Reset Browser Session, etc. * Access the share via the old share url: almost empty page, but there is a dowload button which adds a "/download" to the URL. * Upon clicking, a download.zip is downloaded which contains EVERYTHING from the owncloud directory (of the user who shared the file) * No exception is thrown and no error is logged. This will add a check whether the share is a valid one and also adds unit tests to prevent further regressions in the future. Needs to be backported to ownCloud 8. Adding a proper clean-up of the orphaned shares is out-of-scope and would probably require some kind of FK or so. Fixes https://github.com/owncloud/core/issues/15097
* Move mimetype check from template to controllerJoas Schilling2015-03-161-0/+1
|
* Allow iframes from same domain in share viewLukas Reschke2015-03-101-1/+6
| | | | | | | | This is required because the PDF Viewer itself is embedded using an iframe from the same domain. The default policy is blocking this. Going on further, we have to come up with a solution in the future how to handle previews by applications, one example might be that they call their own endpoint and not the generic share page to allow applications to have full control over how to display previews. Anyways, to test this behaviour use a decent newer browser (such as Chrome 41) and share a PDF file, obviously the PDF viewer needs to be enabled as well. Without this patch publicly shared PDF files should not get previewed and an error is thrown. (if it isn't then your browser is probably not obeying our Content-Security-Policy and you might consider switching to another one ;))
* Merge pull request #14530 from owncloud/revert-14403Thomas Müller2015-02-271-19/+6
|\ | | | | Revert "Updating license headers"
| * Revert "Updating license headers"Morris Jobke2015-02-261-19/+6
| | | | | | | | This reverts commit 6a1a4880f0d556fb090f19a5019fec31916f5c36.
* | Remove uneeded assignmentLukas Reschke2015-02-241-2/+0
|/ | | | 'files' is neither used in the controller or the template
* Updating license headersJenkins for ownCloud2015-02-231-6/+19
|
* Correctly create activities for public downloadsJoas Schilling2015-02-171-13/+35
|
* Merge pull request #13425 from owncloud/phpdoc_cleanupMorris Jobke2015-02-101-1/+1
|\ | | | | Cleanup of PHPDoc return types
| * Cleanup of PHPDoc return typesRobin McCorkell2015-01-161-1/+1
| |
* | use uid provided by setupfs hook to mount server2server shares, otherwise ↵Bjoern Schiessle2015-01-261-1/+0
| | | | | | | | mount will fail for public link shares
* | show animated gifs on public sharing pageGeorg Ehrke2015-01-191-1/+4
|/
* only send activity if we have a valid path and the file is readableBjoern Schiessle2014-12-231-11/+9
|
* Make sure the path passed to the activity manager is normalizedRobin Appelman2014-12-221-1/+1
|
* add activity if public shared file/folder was downloadedBjoern Schiessle2014-12-221-1/+15
|