Bug #105755
closedTCA Type Inline -> Upload sometimes does not work
100%
Description
With TCA Type Inline, the Upload field sometimes does not work if you enter several data records without clicking on Save. However, as soon as you save, the field works again. Please fix this urgently. The editors are already annoyed about this.
Files
Updated by Garvin Hicking about 1 month ago
- Status changed from New to Needs Feedback
Can you give more details to reproduce this? "Sometimes" is hard to get into inspecting this.
If you can provide reproducible steps, maybe errors in the browser console, failed network requests, or the DOM state once it fails to work, or screenshots/videos, that will increase the likelihood that someone will work for free on this.
Updated by Tiran Grigoryan about 1 month ago
Garvin Hicking wrote in #note-1:
Can you give more details to reproduce this? "Sometimes" is hard to get into inspecting this.
If you can provide reproducible steps, maybe errors in the browser console, failed network requests, or the DOM state once it fails to work, or screenshots/videos, that will increase the likelihood that someone will work for free on this.
Sorry, here I have made the steps in video and hope that you can get it reproduced
Updated by Garvin Hicking about 1 month ago
Hm, that video is only 22kb and doesn't load for me? If it's too large maybe you can upload it to vimeo or youtube or so?
Updated by Tiran Grigoryan about 1 month ago
Garvin Hicking wrote in #note-3:
Hm, that video is only 22kb and doesn't load for me? If it's too large maybe you can upload it to vimeo or youtube or so?
https://www.loom.com/share/b213fad0a9384a8b9418a0ca75378533?sid=05754950-1068-4e5c-8b57-325d788efd62
Updated by Tiran Grigoryan about 1 month ago
Tiran Grigoryan wrote in #note-4:
Garvin Hicking wrote in #note-3:
Hm, that video is only 22kb and doesn't load for me? If it's too large maybe you can upload it to vimeo or youtube or so?
https://www.loom.com/share/b213fad0a9384a8b9418a0ca75378533?sid=05754950-1068-4e5c-8b57-325d788efd62
Garvin Hicking wrote in #note-3:
Hm, that video is only 22kb and doesn't load for me? If it's too large maybe you can upload it to vimeo or youtube or so?
Could you reproduce it?
Updated by Garvin Hicking about 1 month ago
I could view the video but didn't find time today. Should be able to have a look tomorrow.
Updated by Garvin Hicking about 1 month ago
- Status changed from Needs Feedback to Accepted
- Priority changed from Should have to Must have
Yes, I can reproduce it now. Thank you!
For TYPO3 v14 main (and v13 below) this can be reproduced via EXT:styleguide:
- Create an element of Type "Form engine - inline expand"
- In "inline_1" click on "Create new"
- New section gets added to the bottom
- Clicking on "Select & upload files" there: works
- Click on "Create new" once more
- Clicking on "Select & upload files" on the second element: DOES NOT WORK
- Clicking on "Select & upload files" on the first element: still works
Feels like wrong DOM element triggering/targetting, needs further inspection. I raised the priority on this.
Updated by Tiran Grigoryan about 1 month ago
Garvin Hicking wrote in #note-7:
Yes, I can reproduce it now. Thank you!
For TYPO3 v14 main (and v13 below) this can be reproduced via EXT:styleguide:
- Create an element of Type "Form engine - inline expand"
- In "inline_1" click on "Create new"
- New section gets added to the bottom
- Clicking on "Select & upload files" there: works
- Click on "Create new" once more
- Clicking on "Select & upload files" on the second element: DOES NOT WORK
- Clicking on "Select & upload files" on the first element: still works
Feels like wrong DOM element triggering/targetting, needs further inspection. I raised the priority on this.
When will this bug be fixed? It's very stupid that customers can't maintain their things, they remain dissatisfied with us. Can I help you to fix it as soon as possible?
Updated by Garvin Hicking about 1 month ago
I cannot tell you this with certainty, when the bug can be fixed. Please also try to refrain from using words like "stupid", because it easily triggers people to feel offended.
Progress is made by volunteers or people who are interested in the product, and by very very few companies that pay their developers to work on TYPO3. They have a lot of tasks (see how many open issues this tracker has). We do our best, and everyone has a special area to work in. Sadly, the javascript of this is a bit more complex.
Me and two other core developers already spent a few hours on this, but there was no simple solution.
Do you or your company have people with JavaScript skills? We could talk and I can give pointers where they can help, indeed! The base problem is that no javascript event is delegated/fored to re-trigger the dragzone init constructor.
(Luckily for your reported bug, there's the mentioned workaround in place for saving a new entry first. Many other open bugs don't have workarounds in place)
Best regards.
Garvin
Updated by Tiran Grigoryan about 1 month ago
Garvin Hicking wrote in #note-9:
I cannot tell you this with certainty, when the bug can be fixed. Please also try to refrain from using words like "stupid", because it easily triggers people to feel offended.
Progress is made by volunteers or people who are interested in the product, and by very very few companies that pay their developers to work on TYPO3. They have a lot of tasks (see how many open issues this tracker has). We do our best, and everyone has a special area to work in. Sadly, the javascript of this is a bit more complex.
Me and two other core developers already spent a few hours on this, but there was no simple solution.
Do you or your company have people with JavaScript skills? We could talk and I can give pointers where they can help, indeed! The base problem is that no javascript event is delegated/fored to re-trigger the dragzone init constructor.
(Luckily for your reported bug, there's the mentioned workaround in place for saving a new entry first. Many other open bugs don't have workarounds in place)
Best regards.
Garvin
Hi Garvin,
Sorry about ‘stupid’ word. My English is not so good. And the translator app translated it that way, I speak German. I didn't mean to offend anyone.
Some customers react very aggressively to a mistake or little thing that doesn't work.
I am a backend developer and also have knowledge of JavaScript. I would also like to support you in the backend + JavaScript tasks, if it should be
Updated by Gerrit Code Review about 1 month ago
- Status changed from Accepted to Under Review
Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87409
Updated by Andreas Kienast about 1 month ago
- Related to Task #102415: Remove jQuery from @typo3/backend/drag-uploader added
Updated by Gerrit Code Review about 1 month ago
Patch set 2 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87409
Updated by Garvin Hicking about 1 month ago
@Tiran Grigoryan Thanks for taking the time to explain this. I understand you get pressure from clients, and we must try to still act without too much pressure.
Andreas Kienast created a patch today which I verified working: https://review.typo3.org/c/Packages/TYPO3.CMS/+/87409
Please have a try and vote on the patch if you can verify it works for you, too.
Updated by Tiran Grigoryan about 1 month ago · Edited
Garvin Hicking wrote in #note-14:
@Tiran Grigoryan Thanks for taking the time to explain this. I understand you get pressure from clients, and we must try to still act without too much pressure.
Andreas Kienast created a patch today which I verified working: https://review.typo3.org/c/Packages/TYPO3.CMS/+/87409
Please have a try and vote on the patch if you can verify it works for you, too.
Works for me, thank you very much.
Updated by Gerrit Code Review about 1 month ago
Patch set 1 for branch 13.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/87440
Updated by Anonymous about 1 month ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 0205629a50c6739a57097225f8ae2b8d2c269d9c.
Updated by Garvin Hicking 3 days ago
- Has duplicate Bug #105918: Image Fields not working correctly with restrictions in IRRE fields added