Incorrect JIT behaviour with unusual PHP syntax

[tyranron]

For situation like

it('falls', function () {
    $pdf = new \mPDF();
});

Kahlan outputs

PHP Parse error:  syntax error, unexpected '}' in /path/to/temp/dir/kahlan/project/vendor/mpdf/mpdf/mpdf.php on line 1981

If we look into this file at this line we will see:

    if ($this->bodyBackgroundColor) {
        $s .= 'q ' .$this->SetFColor($this->bodyBackgroundColor, true)."\n";
        if ($this->bodyBackgroundColor{}0==5) { // RGBa
            $s .= $this->SetAlpha($__KMONKEY__123($this->bodyBackgroundColor{4})/100, 'Normal', true, 'F')."\n";
        }
        else if ($this->bodyBackgroundColor{}0==6) {    // CMYKa
            $s .= $this->SetAlpha($__KMONKEY__124($this->bodyBackgroundColor{5})/100, 'Normal', true, 'F')."\n";
        }
        $s .= $__KMONKEY__125('%.3F %.3F %.3F %.3F re f Q', ($clx*_MPDFK), ($cly*_MPDFK),$clw*_MPDFK,$clh*_MPDFK)."\n";
    }

At the same time the original file is:

    if ($this->bodyBackgroundColor) {
        $s .= 'q ' .$this->SetFColor($this->bodyBackgroundColor, true)."\n";
        if ($this->bodyBackgroundColor{0}==5) { // RGBa
            $s .= $this->SetAlpha(ord($this->bodyBackgroundColor{4})/100, 'Normal', true, 'F')."\n";
        }
        else if ($this->bodyBackgroundColor{0}==6) {    // CMYKa
            $s .= $this->SetAlpha(ord($this->bodyBackgroundColor{5})/100, 'Normal', true, 'F')."\n";
        }
        $s .= sprintf('%.3F %.3F %.3F %.3F re f Q', ($clx*_MPDFK), ($cly*_MPDFK),$clw*_MPDFK,$clh*_MPDFK)."\n";
    }

The problem is in that string:

if ($this->bodyBackgroundColor{}0==5) { // RGBa

Version of Kahlan: 1.1.5 (d78b421)
Version of mPDF: v6.0.0 (a15743d030ce3b5b7be36c6e83f76589b27c3f2c)

[jails]

Nice find.
I fixed it in the JIT repo crysalead/jit@04faf06

[tyranron]

@jails so fast...as usual =) Thank you.
May I ask you to release new minor version? This fix is quite important for me.

[jails]

Since kahlan depends on "crysalead/jit": "~1.0" a simple composer update should update the JIT library to the fixed version.
Anyhow I'll also create a new tag for Kahlan anytime soon.

[tyranron]

@jails didn't notice that possibility...thanks for tip!