refactor: fixed code to pass the flutter analyze check!

IamMuuo · Aug 24, 2024 · 257908b · 257908b
1 parent a065eb3
commit 257908b
Show file tree

Hide file tree

Showing 8 changed files with 16 additions and 16 deletions.
diff --git a/lib/pages/auth/register_page.dart b/lib/pages/auth/register_page.dart
@@ -1,5 +1,4 @@
 import 'package:academia/exports/barrel.dart';
-import 'package:flutter/services.dart';
 import 'package:get/get.dart';
 import 'package:lottie/lottie.dart';
 

diff --git a/lib/pages/courses/course_view_page.dart b/lib/pages/courses/course_view_page.dart
@@ -91,6 +91,7 @@ class CourseViewPage extends StatelessWidget {
                                       .deleteCourseTopic(data);
 
                                   HapticFeedback.heavyImpact().then((value) {
+                                    if (!context.mounted) return;
                                     ScaffoldMessenger.of(context).showSnackBar(
                                         const SnackBar(
                                             content:

diff --git a/lib/pages/tool_page.dart b/lib/pages/tool_page.dart
@@ -24,7 +24,7 @@ class _ToolsPageState extends State<ToolsPage> {
     foundTools = _allTools;
   }
 
-  void _runToolFilter(String enteredKeyWord) {
+  void runToolFilter(String enteredKeyWord) {
     List<Map<String, dynamic>> results = [];
     if (enteredKeyWord.isEmpty) {
       results = _allTools;
@@ -86,10 +86,6 @@ class _ToolsPageState extends State<ToolsPage> {
                 },
                 icon: const Icon(Ionicons.information_circle),
               ),
-              IconButton(
-                onPressed: () {},
-                icon: const Icon(Ionicons.search_circle_outline),
-              )
             ],
           ),
           SliverPadding(

diff --git a/lib/tools/exam_timetable/controllers/exam_timetable_controller.dart b/lib/tools/exam_timetable/controllers/exam_timetable_controller.dart
@@ -45,7 +45,6 @@ class ExamsTimeTableController extends GetxController {
     }
   }
 
-
   void nextQuote() {
     if (quotes.isNotEmpty && index.value < 49) {
       index.value++;
@@ -62,7 +61,7 @@ class ExamsTimeTableController extends GetxController {
     }
   }
 
-   Future<List<Exam>> fetchExams(List<String> units) async {
+  Future<List<Exam>> fetchExams(List<String> units) async {
     const String apiUrl = "http://academia.erick.serv00.net/timetables/exams/";
 
     try {
@@ -104,7 +103,8 @@ class ExamsTimeTableController extends GetxController {
 
         return examData;
       } else {
-        throw Exception("Failed to load exams. Status code: ${response.statusCode}");
+        throw Exception(
+            "Failed to load exams. Status code: ${response.statusCode}");
       }
     } catch (e) {
       throw Exception("Error fetching exams: $e");
@@ -158,7 +158,6 @@ class ExamsTimeTableController extends GetxController {
       List<String> courseTitles = courses
           .map((e) => "${e.unit.replaceAll('-', '')}${e.section.split('-')[0]}")
           .toList();
-      print(courseTitles);
 
       // Fetch from server and store in the database
       exams = await fetchExams(courseTitles);

diff --git a/lib/tools/todo/todo_view_page.dart b/lib/tools/todo/todo_view_page.dart
@@ -38,6 +38,7 @@ class _TodoViewPageState extends State<TodoViewPage> {
       lastDate: DateTime.now().add(const Duration(days: 365)),
     ).then((selectedDate) {
       if (selectedDate != null) {
+        if (!mounted) return;
         showTimePicker(
           context: context,
           initialTime: TimeOfDay.now(),
@@ -214,6 +215,7 @@ class _TodoViewPageState extends State<TodoViewPage> {
                       .then((value) {
                     todoController.getAllTodos();
                     if (value) {
+                      if (!context.mounted) return;
                       showDialog(
                         context: context,
                         builder: (context) => AlertDialog(
@@ -251,6 +253,7 @@ class _TodoViewPageState extends State<TodoViewPage> {
                                 todoController
                                     .deleteTodo(widget.todo!)
                                     .then((value) {
+                                  if (!context.mounted) return;
                                   if (value) {
                                     Navigator.pop(context);
                                     Navigator.pop(context);

diff --git a/lib/tools/todo/widgets/todo_items_section.dart b/lib/tools/todo/widgets/todo_items_section.dart
@@ -1,6 +1,5 @@
 import 'package:academia/exports/barrel.dart';
 import 'package:get/get.dart';
-import 'package:flutter/services.dart';
 
 class TodoItemSection extends StatefulWidget {
   const TodoItemSection({super.key});
@@ -13,7 +12,7 @@ class _TodoItemSectionState extends State<TodoItemSection> {
   final todoController = Get.find<TodoController>();
   late List<Todo> allTodos;
 
-  List<Todo> _getDueTodos() {
+  List<Todo> getDueTodos() {
     DateTime now = DateTime.now();
 
     List<Todo> dueTodos = todoController.allTodos
@@ -54,6 +53,7 @@ class _TodoItemSectionState extends State<TodoItemSection> {
                             todoController
                                 .deleteTodo(currentTodo)
                                 .then((value) {
+                              if (!context.mounted) return;
                               if (value) {
                                 ScaffoldMessenger.of(context).showSnackBar(
                                   const SnackBar(
@@ -90,6 +90,7 @@ class _TodoItemSectionState extends State<TodoItemSection> {
                 ontap: () {
                   HapticFeedback.heavyImpact().then((value) {
                     todoController.updateTodo(currentTodo);
+                    if (!context.mounted) return;
                     ScaffoldMessenger.of(context).showSnackBar(
                       const SnackBar(
                         content: Text("Task updated sucessfully"),

diff --git a/lib/widgets/tool_card.dart b/lib/widgets/tool_card.dart
@@ -39,7 +39,7 @@ class _ToolCardState extends State<ToolCard> {
             color: Theme.of(context).colorScheme.shadow,
           ),
           borderRadius: BorderRadius.circular(4),
-          color: Theme.of(context).colorScheme.background,
+          color: Theme.of(context).colorScheme.surface,
         ),
         child: Column(
           children: [
@@ -62,7 +62,8 @@ class _ToolCardState extends State<ToolCard> {
               alignment: Alignment.centerLeft,
               child: Text(widget.description),
             ),
-            ButtonBar(
+            OverflowBar(
+              alignment: MainAxisAlignment.end,
               children: [
                 TextButton(
                   child: Text(widget.action),

diff --git a/lib/workers/background_worker.dart b/lib/workers/background_worker.dart
@@ -13,7 +13,7 @@ void callbackDispatcher() {
         storyController.fetchStories().then(
           (value) {
             value.fold((l) {
-              print("You pussy $l");
+              debugPrint("You pussy $l");
             }, (r) {
               notificationsController.createInstantNotification(
                 "Hey there you!",
@@ -23,7 +23,7 @@ void callbackDispatcher() {
             });
           },
         );
-        print("hey");
+        debugPrint("hey");
         break;
       default:
     }