Ruleset::processRule(): fix edge case bug - inconsistent handling of empty string array key

Unlikely scenario for real-life, but still a bug.

When an array property is set via the old comma-separated string ruleset format or via an inline `phpcs:set` annotation, and the key for an array element would be explicitly set to an empty string, the resulting array item in the property on the sniff class would be added without a key, resulting in a numeric key.

However, if the (not so) "new" ruleset format using `<element>` nodes is used and an array element key is an empty string, this would result in the array item being added with an empty string for the array key.

This is inconsistent behaviour and is now fixed.

As 2 out of 3 scenarios would result in a numeric key, the behaviour has now been standardized as such. This can be revisited, if needs be, at a later point in time.

Author: jrfnl

src/Ruleset.php

@@ -1215,7 +1215,7 @@ private function processRule($rule, $newSniffs, $depth=0)
                                 }
 
                                 $value = (string) $element['value'];
-                                if (isset($element['key']) === true) {
+                                if (isset($element['key']) === true && trim($element['key']) !== '') {
                                     $key          = (string) $element['key'];
                                     $values[$key] = $value;
                                     $printValue  .= $key.'=>'.$value.',';