?
avatar f-hamel
f-hamel
19 Feb 2019

Steps to reproduce the issue

Just execute /cli/joomla.php on command line

Expected result

Output of the working cli

Actual result

Following php error:

contains 1 abstract method and must therefore be declared abstract or implement the remaining methods

System information (as much as possible)

PHP 7.1

Additional comments

Maybe this caused by following abstract functions in libraries/src/Application/EventAware.php:

/**
         * Get the event dispatcher.
         *
         * @return  DispatcherInterface
         *
         * @since   4.0.0
         * @throws  \UnexpectedValueException May be thrown if the dispatcher has not been set.
         */
        abstract public function getDispatcher();

        /**
         * Get the logger.
         *
         * @return  LoggerInterface
         *
         * @since   4.0.0
         */
        abstract public function getLogger();

I saw we are user this in /libraries/src/Application/ConsoleApplication.php:

use DispatcherAwareTrait, EventAware, IdentityAware, ContainerAwareTrait, ExtensionManagerTrait;

on line 35.

avatar f-hamel f-hamel - open - 19 Feb 2019
avatar joomla-cms-bot joomla-cms-bot - change - 19 Feb 2019
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 19 Feb 2019
avatar mbabker
mbabker - comment - 19 Feb 2019

This would be fixed by #23847 but apparently either I'm on a pull request blacklist or changes that aren't related to the template or web applications causing errors to be thrown in everyone's faces aren't getting any attention.

avatar Quy Quy - change - 20 Feb 2019
Status New Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-02-20 15:06:46
Closed_By Quy
avatar joomla-cms-bot joomla-cms-bot - change - 20 Feb 2019
Status Fixed in Code Base Closed
Closed_By Quy joomla-cms-bot
avatar joomla-cms-bot joomla-cms-bot - close - 20 Feb 2019
avatar joomla-cms-bot
joomla-cms-bot - comment - 20 Feb 2019

Set to "closed" on behalf of @Quy by The JTracker Application at issues.joomla.org/joomla-cms/23941

avatar f-hamel
f-hamel - comment - 20 Feb 2019

/reopen

avatar f-hamel
f-hamel - comment - 20 Feb 2019

Still got the same error.

avatar mbabker
mbabker - comment - 20 Feb 2019

You need to be using the current git branch. Nightly builds are built nightly, not on every commit.

avatar f-hamel
f-hamel - comment - 20 Feb 2019

I merged the current 4.0-dev branche in to my 4.0 and update my command classes to use the new structure.

avatar mbabker
mbabker - comment - 20 Feb 2019

You did a fresh composer install as well to bring in the current dependency tree?

avatar f-hamel
f-hamel - comment - 20 Feb 2019

thanks @mbabker. Miss the composer install

Add a Comment

Login with GitHub to post a comment