Opened 4 years ago
Closed 4 years ago
#32823 closed New feature (duplicate)
Cannot pass extra AJAX settings to admin autocomplete
| Reported by: | Seb G | Owned by: | nobody |
|---|---|---|---|
| Component: | contrib.admin | Version: | 3.2 |
| Severity: | Normal | Keywords: | autocomplete |
| Cc: | Triage Stage: | Unreviewed | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
This ticket is partly linked to #32628 where the bug was reported along with a feature request and got ignored.
The provided JS function $.fn.djangoAdminSelect2(options) allows user to provide extra options that will be passed to select2. But passing extra AJAX options does not work as it rewrites the whole ajax option object instead of extending it.
Example failing call:
$(#my-select).djangoAdminSelect2({ ajax: { processResults: function (data) { console.log("Foo") return data } } });
This call fails because passing an ajax object to $.fn.djangoAdminSelect2 erases the whole default ajax object including the required ajax.data default option.
This failure is caused by the non-recursive call to $.extend and can be easily resolved by adding the recursive parameter to it.
I am available to PR this one quickly if accepted.
Change History (1)
comment:1 by , 4 years ago
| Resolution: | → duplicate |
|---|---|
| Status: | new → closed |
| Type: | Bug → New feature |
Replying to Seb G:
Not ignored but rejected. Again, please don't reopen closed tickets or create duplicates. You can start a discussion on DevelopersMailingList if you don't agree, see triaging guidelines with regards to wontfix tickets.
Duplicate of #32628.