Skip to content

Make it possible to make external storages read only - #8564

Merged
MorrisJobke merged 3 commits into
masterfrom
files_external_read_only
May 1, 2018
Merged

MorrisJobke merged 3 commits into
masterfrom
files_external_read_only

Conversation

@rullzer

@rullzer rullzer commented Feb 27, 2018 •

Copy link
Copy Markdown
Member

Repalces #8315

CC: @Cybso this was the way I suggested. It needs more work but I feel this is way more scaleable and mantainable ;)

Todo:

  • Fix tests
  • Fix sharing (as if you enable sharing and read only it tries to share with to high permissions currently and thus fails).

So it became a bit more.

  1. Added the Read only option in the files external
  2. Added a Read only wrapper
  3. Removed share logic from javascript, we already handle this in dav. And there it should be correct as other clients want to use this as well
  4. Fix the tests

@codecov

codecov Bot commented Mar 8, 2018 •

Copy link
Copy Markdown

Codecov Report

Merging #8564 into master will decrease coverage by 0.39%.
The diff coverage is 35.71%.

@@             Coverage Diff             @@
##             master    #8564     +/-   ##
===========================================
- Coverage     51.76%   51.37%   -0.4%     
+ Complexity    25558    25392    -166     
===========================================
  Files          1627     1541     -86     
  Lines         95939    87509   -8430     
  Branches       1393        0   -1393     
===========================================
- Hits          49667    44954   -4713     
+ Misses        46272    42555   -3717
Impacted Files Coverage Δ Complexity Δ
apps/files_external/lib/Command/ListCommand.php 38.35% <0%> (-0.27%) 39 <0> (ø)
apps/files_external/templates/settings.php 0% <0%> (ø) 0 <0> (ø) ⬇️
apps/dav/lib/Connector/Sabre/Node.php 72.17% <100%> (ø) 48 <0> (+1) ⬆️
lib/private/legacy/util.php 58% <40%> (-0.3%) 243 <2> (+3)
apps/dav/appinfo/routes.php 0% <0%> (ø) 0% <0%> (ø) ⬇️
apps/dav/composer/composer/autoload_classmap.php 0% <0%> (ø) 0% <0%> (ø) ⬇️
remote.php 0% <0%> (ø) 0% <0%> (ø) ⬇️
apps/dav/composer/composer/autoload_static.php 0% <0%> (ø) 1% <0%> (ø) ⬇️
...les_trashbin/composer/composer/autoload_static.php 0% <0%> (ø) 1% <0%> (ø) ⬇️
...s_trashbin/composer/composer/autoload_classmap.php 0% <0%> (ø) 0% <0%> (ø) ⬇️
... and 86 more

@rullzer
rullzer force-pushed the files_external_read_only branch 2 times, most recently from ef13570 to 6ef2da3 Compare March 8, 2018 21:22
@rullzer rullzer added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Mar 9, 2018
$l->t("Check for changes");
$l->t("Never");
$l->t("Once every direct access");
$l->t('Read only');

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

???

Language extraction not working for something?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To be honest I just extended this all. I didn't look into the dark magic of the files_external code yet.

@rullzer
rullzer force-pushed the files_external_read_only branch from 857f378 to ea5393e Compare May 1, 2018 13:20
@rullzer

rullzer commented May 1, 2018

Copy link
Copy Markdown
Member Author

Rebased

@MorrisJobke MorrisJobke left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works 👍

@MorrisJobke
MorrisJobke merged commit e9c6ec4 into master May 1, 2018
@MorrisJobke
MorrisJobke deleted the files_external_read_only branch May 1, 2018 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants