Apply more strict php-cs-fixer rules

Author: Riimu

.php_cs

@@ -7,5 +7,82 @@ $finder = \PhpCsFixer\Finder::create()
 return \PhpCsFixer\Config::create()
     ->setRules([
         '@PSR2' => true,
+
+        'array_syntax' => ['syntax' => 'short'],
+        'binary_operator_spaces' => [
+            'align_equals' => false,
+            'align_double_arrow' => null,
+        ],
+        'blank_line_after_opening_tag' => true,
+        'cast_spaces' => true,
+        'concat_space' => ['spacing' => 'one'],
+        'declare_equal_normalize' => true,
+        'dir_constant' => true,
+        'ereg_to_preg' => true,
+        'function_typehint_space' => true,
+        'hash_to_slash_comment' => true,
+        'heredoc_to_nowdoc' => true,
+        'include' => true,
+        'lowercase_cast' => true,
+        'method_separation' => true,
+        'modernize_types_casting' => true,
+        'native_function_casing' => true,
+        'new_with_braces' => true,
+        'no_alias_functions' => true,
+        'no_blank_lines_after_class_opening' => true,
+        'no_blank_lines_after_phpdoc' => true,
+        'no_empty_comment' => true,
+        'no_empty_phpdoc' => true,
+        'no_empty_statement' => true,
+        'no_extra_consecutive_blank_lines' => true,
+        'no_leading_import_slash' => true,
+        'no_leading_namespace_whitespace' => true,
+        'no_mixed_echo_print' => true,
+        'no_multiline_whitespace_around_double_arrow' => true,
+        'no_php4_constructor' => true,
+        'no_short_bool_cast' => true,
+        'no_singleline_whitespace_before_semicolons' => true,
+        'no_spaces_around_offset' => true,
+        'no_trailing_comma_in_list_call' => true,
+        'no_trailing_comma_in_singleline_array' => true,
+        'no_unneeded_control_parentheses' => true,
+        'no_unused_imports' => true,
+        'no_useless_return' => true,
+        'no_whitespace_before_comma_in_array' => true,
+        'no_whitespace_in_blank_line' => true,
+        'normalize_index_brace' => true,
+        'object_operator_without_whitespace' => true,
+        'ordered_imports' => true,
+        'php_unit_construct' => true,
+        'php_unit_dedicate_assert' => true,
+        'php_unit_strict' => true,
+        'phpdoc_add_missing_param_annotation' => true,
+        'phpdoc_annotation_without_dot' => true,
+        'phpdoc_indent' => true,
+        'phpdoc_inline_tag' => true,
+        'phpdoc_no_access' => true,
+        'phpdoc_no_alias_tag' => true,
+        'phpdoc_no_package' => true,
+        'phpdoc_scalar' => true,
+        'phpdoc_single_line_var_spacing' => true,
+        'phpdoc_summary' => true,
+        'phpdoc_trim' => true,
+        'phpdoc_types' => true,
+        'phpdoc_var_without_name' => true,
+        'pow_to_exponentiation' => true,
+        'psr4' => true,
+        'return_type_declaration' => true,
+        'self_accessor' => true,
+        'short_scalar_cast' => true,
+        'single_blank_line_before_namespace' => true,
+        'single_quote' => true,
+        'space_after_semicolon' => true,
+        'standardize_not_equals' => true,
+        'strict_comparison' => true,
+        'strict_param' => true,
+        'ternary_operator_spaces' => true,
+        'trailing_comma_in_multiline_array' => true,
+        'trim_array_spaces' => true,
+        'whitespace_after_comma_in_array' => true,
     ])
     ->setFinder($finder);

src/JsonStream.php

@@ -134,7 +134,7 @@ public function isSeekable()
      * encoding the entire value, it's not possible to set the cursor using SEEK_END
      * constant and doing so will result in an exception.
      *
-     * @param int $offset The offset for the cursor.
+     * @param int $offset The offset for the cursor
      * @param int $whence Either SEEK_CUR or SEEK_SET to determine new cursor position
      */
     public function seek($offset, $whence = SEEK_SET)
@@ -171,7 +171,7 @@ private function calculatePosition($offset, $whence)
 
     /**
      * Forwards the JSON stream reading cursor to the given position or to the end of stream.
-     * @param int $position The new position of the cursor.
+     * @param int $position The new position of the cursor
      */
     private function forward($position)
     {
@@ -304,7 +304,7 @@ public function getContents()
      * As no underlying PHP stream exists for the JSON stream, this method will
      * always return an empty array or a null.
      *
-     * @param string|null The key of the value to return
+     * @param string|null $key The key of the value to return
      * @return array|null Always returns an empty array or a null
      */
     public function getMetadata($key = null)

tests/tests/JsonEncoderTest.php

@@ -144,6 +144,7 @@ public function testNullOnInvalid()
     }
 
     /**
+     * @param mixed $value The value to encode
      * @dataProvider getSimpleTestValues
      */
     public function testSimpleValues($value)