[PHP-BUG] Req #72382 [NEW]: THIS SHOULD BE CONSIDERED A BUG: __toString() must not throw an exception

[email protected] ("travis dot detert at gmail dot com")
Newsgroups php.standards
Message-ID <[email protected]>
From:             travis dot detert at gmail dot com
Operating system: All OS
PHP version:      Irrelevant
Package:          PHP Language Specification
Bug Type:         Feature/Change Request
Bug description:THIS SHOULD BE CONSIDERED A BUG: __toString() must not throw an exception

Description:
------------
The fundamental behavior of "__toString() must not throw an exception"
needs to be changed.  This problem is completely antithetical to object
oriented design, and just doesn't even make sense.

class Template{
function __construct($path){
$this->path = $path;
}

function __toString(){
if(!file_exists($this->path)){
throw new Exception('Path does not exist');
}else{
//process template
}
}

$Template = new Template('/path/does/not/exist.php');
echo (string)$Template;

There is absolutely no sane reason that this should be a fatal php
error.

Test script:
---------------
class Template{
function __construct($path){
$this->path = $path;
}

function __toString(){
if(!file_exists($this->path)){
throw new Exception('Path does not exist');
}else{
//process template
}
}

$Template = new Template('/path/does/not/exist.php');
echo (string)$Template;


Expected result:
----------------
Raise a regular exception.  Why would raising an exception ever be not
allowed?  What is the point?

Actual result:
--------------
__toString() must not throw an exception

Pretty much the worst result possible.  Unrecoverable fatal error.  Why?
 What is the point?

-- 
Edit bug report at https://bugs.php.net/bug.php?id=72382&edit=1
-- 
Try a snapshot (PHP 5.4):   https://bugs.php.net/fix.php?id=72382&r=trysnapshot54
Try a snapshot (PHP 5.5):   https://bugs.php.net/fix.php?id=72382&r=trysnapshot55
Try a snapshot (trunk):     https://bugs.php.net/fix.php?id=72382&r=trysnapshottrunk
Fixed in SVN:               https://bugs.php.net/fix.php?id=72382&r=fixed
Fixed in release:           https://bugs.php.net/fix.php?id=72382&r=alreadyfixed
Need backtrace:             https://bugs.php.net/fix.php?id=72382&r=needtrace
Need Reproduce Script:      https://bugs.php.net/fix.php?id=72382&r=needscript
Try newer version:          https://bugs.php.net/fix.php?id=72382&r=oldversion
Not developer issue:        https://bugs.php.net/fix.php?id=72382&r=support
Expected behavior:          https://bugs.php.net/fix.php?id=72382&r=notwrong
Not enough info:            https://bugs.php.net/fix.php?id=72382&r=notenoughinfo
Submitted twice:            https://bugs.php.net/fix.php?id=72382&r=submittedtwice
register_globals:           https://bugs.php.net/fix.php?id=72382&r=globals
PHP 4 support discontinued: https://bugs.php.net/fix.php?id=72382&r=php4
Daylight Savings:           https://bugs.php.net/fix.php?id=72382&r=dst
IIS Stability:              https://bugs.php.net/fix.php?id=72382&r=isapi
Install GNU Sed:            https://bugs.php.net/fix.php?id=72382&r=gnused
Floating point limitations: https://bugs.php.net/fix.php?id=72382&r=float
No Zend Extensions:         https://bugs.php.net/fix.php?id=72382&r=nozend
MySQL Configuration Error:  https://bugs.php.net/fix.php?id=72382&r=mysqlcfg
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.