Merge b5d1cd0 into 8da378d

.travis.yml

@@ -1,13 +1,36 @@
 language: python
 
 matrix:
-    include:
-        - python: 2.7
-          env: TOXENV=django110
-        - python: 2.7
-          env: TOXENV=django111
-        - python: 3.5
-          env: TOXENV=django2
+  include:
+    - python: "2.7"
+      env: TOXENV=django111-py27
+
+    - python: "3.5"
+      env: TOXENV=django111-py35
+    - python: "3.5"
+      env: TOXENV=django21-py35
+    - python: "3.5"
+      env: TOXENV=django22-py35
+
+    - python: "3.6"
+      env: TOXENV=django111-py36
+    - python: "3.6"
+      env: TOXENV=django21-py36
+    - python: "3.6"
+      env: TOXENV=django22-py36
+
+    - python: "3.7"
+      sudo: required
+      dist: xenial
+      env: TOXENV=django111-py37
+    - python: "3.7"
+      sudo: required
+      dist: xenial
+      env: TOXENV=django21-py37
+    - python: "3.7"
+      sudo: required
+      dist: xenial
+      env: TOXENV=django22-py37
 
 install:
   - pip install tox

CHANGELOG.rst

@@ -3,7 +3,8 @@ Changelog
 
 next
 ----
-#. Django 2.0 and Python 3.5 support.
+#. Added Django 2.1 and 2.2 along with Python 3.5, 3.6, 3.7 support.
+#. Remove Django 1.10 support.
 
 0.2.3
 -----

README.rst

@@ -35,12 +35,12 @@ Settings
 
 FORMFACTORY["field-types"]
     Control the form fields types that can be selected in Admin.
-    Supports adding none Django fields: ``("<module_for_field>", "<display_name>")``
+    Supports adding non Django fields: ``("<module_for_field>", "<display_name>")``
     eg. ``("formfactory.fields.ParagraphField", "ParagraphField")``
 
 FORMFACTORY["widget-types"]
     Control the form widget types that can be selected in Admin.
-    Supports adding none Django widgets: ``("<module_for_widget>", "<display_name>")``
+    Supports adding non Django widgets: ``("<module_for_widget>", "<display_name>")``
     eg. ``("formfactory.widgets.ParagraphWidget", "ParagraphWidget")``
 
 Widgets and Fields

formfactory/factory.py

@@ -16,6 +16,10 @@
 class FormFactory(forms.Form):
     """Builds a form class from defined fields passed to it by the Form model.
     """
+
+    # TODO: Update uuid in __init__. Forms are readied up during django
+    # startup, meaning the uuid is only set once and not updated for each form
+    # instance.
     uuid = forms.UUIDField(
         initial=str(uuid4()), widget=forms.HiddenInput
     )
@@ -43,6 +47,10 @@ def __init__(self, *args, **kwargs):
         # Models aren't ready when the file is initially processed.
         from formfactory import models
         field_through = models.FieldGroupThrough
