Skip to content

Commit

Permalink
Merge pull request #342 from Rican7/bugfix/php-7-compatibility
Browse files Browse the repository at this point in the history
Bugfix - PHP 7 compatibility
  • Loading branch information
Rican7 authored Nov 1, 2016
2 parents 05a5da6 + 2547866 commit fdf072b
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 8 deletions.
1 change: 0 additions & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ php:

matrix:
allow_failures:
- php: 7.0
- php: nightly
- php: hhvm
fast_finish: true
Expand Down
2 changes: 1 addition & 1 deletion tests/Klein/Tests/Mocks/TestClass.php
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
class TestClass
{

public static function get($r, $r, $a)
public static function get($request, $response, $app)
{
echo 'ok';
}
Expand Down
12 changes: 6 additions & 6 deletions tests/Klein/Tests/RoutingTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -113,20 +113,20 @@ public function testAppReference()

$this->klein_app->respond(
'/',
function ($r, $r, $s, $a) {
$a->state = 'a';
function ($request, $response, $service, $app) {
$app->state = 'a';
}
);
$this->klein_app->respond(
'/',
function ($r, $r, $s, $a) {
$a->state .= 'b';
function ($request, $response, $service, $app) {
$app->state .= 'b';
}
);
$this->klein_app->respond(
'/',
function ($r, $r, $s, $a) {
print $a->state;
function ($request, $response, $service, $app) {
print $app->state;
}
);

Expand Down

0 comments on commit fdf072b

Please sign in to comment.