Skip to content

Implement exists() construct #1530

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

Closed
wants to merge 1 commit into from
Closed

Conversation

flaupretre
Copy link
Contributor

This works targets PHP 7.1. RFC is under way.

@smalyshev smalyshev added the RFC label Sep 22, 2015
@c9s
Copy link
Contributor

c9s commented Sep 25, 2015

Probably rename exists to defined since it might be confusing with array_keys_exists?

@nikic
Copy link
Member

nikic commented Sep 25, 2015

@c9s There is already a defined function. http://php.net/defined

@carusogabriel
Copy link
Contributor

Is there a RFC for this? I couldn’t found it 😞

@cmb69
Copy link
Member

cmb69 commented Sep 8, 2018

This seems to have some overlap with PR #3360.

@Majkl578
Copy link
Contributor

At this moment I think this doesn't need a specialized language construct, that's also why I implemented it as a function optimization in #3360. Even justifying array_key_exists() is 50-50 imho, judging from Dmitry's comment in the other PR (which I can agree with, but mostly because people avoid that function now due to its performance penalty).

@php-pulls
Copy link

Comment on behalf of kalle at php.net:

Closing due to inactivity

@php-pulls php-pulls closed this Sep 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants