Merge pull request #141 from letsintegreat/issue-140-profile-pic

Add a null check on profile pic
OWASP-BLT · Dec 23, 2022 · c4ba996 · c4ba996
2 parents 815556c + 9f5bac7
commit c4ba996
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/assets/default_profile.png b/assets/default_profile.png
diff --git a/lib/src/pages/home/profile.dart b/lib/src/pages/home/profile.dart
@@ -26,6 +26,14 @@ class _UserProfileState extends ConsumerState<UserProfile> {
   late Future<List<Issue>?> getUpvoteList;
   late Future<List<Issue>?> getSavedList;
 
+  ImageProvider<Object> getProfilePicture() {
+    if (currentUser!.pfpLink == null) {
+      return AssetImage("assets/default_profile.png");
+    } else {
+      return NetworkImage(currentUser!.pfpLink!);
+    }
+  }
+
   Future<List<Issue>?> getIssueList(List<int>? idList) async {
     List<Issue>? issueList = null;
     try {
@@ -235,7 +243,7 @@ class _UserProfileState extends ConsumerState<UserProfile> {
                 decoration: BoxDecoration(
                   color: Color(0xFFDC4654),
                   image: DecorationImage(
-                    image: NetworkImage(currentUser!.pfpLink!),
+                    image: getProfilePicture(),
                     fit: BoxFit.cover,
                   ),
                   boxShadow: [BoxShadow(blurRadius: 7.0, color: Colors.black)],