PHP 8.1 while maintaining PHP 7.2 & 7.3 compatibility

[swiffer]

[gildonei]

I think that in SerializableEventTrait.php in serialize and unserialize methods you could call __serialize and __unserialize reducing duplicated code

/**
 * Returns the string representation of this object.
 *
 * @return string
 */
public function serialize()
{
    return $this->__serialize();
}

/**
 * Takes the string representation of this object so it can be reconstructed.
 *
 * @param string $data serialized string
 * @return void
 */
public function unserialize($data)
{
    $this->__unserialize($data);
}

[swiffer]

@gildonei done ;)

[swiffer]

@lorenzo - can this get merged ?

[kyleweishaupt]

Is there anything that can be done to get this merged? I have tons of logs with the following message using PHP 8.1:

AuditStash\Event\AuditUpdateEvent implements the Serializable interface, which is deprecated. Implement __serialize() and __unserialize() instead (or in addition, if support for old PHP versions is necessary)

[gildonei]

Goodwill of Lorenzo to do the merge

[lorenzo]

sorry this took long

[swiffer]

@lorenzo thanks merging. As it was Part of master shouldn't this have been tagged as 3.1.0?🤔

[kyleweishaupt]

@lorenzo Thank you so much!

[lorenzo]

@swiffer thanks, released now

[swiffer]

@swiffer thanks, released now

thanks!