[BUG] \Request\File::getRealType()

[quasipickle]

This is a bug with the documentation that I believe is auto generated from the C-source (which is why I'm not posting this bug in the Docs repository).

Phalcon\Http\Request\File::getRealType() isn't implemented, but nowhere in the documentation does it say that. I had to look through C source code to find that comment.

[phalcon]

Implemented in 1.3.0

[quasipickle]

I just downloaded & installed 1.3.0 this morning and it's not working.

This is the code I'm running:

$File = $this->request->getUploadedFiles()[0];

$Finfo = new Finfo();
$finfo_mime = $Finfo->file($File->getTempName(),FILEINFO_MIME_TYPE);

var_dump($finfo_mime);
var_dump($File->getRealType());

And this is the output:

string(15) "application/pdf"
NULL

[dreamsxin]

Can you test this:

phpunit unit-tests/RequestTest.php

[quasipickle]

Seems fine - this is the output:

PHPUnit 3.7.25 by Sebastian Bergmann.

..

Time: 1.03 seconds, Memory: 2.50Mb

OK (2 tests, 36 assertions)

[dreamsxin]

Can you in RequestTeset.php file find this code?

public function testIssues1442()

[quasipickle]

I just re-downloaded the cphalcon-1.3.0.zip and the RequestTest.php file now has that function. Running it gives me this output:

PHPUnit 3.7.25 by Sebastian Bergmann.

.....F

Time: 226 ms, Memory: 2.50Mb

There was 1 failure:

1) RequestTest::testIssues1442
Failed asserting that 'image/jpeg' matches expected null.

/home/--my username--/cphalcon-1.3.0/unit-tests/RequestTest.php:349

[ghost]

To clarify, did you build Phalcon from ext/?

[quasipickle]

No, I built it from build/, as that's what the documentation says. Re-building it from ext/ makes the test run successfully.

[ghost]

Please build development versions from ext/; build/ is for released versions.

For the development versions build/ and ext/ are often not synchronized (doing so would cause merge conflicts when there is more than one developer working on bug fixes / new features).

[quasipickle]

Thanks - I'll remember to do that in the future.