diagnostic_describe_all_properties
DO reference all public properties in debug methods.
This rule is available as of Dart 2.3.
This rule has a quick fix available.
Details
#DO reference all public properties in debug
method implementations.
Implementers of Diagnosticable
should reference all public properties in a debugFillProperties(...)
or debugDescribeChildren(...)
method implementation to improve debuggability at runtime.
Public properties are defined as fields and getters that are
- not package-private (e.g., prefixed with
_
) - not
static
or overriding - not themselves
Widget
s or collections ofWidget
s
In addition, the "debug" prefix is treated specially for properties in Flutter. For the purposes of diagnostics, a property foo
and a prefixed property debugFoo
are treated as effectively describing the same property and it is sufficient to refer to one or the other.
BAD:
class Absorber extends Widget {
bool get absorbing => _absorbing;
bool _absorbing;
bool get ignoringSemantics => _ignoringSemantics;
bool _ignoringSemantics;
@override
void debugFillProperties(DiagnosticPropertiesBuilder properties) {
super.debugFillProperties(properties);
properties.add(DiagnosticsProperty<bool>('absorbing', absorbing));
// Missing reference to ignoringSemantics
}
}
GOOD:
class Absorber extends Widget {
bool get absorbing => _absorbing;
bool _absorbing;
bool get ignoringSemantics => _ignoringSemantics;
bool _ignoringSemantics;
@override
void debugFillProperties(DiagnosticPropertiesBuilder properties) {
super.debugFillProperties(properties);
properties.add(DiagnosticsProperty<bool>('absorbing', absorbing));
properties.add(DiagnosticsProperty<bool>('ignoringSemantics', ignoringSemantics));
}
}
Usage
#To enable the diagnostic_describe_all_properties
rule, add diagnostic_describe_all_properties
under linter > rules in your analysis_options.yaml
file:
linter:
rules:
- diagnostic_describe_all_properties
Unless stated otherwise, the documentation on this site reflects Dart 3.5.4. Page last updated on 2024-07-03. View source or report an issue.