Make sure only arrays are passed in foreach()

[zkwbbr]

Fix errors like this:

foreach() argument must be of type array|object, string given on /vendor/aura/web/src/Request/Files.php (79)

[harikt]

I think probably the best way is to check is_array before this init method.

[zkwbbr]

Not sure if I follow you, you mean like this?

$tgt[$key] = array();
if (is_array($tgt[$key])
    $this->init($val, $tgt[$key]);

But that will always be an array because of the previous line: $tgt[$key] = array();?

[harikt]

foreach is on the first argument passed. That means it is $val should be an array .

[harikt]

@zkwbbr are you using this package currently ?

[zkwbbr]

@zkwbbr are you using this package currently ?

Yes I am. This issue happens when files are uploaded.

[harikt]

@zkwbbr can you tell me how I can reproduce this error with some basic code you are testing.

According to the error you mentioned is line 79.

foreach() argument must be of type array|object, string given on /vendor/aura/web/src/Request/Files.php (79)

and from the diff of the code it shows only 78 lines. So I would like to do a test before I merge this.

[zkwbbr]

@zkwbbr can you tell me how I can reproduce this error with some basic code you are testing.

According to the error you mentioned is line 79.

foreach() argument must be of type array|object, string given on /vendor/aura/web/src/Request/Files.php (79)

and from the diff of the code it shows only 78 lines. So I would like to do a test before I merge this.

Sorry, the error line # is actually 72.

Here's a simple code where the error occurs upon file upload:

<?php

require __DIR__.'/vendor/autoload.php';

$webFactory = new \Aura\Web\WebFactory($GLOBALS);
$request = $webFactory->newRequest();

?>

<form method="post" enctype="multipart/form-data">
	<input type="file" name="file">
	<input type="submit">
</form>

Upon submit, you should receive an error like this:

Warning: foreach() argument must be of type array|object, string given in /../vendor/aura/web/src/Request/Files.php on line 72

Tested on: PHP 8.1 and 8.2

[harikt]

Released : https://github.com/auraphp/Aura.Web/releases/tag/2.2.1 .

Thank you. And sorry for the delay.

[zkwbbr]

Released : https://github.com/auraphp/Aura.Web/releases/tag/2.2.1 .

Thank you. And sorry for the delay.

No problem, thank you :)