Bug #87570
closed
Path to requirejs-loader.js not correct
100%
Description
Hello,
after updating to 8.7.24, the Frontend-Editing does not work anymore because the path to the requirejs-loader.js is wrong.
HTML extract:
<script src="typo3/sysext/core/Resources/Public/JavaScript/Contrib/require.js?1548170755" type="text/javascript"></script>
<script src="*/usr/www/users/XXXXX*/typo3/sysext/core/Resources/Public/JavaScript/requirejs-loader.js" type="text/javascript"></script>
<script src="typo3/sysext/core/Resources/Public/JavaScript/Contrib/jquery/jquery-3.2.1.min.js?1548170755" type="text/javascript"></script>
The XXXXX is only to mask the customers name.
I tried to correct the path in the file PageRenderer.php . Afterwards, the file is included correctly, but I get warnings in the backend:
Core: Error handler (FE): PHP Warning: call_user_func_array() expects parameter 1 to be a valid callback, class 'TYPO3\CMS\Core\Controller\RequireJsController' does not have a method 'retrievePath' in /usr/www/users/XXXXX/typo3_src-8.7.24/typo3/sysext/core/Classes/Http/Dispatcher.php line 41
Please fix this.
Claudia
Files
Updated by Claudia Nölker about 5 years ago
The stars in the URL are only because I wanted to highlight that part. URL is:
<script src="/usr/www/users/XXXXX/typo3/sysext/core/Resources/Public/JavaScript/requirejs-loader.js" type="text/javascript"></script>
Updated by David Bascom about 5 years ago
Same problem here. Using Typo3 9.5.4 and frontend_editing 1.4.6.
Tested with a fresh installation and only the official Introduction Package installed.
The script-tag gets inserted with an absolute server path of the file.
Correcting it manually makes a call to:
/index.php?eID=requirejs&name=%2Ftypo3%2Fsysext%2Fbackend%2FResources%2FPublic%2FJavaScript%2Fbackend.js
which results in the error attached.
(1/1) ReflectionException Method TYPO3\CMS\Core\Controller\RequireJsController::retrievePath() does not exist
Updated by Oliver Hader about 5 years ago
- Target version set to Candidate for patchlevel
- Is Regression set to Yes
Updated by Gerrit Code Review about 5 years ago
- Status changed from Accepted 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/c/Packages/TYPO3.CMS/+/59810
Updated by Oliver Hader about 5 years ago
- Category set to Frontend
Please test referenced change on review.typo3.org and report back whether that solves the problems you mentioned. Thanks in advance!
Updated by Martin Weymayer about 5 years ago
patch seems not working for 9.5.4 in combination with frontend editing
Updated by Oliver Hader about 5 years ago
Martin Weymayer wrote:
patch seems not working for 9.5.4 in combination with frontend editing
Cleared all caches? Which error messages are shown?
Updated by Martin Weymayer about 5 years ago
- File frontend.jpg frontend.jpg added
yes, cleared all caches. spinning image is shown and does not dissapear. following error in browser console:
ReferenceError: TYPO3 is not defined[Weitere Informationen] Icons.js:13:1794
Error: Load timeout for modules: /typo3/sysext/backend/Resources/Public/JavaScript/backend.js
http://requirejs.org/docs/errors.html#timeout
Updated by Markus Klein about 5 years ago
@martin please vote and comment in the code review https://review.typo3.org/c/Packages/TYPO3.CMS/+/59810
Reviewers will mostly not see your comments here. Thanks.
(I posted a -1 on behalf of you for the time being)
Updated by Gerrit Code Review about 5 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/c/Packages/TYPO3.CMS/+/59810
Updated by Oliver Hader about 5 years ago
Frontend Editing is using some absolute "module" (=path) which was not covered by the new require.js loader:
https://github.com/FriendsOfTYPO3/frontend_editing/blob/master/Resources/Public/JavaScript/GUI.js#L18
Please test again, in my scenario at least frontend editing JavaScript now are loading in frontend context.
Updated by Gerrit Code Review about 5 years ago
Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/59826
Updated by Gerrit Code Review about 5 years ago
Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/59827
Updated by Oliver Hader about 5 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 3d556b40b3a0134aaa36465708f651fec2010bf5.
Updated by Benni Mack almost 5 years ago
- Status changed from Resolved to Closed