openwisp / openwisp-monitoring

Network monitoring system written in Python and Django, designed to be extensible, programmable, scalable and easy to use by end users: once the system is configured, monitoring checks, alerts and metric collection happens automatically.
https://openwisp.io/docs/dev/monitoring/
Other
165 stars 112 forks source link

[bug] WifiSession tabular inline may throw error innecessarily #504

Closed nemesifier closed 1 year ago

nemesifier commented 1 year ago

This bug affects only the development version (current master branch).

If the WiFi session information of a device changes while the user is modifying any detail of the device, the system will throw a validation error:

(Hidden field TOTAL_FORMS) This field is required. (Hidden field INITIAL_FORMS) This field is required.

ManagementForm data is missing or has been tampered with. Missing fields: wifisession_set-TOTAL_FORMS, wifisession_set-INITIAL_FORMS. You may need to file a bug report if the issue persists.

This happens because the WiFi session information is a form.

We have to rewrite this part to avoid using a real inline and simulate it somehow while building the same output without a form. This will allow us to get rid of this nasty error.