php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Request #31627 return statement should return
Submitted: 2005-01-20 19:28 UTC Modified: 2017-10-26 11:27 UTC
Votes:2
Avg. Score:3.0 ± 2.0
Reproduced:1 of 2 (50.0%)
Same Version:1 (100.0%)
Same OS:1 (100.0%)
From: mnemo at minimum dot se Assigned:
Status: Suspended Package: Scripting Engine problem
PHP Version: * OS: *
Private report: No CVE-ID: None
View Add Comment Developer Edit
Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know!
Just going to say 'Me too!'? Don't clutter the database with that please — but make sure to vote on the bug!
Your email address:
MUST BE VALID
Solve the problem:
24 - 2 = ?
Subscribe to this entry?

 
 [2005-01-20 19:28 UTC] mnemo at minimum dot se 
Description:
------------
I wish "return doh" was treated like an expression when though this expression will never be computable.

The reason is that I want to write this:

mysql_connect(blah) or return false;

Just like I do with die etc.

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2011-01-02 02:11 UTC] jani@php.net
-Package: Feature/Change Request +Package: Scripting Engine problem -Operating System: All +Operating System: * -PHP Version: Irrelevant +PHP Version: *
 [2011-07-20 23:36 UTC] binarycleric at gmail dot com 
That really isn't what return does because it's not a conditional.  Similar 
behaviour can be achieved by using: 

return (mysql_connect($blah) !== false) ? true : false;
 [2014-03-12 08:40 UTC] asphp at dsgml dot com 
binarycleric - that doesn't do the same thing in the slightest.

That would cause the return to run either way. What mnemo wants is for the return to happen ONLY if mysql fails.

Otherwise it should continue to the next line.

This works:

function foo() {}

mysql_connect(blah) or foo(false);

But this doesn't:

mysql_connect(blah) or return(false);

And I think it should.
 [2015-01-08 23:23 UTC] ajf@php.net 
This would make sense, I don't see why we couldn't make "return" an expression.
 [2015-03-23 19:00 UTC] cmb@php.net 
Then again, what's wrong with

  if (!mysql_connect(blah)) return false;
  
instead?
 [2016-06-23 23:37 UTC] cmb@php.net 
Related To: Bug #71098
 [2016-06-28 18:55 UTC] cmb@php.net 
Related To: Bug #67184
 [2016-07-13 10:19 UTC] cmb@php.net 
Related To: Bug #72589
 [2017-10-26 11:27 UTC] jhdxr@php.net
-Status: Open +Status: Suspended
 [2017-10-26 11:27 UTC] jhdxr@php.net 
Thank you for your interest in PHP and for submitting a feature request. Please be aware that due to the magnitude of change this request requires, it would be necessary to discuss it on PHP Internals list (internals@lists.php.net) as an RFC. Please read the guide about creating RFCs here:
https://wiki.php.net/rfc/howto
If you haven't had experience with writing RFCs before, it is advised to seek guidance on the Internals list (http://php.net/mailing-lists.php) and/or solicit help from one of the experienced developers. 

Please to not consider this comment as a negative view on the merits of your proposal - every proposal which requires changes of certain magnitude, even the very successful and widely supported ones, must be done through the RFC process. This helps make the process predictable, transparent and accessible to all developers.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Fri Apr 19 09:01:27 2024 UTC