Skip to content

Commit ee22105

Browse files
authored
Merge pull request #226 from novafloss/225-remove-size-limit-item-label
Remove the field size limit for the model field `formidable.models.Item.label`
2 parents e49e931 + 5e55a57 commit ee22105

File tree

4 files changed

+35
-1
lines changed

4 files changed

+35
-1
lines changed

CHANGELOG.rst

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ master (unreleased)
66
===================
77

88
- Added a tox job to update/refresh the swagger-ui related static files (#210 / #213) - including documentation for developers.
9+
- Remove the field size limit for the model field `formidable.models.Item.label` (#225).
910

1011
Release 0.10.0 (2017-04-28)
1112
===========================

demo/tests/test_models.py

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,20 @@ class FormTest(FormidableForm):
2323
)
2424

2525

26+
class FieldItemTestCase(TestCase):
27+
def setUp(self):
28+
super(FieldItemTestCase, self).setUp()
29+
self.formidable = FormTest.to_formidable(label='label')
30+
31+
def test_field_size(self):
32+
field = self.formidable.fields.get(slug='dropdown')
33+
field.items.create(
34+
value='hello',
35+
label="hello" * 800,
36+
order=42,
37+
)
38+
39+
2640
class UnicodeTestCase(TestCase):
2741

2842
def setUp(self):
Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
# -*- coding: utf-8 -*-
2+
from __future__ import unicode_literals
3+
4+
from django.db import migrations, models
5+
6+
7+
class Migration(migrations.Migration):
8+
9+
dependencies = [
10+
('formidable', '0002_remove_access_display'),
11+
]
12+
13+
operations = [
14+
migrations.AlterField(
15+
model_name='item',
16+
name='label',
17+
field=models.TextField(),
18+
),
19+
]

formidable/models.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ def __str__(self):
104104
class Item(models.Model):
105105
field = models.ForeignKey(Field, related_name='items')
106106
value = models.CharField(max_length=256)
107-
label = models.CharField(max_length=256)
107+
label = models.TextField()
108108
order = models.IntegerField()
109109
help_text = models.TextField(blank=True, null=True)
110110

0 commit comments

Comments
 (0)