Rename controller variables in DeviceSearchFilters to be private to control access control.

This commit is contained in:
fkarmoush
2025-03-26 12:40:25 +03:00
parent f670ae78aa
commit 835dfe8785

View File

@ -14,32 +14,32 @@ class DeviceSearchFilters extends StatefulWidget {
class _DeviceSearchFiltersState extends State<DeviceSearchFilters> class _DeviceSearchFiltersState extends State<DeviceSearchFilters>
with HelperResponsiveLayout { with HelperResponsiveLayout {
late final TextEditingController unitNameController; late final TextEditingController _unitNameController;
late final TextEditingController productNameController; late final TextEditingController _productNameController;
@override @override
void initState() { void initState() {
unitNameController = TextEditingController(); _unitNameController = TextEditingController();
productNameController = TextEditingController(); _productNameController = TextEditingController();
super.initState(); super.initState();
} }
@override @override
void dispose() { void dispose() {
unitNameController.dispose(); _unitNameController.dispose();
productNameController.dispose(); _productNameController.dispose();
super.dispose(); super.dispose();
} }
List<Widget> get _widgets => [ List<Widget> get _widgets => [
_buildSearchField( _buildSearchField(
"Space Name", "Space Name",
unitNameController, _unitNameController,
200, 200,
), ),
_buildSearchField( _buildSearchField(
"Device Name / Product Name", "Device Name / Product Name",
productNameController, _productNameController,
300, 300,
), ),
_buildSearchResetButtons(), _buildSearchResetButtons(),
@ -73,8 +73,8 @@ class _DeviceSearchFiltersState extends State<DeviceSearchFilters>
controller: controller, controller: controller,
onSubmitted: () { onSubmitted: () {
final searchDevicesEvent = SearchDevices( final searchDevicesEvent = SearchDevices(
productName: productNameController.text, productName: _productNameController.text,
unitName: unitNameController.text, unitName: _unitNameController.text,
searchField: true, searchField: true,
); );
context.read<DeviceManagementBloc>().add(searchDevicesEvent); context.read<DeviceManagementBloc>().add(searchDevicesEvent);
@ -87,14 +87,14 @@ class _DeviceSearchFiltersState extends State<DeviceSearchFilters>
return SearchResetButtons( return SearchResetButtons(
onSearch: () => context.read<DeviceManagementBloc>().add( onSearch: () => context.read<DeviceManagementBloc>().add(
SearchDevices( SearchDevices(
unitName: unitNameController.text, unitName: _unitNameController.text,
productName: productNameController.text, productName: _productNameController.text,
searchField: true, searchField: true,
), ),
), ),
onReset: () { onReset: () {
unitNameController.clear(); _unitNameController.clear();
productNameController.clear(); _productNameController.clear();
context.read<DeviceManagementBloc>() context.read<DeviceManagementBloc>()
..add(ResetFilters()) ..add(ResetFilters())
..add(FetchDevices(context)); ..add(FetchDevices(context));