Bug #38244: 't3lib_lock' does not exist if $TYPO3_CONF_VARS['SYS']['systemLog'] = 'file,...' - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #38244

closed

't3lib_lock' does not exist if $TYPO3_CONF_VARS['SYS']['systemLog'] = 'file,...'

Added by Simon Apold almost 12 years ago. Updated almost 12 years ago.

Status:

Rejected

Priority:

Won't have this time

Assignee:

Category:

Target version:

4.7.2

Start date:

2012-06-21

Due date:

% Done:

Estimated time:

TYPO3 Version:

4.7

PHP Version:

5.3

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

If the systemLog is configured to file ($TYPO3_CONF_VARS['SYS']['systemLog'] = 'file,...'), the workspace-module won't work.
It will only show an error message: "router - class 't3lib_lock' does not exist".

Files

Patch1.diff (609 Bytes) Patch1.diff

Simon Apold, 2012-06-21 09:42

Related issues 2 (0 open — 2 closed)

Actions

Copy link

Updated by Simon Apold almost 12 years ago

File Patch1.diff Patch1.diff added

I attached a patch, which fixes this issue.

Actions

Copy link

Updated by Philipp Gampe almost 12 years ago

Hi, do you mind pushing the patch to gerrit?

http://wiki.typo3.org/Contribution_Walkthrough_Tutorials

One nitpick for the patch, the comment needs one more tab of indention (always one more than the code).

Actions

Copy link

Updated by Gerrit Code Review almost 12 years ago

Status changed from New to Under Review

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

Actions

Copy link

Updated by Gerrit Code Review almost 12 years ago

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

Actions

Copy link

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/12264

Actions

Copy link

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/12269

Actions

Copy link

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/12275

Actions

Copy link

Updated by Christian Kuhn almost 12 years ago

Is this still reproducible after #36241 is merged in 4.7?

To test please checkout workspaces_4-7 current branch in typo3/sysext/workspaces.

Actions

Copy link

Updated by Simon Apold almost 12 years ago

Your right, with the patch in #36241 this issue is not reproducible any more.
It seems, that any other PHP error message or even notice will cause this behaviour.
Should i abandon this change?

Actions

Copy link

#10