User tests: Successful: Unsuccessful:
Makes the check if a configuration exists in the include files easier to read as it returns early when a condition doesn't exist.
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
Category | ⇒ | Administration |
Status | New | ⇒ | Pending |
Labels |
Added:
PR-5.0-dev
|
I have tested this item
Remove configuration.php - done
Go to the root page of the joomla installation - done
Error message appears: The requested URL was not found on this server.
Can you post a screenshot of the error message and the url you have opened and the url you get redirected?
Can you try with the latest fix?
I guess this is done on purpose as sites can include the index.php in an own script and then define that variable. If you want to change this, then please in another pr, but I'm not sure about the consequences, perhaps @wilsonge can shed some light into this.
I can't shed any proper light. I mean it's always been intended that you can override defines.php rather than index.php at the top level of each app (https://github.com/joomla/joomla-cms/blob/4.3-dev/includes/app.php#L16-L23) so I assume this allows setting JDEBUG there. No clue why you'd choose to set it there. But making it consistent also makes sense to me.
API should be included in this PR as it's based on administrator https://github.com/joomla/joomla-cms/blob/4.3-dev/api/includes/framework.php
Can you try with the latest fix?
Works, expected result.
Can you mark it as a successful test?
Should then API get the same logic as admin and site got in #40509? If yes, can you comment on the original pr? I will then update this one when the rest is done.
Ahh we error out instead of redirecting - one day I'll read code correctly the first time. Yeah just ignore me and I'll get back into my box
Title |
|
Can you mark it as a successful test?
Why these two files? Only file 1 seems to be addressed in this test. I access the root page of the joomla installation through the administrator (administrator/index.php?option=com_joomlaupdate). Or am I not testing it correctly?
C:\xampp\htdocs\bugtesting5\joomla\administrator\includes\framework.php (file 1)
C:\xampp\htdocs\bugtesting5\joomla\includes\framework.php (file 2)
YOu can also test the second file when you go to the administrator subfolder and do the same. You can do it either from the joomla root or the administrator subfolder.
I have tested this item ✅ successfully on 76d6a17
Status | Pending | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-08-21 06:35:56 |
Closed_By | ⇒ | HLeithner |
thanks
Proposal: If you change the following code, both files are identical except for line 4 (administator / site).
Change this code
joomla-cms/administrator/includes/framework.php
Line 81 in e6a7e95
to that of
joomla-cms/includes/framework.php
Line 81 in e6a7e95