mirrors/django
1
0
Fork 0
mirror of https://github.com/django/django.git synced 2025-02-22 07:24:59 +00:00
Code Issues 32 Releases Wiki Activity

[5.2.x] Fixed #31170 -- Added change event trigger to dismissRelatedLookupPopup.

Browse Source 
Backport of 51398f8bd568a6324a8cafe20c068d0974913ad5 from main.
This commit is contained in:
Кайрат Макым 2025-02-18 09:48:13 +01:00 committed by Sarah Boyce
parent 7cf6a34cb2
commit b0d497ada0
2 changed files with 41 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
django/contrib/admin/static/admin/js/admin/RelatedObjectLookups.js
View File

@ -57,6 +57,7 @@
} else {
elem.value = chosenId;
}
$(elem).trigger('change');
const index = window.relatedWindows.indexOf(win);
if (index > -1) {
window.relatedWindows.splice(index, 1);

40
js_tests/admin/RelatedObjectLookups.test.js
View File

@ -36,3 +36,43 @@ QUnit.test('dismissRelatedLookupPopup removes window from relatedWindows array',
window.dismissRelatedLookupPopup(mockWin, '123');
assert.equal(window.relatedWindows.indexOf(mockWin), -1, 'Window should be removed from relatedWindows array');
});
QUnit.test('dismissRelatedLookupPopup triggers change event for single value field', function(assert) {
assert.timeout(1000);
const done = assert.async();
const $ = django.jQuery;
const testId = 'test_id';
const newValue = '123';
const mockWin = {
name: testId,
close: function() {}
};
let changeTriggered = false;
$('#test_id').on('change', function() {
changeTriggered = true;
assert.equal(this.value, newValue, 'Value should be updated');
done();
});
window.dismissRelatedLookupPopup(mockWin, newValue);
assert.true(changeTriggered, 'Change event should be triggered');
});
QUnit.test('dismissRelatedLookupPopup triggers change event for many-to-many field', function(assert) {
assert.timeout(1000);
const $ = django.jQuery;
const testId = 'many_test_id';
const existingValue = '1,2';
const newValue = '3';
$('#many_test_id').val(existingValue);
const mockWin = {
name: testId,
close: function() {}
};
let changeTriggered = false;
$('#many_test_id').on('change', function() {
changeTriggered = true;
assert.equal(this.value, existingValue + ',' + newValue, 'Value should be appended for many-to-many fields');
});
window.dismissRelatedLookupPopup(mockWin, newValue);
assert.true(changeTriggered, 'Change event should be triggered');
});