[TYPO3-core] RFC: #11513: cache_hash table could not be filled because information field (ident) was too short

Michael Stucki michael at typo3.org
Wed Aug 12 11:17:40 CEST 2009


Hi Ingo,

I have fixed the issue now. Since you mentioned that example of "TMPL
CONDITIONS - ALL" again, I remembered that there was a change about this
some time ago.

Well surprise surprise, it was me who fixes this "typo" in 2008 during a
code cleanup.

However, I also wonder why the error occurs in your setup but not in
mine. Here I just get a warning issued by MySQL (which TYPO3 does not
care about at all):

| Warning: #1265 Data truncated for column 'ident' at row 1

So on my end, the bug did never have any consequences. Could this be
depending on MySQL "strict" mode? I have it turned off here.

So what I did was:
- Update the DB field size to 32 bytes
- Cut the string "TMPL CONDITIONS - ALL" again so it fits into the field
  even if the DB was not updated. (and adding a comment so the typo
  won't get "fixed" again)

Thank you very much for finding that issue, it was a really good catch!

* Committed to TYPO3_4-2 (rev. 5778 / 5779)

Regards, Michael

Ingo Schmitt schrieb:
> Hi Michael,
> 
> fine. +1 from me.
> 
> 
> I still want to understand, why "TMPL CONDITIONS - ALL" is stored in the
> dadabase, since inside the CORE there is no get method for this identifier.
> 
> Regards
> 
> Ingo
>> Hi Ingo,
>>
>> this looks pretty simple. Attached is the patch for the change,
>> extending the field size to 32 bytes.
>>
>> There only thing that bugs me a little bit is that this will cause a
>> change to the DB layout during a patch-level upgrade. However, since the
>> update is not explicitely necessary, I would say this is acceptable.
>>
>> - michael
>>
>> Ingo Schmitt schrieb:
>>> Bugtracker is availiabe, so here complete:
>>>
>>>
>>> this is an SVN patch request.
>>>
>>> Type: Bugfix
>>>
>>> Bugtracker references:
>>> 11513
>>>
>>> Solution:
>>> Change the Table-Definition to 25 chars
>>>
>>> See patch in earlier post or bugtracker..
>>>
>>>
>>> Thanks!
>>>
>>> ingo
>>>
>>>
>>> Mit freundlichen Gruessen
>>
>>
> 
> 
> 
> Mit freundlichen Gruessen


-- 
Use a newsreader! Check out
http://typo3.org/community/mailing-lists/use-a-news-reader/


More information about the TYPO3-team-core mailing list