Merge branch 'master' into dialog-improvements

.github/workflows/build.yml

@@ -23,8 +23,3 @@ jobs:
     - name: Test with pytest
       run: |
         pytest
-    
-    - name: Install Cozy
-      run: |
-        meson --prefix=/usr ./build
-        ninja -C build install

README.md

@@ -279,7 +279,7 @@ To nedrichards for the Flatpak.
 ## Help me translate cozy!
 Cozy is on <a href="https://www.transifex.com/geigi/cozy/">Transifex</a>, where anyone can contribute and translate. Can't find your language in the list? Let me know!
 
-If you like this project, consider supporting me on <a href="https://www.patreon.com/bePatron?u=8147127">Patreon</a> :)
+If you like this project, consider supporting me on <a href="https://www.patreon.com/geigi">Patreon</a> :)
 
 ----
 [![Maintainability](https://api.codeclimate.com/v1/badges/fde8cbdff23033adaca2/maintainability)](https://codeclimate.com/github/geigi/cozy/maintainability)

cozy/application.py

@@ -1,3 +1,4 @@
+import gettext
 import locale
 import logging
 import os
@@ -73,9 +74,14 @@ def __init__(self, localedir: str, pkgdatadir: str):
         sys.excepthook = self.handle_exception
         setup_thread_excepthook()
 
-        import gettext
+        # We need to call `locale.*textdomain` to get the strings in UI files translated
+        locale.bindtextdomain('com.github.geigi.cozy', localedir)
+        locale.textdomain('com.github.geigi.cozy')
+
+        # But also `gettext.*textdomain`, to make `_("foo")` in Python work as well
         gettext.bindtextdomain('com.github.geigi.cozy', localedir)
         gettext.textdomain('com.github.geigi.cozy')
+
         gettext.install('com.github.geigi.cozy', localedir)
 
     def do_startup(self):

cozy/control/mpris.py

@@ -31,6 +31,10 @@
 log = logging.getLogger("offline_cache")
 
 
+class UnsupportedProperty(Exception):
+    pass
+
+
 class Server:
     def __init__(self, con, path):
         method_outargs = {}
@@ -84,13 +88,15 @@ def on_method_call(self,
                 invocation.return_value(variant)
             else:
                 invocation.return_value(None)
+        except UnsupportedProperty:
+            invocation.return_dbus_error("{}.Error.NotSupported".format(interface_name), "Unsupported property")
         except Exception as e:
             log.error(e)
             reporter.exception("mpris", e)
             reporter.error("mpris", "MPRIS method call failed with method name: {}".format(method_name))
             if out_args:
                 reporter.error("mpris", "MPRIS method call failed with out_args: {}".format(out_args))
-            invocation.return_value(None)
+            invocation.return_dbus_error("{}.Error.Failed".format(interface_name), "Internal exception occurred")
 
 
 class MPRIS(Server):
@@ -270,7 +276,7 @@ def Get(self, interface, property_name):
              return GLib.Variant("d", self._player.volume)
         else:
             reporter.warning("mpris", "MPRIS required an unknown information: {}".format(property_name))
-            return None
+            raise UnsupportedProperty
 
     def GetAll(self, interface):
         ret = {}

cozy/ui/app_view.py

@@ -24,7 +24,7 @@ def __init__(self, builder: Gtk.Builder):
 
     def _get_ui_elements(self):
         self._main_stack: Gtk.Stack = self._builder.get_object("main_stack")
-        self._navigation_view: Adw.Leaflet = self._builder.get_object("navigation_view")
+        self._navigation_view: Adw.NavigationView = self._builder.get_object("navigation_view")
 
     def _connect_ui_elements(self):
         self._main_stack.connect("notify::visible-child", self._update_view_model_view)