Skip to content

[5.7] remove PHP mail driver #4833

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

Merged
merged 1 commit into from
Dec 14, 2018
Merged

[5.7] remove PHP mail driver #4833

merged 1 commit into from
Dec 14, 2018

Conversation

browner12
Copy link
Contributor

@browner12 browner12 commented Dec 13, 2018

the PHP mail driver is no longer used. It is simply an alias for the sendmail driver.

the 'mail' driver will continue to work (for now), we just shouldn't document it to help avoid confusion.

with the current wording, the programmer may think the native PHP mail() function is still being used, which it is not.

the PHP mail driver is no longer used. It is simply an alias for the sendmail driver.

the 'mail' driver will continue to work (for now), we just shouldn't document it to help avoid confusion.

with the current wording, the program may think the native PHP `mail()` function is still being used, which it is not.
@browner12
Copy link
Contributor Author

@taylorotwell taylorotwell merged commit 43601ad into laravel:5.7 Dec 14, 2018
@browner12 browner12 deleted the patch-1 branch December 14, 2018 14:47
taylorotwell pushed a commit that referenced this pull request Nov 30, 2021
If the callback given to `skipWhile` never returns `false`, then an
empty collection would be returned. Currently the documentation states
the opposite. If the callback never returns `true`, then the whole
collection is returned unfiltered.

```
>>> $collection = collect([1, 2, 3, 4]);
=> Illuminate\Support\Collection {#4833
     all: [
       1,
       2,
       3,
       4,
     ],
   }
>>> $subset = $collection->skipWhile(function ($item) { return false; });
=> Illuminate\Support\Collection {#4835
     all: [
       1,
       2,
       3,
       4,
     ],
   }
>>> $subset = $collection->skipWhile(function ($item) { return true; });
=> Illuminate\Support\Collection {#4824
     all: [],
   }
```
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