GridFieldBulkEditingTools icon indicating copy to clipboard operation
GridFieldBulkEditingTools copied to clipboard

Selecting from assets does not work.

Open martinviolet88 opened this issue 4 years ago • 9 comments

Selecting images from the image libary does not work. After selecting they do appear under the bulk uploader but not in the grifield. After doing a hard ctrl shift r it does seem to work. So my guess it's a caching error? No error's found.

screencapture-localhost-theaterv2-admin-Artiesten-Producent-ArtiestPage-EditForm-field-ArtiestPage-item-35157-edit-2020-02-10-18_10_11 (1)

martinviolet88 avatar Feb 10 '20 17:02 martinviolet88

any update on this?

guyvanbael avatar Nov 09 '20 12:11 guyvanbael

@guyvanbael Nope, having the same problem?

martinviolet88 avatar Nov 13 '20 19:11 martinviolet88

I think this PR might also fix this issue: https://github.com/colymba/GridFieldBulkEditingTools/pull/205. Or possibly https://github.com/colymba/GridFieldBulkEditingTools/pull/206

kinglozzer avatar Nov 23 '20 15:11 kinglozzer

@kinglozzer just used dev-master in my latest project, but issue still occuring

guyvanbael avatar Nov 24 '20 07:11 guyvanbael

same issue here, after some time investigating the problem I didn't manage to find error or something... ended up here :)

EdvJa avatar Dec 02 '20 11:12 EdvJa

Did this ever get resolved? We've just discovered the same problem, also using dev-master. However for us it seems to only be a problem about 50% of the time.

acndr avatar Sep 29 '21 19:09 acndr

After some digging I think I've found the problem to this. In the uploader.js file the bulkUploadFieldReducer reducer is wrapped with a JS DOMContentLoaded. However this file is only requested when a SiteTree page is loaded up in the CMS, meaning the DOM is already loaded and this reducer never gets added. This is only a problem when you load the CMS via /admin then browse to a page to edit - if you refresh on a SiteTree page or route straight to a page (e.g. /admin/pages/edit/show/1) then the correct events are fired and everything works.

I don't know enough about the admin javascript lifecycle to confidently create a PR for this, but perhaps someone else can use this info for a fix?

@kinglozzer

acndr avatar Oct 03 '21 19:10 acndr

Would be awesome if we can get it fixxed;), i'm so annoyed by it that im willing to pay for someone to fix it haha..

martinviolet88 avatar Oct 19 '21 07:10 martinviolet88

need this fixed... but not the jansneeuw namespace changes

mikey-harveycameron avatar Sep 08 '23 03:09 mikey-harveycameron