Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ArrayList support returning references #289

Merged
merged 40 commits into from
Jan 19, 2023
Merged

ArrayList support returning references #289

merged 40 commits into from
Jan 19, 2023

Conversation

duncanxia97
Copy link
Contributor

@duncanxia97 duncanxia97 commented Jan 15, 2023

$arrayList = ArrayList::form([['id'=>1],['id'=>2]]);
// $arrayList => [['id'=>1],['id'=>2]]
foreach($arrayList as &$item)
{
    $item['id'] += 1;
}
unset($item);
// $arrayList => [['id'=>2],['id'=>3]]

duncanxia97 and others added 30 commits January 15, 2023 17:40
$arrayList = ArrayList::form([['id'=>1],['id'=>2]]);
        // $arrayList => [['id'=>1],['id'=>2]]
        foreach($arrayList as &$item)
        {
            $item['id'] += 1;
        }
        unset($item);
        // $arrayList => [['id'=>2],['id'=>3]]
moved from nette/finder
except for ArrayHash::offsetGet() to maintain compatibility with nette/database v3
Since PHP 8.0 ReflectionParameter::getDefaultValue() correctly returns value or throws exception when constant cannot be resolved
@dg dg force-pushed the master branch 8 times, most recently from 489126b to 46e8fe1 Compare January 19, 2023 16:03
@dg dg merged commit cc06620 into nette:master Jan 19, 2023
@dg
Copy link
Member

dg commented Jan 19, 2023

Thanks

dg pushed a commit that referenced this pull request Jan 19, 2023
dg pushed a commit that referenced this pull request Jan 19, 2023
dg pushed a commit that referenced this pull request Jan 19, 2023
dg pushed a commit that referenced this pull request Jan 19, 2023
dg pushed a commit that referenced this pull request Jan 21, 2023
dg pushed a commit that referenced this pull request Jan 21, 2023
dg pushed a commit that referenced this pull request Jan 25, 2023
dg pushed a commit that referenced this pull request Jan 25, 2023
dg pushed a commit that referenced this pull request Jan 26, 2023
dg pushed a commit that referenced this pull request Jan 31, 2023
dg pushed a commit that referenced this pull request Jan 31, 2023
dg pushed a commit that referenced this pull request Jan 31, 2023
dg pushed a commit that referenced this pull request Jan 31, 2023
dg pushed a commit that referenced this pull request Feb 1, 2023
dg pushed a commit that referenced this pull request Feb 1, 2023
dg pushed a commit that referenced this pull request Feb 2, 2023
dg pushed a commit that referenced this pull request Feb 2, 2023
dg pushed a commit that referenced this pull request Feb 2, 2023
dg pushed a commit that referenced this pull request Feb 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants