Callback: fixed bug in is_callable($object, true) [Closes #176]

Author: dg

src/Utils/Callback.php

@@ -107,7 +107,7 @@ public static function toString($callable): string
 		} elseif (is_string($callable) && $callable[0] === "\0") {
 			return '{lambda}';
 		} else {
-			is_callable($callable, true, $textual);
+			is_callable(is_object($callable) ? [$callable, '__invoke'] : $callable, true, $textual);
 			return $textual;
 		}
 	}

tests/Utils/Callback.check.phpt

@@ -26,3 +26,13 @@ Assert::exception(function () {
 Assert::exception(function () {
 	Callback::check('undefined');
 }, Nette\InvalidArgumentException::class, "Callback 'undefined' is not callable.");
+
+
+// PHP bugs - is_callable($object, true) fails
+Assert::exception(function () {
+	Callback::check(new stdClass);
+}, Nette\InvalidArgumentException::class, "Callback 'stdClass::__invoke' is not callable.");
+
+Assert::exception(function () {
+	Callback::check(new stdClass, true);
+}, Nette\InvalidArgumentException::class, 'Given value is not a callable type.');

tests/Utils/Callback.closure.phpt

@@ -205,3 +205,16 @@ test(function () { // magic methods
 		Callback::toReflection(Callback::closure(new Test, 'magic'));
 	}, ReflectionException::class, 'Method Test::magic() does not exist');
 });
+
+
+test(function () { // PHP bugs - is_callable($object, true) fails
+	Assert::exception(function () {
+		Callback::closure(new stdClass);
+	}, Nette\InvalidArgumentException::class, 'Failed to create closure from callable: no array or string given');
+
+	Assert::same('stdClass::__invoke', Callback::toString(new stdClass));
+
+	Assert::exception(function () {
+		Callback::toReflection(new stdClass);
+	}, ReflectionException::class, 'Method stdClass::__invoke() does not exist');
+});