bug fix in has() function.

[nayzo]

No description provided.

[nayzo]

Hello @minwork
Could you merge this fix and tag a new release please ?

[minwork]

Hello,

thanks for the PR. Could you post an example when has is not behaving as expected which is the reason for those changes?
Also this case needs to be added to the tests to ensure it is handled correctly in the next versions.

[nayzo]

Hello,

Sure here's an example:

$array = array('content' => 'some message!');

Arr::has($array, 'content.foo')

==> Warning: array_key_exists() expects parameter 2 to be array, string given

More details with a try catch:

foreach ($keysArray as $key) {
            try {
                if (!array_key_exists($key, $tmp)) {
                    return false;
                }
            } catch (\Exception $e) {
                var_dump($e->getMessage(), $key, $tmp, $keysArray);die;
            }
            $tmp = $tmp[$key];
        }

==>  "Warning: array_key_exists() expects parameter 2 to be array, string given" 
"foo" 
"some message!" 
{ [0]=> "content" [1]=> "foo" }

Hope it's more clear now.
Thanks.

[minwork]

@nayzo Thanks for the example, resolved by #3

[nayzo]

Could you tag a new Release ?

[minwork]

Already did, v1.13.3 (you can find it in tags section)

…

Wiadomość napisana przez Ala Eddine Khefifi ***@***.***> w dniu 05.10.2020, o godz. 10:04:  Could you tag a new Release ? — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.