Merge pull request #485 from zalando-stups/484-check-for-removing-not…

…ifications-should-not-always-run

#484 check for notifications only if there are notification are present

client/lib/yourturn/src/notification-bar/notification-bar.jsx

@@ -34,20 +34,44 @@ class NotificationBar extends React.Component {
     dismiss(id) {
         this.props.dispatch(Actions.removeNotification(id));
     }
+
+    componentWillReceiveProps(nextProps) {
+        const { notifications = [] } = nextProps;
+
+        if (!this.state.interval && notifications.length > 0) {
+            this.startInterval();
+        } else if (this.state.interval && notifications.length == 0) {
+            this.stopInterval();
+        }
+    }
 
     componentDidMount() {
         /**
          * Continually dismiss old notifications.
          */
-        let interval = setInterval(() => {
+        const { notifications = [] } = this.props;
+        if (notifications.length > 0) {
+            this.startInterval();
+        }
+    }
+
+    componentWillUnmount() {
+        this.stopInterval();
+    }
+
+    startInterval() {
+        const interval = setInterval(() => {
             this.props.dispatch(Actions.removeNotificationsOlderThan(5000));
         }, 5000);
         this.setState({
             interval
         });
     }
 
-    componentWillUnmount() {
+    stopInterval() {
+        this.setState({
+            interval: false
+        });
         clearInterval(this.state.interval);
     }