Add NoPatternMatchException and handler with url_launcher package

Author: ivov

lib/helpers/retriever.dart

@@ -22,11 +22,11 @@ Map<String, Map<String, RegExp>> lawRegexes = {
 class Retriever {
   static String lawTextString;
   static Map<String, String> lawContents = {};
-  static String lawPattern;
+  static String lawPattern = "";
 
   static Future<Map<String, String>> retrieveLawText({String url}) async {
     await _getLawTextString(url);
-    _selectLawPattern(); // TODO: if no pattern matches, url_launcher for Infoleg's site
+    _selectLawPattern();
     _parseMostArticles();
     _parseFinalArticle();
     return lawContents;
@@ -39,10 +39,20 @@ class Retriever {
   }
 
   static void _selectLawPattern() {
-    if (lawTextString.contains(lawRegexes["law20305"]["initial"])) {
+    bool isLaw20305 =
+        lawTextString.contains(lawRegexes["law20305"]["initial"]) &&
+            lawTextString.contains(lawRegexes["law20305"]["final"]);
+
+    bool isLaw11723 =
+        lawTextString.contains(lawRegexes["law11723"]["initial"]) &&
+            lawTextString.contains(lawRegexes["law11723"]["final"]);
+
+    if (isLaw20305) {
       lawPattern = "law20305";
-    } else if (lawTextString.contains(lawRegexes["law11723"]["initial"])) {
+    } else if (isLaw11723) {
       lawPattern = "law11723";
+    } else if (lawPattern == "") {
+      throw NoPatternMatchException();
     }
   }
 

lib/screens/law_summary_screen.dart

@@ -1,5 +1,6 @@
 import 'package:flutter/material.dart';
 import 'package:infobootleg/models/law_model.dart';
+import 'package:infobootleg/widgets/alert_box.dart';
 import 'package:material_design_icons_flutter/material_design_icons_flutter.dart';
 
 import 'package:infobootleg/helpers/retriever.dart';
@@ -8,9 +9,14 @@ import 'package:infobootleg/widgets/law_title_card.dart';
 import 'package:infobootleg/widgets/modif_relations_box.dart';
 import 'package:infobootleg/models/search_state_model.dart';
 import 'package:infobootleg/helpers/hex_color.dart';
+import 'package:url_launcher/url_launcher.dart';
 
 enum ModificationType { modifies, isModifiedBy }
 
+class NoPatternMatchException implements Exception {
+  NoPatternMatchException();
+}
+
 class LawSummaryScreen extends StatelessWidget {
   LawSummaryScreen(this.searchState);
 
@@ -105,19 +111,44 @@ class LawSummaryScreen extends StatelessWidget {
     );
   }
 
+  _launchURL(String url) async {
+    if (await canLaunch(url)) {
+      await launch(url);
+    } else {
+      throw 'Could not launch $url';
+    }
+  }
+
+  void _accessLaw(BuildContext context) async {
+    searchState.toggleLoadingState();
+    try {
+      await searchState.updateLawContents();
+      searchState.transitionToScreenVertically(Screen.text);
+      searchState.toggleLoadingState();
+    } on NoPatternMatchException {
+      final bool answer = await AlertBox(
+        title: "Formato desconocido",
+        content:
+            "La Ley ${searchState.activeLaw.number} tiene formato desconocido y la aplicación no puede procesarla. ¿Abrir esta ley en Infoleg en el navagador?",
+        confirmActionText: "Sí",
+        cancelActionText: "No",
+      ).show(context);
+      if (answer) {
+        _launchURL(searchState.activeLaw.link);
+      }
+    } finally {
+      searchState.toggleLoadingState();
+    }
+  }
+
   _buildLawAccessButton(BuildContext context, bool isLoading) {
     if (isLoading == true) {
       return CircularProgressIndicator();
     }
 
     return RaisedButton(
+      onPressed: () => _accessLaw(context),
       elevation: 10.0,
-      onPressed: () async {
-        searchState.toggleLoadingState();
-        await searchState.updateLawContents();
-        searchState.transitionToScreenVertically(Screen.text);
-        searchState.toggleLoadingState();
-      },
       autofocus: true,
       color: hexColor("2c7873"),
       shape: RoundedRectangleBorder(

lib/screens/law_text_screen.dart

@@ -97,18 +97,6 @@ class _LawTextScreenState extends State<LawTextScreen> {
     );
   }
 
-  bool _getStarredStatus(String articleNumber) {
-    String dotlessLawNumber =
-        widget.searchState.activeLaw.number.replaceAll(".", "");
-    String cardName = dotlessLawNumber + "&" + articleNumber;
-    if (userFavorites != null) {
-      if (userFavorites.keys.toList().contains(cardName)) {
-        return true;
-      }
-    }
-    return false;
-  }
-
   Widget _buildListItem(
       {@required String articleNumber, @required BuildContext context}) {
     if (articleNumber == "0") {
@@ -138,6 +126,18 @@ class _LawTextScreenState extends State<LawTextScreen> {
     );
   }
 
+  bool _getStarredStatus(String articleNumber) {
+    String dotlessLawNumber =
+        widget.searchState.activeLaw.number.replaceAll(".", "");
+    String cardName = dotlessLawNumber + "&" + articleNumber;
+    if (userFavorites != null) {
+      if (userFavorites.keys.toList().contains(cardName)) {
+        return true;
+      }
+    }
+    return false;
+  }
+
   void _onYesAtSave(Favorite favorite, BuildContext context) async {
     await widget.dbService.saveFavorite(favorite);
     _updateUserFavorites();

lib/services/database_service.dart

@@ -67,36 +67,3 @@ class DatabaseService {
     });
   }
 }
-
-// https://firebase.google.com/docs/firestore/manage-data/add-data
-// https://firebase.google.com/docs/firestore/manage-data/delete-data
-
-// @override
-// Future<void> createJob(Job job) async => _setData(
-//       path: APIPath.job(userId, "job_abc"),
-//       data: job.toMap(),
-//     );
-
-// single entry point for all writes to Firestore
-// Future<void> _setData({String path, Map<String, dynamic> data}) async {
-//   final reference = Firestore.instance.document(path);
-//   await reference.setData(data);
-// }
-
-// Stream<List<Job>> jobsStream() => _collectionStream(
-//       path: APIPath.jobs(userId),
-//       builder: (data) => Job.fromMap(data),
-//     );
-
-// Stream<List<T>> _collectionStream<T>({
-//   @required String path,
-//   @required T builder(Map<String, dynamic> data),
-// }) {
-//   final reference = Firestore.instance.collection(path);
-//   final snapshots = reference.snapshots();
-
-//   return snapshots.map(
-//     (snapshot) =>
-//         snapshot.documents.map((snapshot) => builder(snapshot.data)).toList(),
-//   );
-// }

pubspec.lock

@@ -109,6 +109,11 @@ packages:
     description: flutter
     source: sdk
     version: "0.0.0"
+  flutter_web_plugins:
+    dependency: transitive
+    description: flutter
+    source: sdk
+    version: "0.0.0"
   flutter_widgets:
     dependency: "direct main"
     description:
@@ -200,6 +205,13 @@ packages:
       url: "https://pub.dartlang.org"
     source: hosted
     version: "2.4.0"
+  plugin_platform_interface:
+    dependency: transitive
+    description:
+      name: plugin_platform_interface
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "1.0.1"
   provider:
     dependency: "direct main"
     description:
@@ -268,6 +280,34 @@ packages:
       url: "https://pub.dartlang.org"
     source: hosted
     version: "1.1.6"
+  url_launcher:
+    dependency: "direct main"
+    description:
+      name: url_launcher
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "5.4.1"
+  url_launcher_macos:
+    dependency: transitive
+    description:
+      name: url_launcher_macos
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "0.0.1+2"
+  url_launcher_platform_interface:
+    dependency: transitive
+    description:
+      name: url_launcher_platform_interface
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "1.0.5"
+  url_launcher_web:
+    dependency: transitive
+    description:
+      name: url_launcher_web
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "0.1.0+2"
   vector_math:
     dependency: transitive
     description:
@@ -284,4 +324,4 @@ packages:
     version: "3.5.0"
 sdks:
   dart: ">=2.4.0 <3.0.0"
-  flutter: ">=1.9.1 <2.0.0"
+  flutter: ">=1.12.8 <2.0.0"

pubspec.yaml

@@ -20,6 +20,7 @@ dependencies:
   http:
   html:
   flutter_widgets:
+  url_launcher:
 
 dev_dependencies:
   flutter_test: