Project

General

Profile

Actions
Copy link

Bug #38691

closed

Exclude E_STRICT on PHP 5.4

Added by Philipp Gampe almost 12 years ago. Updated over 11 years ago.

Status:
Closed
Priority:
Must have
Assignee:
Philipp Gampe
Category:
Code Cleanup
Target version:
-
Start date:
2012-07-05
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
4.5
PHP Version:
5.4
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Some pleases in TYPO3, e.g. typo3/sysext/cms/tslib/index_ts.php do not yet exclude E_STRICT from error reporting.

Adopt those places to look all the same (like in index.php).

As my patch will be a follow up for the E_DEPRECATED not defined on PHP 5.2, this will only apply to 4.5.

4.6 and 4.7 needs to be checked too.

Related issues 3 (0 open — 3 closed)

Related to TYPO3 Core - Bug #38645: New error_reporting settings cause notice "undefined constant E_DEPRECATED " with PHP 5.2Closed2012-07-04

Actions
Related to TYPO3 Core - Bug #39232: PHP error_reporting cannot be set properly, E_STRICT errors shown on PHP 5.4Closed2012-07-25

Actions
Is duplicate of TYPO3 Core - Bug #35154: Exclude E_STRICT from exceptionalErrors in PHP 5.4Closed2012-03-22

Actions
Actions
Copy link
 #1

Updated by Markus Klein almost 12 years ago

4.7.2 misses:
  • t3lib/thumbs.php
  • typo3/sysext/cms/tslib/showpic.php
  • typo3/sysext/install/mod/class.tx_install_ajax.php
4.6.10 misses:
  • t3lib/class.t3lib_superadmin.php
  • t3lib/thumbs.php
  • typo3/sysext/cms/tslib/index_ts.php
  • typo3/sysext/cms/tslib/showpic.php
  • typo3/sysext/install/mod/class.tx_install_ajax.php
Actions
Copy link
 #2

Updated by Philipp Gampe almost 12 years ago

Thanks Markus. I will take care tonight or tomorrow.

Actions
Copy link
 #3

Updated by Gerrit Code Review almost 12 years ago

  • Status changed from New to Under Review

Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at http://review.typo3.org/12656

Actions
Copy link
 #4

Updated by Philipp Gampe almost 12 years ago

Other branches will follow in a couple hours ... I have to to university now ;)

Actions
Copy link
 #5

Updated by Oliver Hader almost 12 years ago

... (ignore this comment) ...

Actions
Copy link
 #6

Updated by Gerrit Code Review almost 12 years ago

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12689

Actions
Copy link
 #7

Updated by Gerrit Code Review almost 12 years ago

Patch set 1 for branch TYPO3_4-7 has been pushed to the review server.
It is available at http://review.typo3.org/12690

Actions
Copy link
 #8

Updated by Gerrit Code Review almost 12 years ago

Patch set 1 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/12691

Actions
Copy link
 #9

Updated by Philipp Gampe almost 12 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions
Copy link
 #10

Updated by Gerrit Code Review almost 12 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_4-7 has been pushed to the review server.
It is available at http://review.typo3.org/13206
(irrelevant/accidental commit, pls ignore, Peter Niederlag)

Actions
Copy link
 #11

Updated by Helmut Hummel almost 12 years ago

  • Status changed from Under Review to Resolved
Actions
Copy link
 #12

Updated by Gerrit Code Review almost 12 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/13313

Actions
Copy link
 #13

Updated by Stefan Galinski over 11 years ago

  • Status changed from Under Review to Closed
Actions
Copy link

Also available in: Atom PDF