Skip to content

Commit

Permalink
fix(cs): fixed code-style and static analysis
Browse files Browse the repository at this point in the history
  • Loading branch information
alekitto committed Apr 25, 2024
1 parent 4ee5cc9 commit 57cf7a5
Show file tree
Hide file tree
Showing 6 changed files with 20 additions and 8 deletions.
4 changes: 2 additions & 2 deletions src/Context.php
Original file line number Diff line number Diff line change
Expand Up @@ -52,13 +52,13 @@ public function __clone()
*
* @return static
*/
public static function create(): self
public static function create(): static
{
return new static();
}

/** @param array<string, mixed> $attributes */
public function createChildContext(array $attributes = []): self
public function createChildContext(array $attributes = []): static
{
if (! $this->initialized) {
throw new LogicException('Cannot create a child context of an uninitialized context.');
Expand Down
1 change: 1 addition & 0 deletions src/Debug/TraceableHandlerRegistry.php
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@
use Throwable;

use function call_user_func_array;
use function class_exists;
use function func_get_args;
use function is_array;
use function is_callable;
Expand Down
3 changes: 2 additions & 1 deletion src/DeserializationContext.php
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,8 @@ class DeserializationContext extends Context
public bool $ignoreCase = false;
private int $depth = 0;

public function createChildContext(array $attributes = []): Context
/** @inheritDoc */
public function createChildContext(array $attributes = []): static
{
$context = parent::createChildContext($attributes);
$context->ignoreCase = $this->ignoreCase;
Expand Down
5 changes: 5 additions & 0 deletions src/GenericDeserializationVisitor.php
Original file line number Diff line number Diff line change
Expand Up @@ -12,14 +12,19 @@
use Kcs\Serializer\Type\Type;
use UnitEnum;

use function array_change_key_case;
use function array_key_exists;
use function assert;
use function gettype;
use function is_array;
use function is_subclass_of;
use function mb_convert_case;
use function sprintf;
use function var_export;

use const CASE_LOWER;
use const MB_CASE_LOWER;

/**
* Generic Deserialization Visitor.
*/
Expand Down
5 changes: 1 addition & 4 deletions src/SerializationContext.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@
use Kcs\Serializer\Type\Type;
use SplObjectStorage;

use function assert;
use function gettype;
use function is_object;

Expand All @@ -31,11 +30,9 @@ public function initialize(
/**
* {@inheritDoc}
*/
public function createChildContext(array $attributes = []): self
public function createChildContext(array $attributes = []): static
{
$obj = parent::createChildContext($attributes);
assert($obj instanceof self);

$obj->visitingSet = $this->visitingSet;

return $obj;
Expand Down
10 changes: 9 additions & 1 deletion src/XmlDeserializationVisitor.php
Original file line number Diff line number Diff line change
Expand Up @@ -17,21 +17,29 @@
use SimpleXMLElement;
use UnitEnum;

use function array_change_key_case;
use function array_combine;
use function array_key_exists;
use function array_keys;
use function assert;
use function in_array;
use function is_string;
use function libxml_use_internal_errors;
use function mb_convert_case;
use function reset;
use function Safe\libxml_get_last_error;
use function Safe\preg_replace;
use function Safe\simplexml_load_string;
use function sprintf;
use function str_replace;
use function stripos;
use function strtolower;
use function substr;
use function var_export;

use const CASE_LOWER;
use const MB_CASE_LOWER;

class XmlDeserializationVisitor extends GenericDeserializationVisitor
{
/** @var string[] */
Expand Down Expand Up @@ -259,7 +267,7 @@ protected function visitProperty(PropertyMetadata $metadata, mixed $data, Contex

$node = $children[0];
} else {
if (!isset($data->$name)) {
if (! isset($data->$name)) {
return null;
}

Expand Down

0 comments on commit 57cf7a5

Please sign in to comment.