Add option to enable/disable sound notification; closes #4 (#15)

Author: hsbasu

usr/lib/battery-monitor/Notification.py

@@ -110,6 +110,10 @@ def load_config(self):
 				self.critical_battery = int(self.config['settings']['critical_battery'])
 			except ValueError:
 				self.critical_battery = 10
+			try:
+				self.use_sound = int(self.config['settings']['use_sound'])
+			except ValueError:
+				self.use_sound = 1
 			try:
 				self.notification_stability = int(self.config['settings']['notification_stability'])
 			except ValueError:
@@ -123,6 +127,7 @@ def load_config(self):
 			self.third_custom_warning = -3
 			self.low_battery = 30
 			self.critical_battery = 10
+			self.use_sound = 1
 			self.notification_stability = 5
 
 	def show_notification(self, notiftype: str, battery_percentage: int,
@@ -138,7 +143,8 @@ def show_notification(self, notiftype: str, battery_percentage: int,
 				if ("charging" or "discharging") in notiftype:
 					notification = self.notifier.new(head, body, icon)
 					notification.show()
-					os.system("paplay /usr/share/sounds/Yaru/stereo/complete.oga")
+					if self.use_sound:
+						os.system("paplay /usr/share/sounds/Yaru/stereo/complete.oga")
 				else:
 					self.monitor.is_updated()
 					info = self.monitor.get_processed_battery_info()
@@ -147,7 +153,8 @@ def show_notification(self, notiftype: str, battery_percentage: int,
 						continue
 					notification = self.notifier.new(head, body, icon)
 					notification.show()
-					os.system("paplay /usr/share/sounds/Yaru/stereo/complete.oga")
+					if self.use_sound:
+						os.system("paplay /usr/share/sounds/Yaru/stereo/complete.oga")
 					time.sleep(self.notification_stability)
 
 		except GLib.GError as e:

usr/lib/battery-monitor/SettingsWindow.py

@@ -77,7 +77,8 @@ def __init__(self, application):
 		self.critical_battery_entry = self.builder.get_object("critical_battery")
 
 		## Sound configuration page
-		self.mute_sound_entry = self.builder.get_object("mute_sound")
+		self.label_sound_switch = self.builder.get_object("label_sound_switch")
+		self.sound_switch = self.builder.get_object("sound_switch")
 		self.sound_file_entry = self.builder.get_object("sound_file")
 
 		## Notification configuration page
@@ -148,6 +149,7 @@ def __load_config(self):
 			self.third_custom_warning = self.config['settings']['third_custom_warning']
 			self.low_battery = self.config['settings']['low_battery']
 			self.critical_battery = self.config['settings']['critical_battery']
+			self.use_sound = int(self.config['settings']['use_sound'])
 			self.notification_stability = self.config['settings']['notification_stability']
 		except:
 			print('Config file is missing or not readable. Using default configurations.')
@@ -158,6 +160,7 @@ def __load_config(self):
 			self.third_custom_warning = '40'
 			self.low_battery = '30'
 			self.critical_battery = '15'
+			self.use_sound = 1
 			self.notification_stability = '5'
 
 		self.success_shown_entry.set_text(self.success_shown)
@@ -167,6 +170,7 @@ def __load_config(self):
 		self.third_custom_warning_entry.set_text(self.third_custom_warning)
 		self.low_battery_entry.set_text(self.low_battery)
 		self.critical_battery_entry.set_text(self.critical_battery)
+		self.sound_switch.set_active(self.use_sound)
 		self.notify_duration_entry.set_text(self.notification_stability)
 
 	def __save_config(self, widget):
@@ -180,6 +184,11 @@ def __save_config(self, widget):
 		else:
 			os.makedirs(self.config_dir)
 
+		if self.sound_switch.get_active():
+			use_sound = 1
+		else:
+			use_sound = 0
+		
 		self.config['settings'] = {
 			'success_shown': self.success_shown_entry.get_text(),
 			'upper_threshold_warning': self.upper_threshold_warning_entry.get_text(),
@@ -188,6 +197,7 @@ def __save_config(self, widget):
 			'third_custom_warning': self.third_custom_warning_entry.get_text(),
 			'low_battery': self.low_battery_entry.get_text(),
 			'critical_battery': self.critical_battery_entry.get_text(),
+			'use_sound': use_sound,
 			'notification_stability': self.notify_duration_entry.get_text()
 		}