Skip to content

Commit b993a54

Browse files
authored
Add whitespace check for required validator (#779)
* Add whitespace check for required validator * Format code_page file
1 parent 4d11ee8 commit b993a54

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

example/lib/code_page.dart

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,4 +36,3 @@ class _CodePageState extends State<CodePage> {
3636
);
3737
}
3838
}
39-

lib/src/form_builder_validators.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ class FormBuilderValidators {
2828
}) {
2929
return (T? valueCandidate) {
3030
if (valueCandidate == null ||
31-
(valueCandidate is String && valueCandidate.isEmpty) ||
31+
(valueCandidate is String && valueCandidate.trim().isEmpty) ||
3232
(valueCandidate is Iterable && valueCandidate.isEmpty) ||
3333
(valueCandidate is Map && valueCandidate.isEmpty)) {
3434
return errorText ??

test/form_builder_validators_test.dart

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
import 'package:flutter/cupertino.dart';
22
import 'package:flutter/material.dart';
3+
import 'package:flutter_form_builder/flutter_form_builder.dart';
34
import 'package:flutter_localizations/flutter_localizations.dart';
45
import 'package:flutter_test/flutter_test.dart';
5-
import 'package:flutter_form_builder/flutter_form_builder.dart';
66

77
/// Test Harness for running Validations
88
Future<void> testValidations(
@@ -69,6 +69,7 @@ void main() {
6969
// Fail
7070
expect(validatorString(null), isNotNull);
7171
expect(validatorString(''), isNotNull);
72+
expect(validatorString(' '), isNotNull);
7273

7374
final validatorList =
7475
FormBuilderValidators.required<List<int>>(context);

0 commit comments

Comments
 (0)