[bug] admin.js file loaded from site side #976

New Issue

Closed

opened 2023-02-15 18:53:48 +00:00 by jcodewalker · 1 comment

jcodewalker commented

2023-02-15 18:53:48 +00:00

Steps to reproduce the issue

add javascript to component

Expected result

admin.js loaded from "/administrator/.../assets/js/admin.js

Actual result

admin.js loaded from "/components/.../assets/js/admin.js

System information (as much as possible)

OS Name & Version:
MySql Version:
Apache Version:
PHP Version: 7.4
Joomla Version: 3.10.11
JCB Version: 3.1.13
Browser:

Additional comments

Same as admin.css

### Steps to reproduce the issue add javascript to component ### Expected result admin.js loaded from "/administrator/.../assets/js/admin.js ### Actual result admin.js loaded from "/components/.../assets/js/admin.js ### System information (as much as possible) - OS Name & Version: - MySql Version: - Apache Version: - PHP Version: 7.4 - Joomla Version: 3.10.11 - JCB Version: 3.1.13 - Browser: ### Additional comments Same as admin.css

Llewellyn commented

2023-02-15 20:03:23 +00:00

Owner

Please note that the script is referenced as follows:

<script src="components/com_demo/assets/js/admin.js"></script>

It is not referenced as:

<script src="/components/com_demo/assets/js/admin.js"></script>

As you can see from the attached image, this is a relative path and does successfully load the admin area JavaScript file.

While we could make it a root-relative link, using a relative path to the application entry-point is more appropriate, especially as we move towards potentially implementing a dynamic admin path.

If you have any further questions or concerns, please let me know and I will be happy to address them.

Please note that the script is referenced as follows: ``` <script src="components/com_demo/assets/js/admin.js"></script> ``` It is not referenced as: ``` <script src="/components/com_demo/assets/js/admin.js"></script> ``` ![image](/attachments/416d00a5-d797-4521-97b7-a31b6bbe2581) As you can see from the attached image, this is a relative path and does successfully load the admin area JavaScript file. While we could make it a root-relative link, using a relative path to the application entry-point is more appropriate, especially as we move towards potentially implementing a dynamic admin path. If you have any further questions or concerns, please let me know and I will be happy to address them.

image.png

33 KiB