fixed reporting UI issue

clragon · Dec 14, 2021 · 6b80f38 · 6b80f38
1 parent bf966d4
commit 6b80f38
Show file tree

Hide file tree

Showing 4 changed files with 21 additions and 10 deletions.
diff --git a/lib/client/client.dart b/lib/client/client.dart
@@ -727,3 +727,12 @@ Future<bool> validateCall(Future<void> Function() call) async {
     return false;
   }
 }
+
+Future<bool> validateRawCall(Future<void> Function() call) async {
+  try {
+    await call();
+    return true;
+  } on DioError catch (e) {
+    return e.response?.statusCode == 302;
+  }
+}
diff --git a/lib/ticket/widgets/comment.dart b/lib/ticket/widgets/comment.dart
@@ -12,7 +12,7 @@ class CommentReportScreen extends StatelessWidget {
   Widget build(BuildContext context) {
     return ReasonReportScreen(
       title: Text('Comment #${comment.id}'),
-      onReport: (reason) => validateCall(
+      onReport: (reason) => validateRawCall(
         () => client.reportComment(
           comment.id,
           reason,

diff --git a/lib/ticket/widgets/post.dart b/lib/ticket/widgets/post.dart
@@ -93,19 +93,20 @@ class _PostReportScreenState extends State<PostReportScreen> {
                         duration: defaultAnimationDuration,
                         curve: Curves.easeInOut,
                       );
-                      try {
-                        await client.reportPost(
+                      if (await validateRawCall(
+                        () => client.reportPost(
                           widget.post.id,
                           reportIds[type!]!,
                           reasonController.text.trim(),
-                        );
+                        ),
+                      )) {
                         ScaffoldMessenger.of(context).showSnackBar(SnackBar(
                           duration: Duration(seconds: 1),
                           content: Text('Reported post #${widget.post.id}'),
                           behavior: SnackBarBehavior.floating,
                         ));
                         Navigator.maybePop(context);
-                      } on DioError {
+                      } else {
                         ScaffoldMessenger.of(context).showSnackBar(SnackBar(
                           duration: Duration(seconds: 1),
                           content:
@@ -200,19 +201,20 @@ class _PostFlagScreenState extends State<PostFlagScreen> {
                         duration: defaultAnimationDuration,
                         curve: Curves.easeInOut,
                       );
-                      try {
-                        await client.flagPost(
+                      if (await validateRawCall(
+                        () => client.flagPost(
                           widget.post.id,
                           flagName[type]!,
                           parent: int.tryParse(parentController.text),
-                        );
+                        ),
+                      )) {
                         ScaffoldMessenger.of(context).showSnackBar(SnackBar(
                           duration: Duration(seconds: 1),
                           content: Text('Flagged post #${widget.post.id}'),
                           behavior: SnackBarBehavior.floating,
                         ));
                         Navigator.maybePop(context);
-                      } on DioError {
+                      } else {
                         ScaffoldMessenger.of(context).showSnackBar(SnackBar(
                           duration: Duration(seconds: 1),
                           content:

diff --git a/lib/ticket/widgets/user.dart b/lib/ticket/widgets/user.dart
@@ -15,7 +15,7 @@ class UserReportScreen extends StatelessWidget {
   Widget build(BuildContext context) {
     return ReasonReportScreen(
       title: Text('User #${user.id}'),
-      onReport: (reason) => validateCall(
+      onReport: (reason) => validateRawCall(
         () => client.reportUser(
           user.id,
           reason,