Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.

Commit 1e5efd1

Browse files
[various] Enable use_build_context_synchronously (#6585)
Enables the `use_build_context_synchronously` lint, and fixes violations. Part of flutter/flutter#76229
1 parent 8bab180 commit 1e5efd1

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

41 files changed

+304
-204
lines changed

analysis_options.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -241,7 +241,7 @@ linter:
241241
- unnecessary_to_list_in_spreads
242242
- unrelated_type_equality_checks
243243
- unsafe_html
244-
# - use_build_context_synchronously # LOCAL CHANGE - Needs to be enabled and violations fixed.
244+
- use_build_context_synchronously
245245
# - use_colored_box # not yet tested
246246
# - use_decorated_box # not yet tested
247247
# - use_enums # not yet tested

packages/file_selector/file_selector/CHANGELOG.md

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
## NEXT
22

3+
* Updates example code for `use_build_context_synchronously` lint.
34
* Updates minimum Flutter version to 3.0.
45

56
## 0.9.2+2

packages/file_selector/file_selector/example/lib/get_directory_page.dart

+6-4
Original file line numberDiff line numberDiff line change
@@ -22,10 +22,12 @@ class GetDirectoryPage extends StatelessWidget {
2222
// Operation was canceled by the user.
2323
return;
2424
}
25-
await showDialog<void>(
26-
context: context,
27-
builder: (BuildContext context) => TextDisplay(directoryPath),
28-
);
25+
if (context.mounted) {
26+
await showDialog<void>(
27+
context: context,
28+
builder: (BuildContext context) => TextDisplay(directoryPath),
29+
);
30+
}
2931
}
3032

3133
@override

packages/file_selector/file_selector/example/lib/open_image_page.dart

+6-4
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,12 @@ class OpenImagePage extends StatelessWidget {
2929
final String fileName = file.name;
3030
final String filePath = file.path;
3131

32-
await showDialog<void>(
33-
context: context,
34-
builder: (BuildContext context) => ImageDisplay(fileName, filePath),
35-
);
32+
if (context.mounted) {
33+
await showDialog<void>(
34+
context: context,
35+
builder: (BuildContext context) => ImageDisplay(fileName, filePath),
36+
);
37+
}
3638
}
3739

3840
@override

packages/file_selector/file_selector/example/lib/open_multiple_images_page.dart

+6-4
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,12 @@ class OpenMultipleImagesPage extends StatelessWidget {
3232
// Operation was canceled by the user.
3333
return;
3434
}
35-
await showDialog<void>(
36-
context: context,
37-
builder: (BuildContext context) => MultipleImagesDisplay(files),
38-
);
35+
if (context.mounted) {
36+
await showDialog<void>(
37+
context: context,
38+
builder: (BuildContext context) => MultipleImagesDisplay(files),
39+
);
40+
}
3941
}
4042

4143
@override

packages/file_selector/file_selector/example/lib/open_text_page.dart

+6-4
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,12 @@ class OpenTextPage extends StatelessWidget {
3232
final String fileName = file.name;
3333
final String fileContent = await file.readAsString();
3434

35-
await showDialog<void>(
36-
context: context,
37-
builder: (BuildContext context) => TextDisplay(fileName, fileContent),
38-
);
35+
if (context.mounted) {
36+
await showDialog<void>(
37+
context: context,
38+
builder: (BuildContext context) => TextDisplay(fileName, fileContent),
39+
);
40+
}
3941
}
4042

4143
@override

packages/file_selector/file_selector_ios/CHANGELOG.md

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
## NEXT
22

3+
* Updates example code for `use_build_context_synchronously` lint.
34
* Updates minimum Flutter version to 3.0.
45

56
## 0.5.0+2

packages/file_selector/file_selector_ios/example/lib/open_image_page.dart

+6-4
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,12 @@ class OpenImagePage extends StatelessWidget {
2929
final String fileName = file.name;
3030
final String filePath = file.path;
3131

32-
await showDialog<void>(
33-
context: context,
34-
builder: (BuildContext context) => ImageDisplay(fileName, filePath),
35-
);
32+
if (context.mounted) {
33+
await showDialog<void>(
34+
context: context,
35+
builder: (BuildContext context) => ImageDisplay(fileName, filePath),
36+
);
37+
}
3638
}
3739

3840
@override

packages/file_selector/file_selector_ios/example/lib/open_multiple_images_page.dart

+6-4
Original file line numberDiff line numberDiff line change
@@ -34,10 +34,12 @@ class OpenMultipleImagesPage extends StatelessWidget {
3434
// Operation was canceled by the user.
3535
return;
3636
}
37-
await showDialog<void>(
38-
context: context,
39-
builder: (BuildContext context) => MultipleImagesDisplay(files),
40-
);
37+
if (context.mounted) {
38+
await showDialog<void>(
39+
context: context,
40+
builder: (BuildContext context) => MultipleImagesDisplay(files),
41+
);
42+
}
4143
}
4244

4345
@override

packages/file_selector/file_selector_ios/example/lib/open_text_page.dart

