Left: | ||
Right: |
OLD | NEW |
---|---|
(Empty) | |
1 def subscription_sort(value, prioritizeRecommended=True): | |
Sebastian Noack
2015/02/13 17:51:49
Nit: Please use underscores instead camelcase in P
kzar
2015/02/20 14:55:18
Done.
| |
2 value = value[:] # create a copy of the list | |
3 if prioritizeRecommended: | |
4 value.sort(lambda a, b: ( | |
5 cmp(a.get("type"), b.get("type")) or | |
6 cmp(a.get("deprecated"), b.get("deprecated")) or | |
7 cmp(b.get("catchall"), a.get("catchall")) or | |
8 cmp(b.get("recommendation") != None, a.get("recommendation") != None) or | |
Sebastian Noack
2015/02/13 17:51:49
None is a singleton. So please always use the "is
kzar
2015/02/20 14:55:18
Done.
| |
9 cmp(a.get("name").lower(), b.get("name").lower()) | |
Sebastian Noack
2015/02/13 17:51:49
This code will raise an exception anyway if "name"
kzar
2015/02/20 14:55:18
Done.
| |
10 )) | |
11 else: | |
12 value.sort(lambda a, b: ( | |
13 cmp(a.get("type"), b.get("type")) or | |
14 cmp(a.get("deprecated"), b.get("deprecated")) or | |
15 cmp(a.get("name").lower(), b.get("name").lower()) | |
16 )) | |
17 return value | |
OLD | NEW |