From d16866e0f0e97d1e01502b83c5c58361d169e35f Mon Sep 17 00:00:00 2001
From: Manos Pitsidianakis <el13635@mail.ntua.gr>
Date: Wed, 13 Jan 2021 22:34:19 +0200
Subject: [PATCH] notifications: run update_xbiff even if notifications
 disabled

---
 src/components/notifications.rs | 42 ++++++++++++++++-----------------
 1 file changed, 20 insertions(+), 22 deletions(-)

diff --git a/src/components/notifications.rs b/src/components/notifications.rs
index c3a828b0b..0aa47cd9f 100644
--- a/src/components/notifications.rs
+++ b/src/components/notifications.rs
@@ -188,29 +188,27 @@ impl Component for NotificationCommand {
     fn draw(&mut self, _grid: &mut CellBuffer, _area: Area, _context: &mut Context) {}
 
     fn process_event(&mut self, event: &mut UIEvent, context: &mut Context) -> bool {
-        if !context.settings.notifications.enable {
-            return false;
-        }
-
         if let UIEvent::Notification(ref title, ref body, ref kind) = event {
-            if let Some(ref bin) = context.settings.notifications.script {
-                match Command::new(bin)
-                    .arg(&kind.map(|k| k.to_string()).unwrap_or_default())
-                    .arg(title.as_ref().map(String::as_str).unwrap_or("meli"))
-                    .arg(body)
-                    .stdin(Stdio::piped())
-                    .stdout(Stdio::piped())
-                    .spawn()
-                {
-                    Ok(child) => {
-                        context.children.push(child);
-                    }
-                    Err(err) => {
-                        log(
-                            format!("Could not run notification script: {}.", err.to_string()),
-                            ERROR,
-                        );
-                        debug!("Could not run notification script: {:?}", err);
+            if context.settings.notifications.enable {
+                if let Some(ref bin) = context.settings.notifications.script {
+                    match Command::new(bin)
+                        .arg(&kind.map(|k| k.to_string()).unwrap_or_default())
+                        .arg(title.as_ref().map(String::as_str).unwrap_or("meli"))
+                        .arg(body)
+                        .stdin(Stdio::piped())
+                        .stdout(Stdio::piped())
+                        .spawn()
+                    {
+                        Ok(child) => {
+                            context.children.push(child);
+                        }
+                        Err(err) => {
+                            log(
+                                format!("Could not run notification script: {}.", err.to_string()),
+                                ERROR,
+                            );
+                            debug!("Could not run notification script: {:?}", err);
+                        }
                     }
                 }
             }