[TASK] Update RuleSet::comparePositionable to use new methods (#1283)

`getLineNo` and `getColNo` are deprecated.
When the titled method was extracted, use of the above-mentioned methods was
retained to ease backporting and transition to their replacement counterparts:
`getLineNumber` and `getColumnNumber`,
which differ by returning `null` in the case of 'not set'.

This replaces all instances of calls to `getColNo`.

Part of #974

Author: JakeQZ

src/RuleSet/RuleSet.php

@@ -326,13 +326,33 @@ protected function renderRules(OutputFormat $outputFormat): string
 
     /**
      * @return int negative if `$first` is before `$second`; zero if they have the same position; positive otherwise
+     *
+     * @throws \UnexpectedValueException if either argument does not have a valid position, which should never happen
      */
     private static function comparePositionable(Positionable $first, Positionable $second): int
     {
-        if ($first->getLineNo() === $second->getLineNo()) {
-            return $first->getColNo() - $second->getColNo();
+        $firstsLineNumber = $first->getLineNumber();
+        $secondsLineNumber = $second->getLineNumber();
+        if (!\is_int($firstsLineNumber) || !\is_int($secondsLineNumber)) {
+            throw new \UnexpectedValueException(
+                'A Rule without a line number was passed to comparePositionable',
+                1750637683
+            );
         }
-        return $first->getLineNo() - $second->getLineNo();
+
+        if ($firstsLineNumber === $secondsLineNumber) {
+            $firstsColumnNumber = $first->getColumnNumber();
+            $secondsColumnNumber = $second->getColumnNumber();
+            if (!\is_int($firstsColumnNumber) || !\is_int($secondsColumnNumber)) {
+                throw new \UnexpectedValueException(
+                    'A Rule without a column number was passed to comparePositionable',
+                    1750637761
+                );
+            }
+            return $firstsColumnNumber - $secondsColumnNumber;
+        }
+
+        return $firstsLineNumber - $secondsLineNumber;
     }
 
     private function hasRule(Rule $rule): bool