feat(ISSUE-8): Formatting m2m confirmation values as html lists

Makefile

@@ -18,10 +18,10 @@ package:
 	python3 setup.py sdist bdist_wheel
 
 upload-testpypi:
-	python3 -m twine upload --repository testpypi dist/django_admin_confirm-$(VERSION)*
+	python3 -m twine upload --repository testpypi dist/django_admin_confirm-$(VERSION)
 
 i-have-tested-with-testpypi-and-am-ready-to-release:
-	python3 -m twine upload --repository pypi dist/django_admin_confirm-$(VERSION)*
+	python3 -m twine upload --repository pypi dist/django_admin_confirm-$(VERSION)
 
 install-testpypi:
-	python -m pip install --index-url https://test.pypi.org/simple/ django_admin_confirm
+	python -m pip install --index-url https://test.pypi.org/simple/ django_admin_confirm==${VERSION}

admin_confirm/admin.py

@@ -130,7 +130,7 @@ def _get_changed_data(
                     field_object = model._meta.get_field(name)
                     initial_value = getattr(obj, name)
                     if isinstance(field_object, ManyToManyField):
-                        initial_value = field_object.value_to_string(obj)
+                        initial_value = field_object.value_from_object(obj)
 
                     if initial_value != new_value:
                         changed_data[name] = [initial_value, new_value]

admin_confirm/templatetags/formatting.py

@@ -1,12 +1,25 @@
 from django import template
 from django.db.models.query import QuerySet
+from django.utils.html import escape
+from django.utils.safestring import mark_safe
 
 register = template.Library()
 
 
 @register.filter
 def format_change_data_field_value(field_value):
-    if isinstance(field_value, QuerySet):
-        return list(field_value)
+    if isinstance(field_value, str):
+        return field_value
+    try:
+        output = "<ul>"
+        for value in iter(field_value):
+            output += "<li>" + escape(value) + "</li>"
+        output += "</ul>"
+        return mark_safe(output)
+    except:
+        return field_value
 
-    return field_value
+    # if isinstance(field_value, QuerySet):
+    #     return list(field_value)
+
+    # return field_value

pytest.ini

@@ -1,4 +1,4 @@
 [pytest]
 DJANGO_SETTINGS_MODULE=tests.test_project.settings.test
-addopts = --doctest-modules -ra -l --tb=short --show-capture=log --color=yes
+addopts = --doctest-modules -ra -l --tb=short --show-capture=stdout --color=yes
 testpaths = admin_confirm

setup.py

@@ -6,7 +6,7 @@
 
 setup(
     name="django-admin-confirm",
-    version="0.2.3.dev2",
+    version="0.2.3.dev3",
     packages=["admin_confirm"],
     description="Adds confirmation to Django Admin changes, additions and actions",
     long_description_content_type="text/markdown",

tests/test_project/settings/__init__.py

@@ -1 +1,6 @@
-from .local import *  # noqa
+# For some reason the working directory of tests
+# And running the server are different
+# (Possibly due to test_project being within a subfolder)
+# This defaults settings to local unless
+# DJANGO_SETTINGS is specified.
+from .local import *