+6-4
Original file line numberDiff line numberDiff line change
@@ -26,10 +26,12 @@ class OpenTextPage extends StatelessWidget {
2626
final String fileName = file.name;
2727
final String fileContent = await file.readAsString();
2828

29-
await showDialog<void>(
30-
context: context,
31-
builder: (BuildContext context) => TextDisplay(fileName, fileContent),
32-
);
29+
if (context.mounted) {
30+
await showDialog<void>(
31+
context: context,
32+
builder: (BuildContext context) => TextDisplay(fileName, fileContent),
33+
);
34+
}
3335
}
3436

3537
@override

packages/file_selector/file_selector_linux/CHANGELOG.md

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
## NEXT
22

3+
* Updates example code for `use_build_context_synchronously` lint.
34
* Updates minimum Flutter version to 3.0.
45

56
## 0.9.1

packages/file_selector/file_selector_linux/example/lib/get_directory_page.dart

+6-4
Original file line numberDiff line numberDiff line change
@@ -21,10 +21,12 @@ class GetDirectoryPage extends StatelessWidget {
2121
// Operation was canceled by the user.
2222
return;
2323
}
24-
await showDialog<void>(
25-
context: context,
26-
builder: (BuildContext context) => TextDisplay(directoryPath),
27-
);
24+
if (context.mounted) {
25+
await showDialog<void>(
26+
context: context,
27+
builder: (BuildContext context) => TextDisplay(directoryPath),
28+
);
29+
}
2830
}
2931

3032
@override

packages/file_selector/file_selector_linux/example/lib/get_multiple_directories_page.dart

+7-4
Original file line numberDiff line numberDiff line change
@@ -21,10 +21,13 @@ class GetMultipleDirectoriesPage extends StatelessWidget {
2121
// Operation was canceled by the user.
2222
return;
2323
}
24-
await showDialog<void>(
25-
context: context,
26-
builder: (BuildContext context) => TextDisplay(directoryPaths.join('\n')),
27-
);
24+
if (context.mounted) {
25+
await showDialog<void>(
26+
context: context,
27+
builder: (BuildContext context) =>
28+
TextDisplay(directoryPaths.join('\n')),
29+
);
30+
}
2831
}
2932

3033
@override

packages/file_selector/file_selector_linux/example/lib/open_image_page.dart

+6-4
Original file line numberDiff line numberDiff line change
@@ -28,10 +28,12 @@ class OpenImagePage extends StatelessWidget {
2828
final String fileName = file.name;
2929
final String filePath = file.path;
3030

31-
await showDialog<void>(
32-
context: context,
33-
builder: (BuildContext context) => ImageDisplay(fileName, filePath),
34-
);
31+
if (context.mounted) {
32+
await showDialog<void>(
33+
context: context,
34+
builder: (BuildContext context) => ImageDisplay(fileName, filePath),
35+
);
36+
}
3537
}
3638

3739
@override

packages/file_selector/file_selector_linux/example/lib/open_multiple_images_page.dart

+6-4
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,12 @@ class OpenMultipleImagesPage extends StatelessWidget {
3232
// Operation was canceled by the user.
3333
return;
3434
}
35-
await showDialog<void>(
36-
context: context,
37-
builder: (BuildContext context) => MultipleImagesDisplay(files),
38-
);
35+
if (context.mounted) {
36+
await showDialog<void>(
37+
context: context,
38+
builder: (BuildContext context) => MultipleImagesDisplay(files),
39+
);
40+
}
3941
}
4042

4143
@override

packages/file_selector/file_selector_linux/example/lib/open_text_page.dart

+6-4
Original file line numberDiff line numberDiff line change
@@ -25,10 +25,12 @@ class OpenTextPage extends StatelessWidget {
2525
final String fileName = file.name;
2626
final String fileContent = await file.readAsString();
2727

28-
await showDialog<void>(
29-
context: context,
30-
builder: (BuildContext context) => TextDisplay(fileName, fileContent),
31-
);
28+
if (context.mounted) {
29+
await showDialog<void>(
30+
context: context,
31+
builder: (BuildContext context) => TextDisplay(fileName, fileContent),
32+
);
33+
}
3234
}
3335

3436
@override

packages/file_selector/file_selector_macos/CHANGELOG.md

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
## NEXT
22

3+
* Updates example code for `use_build_context_synchronously` lint.
34
* Updates minimum Flutter version to 3.0.
45

56
## 0.9.0+4

packages/file_selector/file_selector_macos/example/lib/get_directory_page.dart

+6-4
Original file line numberDiff line numberDiff line change
@@ -21,10 +21,12 @@ class GetDirectoryPage extends StatelessWidget {
2121
// Operation was canceled by the user.
2222
return;
2323
}
24-
await showDialog<void>(
25-
context: context,
26-
builder: (BuildContext context) => TextDisplay(directoryPath),
27-
);
24+
if (context.mounted) {
25+
await showDialog<void>(
26+
context: context,
27+
builder: (BuildContext context) => TextDisplay(directoryPath),
28+
);
29+
}
2830
}
2931

3032
@override

packages/file_selector/file_selector_macos/example/lib/open_image_page.dart

