Remove prints (#110)

analysis_options.yaml

@@ -4,4 +4,7 @@ analyzer:
   errors:
     undefined_prefixed_name: ignore
     omit_local_variable_types: ignore
-    unsafe_html: ignore
+    unsafe_html: ignore
+linter:
+  rules:
+    - avoid_print

lib/models/rules/rule.dart

@@ -61,7 +61,6 @@ class Rules {
         final result = rule.apply(delta, index,
             len: len, data: data, attribute: attribute);
         if (result != null) {
-          print('Rule $rule applied');
           return result..trim();
         }
       } catch (e) {

lib/widgets/controller.dart

@@ -78,12 +78,7 @@ class QuillController extends ChangeNotifier {
 
     Delta? delta;
     if (len > 0 || data is! String || data.isNotEmpty) {
-      try {
+      delta = document.replace(index, len, data);
-        delta = document.replace(index, len, data);
-      } catch (e) {
-        print('document.replace failed: $e');
-        rethrow;
-      }
       bool shouldRetainDelta = toggledStyle.isNotEmpty &&
           delta.isNotEmpty &&
           delta.length <= 2 &&
@@ -112,23 +107,18 @@ class QuillController extends ChangeNotifier {
       if (delta == null || delta.isEmpty) {
         _updateSelection(textSelection, ChangeSource.LOCAL);
       } else {
-        try {
+        Delta user = Delta()
-          Delta user = Delta()
+          ..retain(index)
-            ..retain(index)
+          ..insert(data)
-            ..insert(data)
+          ..delete(len);
-            ..delete(len);
+        int positionDelta = getPositionDelta(user, delta);
-          int positionDelta = getPositionDelta(user, delta);
+        _updateSelection(
-          _updateSelection(
+          textSelection.copyWith(
-            textSelection.copyWith(
+            baseOffset: textSelection.baseOffset + positionDelta,
-              baseOffset: textSelection.baseOffset + positionDelta,
+            extentOffset: textSelection.extentOffset + positionDelta,
-              extentOffset: textSelection.extentOffset + positionDelta,
+          ),
-            ),
+          ChangeSource.LOCAL,
-            ChangeSource.LOCAL,
+        );
-          );
-        } catch (e) {
-          print('getPositionDelta or getPositionDelta error: $e');
-          rethrow;
-        }
       }
     }
     notifyListeners();

lib/widgets/toolbar.dart

@@ -4,7 +4,6 @@ import 'package:file_picker/file_picker.dart';
 import 'package:filesystem_picker/filesystem_picker.dart';
 import 'package:flutter/foundation.dart';
 import 'package:flutter/material.dart';
-import 'package:flutter/services.dart';
 import 'package:flutter_colorpicker/flutter_colorpicker.dart';
 import 'package:flutter_quill/models/documents/attribute.dart';
 import 'package:flutter_quill/models/documents/nodes/embed.dart';
@@ -520,46 +519,25 @@ class _ImageButtonState extends State<ImageButton> {
 
     final File file = File(pickedFile.path);
 
-    // We simply return the absolute path to selected file.
+    return widget.onImagePickCallback!(file);
-    try {
-      String url = await widget.onImagePickCallback!(file);
-      print('Image uploaded and its url is $url');
-      return url;
-    } catch (error) {
-      print('Upload image error $error');
-    }
-    return null;
   }
 
   Future<String?> _pickImageWeb() async {
-    try {
+    _paths = (await FilePicker.platform.pickFiles(
-      _paths = (await FilePicker.platform.pickFiles(
+      type: _pickingType,
-        type: _pickingType,
+      allowMultiple: false,
-        allowMultiple: false,
+      allowedExtensions: (_extension?.isNotEmpty ?? false)
-        allowedExtensions: (_extension?.isNotEmpty ?? false)
+          ? _extension?.replaceAll(' ', '').split(',')
-            ? _extension?.replaceAll(' ', '').split(',')
+          : null,
-            : null,
+    ))
-      ))
+        ?.files;
-          ?.files;
-    } on PlatformException catch (e) {
-      print('Unsupported operation' + e.toString());
-    } catch (ex) {
-      print(ex);
-    }
     var _fileName =
         _paths != null ? _paths!.map((e) => e.name).toString() : '...';
 
     if (_paths != null) {
       File file = File(_fileName);
       // We simply return the absolute path to selected file.
-      try {
+      return widget.onImagePickCallback!(file);
-        String url = await widget.onImagePickCallback!(file);
-        print('Image uploaded and its url is $url');
-        return url;
-      } catch (error) {
-        print('Upload image error $error');
-      }
-      return null;
     } else {
       // User canceled the picker
     }
@@ -567,23 +545,16 @@ class _ImageButtonState extends State<ImageButton> {
   }
 
   Future<String> _pickImageDesktop() async {
-    try {
+    var filePath = await FilesystemPicker.open(
-      var filePath = await FilesystemPicker.open(
+      context: context,
-        context: context,
+      rootDirectory: await getApplicationDocumentsDirectory(),
-        rootDirectory: await getApplicationDocumentsDirectory(),
+      fsType: FilesystemType.file,
-        fsType: FilesystemType.file,
+      fileTileSelectMode: FileTileSelectMode.wholeTile,
-        fileTileSelectMode: FileTileSelectMode.wholeTile,
+    );
-      );
+    if (filePath != null && filePath.isEmpty) return '';
-      if (filePath != null && filePath.isEmpty) return '';
+
-
+    final File file = File(filePath!);
-      final File file = File(filePath!);
+    return widget.onImagePickCallback!(file);
-      String url = await widget.onImagePickCallback!(file);
-      print('Image uploaded and its url is $url');
-      return url;
-    } catch (error) {
-      print('Upload image error $error');
-    }
-    return '';
   }
 
   @override