tooreht / django-jsonsuit Goto Github PK

The widget looks very useful but it is very difficult to understand how to use it w/o working examples.

• django-jsonsuit version: 0.4.4
• Django version: 3.0.5
• Python version: 3.8.1
• Operating System: Mac Os Catalina

Description

Disabling change permissions for the admin user in my ModelAdmin results in JSONSuit stopping to work.

What I Did

I tried setting up my fields with either the JSONSuit or the ReadOnlyJSONSuit widgets.

There is a workaround which involves faking the change form layout, but for obvious reasons I would avoid going down that path, if possible.

• django-jsonsuit version: 0.4.2
• Django version: 1,11,2 final
• Python version: 3.6
• Operating System: windows

Description

do not accept any parameter name except "data"
{% jsonsuit data %}

What I Did

views.py
...
return render(request, 'sensor_list.html', {'sensor': sensorTypeModel.jsonTemplate})
...
expect:
in file sensor_list.html
{% jsonsuit sensor %} is working

but I have error
is not JSON serializable

Paste the command(s) you ran and the output.
If there was a crash, please include the traceback here.

• django-jsonsuit version: 0.4.4
• Django version: 3.1.7
• Python version: 3.8.5
• Operating System: WSL Ubuntu

Description

When trying to add multiple instances of the same form to a page, controls from a JSON field on any instance of the form operate the same field on the first instance of a form.

Steps to Reproduce

1. Create a model class having a corresponding form with a JSONSuit field.
2. Pass an array of form instances representing said model class to a template and render them using a loop, e.g.

{% for form in model_forms %}
    {{ form }}
{% endfor %}

3. Attempt to use the controls on an instance of the JSONSuit field.

What I Expected

When clicking on "Raw", I expect it to switch that field to raw mode.

What Actually Happened

When clicking on "Raw", the corresponding field on the first instance of the form on that page switched to raw mode.

Initial Findings

Right now the code for JsonSuit assumes the field will have an id of id_{field.name}, which is generally correct. However, in the case of more than one instance of the form, it selects the first element matching that ID, which will be on the first form.

Generally you could mitigate this by changing the auto_id parameter on the Form object, which alters the ID generated for all fields (its default value is id_%s). However, because JSONSuit doesn't use this when calculating the ID it expects for a given field, changing that value will break JSONSuit entirely.