[#15843] - Allow Joomla to be extended with JDEBUG defined by developers
- Fixed in Code Base
- 13 Nov 2018
- Medium
- Build: staging
- # 15843
- Diff
- PhilETaylor:joomla-cms-15827
User tests: Successful: Unsuccessful:
Pull Request for Issue #15827 .
Summary of Changes
Allow Joomla to be extended with JDEBUG defined by developers when in a sub folder without FOF moaning and fatal erroring.
Testing Instructions
Apply PR and check Joomla operates as normal.
Create api.php in Joomla tmp folder with content
<?php
define('_JEXEC', 1);
define('JPATH_BASE', realpath('../'));
ini_set('display_errors', '1');
error_reporting(E_ALL);
define('JDEBUG', 1);
include_once JPATH_BASE . '/includes/defines.php';
include_once JPATH_BASE . '/includes/framework.php';
include_once JPATH_BASE . '/libraries/joomla/factory.php';Expected result
No errors on running
Actual result
PHP Fatal error: Uncaught Error: Call to a member function logAddLogger() on null in /Users/phil/Sites/joomla-cms/libraries/fof/include.php:26
Stack trace:
#0 /Users/phil/Sites/joomla-cms/libraries/cms.php(45): include_once()
#1 /Users/phil/Sites/joomla-cms/includes/framework.php(23): require_once('/Users/phil/Sit...')
#2 /Users/phil/Sites/joomla-cms/tmp/api.php(8): include_once('/Users/phil/Sit...')
#3 {main}
thrown in /Users/phil/Sites/joomla-cms/libraries/fof/include.php on line 26
Documentation Changes Required
None
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
@PhilETaylor i have Debug enabled + api.php in tmp-Folder but didn't get PHP Fatal error:.
Is your error_reporting in PHP disabled or set to E_ALL ? is display_errors=on in your php.ini?
even with the latest staging I still get this when running the example code:
Fatal error: Uncaught Error: Call to a member function logAddLogger() on null in /Users/phil/Sites/joomla-cms/libraries/fof/include.php:26 Stack trace: #0 /Users/phil/Sites/joomla-cms/libraries/cms.php(45): include_once() #1 /Users/phil/Sites/joomla-cms/includes/framework.php(23): require_once('/Users/phil/Sit...') #2 /Users/phil/Sites/joomla-cms/tmp/api.php(8): include_once('/Users/phil/Sit...') #3 {main} thrown in /Users/phil/Sites/joomla-cms/libraries/fof/include.php on line 26
@PhilETaylor thanks for Answer. As i don't know how to set this Values this Issue is about my Knowledge.
| Labels |
Added:
?
|
||
@franz-wohlkoenig this just needs another test to get to RTC if you have time please :) (@or @SharkyKZ as you seem active presently)
As i don't know how to set this Values this Issue is about my Knowledge.
The testing instructions are in the PR text, under "testing instructions"....
i meant comments starting #15843 (comment)
What Joomla version are you testing against?
Actually i'm testing not as the "how to set this Values this Issue is about my Knowledge".
I have tested this item
Changes applied manually.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15843.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15843.
mbabker
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-11-13 00:01:16 |
| Closed_By | ⇒ | mbabker | |
| Labels |
Added:
?
|
||
I have tested this item✅ successfully on ec7f86a
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15843.