Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
When using a file widget (uploading a file) after selecting the progress bar as progress indicator in the widget settings of a file field, an AJAX error is triggered (basically a 404 Not Found) because the argument to \Drupal\file\Controller\FileWidgetAjaxController::progress
was skipped in file.routing.yml
Proposed resolution
Fix file.routing.yml by adding the argument to the path
Remaining tasks
Apply and commit patch from #1
User interface changes
None.
API changes
None.
Related Issues
None.
(Discovered issue while I was working on #1751382: Selectors clean-up: file module)
Comment | File | Size | Author |
---|---|---|---|
#8 | Screenshot from 2013-11-27 12:26:56.png | 12.66 KB | Jelle_S |
#3 | file-uploadprogress-before.png | 9.99 KB | idebr |
#1 | 2097147-1-file-widget-progress-bar.patch | 456 bytes | Jelle_S |
Comments
Comment #1
Jelle_SAnd the corresponding patch:
Comment #1.0
Jelle_SUpdated issue summary.
Comment #2
johnmcc CreditAttribution: johnmcc commentedI was able to replicate the issue and the patch in #1 resolved it.
Comment #2.0
johnmcc CreditAttribution: johnmcc commentedUpdated issue summary.
Comment #3
idebr CreditAttribution: idebr commentedI can confirm the bug report, see file-uploadprogress-before.png
Patch in #1 resolves this issue
Comment #4
Xano1: 2097147-1-file-widget-progress-bar.patch queued for re-testing.
Comment #6
Jelle_S1: 2097147-1-file-widget-progress-bar.patch queued for re-testing.
Comment #8
Jelle_SI'm guessing this is testbot acting up? :/
Only one fail (screenshot attached). Doesn't seem to have to do anything with the progressbar functionality...
Comment #9
Jelle_S1: 2097147-1-file-widget-progress-bar.patch queued for re-testing.
Comment #10
Jelle_SThere we go. Good boy, testbot, good boy! Have a cookie!
Comment #11
slashrsm CreditAttribution: slashrsm commentedLooks ok. Could we that test coverage for that?
Comment #12
slashrsm CreditAttribution: slashrsm commentedComment #13
Jelle_SNot sure what to add test coverage for here? To check if the argument in the yaml file is set?
Comment #22
AltaGrade CreditAttribution: AltaGrade commentedEven-though #2587755: AJAX error when using progress bar on file field widget was created later than this thread, it has an already committed solution for the same problem.