Merge branch 'develop' of github.com:bee-mar/mmpm

Author: Bee-Mar

CHANGELOG.md

@@ -240,3 +240,7 @@
 # Version 2.03
 
 - fixed small typos for the keyword `upgradeable`
+
+# Version 2.04
+
+- Bug fix (#44): fixed screen rotation from GUI due to key in dictionary not being cast to int from string

gui/src/app/components/magic-mirror-control-center/magic-mirror-control-center.component.ts

@@ -189,7 +189,7 @@ export class MagicMirrorControlCenterComponent implements OnInit {
       rows: 1,
       badge: null,
       url: URLS.POST.RASPBERRYPI.ROTATE_SCREEN,
-      message: "The screen will be rotated with sudo permissions. The RaspberryPi must be restarted after",
+      message: "The screen will be rotated with sudo permissions, and the RaspberryPi must be restarted to reflect changes.",
       disabled: false,
       dialogWidth: "50vw",
     },
@@ -234,7 +234,11 @@ export class MagicMirrorControlCenterComponent implements OnInit {
             if (!response) { return; }
 
             this.api.rotateRaspberryPiScreen(value).then((error: any) => {
-              if (error?.error) { this.snackbar.error(error.error); }
+              if (error?.error) {
+                this.snackbar.error(error.error);
+              } else {
+                this.snackbar.success("Screen rotation value updated. Please restart your RaspberryPi.");
+              }
             });
           });
         });

mmpm/core.py

@@ -2239,7 +2239,7 @@ def rotate_raspberrypi_screen(degrees: int, assume_yes: bool = False) -> str: #p
     os.system(f'sudo cp {boot_config} {boot_config}.bak')
     print(mmpm.consts.GREEN_CHECK_MARK)
 
-    desired_setting: int = rotation_map[degrees]
+    desired_setting: int = rotation_map[int(degrees)]
 
     grep: subprocess.CompletedProcess = subprocess.run(['grep', '--color=never', 'display_rotate', boot_config], stdout=subprocess.PIPE)
     output: str = grep.stdout.decode('utf-8').strip()

mmpm/mmpm.py

@@ -15,7 +15,7 @@
 MagicMirrorPackage = mmpm.models.MagicMirrorPackage
 get_env = mmpm.utils.get_env
 
-__version__ = 2.03
+__version__ = 2.04
 
 
 def main(argv):

mmpm/utils.py

@@ -533,7 +533,7 @@ def kill_magicmirror_processes() -> None:
 
     processes = ['electron']
 
-    log.info('Killing processes associated with MagicMirror: {processes}')
+    log.info(f'Killing processes associated with MagicMirror: {processes}')
 
     for process in processes:
         kill_pids_of_process(process)