J4 Issue ?
avatar SharkyKZ
SharkyKZ
21 Jul 2019

Steps to reproduce the issue

Enable Debug System in Global Configuration.
Enable System - Debug plugin.
In plugin, enable Log Deprecated API.
Inspect the Debug bar.

Expected result

100s of deprecation messages.

Actual result

Only 1 message.

Additional comments

Caused by #20547 (CC @laoneo). At first glimpse this call could be moved from constructor to an event but I'm not sure if this is the correct solution, plus it might miss some messages this way:

$this->setupLogging();

avatar SharkyKZ SharkyKZ - open - 21 Jul 2019
avatar joomla-cms-bot joomla-cms-bot - change - 21 Jul 2019
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 21 Jul 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 21 Jul 2019
Labels Added: J4 Issue
avatar franz-wohlkoenig franz-wohlkoenig - labeled - 21 Jul 2019
avatar SharkyKZ SharkyKZ - change - 21 Jul 2019
Title
[4.0] Deprecation logging in Debug plugin is broken
[4.0] Logging in Debug plugin is broken
avatar SharkyKZ SharkyKZ - edited - 21 Jul 2019
avatar SharkyKZ SharkyKZ - change - 21 Jul 2019
The description was changed
avatar SharkyKZ SharkyKZ - edited - 21 Jul 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 21 Jul 2019
Status New Discussion
avatar laoneo
laoneo - comment - 24 Jul 2019

I will have a look on the debug plugin as there are some other things which need to be changed.

avatar infograf768
infograf768 - comment - 24 Jul 2019

there are some other things which need to be changed.

I could not login this morning after updating a dev test site (which was set to debug) to latest -dev via eclipse and running npm ci.
Joomla\Plugin\System\Debug\Storage\FileStorage; not found.

After adding /use Joomla\Plugin\System\Debug\Storage\FileStorage; in
/plugins/system/debug/Storage/FileStorage.php

I could log in. Thereafter I commented that line and it worked fine again.
Hope it helps.

avatar SharkyKZ
SharkyKZ - comment - 26 Jul 2019

@laoneo Thanks.

@infograf768 You needed to delete the libraries/autoload_psr4.php file. Maybe we should have it regenerated automatically after updating Joomla!

avatar laoneo
laoneo - comment - 11 Sep 2019

Please test #26264.

avatar franz-wohlkoenig franz-wohlkoenig - change - 11 Sep 2019
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2019-09-11 13:24:13
Closed_By franz-wohlkoenig
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 11 Sep 2019

Closed as having Pull Request #26264

avatar franz-wohlkoenig franz-wohlkoenig - close - 11 Sep 2019
avatar infograf768 infograf768 - change - 6 Feb 2020
Status Closed New
Closed_Date 2019-09-11 13:24:13
Closed_By franz-wohlkoenig
avatar infograf768 infograf768 - reopen - 6 Feb 2020
avatar infograf768
infograf768 - comment - 6 Feb 2020

reopened as #26264 did not solve it.

avatar infograf768 infograf768 - change - 6 Feb 2020
Labels Added: ?
avatar infograf768 infograf768 - labeled - 6 Feb 2020
avatar jwaisner jwaisner - change - 17 Feb 2020
Priority Medium Urgent
avatar joomla-cms-bot joomla-cms-bot - change - 17 Feb 2020
Labels Removed: ?
avatar joomla-cms-bot joomla-cms-bot - unlabeled - 17 Feb 2020
avatar jwaisner
jwaisner - comment - 17 Feb 2020

Increasing priority as per guidelines for beta-blockers.


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

avatar jwaisner jwaisner - change - 17 Feb 2020
Labels Added: ?
avatar jwaisner jwaisner - labeled - 17 Feb 2020
avatar jwaisner jwaisner - change - 18 Feb 2020
Status New Confirmed
avatar joomla-cms-bot joomla-cms-bot - change - 18 Feb 2020
Labels Removed: ?
avatar joomla-cms-bot joomla-cms-bot - unlabeled - 18 Feb 2020
avatar jwaisner jwaisner - change - 18 Feb 2020
Labels Added: ?
avatar jwaisner jwaisner - labeled - 18 Feb 2020
avatar wilsonge wilsonge - change - 20 Feb 2020
Labels Added: ?
Removed: ?
avatar wilsonge wilsonge - unlabeled - 20 Feb 2020
avatar wilsonge wilsonge - labeled - 20 Feb 2020
avatar alikon
alikon - comment - 28 Mar 2020

i would not add another issue, but also query explain is broken in the debug plugin as we are now quite converted all SQL to prepared statements
Screenshot from 2020-03-28 09-26-38

avatar Fedik
Fedik - comment - 8 Aug 2020

Please test #30317

@alikon about db "explain", I think it may be better to open a new issue, it more related to DB layer.

avatar alikon
alikon - comment - 8 Aug 2020

it can be better,
or wait ....
before to switch on this new thing maybe someone would had been to think a little bit more , we are losing a too much feature from j3 compared to j4 for what ? .... a more fancy display
your friendly complainer...

avatar Fedik
Fedik - comment - 8 Aug 2020

we are losing a too much feature from j3 compared to j4 for what ? .... a more fancy display

No, it not because fancy display, it because DB layer limitation.
Later I will open new issue with some explanation.

avatar richard67
richard67 - comment - 8 Aug 2020

So what about this issue here? Can it be closed because PR #30317 will solve it?

avatar Fedik
Fedik - comment - 8 Aug 2020

Wait, I will open new issue about database, then we can close this ?

avatar richard67
richard67 - comment - 8 Aug 2020

Ok, thanks. No need to hurry, just ping me when done.

avatar Fedik
Fedik - comment - 8 Aug 2020

there it is #30318

avatar richard67
richard67 - comment - 8 Aug 2020

Closing as having 1 pull request covering the main issue here, and one issue to keep track of the remaining one:

Thanks @Fedik .

avatar richard67 richard67 - change - 8 Aug 2020
Status Confirmed Closed
Closed_Date 0000-00-00 00:00:00 2020-08-08 11:10:32
Closed_By richard67
Labels Added: ?
Removed: ?
avatar richard67 richard67 - close - 8 Aug 2020
avatar alikon
alikon - comment - 9 Aug 2020

please don't get me wrong
i really appreciate your efforts
but we still miss something "Call Stack" for example
j3
Screenshot from 2020-08-09 11-28-39

j4

Screenshot from 2020-08-09 11-27-57

avatar HLeithner
HLeithner - comment - 9 Aug 2020

but we still miss something "Call Stack" for example

You have to activate Call Stack, Explain and Profile in the plugin first

avatar wilsonge wilsonge - change - 18 Dec 2020
Labels Removed: ?
avatar wilsonge wilsonge - unlabeled - 18 Dec 2020

Add a Comment

Login with GitHub to post a comment