TinCanPHP

TinCanPHP copied to clipboard

Tincan error in the `set_error_handler` method due to incompatibility with PHP 8.0

1

As of PHP 8.0, the native PHP method `set_error_handler` now only accepts 4 parameters instead of 5 due to the variable `$errcontext` being removed. See PHP official Docs: https://www.php.net/manual/en/function.set-error-handler.php ```...

GrahamStaveleyBerry

#111: Resolve PHP 8.0 error when calling set_error_handler

1

GrahamStaveleyBerry

Namshi/jose dependency

1

Hello, here https://github.com/namshi/jose I've found a "Deprecation notice" and a suggestion to use a maintained library, eg: https://github.com/firebase/php-jwt do you plan to change the dependency? thanks

furlo82

PHP 8 compatibility

11

the packagist / composer instance tops out at php 7. Are there plans to be compatible with php 8?

scottshipman

On Php 7.3 you get this exception: ErrorException(code: 0): count(): Parameter must be an array or an object that implements Countable at rusticisoftware/tincan/src/Map.php:49 This commit fixes it.

il-santo

There is a [HHVM blog post](https://hhvm.com/blog/2018/09/12/end-of-php-support-future-of-hack.html) that says HHVM v3.30 will be the last version to support PHP. Maybe TinCanPHP should drop support of HHVM? Or maybe add a note...

pondermatic

Fix testing PHP 7.0

1

* Testing with PHP 7.0 was failing because PHPUnit 6.4 was used, which breaks compatibility with PHPUnit 4.8.* and 5.4.*. * Updated tests for PHP 7.1 change to handling "too...

pondermatic

pondermatic

Fix ContextTest comparison of ContextActivities signatures.

1

The grouping array was sent as argument 2 to the $contextActivities2 constructor. ContextActivities->__construct() only takes one argument and will not assign ANY properties if there is not exactly one argument....

pondermatic

Fix code smells

1

Fix issues that are considered code smells: * method calls with mismatched number of parameters * inconsistent return points in methods with missing return statements (explicity return null)

pondermatic