Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Hi! Do you have any plans about Drupal 8.x version?
Comment | File | Size | Author |
---|---|---|---|
#22 | filefield_sources-drupal_8.x-2354925-22.patch | 233.51 KB | tien.xuan.vo |
Comments
Comment #1
tien.xuan.vo CreditAttribution: tien.xuan.vo commentedCurrently I'm working on 8.x version of this module. You can checkout it here filefield_sources 8.x
I'm almost done. I'll create a patch when every things are done.
Comment #2
tien.xuan.vo CreditAttribution: tien.xuan.vo commentedHere is the patch. Currently, Clipboard is not working on firefox. Imce is not working at all because imce module is not ported yet.
Comment #3
tien.xuan.vo CreditAttribution: tien.xuan.vo commentedHere is new patch which includes:
Sorry, I can not create interdiff text file.
Comment #4
tien.xuan.vo CreditAttribution: tien.xuan.vo commentedMore coding standards warnings and errors fixed.
Comment #5
rootworkHow's progress going on this? Would love to see a new commit in line with recent betas. Currently the most recent version installs cleanly and things can be enabled (took me a minute to find it there on "manage form display" instead of in the field settings themselves, but it's there).
But transferring files doesn't appear to work -- it starts the transfer and then just stops, with no error message or anything written to the log. Reference existing doesn't work either, and in the log I got this message:
I didn't try the other options.
Let us know if there's anything we can do in terms of testing or providing feedback. Thanks so much for working on this incredibly useful module!
Comment #6
rootworkSorry, should have specified that I tried this on Drupal 8.0.0-beta9, which is the most recent beta as of this writing.
Comment #7
tien.xuan.vo CreditAttribution: tien.xuan.vo commentedHere is new patch which includes:
I wonder if how we can enable automated testing for 8.x branch of this module.
Comment #8
tien.xuan.vo CreditAttribution: tien.xuan.vo commentedChange issue status.
Comment #9
DamienMcKennaComment #10
profak CreditAttribution: profak as a volunteer commentedHello @tien.xuan.vo!
What's with your Drupal 8 port? Are you still working on it?
Comment #11
jonathanshawWe have it working with beta 14, upating the work of @tien.xuan.vo. I'll ask our developer to make a pull request to his github.
Comment #12
rivimeyYes please! even a -dev version would be helpful!
Comment #13
profak CreditAttribution: profak as a volunteer commented@jonathanjfshaw
Nice, i will wait for any changes!
Comment #14
jonathanshawOK, our work on D8 Filefield_sourceshas been shared as a PR with tien.xuan.vo's Github repo:
https://github.com/tienvx/filefield_sources/pull/1
It was done in D8 beta 14, I don't know if anything needs doing to make it compatible with RC1. Huge thanks to @tien.xuan.vo!
We've also created a submodule Filefield_sources_flysystem that extends Filefield_sources to have as a source remote filesystems through the Flysystem API. This makes it dead easy to load files from Dropbox, Amazon S3, and many other systems.
Best of all, there's an option "Serve file from current location". This only works when you also have the same Flysystem filesystem as both the source and the storage destination for the file field. What it does is it serves the file to end users from the remote, without storing it on your Drupal server. So you can pick a file on Dropbox, and serve it to visitors, without the file ever touching your site.
If you're interested, pls check it out and help with the review process to get the project upgraded from sandbox:
https://www.drupal.org/node/2604294
You can get the module here:
https://www.drupal.org/sandbox/arit_nath/2604164
Comment #15
Todd Young CreditAttribution: Todd Young commentedProject page indicates this has been included in D8 core? (scratches head)
Comment #16
akalata CreditAttribution: akalata commented@ToddYoung that project page message is generated by contrib_tracker info, which is incorrect.
Comment #17
tien.xuan.vo CreditAttribution: tien.xuan.vo commentedThe MR on #14 has been merged. Many thanks to aritnath1990. Currently all sources are working except Imce. We also need to add test cases for Imce too. MRs are welcome.
Comment #18
giorgio79 CreditAttribution: giorgio79 commentedI would love to see additional sources, such as Google Drive, Facebook etc. This D8 module already integrates with remote providers https://www.drupal.org/project/avatars
Or the patch here could be ported to D8 #1798282: Source: Google Drive
Comment #19
giorgio79 CreditAttribution: giorgio79 commentedChanged issue summary by accident.
Comment #20
idflood CreditAttribution: idflood at Stimul.ch commentedI've enabled this module on a website and it works perfectly except for one error. Here is the error displayed in the log entries:
To reproduce this error:
- enable at least 2 languages
- On an unlimited image field enable translation of the "alt" text.
- upload a file on this field, define an alt
- translate the node
- then finally try to remove the image. This will not work, there is no way to remove the image since the exception happen.
Maybe this can be a followup once the module is officially available for drupal 8.
Comment #21
jonathanshaw#18 @giorgio79 Regarding Google Drive, consider accessing it via Flysystem using Filefield_sources_Flysystem
https://github.com/thephpleague/flysystem/issues/192
https://packagist.org/packages/ignited/flysystem-google-drive
https://www.drupal.org/sandbox/arit_nath/2604164
Comment #22
tien.xuan.vo CreditAttribution: tien.xuan.vo commentedWhat's new in this patch:
Comment #23
idflood CreditAttribution: idflood at Stimul.ch commented@tien.xuan.vo you are right, my issue was eventually caused by another module (inline entity form). I really should have tested on a clean install : /
Comment #24
charos CreditAttribution: charos commentedIsn't the source material enough to release at least an alpha version?
Comment #25
arknoll CreditAttribution: arknoll commentedI think we should at least push this to an 8.x branch at the very least.
Comment #26
grahl+1 for pushing tien.xuan.vo's branch to d.o
I am aware of a bug in the widget though, which I haven't been able to resolve yet. The ajax handler doesn't get attached properly so that regular uploads require a save before the actual uploading happens. Note that this also applies to fields where only the upload source is active, when the module is enabled.
Comment #27
jonathanshawLooks like @profak got maintainer access after @quicksketch focused on Backdrop, but he has not been very active.
Would someone else be up for creating a D8 branch and committing patches, if they were given the necessary permissions?
tien.xuan.vo has been the one really looking after this d8 port.
Comment #28
profak CreditAttribution: profak as a volunteer commentedHi @jonathanjfshaw,
i'm here :) ll create 8.x branch.
Comment #29
jonathanshawAwesome! Thanks :)
Comment #30
Fidelix CreditAttribution: Fidelix as a volunteer commented@profak, when can we expect that?
Thanks.
Comment #31
profak CreditAttribution: profak as a volunteer commented@Fidelix,
branch is available.
I've just started to review code.
Comment #32
gnugetI'm working on a project which required this module be at least beta, is there something to I can do to help moving forward this to reach the first beta?
besides #2755707: WSOD: array_slice($element['#array_parents'], 0, -1); is null it seems to there is nothing critical in the queue.
BTW, Awesome work!
Thanks!
Comment #33
hanoiiWhat's the status of this? There's not much active development, is there any replacement out there?
Comment #34
jonathanshawThere is an 8.x-1.x branch, it's just that it doesn't show up on the project home page.
@profak you can you fix this, and cut an alpha release, even if you've not had time to review the code. People are probably using it in production anyway ...
Comment #35
Shane Birley CreditAttribution: Shane Birley commentedI have been testing out Drupal 8 dev release for the last couple of months and it seems to work without issue. I am able to do everything that I have done on Drupal 7. I will keep pushing it to see if I can break it.
Comment #36
mlncn CreditAttribution: mlncn at Agaric for Drutopia, Teachers with GUTS, National Institute for Children's Health Quality commentedIn addition to the comments noting it is working, we've been using the 8.x-dev branch on a couple production sites for months. This is ready to be tagged with a release!
Comment #37
rivimey@profak / @quicksketch - could you push a release please?
Comment #38
k_zoltan CreditAttribution: k_zoltan at Cylex commentedPlease please please create a release.
Comment #39
k_zoltan CreditAttribution: k_zoltan at Cylex commentedComment #40
GiorgosKOnly thing that does not seem to work is
Reference existing: Search all: Yes
but other than that it seems everything else is working as it should
Comment #41
gnugetIdeally we could create an issue to fix #40 and create the release, yes?
Comment #42
jonathanshawSomeone needs to step up to be a new maintainer it seems
Comment #43
gnugetI sent a message to profak to see if he can help us to create an alpha release of this module.
Let's see if he is around before to start looking for new maintainers.
Comment #44
darrenw16 CreditAttribution: darrenw16 commentedHey installed this module and go to Structure ->Content types->Collective Story->Manage fields try to edit an image field so it can take urls bu the file field options to do this are not there. Is this module working correctly for anyone I installed the 8.x-1.x dev branch.
Comment #45
gnugetI've been using the module for a while.
The settings are now in the "display field settings"
Comment #46
darrenw16 CreditAttribution: darrenw16 commented@gnuget where do I find the display field settings are they still under the Structure area ? Cheers for the reply mate
Its ok found it under Manage -> Structure -> Content types and click "Manage form display" edit an image field. Then you should see file sources. Thanks for the help
Comment #47
gnugetGlad to help!
Comment #48
gnugetI just created the first alpha release of this module!!! :-D
I'm going to close this issue.
Thanks for all the work!
Comment #49
mlncn CreditAttribution: mlncn at Agaric for Drutopia, Teachers with GUTS, National Institute for Children's Health Quality commentedWhoohoo! Congratulations!
Comment #50
OFF CreditAttribution: OFF commentedCool!
Comment #51
DamienMcKenna@gnuget: thanks for stepping up to comaintain this module!