Skip to content

Comments

Use === and !== in lib#2979

Closed
Niduroki wants to merge 13 commits intomasterfrom
===_and_!==_in_lib
Closed

Use === and !== in lib#2979
Niduroki wants to merge 13 commits intomasterfrom
===_and_!==_in_lib

Conversation

@Niduroki
Copy link
Member

Mostly automated. Did some testing.

More testing would be nice.

cc @tomneedham @bartv2 @MTGap @karlitschek @LukasReschke @icewind1991 @DeepDiver1975 @tanghus @eMerzh

@LukasReschke
Copy link
Member

@Kondou-ger Build failed

@LukasReschke
Copy link
Member

;-)

@DeepDiver1975
Copy link
Member

@Kondou-ger @butonic rebase required - THX

@jancborchardt
Copy link
Member

Also here too, can the referred people take a look at this? cc @icewind1991 @LukasReschke @bartv2 @butonic @DeepDiver1975

@bartv2
Copy link
Contributor

bartv2 commented Jul 21, 2013

👍 looks good

@ghost
Copy link

ghost commented Sep 15, 2013

Conflicts:
	lib/user/http.php
@ghost
Copy link

ghost commented Sep 30, 2013

Conflicts:
	lib/base.php
	lib/private/app.php
	lib/private/connector/sabre/file.php
	lib/private/helper.php
	lib/vcategories.php
@ghost
Copy link

ghost commented Oct 11, 2013

@karlitschek
Copy link
Contributor

I'm actually not sure if this is the right approach. == and === makes a difference in PHP. You can't always assume that the logic doesn'T break if you replace this automatically with a script. Sorry but we have to do this manually or keep it as is is. Closing this for now.

@Niduroki
Copy link
Member Author

You can't always assume that the logic doesn'T break if you replace this automatically with a script.

That's why this has to be tested 😉

@Niduroki Niduroki deleted the ===_and_!==_in_lib branch October 22, 2013 15:42
@karlitschek
Copy link
Contributor

Sure. But it's a challenge to do a full test. :-) I think we have to do this step by step :-)

@lock lock bot locked as resolved and limited conversation to collaborators Aug 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants