Sort Quick Fixes. Make lower priorities more relavant than higher.

pkg/analysis_server/lib/src/services/correction/fix.dart

 // Copyright (c) 2014, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library services.correction.fix;
 
 import 'package:analysis_server/src/protocol.dart' show SourceChange;
 import 'package:analysis_server/src/services/correction/fix_internal.dart';
 import 'package:analysis_server/src/services/search/search_engine.dart';
 import 'package:analyzer/src/generated/ast.dart';
 import 'package:analyzer/src/generated/error.dart';
 
 
 /**
  * Computes [Fix]s for the given [AnalysisError].
  *
  * Returns the computed [Fix]s, not `null`.
  */
 List<Fix> computeFixes(SearchEngine searchEngine, CompilationUnit unit,
     AnalysisError error) {
   var processor = new FixProcessor(searchEngine, unit, error);
-  return processor.compute();
+  List<Fix> fixes = processor.compute();
+  fixes.sort((Fix a, Fix b) {
+    return a.kind.relevance - b.kind.relevance;
+  });
+  return fixes;
 }
 
 
 /**
  * A description of a single proposed fix for some problem.
  */
 class Fix {
   final FixKind kind;
   final SourceChange change;
 
   Fix(this.kind, this.change);
 
   @override
   String toString() {
     return '[kind=$kind, change=$change]';
   }
 }
 
 
 /**
  * An enumeration of possible quick fix kinds.
  */
 class FixKind {
   static const ADD_PACKAGE_DEPENDENCY =
       const FixKind('ADD_PACKAGE_DEPENDENCY', 50, "Add dependency on package '{0}'");
   static const ADD_SUPER_CONSTRUCTOR_INVOCATION = const FixKind(
       'ADD_SUPER_CONSTRUCTOR_INVOCATION',
       50,
       "Add super constructor {0} invocation");
-  static const CHANGE_TO = const FixKind('CHANGE_TO', 51, "Change to '{0}'");
+  static const CHANGE_TO = const FixKind('CHANGE_TO', 49, "Change to '{0}'");
   static const CHANGE_TO_STATIC_ACCESS = const FixKind(
       'CHANGE_TO_STATIC_ACCESS',
       50,
       "Change access to static using '{0}'");
   static const CREATE_CLASS =
       const FixKind('CREATE_CLASS', 50, "Create class '{0}'");
   static const CREATE_CONSTRUCTOR =
       const FixKind('CREATE_CONSTRUCTOR', 50, "Create constructor '{0}'");
   static const CREATE_CONSTRUCTOR_SUPER = const FixKind(
       'CREATE_CONSTRUCTOR_SUPER',
       50,
       "Create constructor to call {0}");
   static const CREATE_FIELD =
       const FixKind('CREATE_FIELD', 50, "Create field '{0}'");
   static const CREATE_FILE =
       const FixKind('CREATE_FILE', 50, "Create file '{0}'");
   static const CREATE_FUNCTION =
-      const FixKind('CREATE_FUNCTION', 49, "Create function '{0}'");
+      const FixKind('CREATE_FUNCTION', 51, "Create function '{0}'");
   static const CREATE_LOCAL_VARIABLE =
       const FixKind('CREATE_LOCAL_VARIABLE', 50, "Create local variable '{0}'");
   static const CREATE_METHOD =
       const FixKind('CREATE_METHOD', 50, "Create method '{0}'");
   static const CREATE_MISSING_OVERRIDES = const FixKind(
       'CREATE_MISSING_OVERRIDES',
       50,
       "Create {0} missing override(s)");
   static const CREATE_NO_SUCH_METHOD =
-      const FixKind('CREATE_NO_SUCH_METHOD', 49, "Create 'noSuchMethod' method");
+      const FixKind('CREATE_NO_SUCH_METHOD', 51, "Create 'noSuchMethod' method");
   static const IMPORT_LIBRARY_PREFIX = const FixKind(
       'IMPORT_LIBRARY_PREFIX',
       51,
       "Use imported library '{0}' with prefix '{1}'");
   static const IMPORT_LIBRARY_PROJECT =
-      const FixKind('IMPORT_LIBRARY_PROJECT', 51, "Import library '{0}'");
+      const FixKind('IMPORT_LIBRARY_PROJECT', 49, "Import library '{0}'");
   static const IMPORT_LIBRARY_SDK =
-      const FixKind('IMPORT_LIBRARY_SDK', 51, "Import library '{0}'");
+      const FixKind('IMPORT_LIBRARY_SDK', 49, "Import library '{0}'");
   static const IMPORT_LIBRARY_SHOW =
-      const FixKind('IMPORT_LIBRARY_SHOW', 51, "Update library '{0}' import");
+      const FixKind('IMPORT_LIBRARY_SHOW', 49, "Update library '{0}' import");
   static const INSERT_SEMICOLON =
       const FixKind('INSERT_SEMICOLON', 50, "Insert ';'");
   static const MAKE_CLASS_ABSTRACT =
       const FixKind('MAKE_CLASS_ABSTRACT', 50, "Make class '{0}' abstract");
   static const REMOVE_PARAMETERS_IN_GETTER_DECLARATION = const FixKind(
       'REMOVE_PARAMETERS_IN_GETTER_DECLARATION',
       50,
       "Remove parameters in getter declaration");
   static const REMOVE_PARENTHESIS_IN_GETTER_INVOCATION = const FixKind(
       'REMOVE_PARENTHESIS_IN_GETTER_INVOCATION',
@@ skipping 23 matching lines @@
 
   final name;
   final int relevance;
   final String message;
 
   const FixKind(this.name, this.relevance, this.message);
 
   @override
   String toString() => name;
 }