fix two of the remaining todos :D

Author: techno-disaster

lib/bloc/process_bloc/process_bloc.dart

@@ -200,6 +200,8 @@ class ProcessBloc extends Bloc<ProcessEvent, ProcessState> {
       );
     } else if (event is ProcessError) {
       yield state.copyWith(current: state.current, exitCode: event.exitCode);
+    } else if (event is ResetProcessError) {
+      yield state.copyWith(current: state.current, exitCode: 0);
     } else if (event is ProcessOnNetwork) {
       yield* _extractOnNetwork(
           event.type, event.location, event.tcppassword, event.tcpdesc);

lib/bloc/process_bloc/process_event.dart

@@ -13,36 +13,41 @@ class ProcessFilesSubmitted extends ProcessEvent {
 
 class ProcessStarted extends ProcessEvent {}
 
-/// ProcessStopped stops all the files from processing after finishing the
-/// current one. Maybe use process.kill for this?
+/// ProcessStopped stops all the files from processing.
 class ProcessStopped extends ProcessEvent {}
 
+/// Removes pending file from state queue
 class ProcessFileRemoved extends ProcessEvent {
   final XFile file;
 
   const ProcessFileRemoved(this.file);
 }
 
+/// Stops ccx on a file using .kill()
 class ProcessKill extends ProcessEvent {
   final XFile file;
 
   const ProcessKill(this.file);
 }
 
+/// Remvoes all files from selected and stops ccx on any running file
 class ProcessRemoveAll extends ProcessEvent {}
 
+/// Used in callback when ccx emits logs to stderr
 class ProcessFileExtractorOutput extends ProcessEvent {
   final String log;
 
   const ProcessFileExtractorOutput(this.log);
 }
 
+/// Used in callback when ccx emits video details to stderr
 class ProcessFileVideoDetails extends ProcessEvent {
   final List<String> videoDetails;
 
   ProcessFileVideoDetails(this.videoDetails);
 }
 
+/// Used in callback when ccx emits progress details to stderr
 class ProcessFileExtractorProgress extends ProcessEvent {
   final String progress;
 
@@ -55,11 +60,13 @@ class ProcessFileComplete extends ProcessEvent {
   const ProcessFileComplete(this.file);
 }
 
+// emits a state with exit codes to show in snackbar
 class ProcessError extends ProcessEvent {
   final int exitCode;
   const ProcessError(this.exitCode);
 }
 
+// Runs tcp udp stuff in ccx
 class ProcessOnNetwork extends ProcessEvent {
   final String type;
   final String location;
@@ -73,4 +80,8 @@ class ProcessOnNetwork extends ProcessEvent {
       required this.tcpdesc});
 }
 
+// Gets the ccextractor version using `--version`
 class GetCCExtractorVersion extends ProcessEvent {}
+
+// Resets exitcode to 0, ideally should be null but wont emit a new state then
+class ResetProcessError extends ProcessEvent {}

lib/main.dart

@@ -66,11 +66,11 @@ class MyApp extends StatelessWidget {
 
 // Logs events and states during transition
 class SimpleBlocObserver extends BlocObserver {
-  // @override
-  // void onTransition(Bloc bloc, Transition transition) {
-  //   print(transition);
-  //   super.onTransition(bloc, transition);
-  // }
+  @override
+  void onTransition(Bloc bloc, Transition transition) {
+    print(transition);
+    super.onTransition(bloc, transition);
+  }
 
   @override
   void onError(BlocBase bloc, Object error, StackTrace stackTrace) {

lib/repositories/settings_repository.dart

@@ -36,17 +36,16 @@ class SettingsRepository {
           throw Exception('Setting key not found');
         }
       }
-      // This checks if all the values are of the intended datatype. TODO
-      if (outputFormats.contains(data['out']) &&
-          data['outputfilename'] is String &&
-          data['append'] is bool &&
-          data['autoprogram'] is bool) {
-      } else {
-        throw Exception('Settings value has mismatched datatype');
+      // This checks if all the values are of the intended datatype, by setting the current data to a settingsModel object
+      try {
+        // ignore: unused_local_variable
+        final _settingsModel = SettingsModel.fromJson(data);
+      } catch (e) {
+        throw Exception('Settings value has mismatched datatype. Error $e');
       }
       return true;
     } catch (e) {
-      print('Rewriting config.json file');
+      print('Rewriting config.json file. Error $e');
       await file.writeAsString(jsonEncode(SettingsModel()));
       return false;
     }

lib/screens/dashboard/dashboard.dart

@@ -52,12 +52,11 @@ class ClearFilesButton extends StatelessWidget {
   @override
   Widget build(BuildContext context) {
     return BlocConsumer<ProcessBloc, ProcessState>(
-      listenWhen: (_, __) => true,
       listener: (context, processState) {
-        //TODO: fix only works the first time, bloc wont emit same state
-        if (CCExtractor.exitCodes[processState.exitCode] != null) {
+        if (CCExtractor.exitCodes[processState.exitCode] != null && processState.exitCode != 0) {
           CustomSnackBarMessage.show(
               context, CCExtractor.exitCodes[processState.exitCode]!);
+          context.read<ProcessBloc>().add(ResetProcessError()); 
         }
       },
       builder: (context, processState) {