Bug fix pallets#793; use the first defined long name

ericfrederich · May 19, 2017 · 4cd18db · 4cd18db
1 parent 2ab5f7e
commit 4cd18db
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/click/core.py b/click/core.py
@@ -1548,8 +1548,8 @@ def _parse_decls(self, decls, expose_value):
                     opts.append(decl)
 
         if name is None and possible_names:
-            possible_names.sort(key=lambda x: len(x[0]))
-            name = possible_names[-1][1].replace('-', '_').lower()
+            possible_names.sort(key=lambda x: -len(x[0]))  # group long options first
+            name = possible_names[0][1].replace('-', '_').lower()
             if not isidentifier(name):
                 name = None