refactor: Re-order grouping methods for consistency with sorting

Author: claremacrae

src/Query/Filter/Field.ts

@@ -264,26 +264,6 @@ export abstract class Field {
         return false;
     }
 
-    /**
-     * Return a function to get a list of a task's group names, for use in grouping by this field's value.
-     *
-     * See {@link supportsGrouping} for what to do, to enable support of grouping in a
-     * particular {@link Field} implementation.
-     */
-    public grouper(): GrouperFunction {
-        throw Error(`grouper() unimplemented for ${this.fieldNameSingular()}`);
-    }
-
-    /**
-     * Create a {@link Grouper} object for grouping tasks by this field's value.
-     *
-     * For now, parsing of `group by` lines is currently done in {@link FilterParser.parseGrouper()}.
-     * Later, this will probably be moved to the {@link Field} classes.
-     */
-    public createGrouper(reverse: boolean): Grouper {
-        return new Grouper(this.fieldNameSingular(), this.grouper(), reverse);
-    }
-
     /**
      * Parse a 'group by' line and return a {@link Grouper} object.
      *
@@ -365,6 +345,26 @@ export abstract class Field {
         return new RegExp(`^group by ${this.fieldNameSingularEscaped()}( reverse)?$`);
     }
 
+    /**
+     * Return a function to get a list of a task's group names, for use in grouping by this field's value.
+     *
+     * See {@link supportsGrouping} for what to do, to enable support of grouping in a
+     * particular {@link Field} implementation.
+     */
+    public grouper(): GrouperFunction {
+        throw Error(`grouper() unimplemented for ${this.fieldNameSingular()}`);
+    }
+
+    /**
+     * Create a {@link Grouper} object for grouping tasks by this field's value.
+     *
+     * For now, parsing of `group by` lines is currently done in {@link FilterParser.parseGrouper()}.
+     * Later, this will probably be moved to the {@link Field} classes.
+     */
+    public createGrouper(reverse: boolean): Grouper {
+        return new Grouper(this.fieldNameSingular(), this.grouper(), reverse);
+    }
+
     /**
      * Create a {@link Grouper} object for grouping tasks by this field's value,
      * in the standard/normal group order for this field.