Summary

Add typing for ConditionsWidget.

Review Request #14273 — Created Dec. 16, 2024 and updated Dec. 16, 2024, 9:26 a.m.

Information

Owner

chipx86

Repository

Djblets

Branch

release-5.x

Bugs

Depends On

Reviewers

Groups

djblets

People

Description

This adds typing throughout ConditionsWidget, and fixes some issues
that this unveiled. We had typing assumptions, potential unbound
variables, and bad logic when choices or operators were None.

This is in preparation for some upcoming work on the widget for better
extensibility.

Testing Done

Unit tests pass.

Commits

Summary	ID
Add typing for ConditionsWidget. This adds typing throughout `ConditionsWidget`, and fixes some issues that this unveiled. We had typing assumptions, potential unbound variables, and bad logic when choices or operators were `None`. This is in preparation for some upcoming work on the widget for better extensibility.	018751b9d89de50f5b811bc08bf0f90b8474ea2c

Issues

Description	From	Last Updated
This should be documented with a "Version Changed" in the method docstring.	maubin	Dec. 16, 2024, 9:26 a.m.

flake8 passed.

JSHint passed.

Ship it! Fix it!

djblets/forms/widgets.py (Diff revision 1)

An issue was opened. Show all issues

This should be documented with a "Version Changed" in the method docstring.