Remove try/catch in Widget#queryBool() (fixes #69)

discoveryjs · Nov 3, 2020 · 845d9d4 · 845d9d4
1 parent 3d2de4c
commit 845d9d4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -3,6 +3,7 @@
 - Fixed dark mode styles of action buttons in `struct` view for touch devices
 - Fixed context for nav to include route attributes (#70)
 - Added `nav.remove(name)` method to remove nav items (#72)
+- Removed try/catch in `Widget#queryBool()`, so now `when` and `whenData` throws instead of silently fail (#69)
 
 ## 1.0.0-beta.47 (02-11-2020)
 

diff --git a/src/widget/index.js b/src/widget/index.js
@@ -348,11 +348,7 @@ export default class Widget extends Emitter {
     }
 
     queryBool(...args) {
-        try {
-            return jora.buildin.bool(this.query(...args));
-        } catch (e) {
-            return false;
-        }
+        return jora.buildin.bool(this.query(...args));
     }
 
     queryToConfig(view, query) {