?
avatar mbabker
mbabker
9 Feb 2019

Steps to reproduce the issue

The Joomla\CMS\Application\ApplicationHelper::getClientInfo() method identifies each of the core clients (applications), however fails to recognize the console application. The console application should be included in this list.

As far as API design goes, there is also an issue with how applications are registered to this method (which becomes obvious when considering the console and proposed API applications). It is a static list that requires one to manually call Joomla\CMS\Application\ApplicationHelper::addClientInfo() to register a new application and core contributors to be aware of and actually update as necessary. The logic in the getClientInfo() method should ideally be refactored to emit an event at the first call to the method to get all of the information from known applications dynamically (how this would work is to be determined). This would then ensure that clients are automatically registered at the first request for client information.

avatar mbabker mbabker - open - 9 Feb 2019
avatar joomla-cms-bot joomla-cms-bot - change - 9 Feb 2019
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 9 Feb 2019
avatar mbabker mbabker - change - 19 Feb 2019
Status New Closed
Closed_Date 0000-00-00 00:00:00 2019-02-19 01:59:06
Closed_By mbabker
avatar mbabker mbabker - close - 19 Feb 2019

Add a Comment

Login with GitHub to post a comment