1
0
mirror of https://github.com/django/django.git synced 2025-10-04 12:29:11 +00:00

Fixed #36587 -- Clarified usage of list.insert() for upload handlers.

Thanks Baptiste Mispelon for the report

Co-authored-by: Natalia <124304+nessita@users.noreply.github.com>
This commit is contained in:
okaybro 2025-09-01 23:09:41 +00:00 committed by Sarah Boyce
parent e8190b370e
commit afe6634146
2 changed files with 5 additions and 3 deletions

View File

@ -209,6 +209,7 @@ answer newbie questions, and generally made Django that much better:
Carlton Gibson <carlton.gibson@noumenal.es> Carlton Gibson <carlton.gibson@noumenal.es>
cedric@terramater.net cedric@terramater.net
Chad Whitman <chad.whitman@icloud.com> Chad Whitman <chad.whitman@icloud.com>
Chaitanya Keyal <chaitanyakeyal@gmail.com>
ChaosKCW ChaosKCW
Charlie Leifer <coleifer@gmail.com> Charlie Leifer <coleifer@gmail.com>
charly.wilhelm@gmail.com charly.wilhelm@gmail.com

View File

@ -289,9 +289,10 @@ this handler to your upload handlers like this::
request.upload_handlers.insert(0, ProgressBarUploadHandler(request)) request.upload_handlers.insert(0, ProgressBarUploadHandler(request))
You'd probably want to use ``list.insert()`` in this case (instead of Using ``list.insert()``, as shown above, ensures that the progress bar handler
``append()``) because a progress bar handler would need to run *before* any is placed at the beginning of the list. Since upload handlers are executed in
other handlers. Remember, the upload handlers are processed in order. order, this placement guarantees that the progress bar handler runs before the
default handlers, allowing it to track progress across the entire upload.
If you want to replace the upload handlers completely, you can assign a new If you want to replace the upload handlers completely, you can assign a new
list:: list::