? ? Pending

User tests: Successful: Unsuccessful:

avatar SniperSister
SniperSister
30 Mar 2018

Summary of Changes

Removed the hardcoded "maxmium, shout all the stuff out there" error-reporting level from the CLI scripts to respect the php.ini value.

Testing Instructions

Add the line brokenCode(); in front of $app = \Joomla\CMS\Factory::getContainer()->get(\Joomla\Console\Application::class); of the /cli/joomla.php. Run via CLI, find a fatal error outputted. Apply patch, run again - system now uses php.ini values for error reporting.

Expected result

php.ini settings for error reporting being used.

Actual result

Forced output of error messages.

Documentation Changes Required

None

avatar SniperSister SniperSister - open - 30 Mar 2018
avatar SniperSister SniperSister - change - 30 Mar 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 30 Mar 2018
Category CLI
avatar brianteeman
brianteeman - comment - 30 Mar 2018

I have tested this item successfully on 4407635

Tested ok

I wonder why it was put there in the first place?


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/20027.

avatar brianteeman brianteeman - test_item - 30 Mar 2018 - Tested successfully
avatar astridx
astridx - comment - 30 Mar 2018

I have tested this item successfully on 4407635

During my first attempts with Joomla CLI I tested this successfully.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/20027.

avatar astridx astridx - test_item - 30 Mar 2018 - Tested successfully
avatar astridx astridx - test_item - 30 Mar 2018 - Tested successfully
avatar Quy Quy - change - 30 Mar 2018
Status Pending Ready to Commit
avatar Quy
Quy - comment - 30 Mar 2018

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/20027.

avatar zero-24 zero-24 - change - 30 Mar 2018
Labels Added: ?
avatar wilsonge wilsonge - close - 31 Mar 2018
avatar wilsonge wilsonge - merge - 31 Mar 2018
avatar wilsonge wilsonge - change - 31 Mar 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-03-31 19:19:42
Closed_By wilsonge
Labels Added: ?

Add a Comment

Login with GitHub to post a comment