Skip to content

Commit 4f0172f

Browse files
committed
Rename widget_id to field_id to increase consistency
1 parent 270e5b5 commit 4f0172f

File tree

3 files changed

+7
-8
lines changed

3 files changed

+7
-8
lines changed

django_select2/forms.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -221,6 +221,7 @@ def __init__(self, attrs=None, choices=(), **kwargs):
221221
self.attrs = {}
222222

223223
self.uuid = str(uuid.uuid4())
224+
self.field_id = signing.dumps(self.uuid)
224225
self.data_view = kwargs.pop('data_view', None)
225226
self.data_url = kwargs.pop('data_url', None)
226227

@@ -253,9 +254,7 @@ def build_attrs(self, base_attrs, extra_attrs=None):
253254

254255
attrs = super().build_attrs(default_attrs, extra_attrs=extra_attrs)
255256

256-
self.widget_id = signing.dumps(self.uuid)
257-
258-
attrs['data-field_id'] = self.widget_id
257+
attrs['data-field_id'] = self.field_id
259258

260259
attrs['class'] += ' django-select2-heavy'
261260
return attrs

tests/test_forms.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -388,7 +388,7 @@ def test_ajax_view_registration(self, client):
388388
widget.render('name', 'value')
389389
url = reverse('django_select2:auto-json')
390390
genre = Genre.objects.last()
391-
response = client.get(url, data=dict(field_id=widget.widget_id,
391+
response = client.get(url, data=dict(field_id=widget.field_id,
392392
term=genre.title))
393393
assert response.status_code == 200, response.content
394394
data = json.loads(response.content.decode('utf-8'))

tests/test_views.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ def test_get(self, client, artists):
2020
artist = artists[0]
2121
form = AlbumModelSelect2WidgetForm()
2222
assert form.as_p()
23-
field_id = form.fields['artist'].widget.widget_id
23+
field_id = form.fields['artist'].widget.field_id
2424
url = reverse('django_select2:auto-json')
2525
response = client.get(url, {'field_id': field_id, 'term': artist.title})
2626
assert response.status_code == 200
@@ -55,7 +55,7 @@ def test_pagination(self, genres, client):
5555
search_fields=['title__icontains']
5656
)
5757
widget.render('name', None)
58-
field_id = widget.widget_id
58+
field_id = widget.field_id
5959

6060
response = client.get(url, {'field_id': field_id, 'term': ''})
6161
assert response.status_code == 200
@@ -76,7 +76,7 @@ def test_label_from_instance(self, artists, client):
7676
form = AlbumModelSelect2WidgetForm()
7777
form.fields['artist'].widget = ArtistCustomTitleWidget()
7878
assert form.as_p()
79-
field_id = form.fields['artist'].widget.widget_id
79+
field_id = form.fields['artist'].widget.field_id
8080

8181
artist = artists[0]
8282
response = client.get(url, {'field_id': field_id, 'term': artist.title})
@@ -90,7 +90,7 @@ def test_url_check(self, client, artists):
9090
artist = artists[0]
9191
form = AlbumModelSelect2WidgetForm()
9292
assert form.as_p()
93-
field_id = form.fields['artist'].widget.widget_id
93+
field_id = form.fields['artist'].widget.field_id
9494
cache_key = form.fields['artist'].widget._get_cache_key()
9595
widget_dict = cache.get(cache_key)
9696
widget_dict['url'] = 'yet/another/url'

0 commit comments

Comments
 (0)