Project

General

Profile

Actions

Bug #17095

closed

Typolink : cHash isn't created if there's no additionalParams

Added by Popy no-lastname-given over 17 years ago. Updated over 13 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Communication
Target version:
-
Start date:
2007-03-08
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
4.0
PHP Version:
4.3
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

If the typolink is configured without additionalParams and without addQueryString, the cHash is not generated, even if we need it because of linkVars

File class.tslib_content.php, line 5219 :

if (substr($addQueryParams,0,1)!='&') {
$addQueryParams = '';
} elseif ($conf['useCacheHash']) {
$pA = t3lib_div::cHashParams($GLOBALS['TSFE']->linkVars.$addQueryParams);
$addQueryParams.= '&cHash='.t3lib_div::shortMD5(serialize($pA));
}

I propose this patch :

if (substr($addQueryParams,0,1)!='&') {
$addQueryParams = '';
}
if ($conf['useCacheHash'] && trim($GLOBALS['TSFE']->linkVars.$addQueryParams)) {
$pA = t3lib_div::cHashParams($GLOBALS['TSFE']->linkVars.$addQueryParams);
$addQueryParams.= '&cHash='.t3lib_div::shortMD5(serialize($pA));
}

(issue imported from #M5161)


Files

class.tslib_content.php.diff (604 Bytes) class.tslib_content.php.diff Administrator Admin, 2007-05-02 10:52

Related issues 3 (0 open3 closed)

Related to TYPO3 Core - Bug #17070: typolink: cHash corrupted due to linkVarsClosedDmitry Dulepov2007-03-02

Actions
Related to TYPO3 Core - Bug #17002: cHash is not recreated on disposal in config.linkVarsClosed2007-02-17

Actions
Related to TYPO3 Core - Bug #19003: double cache or no cache through cHash variants of the same URLClosed2008-06-23

Actions
Actions #1

Updated by Popy no-lastname-given over 17 years ago

Patch file also resolve bug #0005117

Actions #2

Updated by Popy no-lastname-given over 17 years ago

Did somebody read the patch and test it ?

Actions #3

Updated by Sareen Millet almost 17 years ago

Hey Popy,

I did test the patch and it really helps some problem. But in the new Typo 4.15 again the cache is confusing and there must be some more bugs. - proper caching seams to be relly tough.

Sareen

Actions #4

Updated by Popy no-lastname-given over 15 years ago

Bug still exists in 4.2.6

It is sad that core source code is breaking my cache :/

Actions #5

Updated by Rik Willems about 15 years ago

Hi Popy,
While searching for this problem I read your issue. Soon I found the following in the typoscript documentation that at least solved my problem:

typolink.useCacheHash = 1

Greets!

Actions #6

Updated by Popy no-lastname-given over 14 years ago

Reminder : Bug still present in latests releases (v4.1, 4.2, 4.3).

I can build new pacth file if needed and if it has a chance to be taken in account.

Actions #7

Updated by Popy no-lastname-given over 14 years ago

Patch included in #17070

Actions #8

Updated by Mathias Schreiber over 14 years ago

still valid?

Actions #9

Updated by Popy no-lastname-given over 14 years ago

Fixed in #17070

Actions #10

Updated by Steffen Kamper over 14 years ago

fixed in #17070

Actions #11

Updated by Popy no-lastname-given about 14 years ago

Bug remain in trunk

Actions #12

Updated by Popy no-lastname-given about 14 years ago

Sorry, wasn't looking the good svn repository...

Actions #13

Updated by Jo Hasenau almost 14 years ago

So is it still valid or can it be closed?

Actions #14

Updated by Ernesto Baschny almost 14 years ago

No further feedback, so I consider it as closed.

Actions #15

Updated by Susanne Moog over 13 years ago

  • Target version deleted (4.5.0)
Actions

Also available in: Atom PDF