Skip to content

Commit 2a3ff1d

Browse files
authored
Merge pull request #17 from vemaeg/feature/display_errors_to_stderr
1.5.15-patched: Feature/display errors to stderr
2 parents 36daf0b + 70db22f commit 2a3ff1d

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

WHATS_NEW.md

+3-2
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,8 @@ This allow you to remove all stylesheets and/or all javascripts added into the g
165165
Configuration
166166
-------------
167167

168-
A new configuration `sf_upload_dir_name` contains 'uploads' has been added.
168+
* A new configuration `sf_upload_dir_name` contains 'uploads' has been added.
169+
* A new settings `sf_display_errors` has been added to change display_errors from stdout to stderr for example.
169170

170171
Performance
171172
-----------
@@ -244,4 +245,4 @@ all:
244245
monolog.handler.file:
245246
class: Monolog\Handler\StreamHandler
246247
arguments: [/the/file/path/of/your/file.log]
247-
```
248+
```

lib/config/sfApplicationConfiguration.class.php

+2-1
Original file line numberDiff line numberDiff line change
@@ -143,7 +143,8 @@ public function initConfiguration()
143143
}
144144

145145
// error settings
146-
ini_set('display_errors', $this->isDebug() ? 'on' : 'off');
146+
// PHP >= 5.2.4 supports stdout/stderr instead of bool here: https://www.php.net/manual/en/errorfunc.configuration.php#ini.display-errors
147+
ini_set('display_errors', $this->isDebug() ? sfConfig::get('sf_display_errors', 'stdout') : '');
147148
error_reporting(sfConfig::get('sf_error_reporting'));
148149

149150
// initialize plugin configuration objects

0 commit comments

Comments
 (0)