+6-4
Original file line numberDiff line numberDiff line change
@@ -28,10 +28,12 @@ class OpenImagePage extends StatelessWidget {
2828
final String fileName = file.name;
2929
final String filePath = file.path;
3030

31-
await showDialog<void>(
32-
context: context,
33-
builder: (BuildContext context) => ImageDisplay(fileName, filePath),
34-
);
31+
if (context.mounted) {
32+
await showDialog<void>(
33+
context: context,
34+
builder: (BuildContext context) => ImageDisplay(fileName, filePath),
35+
);
36+
}
3537
}
3638

3739
@override

packages/file_selector/file_selector_macos/example/lib/open_multiple_images_page.dart

+6-4
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,12 @@ class OpenMultipleImagesPage extends StatelessWidget {
3232
// Operation was canceled by the user.
3333
return;
3434
}
35-
await showDialog<void>(
36-
context: context,
37-
builder: (BuildContext context) => MultipleImagesDisplay(files),
38-
);
35+
if (context.mounted) {
36+
await showDialog<void>(
37+
context: context,
38+
builder: (BuildContext context) => MultipleImagesDisplay(files),
39+
);
40+
}
3941
}
4042

4143
@override

packages/file_selector/file_selector_macos/example/lib/open_text_page.dart

+6-4
Original file line numberDiff line numberDiff line change
@@ -25,10 +25,12 @@ class OpenTextPage extends StatelessWidget {
2525
final String fileName = file.name;
2626
final String fileContent = await file.readAsString();
2727

28-
await showDialog<void>(
29-
context: context,
30-
builder: (BuildContext context) => TextDisplay(fileName, fileContent),
31-
);
28+
if (context.mounted) {
29+
await showDialog<void>(
30+
context: context,
31+
builder: (BuildContext context) => TextDisplay(fileName, fileContent),
32+
);
33+
}
3234
}
3335

3436
@override

packages/file_selector/file_selector_windows/CHANGELOG.md

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
## NEXT
22

3+
* Updates example code for `use_build_context_synchronously` lint.
34
* Updates minimum Flutter version to 3.0.
45

56
## 0.9.1+4

packages/file_selector/file_selector_windows/example/lib/get_directory_page.dart

+6-4
Original file line numberDiff line numberDiff line change
@@ -21,10 +21,12 @@ class GetDirectoryPage extends StatelessWidget {
2121
// Operation was canceled by the user.
2222
return;
2323
}
24-
await showDialog<void>(
25-
context: context,
26-
builder: (BuildContext context) => TextDisplay(directoryPath),
27-
);
24+
if (context.mounted) {
25+
await showDialog<void>(
26+
context: context,
27+
builder: (BuildContext context) => TextDisplay(directoryPath),
28+
);
29+
}
2830
}
2931

3032
@override

packages/file_selector/file_selector_windows/example/lib/open_image_page.dart

+6-4
Original file line numberDiff line numberDiff line change
@@ -28,10 +28,12 @@ class OpenImagePage extends StatelessWidget {
2828
final String fileName = file.name;
2929
final String filePath = file.path;
3030

31-
await showDialog<void>(
32-
context: context,
33-
builder: (BuildContext context) => ImageDisplay(fileName, filePath),
34-
);
31+
if (context.mounted) {
32+
await showDialog<void>(
33+
context: context,
34+
builder: (BuildContext context) => ImageDisplay(fileName, filePath),
35+
);
36+
}
3537
}
3638

3739
@override

packages/file_selector/file_selector_windows/example/lib/open_multiple_images_page.dart

+6-4
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,12 @@ class OpenMultipleImagesPage extends StatelessWidget {
3232
// Operation was canceled by the user.
3333
return;
3434
}
35-
await showDialog<void>(
36-
context: context,
37-
builder: (BuildContext context) => MultipleImagesDisplay(files),
38-
);
35+
if (context.mounted) {
36+
await showDialog<void>(
37+
context: context,
38+
builder: (BuildContext context) => MultipleImagesDisplay(files),
39+
);
40+
}
3941
}
4042

4143
@override

packages/file_selector/file_selector_windows/example/lib/open_text_page.dart

+6-4
Original file line numberDiff line numberDiff line change
@@ -25,10 +25,12 @@ class OpenTextPage extends StatelessWidget {
2525
final String fileName = file.name;
2626
final String fileContent = await file.readAsString();
2727

28-
await showDialog<void>(
29-
context: context,
30-
builder: (BuildContext context) => TextDisplay(fileName, fileContent),
31-
);
28+
if (context.mounted) {
29+
await showDialog<void>(
30+
context: context,
31+
builder: (BuildContext context) => TextDisplay(fileName, fileContent),
32+
);
33+
}
3234
}
3335

3436
@override

packages/image_picker/image_picker_windows/CHANGELOG.md

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
1-
## NEXT
1+
## 0.1.0+4
22

3+
* Updates example code for `use_build_context_synchronously` lint.
34
* Updates minimum Flutter version to 3.0.
45

56
## 0.1.0+3

0 commit comments

Comments
 (0)