Skip to content

Update downloader_menu.dart - #303

Open
s-b-repo wants to merge 2 commits into
quickemu-project:mainfrom
s-b-repo:main
Open

s-b-repo wants to merge 2 commits into
quickemu-project:mainfrom
s-b-repo:main

Conversation

@s-b-repo

@s-b-repo s-b-repo commented Jan 3, 2026

Copy link
Copy Markdown

fixes:

1.Added _currentPath state variable to safely store the path for display
2.Added null/empty check before setting Directory.current in initState
3.Added Directory(pref).existsSync() check to verify path exists
4.Use _currentPath in the button text instead of Directory.current.path
5.Save folder directly instead of Directory.current.path after setting****

Refactor loadOperatingSystems and getIcons functions for better error handling and code clarity. 

fixes:

    1.Wrapped getIcons() in try-catch to handle AssetManifest.json load failures gracefully
    2.Added await to getIcons() call for proper async error handling
    3.Added try-catch around PackageInfo.fromPlatform() (can also fail with sudo)
    4.Uses debugPrint for warnings instead of crashing
@s-b-repo

s-b-repo commented Jan 3, 2026

Copy link
Copy Markdown
Author

fixes:

1.Wrapped getIcons() in try-catch to handle AssetManifest.json load failures gracefully
2.Added await to getIcons() call for proper async error handling
3.Added try-catch around PackageInfo.fromPlatform() (can also fail with sudo)
4.Uses debugPrint for warnings instead of crashing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant