Project

General

Profile

Actions

Bug #47828

closed

Short PHP opening/closing Tags in temp_CACHED files cause errors

Added by Gabriel Kaufmann / Typoworx NewMedia over 11 years ago. Updated almost 10 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Caching
Target version:
-
Start date:
2013-05-02
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
4.7
PHP Version:
5.3
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

The "temp_CACHED"-Files written into "typo3conf/" contain the content of a list of files. Closing PHP tags and long opening PHP tags

<?php ?>
are stripped from these files.

However, I noticed that short PHP opening-/closing Tags

<?...?>
are not stripped, but included for every included file.

This is absolutely unnesseccary and can also lead into problems.

Some developers begin to strip-off the closing-tags at the end of their PHP-Files to avoid writing blanks to the PHP-Output.

The blanks may lead to invalid xHTML (for the W3C Validator).

As the additional opening-/closing tags are just overload for the temp_CACHE files they should be simply stripped of except the first one. The closing-tag should also be stripped off.

Actions #1

Updated by Gabriel Kaufmann / Typoworx NewMedia over 11 years ago

Update:

Sorry I noticed that TYPO3 still seems to strip off additional PHP-Tags.

But I noticed that TYPO3 does this only with "

The TYPO3 Cache-Parse writing the temp_CACHE PHP-Files should also take care of the short-open-tag for PHP.

So please take note of this change of my issue.

Actions #2

Updated by Chris topher over 11 years ago

  • Subject changed from temp_CACHED Files contain a bunch of PHP opening/closing Tags that cause errors to Short PHP opening/closing Tags in temp_CACHED files cause errors
Actions #3

Updated by Gabriel Kaufmann / Typoworx NewMedia over 11 years ago

By the way

I think it would make sense to wrap each File-Incluion in temp_CACHE-Files into try/catch to fetch errors and try to avoid a total crash of the CORE if one extension is breaking something in there?

Actions #4

Updated by Mathias Schreiber almost 10 years ago

  • Status changed from New to Closed
  • Is Regression set to No

won't fix in 4.5 and no longer necessary in 6.x+

Actions

Also available in: Atom PDF