This PR was merged into the 1.x branch.
Discussion
----------
Fix implicit float to int casts
PHP 8.1 is picky about implicit float-to-int casts if we lose precision because of that cast. This PR attempts to fix all issues on the 1.x branch.
Commits
-------
de9cd76d Fix implicit flot to int casts
This PR was merged into the 2.x branch.
Discussion
----------
Add ReturnTypeWillChange to JsonSerializable implementation
On PHP 8.1, implementations of `JsonSerializable` should declare a return type.
In order to avoid a deprecation warning and maintain backwards compatibility, I've added the `ReturnTypeWillChange` attribute to declare that our implementation will add the necessary return type in a future release.
Commits
-------
507f78d9 Add ReturnTypeWillChange to JsonSerializable implementation
This PR was merged into the 1.x branch.
Discussion
----------
Replace secure.php.net with www.php.net
It is no longer necessary to use the _secure_ subdomain.
See: https://news-web.php.net/php.internals/104533.
> Instead of having mirrors, we are moving all of PHP.net to HTTPS (and get rid of https://secure.php.net)
Commits
-------
1783298b Replace secure.php.net with www.php.net
This PR was submitted for the 3.x branch but it was squashed and merged into the 1.x branch instead.
Discussion
----------
fix file permissions
`drupal_test.sh` was executable and when installing the plugin with `composer` it retains these permissions.
Better to call this script as `bash ./drupal_test.sh` and have `100644` as permissions.
I could only find one reference to this script and it was commented out.
Updated it anyway as it might be revived at some point.
Commits
-------
bd706520 fix file permissions
This PR was squashed before being merged into the 3.x branch.
Discussion
----------
Some code style fixes
Fixes that:
- Type cast is unnecessary
- Unnecessary curly braces syntax for variable
- Operations priority might differ that except
Commits
-------
17c6371f Some code style fixes