Support validating cache backends on Django 1.6 and 1.11.

Review Request #10506 — Created April 3, 2019 and submitted — Latest diff uploaded

Information

Review Board
release-4.0.x

Reviewers

This adds support for validating the cache backend in the General
Settings page on Django 1.11. We no longer have access to get_cache(),
which was present on Django 1.6, so we now duplicate the functionality
by importing the backend manually and then constructing an instance of
the backend.

In order to test this on Django 1.11, a change needed to be made to the
cache field display logic for the form. Modern versions set the
required attribute on required fields, which meant that the form
couldn't be submitted due to the hidden fields for other backends. We
now dynamically set the required field based on the current backend
and the presence of a required CSS class on the label (which is
available on both).

Tested the form on Django 1.6 and 1.11. Checked for both validation
failures and successful validation.

Commits

Files

    Loading...