Closed (fixed)
Project:
Stage File Proxy
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
15 Nov 2013 at 15:19 UTC
Updated:
21 Jan 2017 at 07:14 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
chi commentedComment #2
gregglesComment #3
gregglesI just reviewed this and its an interesting idea. One of the theories of this module is that it's a proxy and only downloads things as they are needed.
That said, I could see the benefit of the solution you provided in a few scenarios.
Anyone else have opinions on this proposal?
Comment #4
pere orgaI think that would be a great addition
Comment #5
jelle_sI wrote a similar script that can be used after configuring the module. Just 5 lines of PHP (just for other people who might be looking for this):
Note that depending on how many files there are this might take quite a while, and it might be (or will be) better to convert it to a drupal batch operation or something similar.
Comment #6
sylvainm commentedI think it is a good idea and wrote also on my side this drush command : see attached file.
Comment #7
asrobI've tested SylvainM's patch and it seems to me that will be good. There were some files that couldn't be downloaded but I think it's site/server issue because I couldn't find those files.
Comment #8
greggles@asrob, there are two proposals here: 1 is in a zip and 1 in a patch. Any chance you could review them both and compare/contrast?
Comment #9
BarisW commentedI reviewed both proposals; the patch is created for stage_file_proy while the zip is a more generic approach.
I renamed the drush command to stage-file-proxy-dl instead of stage-file-proxy-download-all, since the drush command overview truncated the latter to stage-file-proxy-dow.
Comment #10
BarisW commentedComment #12
BarisW commentedCommitted to 7.x-1.x-dev. If someone would port this to D8, that would be awesome!
Comment #14
sylvainm commentedHere is a D8 patch
Comment #16
BarisW commentedTested the patch, it works great. Thanks for porting it SylvainM!
Comment #18
BarisW commented