replace weird boolean logic with ternary

b6b7915a · Michael Ficarra · 7837fd1f · b6b7915a
Commit b6b7915a authored Oct 07, 2014 by Michael Ficarra
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 4 deletions

architecture-examples/backbone/js/views/todo-view.js architecture-examples/backbone/js/views/todo-view.js +3 -4

No files found.
--- a/architecture-examples/backbone/js/views/todo-view.js
+++ b/architecture-examples/backbone/js/views/todo-view.js
@@ -60,10 +60,9 @@ var app = app || {};
 		},
 		isHidden: function () {
-			var isCompleted = this.model.get('completed');
+			return this.model.get('completed') ?
-			return (// hidden cases only
+				app.TodoFilter === 'active' :
-				(!isCompleted && app.TodoFilter === 'completed') ||
+				app.TodoFilter === 'completed';
-				(isCompleted && app.TodoFilter === 'active')
 			);
 		},