Bugfix: Fix is_equal_configuration() "off" comparison

diff --git a/README.md b/README.md
index 4dbb0c5e1c533ec000cc37a764211c7e284ce5fc..643cac912bd5ff37b9dd5977fd740fe5ee17d8e4 100644 (file)
--- a/README.md
+++ b/README.md
@@ -213,6 +213,7 @@ profiles matching multiple (or any) monitors.
 * *2018-04-19* Bugfix: Do not load default profile unless --change is set
 * *2018-04-30* Added a `AUTORANDR_MONITORS` variable to hooks (by @bricewge, #106)
 * *2018-06-29* Fix detection of current configuration if extra monitors are active
+* *2018-07-11* Bugfix in the latest change: Correctly handle "off" minitors when comparing
 
 **autorandr 1.5**
 

diff --git a/autorandr.py b/autorandr.py
index 2a4ddb7f3baffdf10f6c6d3847ed90c18afec40f..30fa225ade5599340d27e59fe052576c8c8b69fe 100755 (executable)
--- a/autorandr.py
+++ b/autorandr.py
@@ -761,7 +761,7 @@ def is_equal_configuration(source_configuration, target_configuration):
     """
     for output in target_configuration.keys():
         if "off" in target_configuration[output].options:
-            if (output in source_configuration or "off" not in source_configuration[output].options):
+            if (output in source_configuration and "off" not in source_configuration[output].options):
                 return False
         else:
             if (output not in source_configuration) or (source_configuration[output] != target_configuration[output]):