Bug #28255
Warning in log when displaying report
| Status: | Closed | Start date: | 2011-07-16 | |
|---|---|---|---|---|
| Priority: | Must have | Due date: | ||
| Assignee: | - | % Done: | 100% |
|
| Category: | - | |||
| Target version: | Core - 4.5.4 | |||
| TYPO3 Version: | Tags: | |||
| PHP Version: | Complexity: | |||
| Votes: | 0 |
Description
When displaying the report the TYPO3 log is filled with warnings like:
Core: Error handler (BE): PHP Warning: filesize(): stat failed for /var/www/conet/htdocs/typo3/atibility.js in /var/www/conet/htdocs/typo3_src-4.5.2/t3lib/class.t3lib_compressor.php line 341
I described this earlier in #25303 but assumed that this is an error in the template class. Investigating this again, I found out that it's actually an error in linkvalidator.
modfuncreport/class.tx_linkvalidator_modfuncreport.php creates a template object in line 295. In the next line, setModuleTemplate is called which internally adds the js/iecompatiblity.js to the renderer. To do this properly, it needs the backPath. BUT the backPath of the template object is set only one line later.
So, the solution is to swap lines 296 and 297, thus setting the backPath immediately after the creating of the template object.
Related issues
| duplicates Core - Bug #25303: js/iecompatibility.js inclusion triggers compressor bug | Resolved | 2011-03-10 | ||
| duplicated by Linkvalidator - Bug #13370: PHP Warnings concerning typo3/atibility.js | Closed | 2011-02-26 |
Associated revisions
[BUGFIX] Correct initialisation of BACK_PATH
BACK_PATH needs to be set before calling setModuleTemplate().
Thanks to Daniel Minder.
Change-Id: Ia5a4c332ef50785150b58e2e4d7deff0a9544e26
Resolves: #28255
Releases: 4.5, 4.6
[BUGFIX] Correct initialisation of BACK_PATH
BACK_PATH needs to be set before calling setModuleTemplate().
Thanks to Daniel Minder.
Change-Id: I24bee765b97074817cc2914f9d1ad174d656cf0e
Resolves: #28255
Releases: 4.5, 4.6
History
Updated by Mr. Hudson almost 2 years ago
Patch set 1 of change Ia5a4c332ef50785150b58e2e4d7deff0a9544e26 has been pushed to the review server.
It is available at http://review.typo3.org/3377
Updated by Mr. Hudson almost 2 years ago
Patch set 1 of change I24bee765b97074817cc2914f9d1ad174d656cf0e has been pushed to the review server.
It is available at http://review.typo3.org/3382
Updated by Stefan Neufeind almost 2 years ago
- Status changed from New to Resolved
- % Done changed from 0 to 100
Applied in changeset 73570b348ff7a124faa7c717a996319e10230cf9.
Updated by Chris topher almost 2 years ago
- Target version set to 4.5.4
Updated by Chris topher about 1 year ago
- Status changed from Resolved to Closed