Skip to content

Conversation

@lucasmichot
Copy link
Contributor

No need to call isset twice.

taylorotwell added a commit that referenced this pull request Nov 30, 2015
@taylorotwell taylorotwell merged commit 8f7d46d into laravel:5.1 Nov 30, 2015
@taylorotwell
Copy link
Member

TIL 😲

@mdwheele
Copy link

Oh lawd... that's pretty boss.

@davzie
Copy link

davzie commented Nov 30, 2015

Thank you! This is incredibly useful to know @lucasmichot

@joshbrw
Copy link

joshbrw commented Nov 30, 2015

😮

@brocktaylor7
Copy link

whoa, how have I never known that was possible?! Great to know!

@zaynali53
Copy link

😲

@unicodeveloper
Copy link
Contributor

😮

@idiscussforum
Copy link

What? That's possible?

😮

@lucasmichot
Copy link
Contributor Author

Yes same for me guys
And @taylorotwell TIL was a surprise too

@GrahamCampbell GrahamCampbell changed the title Remove a useless isset [5.1] Remove a useless isset Nov 30, 2015
@vlakoff
Copy link
Contributor

vlakoff commented Dec 1, 2015

I knew it 😛

@abrahamgreyson
Copy link
Contributor

blow my mind.

@mateusjatenee
Copy link
Contributor

Holy. Fucking. Shit. 😨

@lucasmichot lucasmichot deleted the feature/5.1/useless-isset branch December 2, 2015 04:59
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.