+
+        # TODO: Duplicate field groups and fields allowed. Has possible
+        # unintended side effect that only the last field value is ever stored
+        # using the default store data action.
         for field_group in defined_field_groups:
 
             fields = utils.order_by_through(

formfactory/fields.py

@@ -1,9 +1,14 @@
 import markdown
 
+import django
 from django.forms.fields import Field
-from django.utils.text import mark_safe
 from django.utils.translation import ugettext as _
 
+if django.VERSION[0] < 2:
+    from django.utils.text import mark_safe
+else:
+    from django.utils.safestring import mark_safe
+
 from formfactory import widgets
 
 

formfactory/models.py

@@ -1,18 +1,30 @@
 import importlib
 import markdown
 
+import django
 from django import forms
 from django.contrib.contenttypes.fields import GenericForeignKey
 from django.contrib.contenttypes.models import ContentType
 from django.db import models
 from django.urls import reverse
-from django.utils.text import mark_safe
 from django.utils.translation import ugettext as _
 
+if django.VERSION[0] < 2:
+    from django.utils.text import mark_safe
+else:
+    from django.utils.safestring import mark_safe
+
+
 from simplemde.fields import SimpleMDEField
 
 from formfactory import (
-    _registry, actions, clean_methods, factory, SETTINGS, validators, utils
+    SETTINGS,
+    _registry,
+    actions,
+    clean_methods,
+    factory,
+    utils,
+    validators,
 )
 
 
@@ -29,8 +41,10 @@ def _FIELD_TYPES():
             fields = fields + ((content_type, field),)
     return fields
 
+
 FIELD_TYPES = _FIELD_TYPES()
 
+
 def _WIDGET_TYPES():
     widgets = ()
     for content_type, widget in SETTINGS["widget-types"]:
@@ -39,6 +53,7 @@ def _WIDGET_TYPES():
             widgets = widgets + ((content_type, widget),)
     return widgets
 
+
 WIDGET_TYPES = _WIDGET_TYPES()
 
 ERROR_MESSAGES = tuple(
@@ -71,6 +86,8 @@ class Meta(object):
     def __unicode__(self):
         return "%s (%s)" % (self.form.title, self.uuid)
 
+    __str__ = __unicode__
+
 
 class FormDataItem(models.Model):
     """A basic store for form data items.
@@ -90,6 +107,8 @@ class Action(models.Model):
     def __unicode__(self):
         return self.action
 
+    __str__ = __unicode__
+
     @property
     def as_function(self):
         return _registry["actions"][self.action]
@@ -103,6 +122,8 @@ class Validator(models.Model):
     def __unicode__(self):
         return self.validator
 
+    __str__ = __unicode__
+
     @property
     def as_function(self):
         return _registry["validators"][self.validator]
@@ -116,6 +137,8 @@ class CleanMethod(models.Model):
     def __unicode__(self):
         return self.clean_method
 
+    __str__ = __unicode__
+
     @property
     def as_function(self):
         return _registry["clean_methods"][self.clean_method]
@@ -132,6 +155,8 @@ class Meta(object):
     def __unicode__(self):
         return "%s: %s" % (self.key, self.value)
 
+    __str__ = __unicode__
+
 
 class ActionParam(models.Model):
     """Defines a constant that can be passed to the action function.
@@ -145,6 +170,8 @@ class ActionParam(models.Model):
     def __unicode__(self):
         return "%s:%s" % (self.key, self.value)
 
+    __str__ = __unicode__
+
 
 class FormActionThrough(models.Model):
     """Through table for form actions which defines an order.
@@ -161,6 +188,8 @@ class Meta(object):
     def __unicode__(self):
         return "%s (%s)" % (self.action.action, self.order)
 
+    __str__ = __unicode__
+
 
 class BaseFormModel(models.Model):
     """Form model which encompasses a set of form fields and defines an action
@@ -214,6 +243,8 @@ class Meta(object):
     def __unicode__(self):
         return self.title
 
+    __str__ = __unicode__
+
     def get_absolute_url(self):
         if self.enable_csrf:
             return reverse(
@@ -265,6 +296,8 @@ class Wizard(BaseFormModel):
     def __unicode__(self):
         return self.title
 
+    __str__ = __unicode__
+
     def absolute_url(self):
         return self.get_absolute_url()
 
@@ -287,6 +320,8 @@ class Meta(object):
     def __unicode__(self):
         return "%s (%s)" % (self.form.title, self.order)
 
+    __str__ = __unicode__
+
 
 class WizardActionThrough(models.Model):
     """Through table for wizard actions with a defined order.
@@ -303,6 +338,8 @@ class Meta(object):
     def __unicode__(self):
         return "%s (%s)" % (self.action.action, self.order)
 
+    __str__ = __unicode__
+
 
 class FieldChoice(models.Model):
     """Defines options for select or multiselect field types.
@@ -316,6 +353,8 @@ class Meta(object):
     def __unicode__(self):
         return "%s:%s" % (self.label, self.value)
 
+    __str__ = __unicode__
+
 
 class FormFieldGroup(models.Model):
     """Enable the grouping of fields and how that field should be rendered.
@@ -334,6 +373,8 @@ class FormFieldGroup(models.Model):
     def __unicode__(self):
         return self.title
 
+    __str__ = __unicode__
+
 
 class FieldGroupFormThrough(models.Model):
     """Through table for field groups forms with a defined order.
@@ -350,6 +391,8 @@ class Meta(object):
     def __unicode__(self):
         return "%s (%s)" % (self.field_group.title, self.order)
 
+    __str__ = __unicode__
+
 
 class FormField(models.Model):
     """Defines a form field with all options and required attributes.
@@ -399,6 +442,8 @@ class FormField(models.Model):
     def __unicode__(self):
         return self.title
 
+    __str__ = __unicode__
+
     @property
     def get_field_meta(self):
         """
@@ -452,6 +497,8 @@ class Meta(object):
     def __unicode__(self):
         return "%s (%s)" % (self.field.title, self.order)
 
+    __str__ = __unicode__
+
 
 class FormFieldErrorMessageProxy(FormField.error_messages.through):
     class Meta:
@@ -461,6 +508,8 @@ class Meta:
     def __unicode__(self):
         return str(self.customerrormessage)
 
+    __str__ = __unicode__
+
 
 class FormFieldValidatorProxy(FormField.additional_validators.through):
     class Meta:
@@ -469,3 +518,5 @@ class Meta:
 
     def __unicode__(self):
         return str(self.validator)
+
+    __str__ = __unicode__

formfactory/tests/requirements/111.txt

@@ -1,5 +1,15 @@
 Django>=1.11,<2.0
-django-formtools==2.0
+#django-formtools==2.0
+#django-test-without-migrations==0.6
+#Markdown==2.6.7
+#django-simplemde==0.0.9
+
+
+django-formtools==2.1
 django-test-without-migrations==0.6
 Markdown==2.6.7
-django-simplemde==0.0.9
+django-simplemde==0.1.2
+djangorestframework-extras>=0.2.1,<0.3
+#djangorestframework-jwt==1.8.0
+djangorestframework==3.8.2
+six==1.10.0

formfactory/tests/requirements/2.txt → formfactory/tests/requirements/21.txt

@@ -1,10 +1,9 @@
-Django>=2.0rc1<3.0
+Django>=2.1<2.2
 django-formtools==2.1
 django-test-without-migrations==0.6
 Markdown==2.6.7
-django-simplemde==0.1.2
+django-simplemde==0.1.3
 djangorestframework-extras>=0.2.1,<0.3
 #djangorestframework-jwt==1.8.0
 djangorestframework==3.8.2
 six==1.10.0
-

formfactory/tests/requirements/22.txt

@@ -0,0 +1,9 @@
+Django>=2.2<3.0
+django-formtools==2.1
+django-test-without-migrations==0.6
+Markdown==2.6.7
+django-simplemde==0.1.3
+djangorestframework-extras>=0.2.1,<0.3
+#djangorestframework-jwt==1.8.0
+djangorestframework==3.8.2
+six==1.10.0