Bug #21396
closedsys_refindex can't handle linkhandler links correctly
100%
Description
Problem:
The "t3lib_softrefproc" handle the linkhandler values as type of "_STRING" and that is wrong.
The linkhandler should provide the logic to extract the reference to their records if there any.
Solution:
The "t3lib_softrefproc" needs a detection for linkhandler references so that they can handled by the linkhandler itself.
Therefore i suggest two hooks and a new small method to detect linkhandler values.
Note:
If there is no way to keep the sys_refindex up to date you can't clean your environment using the lowlevel_cleaner.
(issue imported from #M12389)
Files
Updated by Oliver Hader about 15 years ago
Hi Mick,
could you please provide a short information on how to test this issue? I don't think that it would be easy to write unit tests in this scenario... Thanks in advance!
Updated by Michael Klapper about 15 years ago
How to reprocue it:
===============
- Use current linkhandler version from TER http://typo3.org/extensions/repository/view/linkhandler/current/
1) Go to your tt_content element with the uid XY and create a link to an record using the linkhandler extension (e.q. record:tt_news:10)
2) Open you bash and run "time php -d memory_limit=512M typo3/cli_dispatch.phpsh lowlevel_refindex -e"
3) Open the info dialog on the tt_content element with the uid XY where you created the linkhandler link. There you can see the link entry as type "_STRING", this means that no valid relation from the record to the tt_news record exist in the table sys_refindex.
After applying the patch or just installing the latest linkhandler from trunk https://svn.typo3.org/TYPO3v4/Extensions/linkhandler/trunk/ (the linkhandler provide a XCLASS to add the hooks).
1) Run the "lowlevel_refindex " again and refresh the info dialog. Now you can see that there a valid reference to the tt_news record with the uid 10 exists.
2) Go to the tt_news record with the uid 10 open the info dialog of that record and you should see the back reference to the tt_news record XY who linked te the this record.
=======================================
If you need more details just add me a note.
Thanks,
Mick
Updated by Alexander Opitz over 11 years ago
- Status changed from New to Needs Feedback
- Target version deleted (
0)
The issue is very old, does this issue exists in newer versions of TYPO3 CMS (4.5 or 6.1)?
Updated by Ralf Hettinger over 11 years ago
Updated by Alexander Opitz over 11 years ago
Which TYPO3 CMS version did you test?
Updated by Alexander Opitz about 11 years ago
- Is Regression set to No
Can you tell me, with which version do you test the issue?
Updated by Ralf Hettinger about 11 years ago
There is currently no proper test setup, since the linkhandler extension is afaik not compatble with current versions. This extension is hopefully meant to be integrated into the Core (#46140). Before this isn't the case, it is only testable with 4.5 using the linkhandler from Ext Manager.
So I'd suggest to leave this one here open until #46140 is solved to not forget compatiblity with the Reference Index.
Updated by Alexander Opitz about 11 years ago
- Category set to Backend API
- Status changed from Needs Feedback to New
Updated by Gerrit Code Review almost 11 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27746
Updated by Alexander Stehlik almost 11 years ago
Example implementation for testing can be found here:
Updated by Gerrit Code Review over 10 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27746
Updated by Gerrit Code Review over 10 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27746
Updated by Gerrit Code Review over 10 years ago
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/31423
Updated by Alexander Stehlik over 10 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 3a8ffa20c9d5de197ba4840b94bb809cd6877035.