Project

General

Profile

Actions

Bug #87570

closed

Path to requirejs-loader.js not correct

Added by Claudia Nölker almost 6 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Frontend
Start date:
2019-01-29
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
9
PHP Version:
7.2
Tags:
Complexity:
Is Regression:
Yes
Sprint Focus:

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

01.png (498 KB) 01.png David Bascom, 2019-02-15 21:58
frontend.jpg (123 KB) frontend.jpg Martin Weymayer, 2019-02-28 08:02
Actions #1

Updated by Claudia Nölker almost 6 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>

Actions #2

Updated by David Bascom almost 6 years ago

  • File 01.png 01.png added
  • TYPO3 Version changed from 8 to 9

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

Actions #3

Updated by Oliver Hader over 5 years ago

  • Target version set to Candidate for patchlevel
  • Is Regression set to Yes
Actions #4

Updated by Oliver Hader over 5 years ago

  • Status changed from New to Accepted
Actions #5

Updated by Gerrit Code Review over 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

Actions #6

Updated by Oliver Hader over 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!

Actions #7

Updated by Martin Weymayer over 5 years ago

patch seems not working for 9.5.4 in combination with frontend editing

Actions #8

Updated by Oliver Hader over 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?

Actions #9

Updated by Martin Weymayer over 5 years ago

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

Actions #10

Updated by Markus Klein over 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)

Actions #11

Updated by Gerrit Code Review over 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

Actions #12

Updated by Oliver Hader over 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.

Actions #13

Updated by Gerrit Code Review over 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

Actions #14

Updated by Gerrit Code Review over 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

Actions #15

Updated by Oliver Hader over 5 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #16

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF