Object of class _Request could not be converted to string

[j-hernandez]

in klein.php on line 21 - according to Apache.

respond('GET', '/posts', function($request)
{
  echo "hello";
});

This code throws a 500 error.

[codingkevin]

Get this as well:

PHP Catchable fatal error: Object of class _Request could not be converted to string

That is apache error log output, same line.

Also, was getting a ton of these warnings:

PHP Notice: Undefined variable: namespace in klein.php on line 21

Is that variable suppose to be __namespace?

[j-hernandez]

@codingkevin Yeah, I was able to fix that error by changing namespace to __namespace on line 21, the original error persists after that.

[samwierema]

This issue was introduced in commit 38ca436.

In commit e8357bf aliases were introduced for the respond() function (see lines 39 through 42). This means that for all GET, POST and PUT requests the $callback variable will always be converted to the $method, since those methods are callable (aliases, remember?). I've solved this issue by commenting out that part of the function, like so:

<?php
/*if (is_callable($method)) {
    $callback = $method;
    $method = null;
    $count_match = false;
} else*/if (is_callable($route)) {
    $callback = $route;
    $route = $method;
    $method = null;
}

This fixes the issue. However I can not tell whether I've broken any of the other code by doing so. It's very untested...

[chriso]

Apologies guys, should have tested this stuff before pushing it. @samwierema was right, is_callable($method) was picking up the aliases get(), post(), etc. and going down the wrong branch. It should all be fixed as of this commit.