Issue 5844 - Remove redundant parentheses in sitescripts and abpsembly

sitescripts/templateFilters.py

Index: sitescripts/templateFilters.py
===================================================================
--- a/sitescripts/templateFilters.py
+++ b/sitescripts/templateFilters.py
@@ -123,19 +123,19 @@
 def subscriptionSort(value, prioritizeRecommended=True):
     value = value[:]  # create a copy of the list
     if prioritizeRecommended:
-        value.sort(lambda a, b: (
-            cmp(a.type, b.type) or
-            cmp(a.deprecated, b.deprecated) or
-            cmp(b.catchall, a.catchall) or
-            cmp(b.recommendation != None, a.recommendation != None) or
-            cmp(a.name.lower(), b.name.lower())
-        ))
+        value.sort(lambda a, b:
+                   cmp(a.type, b.type) or
+                   cmp(a.deprecated, b.deprecated) or
+                   cmp(b.catchall, a.catchall) or
+                   cmp(b.recommendation != None, a.recommendation != None) or
+                   cmp(a.name.lower(), b.name.lower())
+                   )

[Sebastian Noack, 2018/01/17 01:43:51]

If you indent code without a newline after the opening parenthesis, please don't
put the closing parenthesis on a new line either:

  value.sort(lambda a, b:
             ...
             ...)

Alternatively, following indention would be acceptable, and compliant with
flake8, too, and while it requires 2 more lines it looks a bit more structured:

  value.sort(
      lambda a, b:
          ...
  )

FWIW, one could also argue that neither is an improvement to the original code.
However, if we just remove the parenthesis, without changing the indentation,
we'd get an E128. Not sure if it's worth to add a (potentially complex)
exception to the A111 rule, though. What would the exception be, anyway?

[Vasily Kuznetsov, 2018/01/18 10:46:07]

Just for the record, I prefer Sebastian's second version. I also think it's more
readable than the original code.

[rosie, 2018/01/25 16:13:48]

Yeah, the second version also seems more readable to me.
Done.

     else:
-        value.sort(lambda a, b: (
-            cmp(a.type, b.type) or
-            cmp(a.deprecated, b.deprecated) or
-            cmp(a.name.lower(), b.name.lower())
-        ))
+        value.sort(lambda a, b:
+                   cmp(a.type, b.type) or
+                   cmp(a.deprecated, b.deprecated) or
+                   cmp(a.name.lower(), b.name.lower())
+                   )
     return value