J4 Issue ?
avatar skurvish
skurvish
6 Nov 2019

Steps to reproduce the issue

From a command prompt run finder_indexer.php

Expected result

Not sure but it fails

Actual result

Fatal error: Class FinderCli contains 2 abstract methods and must therefore be declared abstract or implement the remaining methods (Joomla\CMS\Application\CMSApplicationInterface::getName, Joomla\CMS\Application\CMSApplicationInterface::getMenu) in /var/www/html/cli/finder_indexer.php on line 59

System information (as much as possible)

Alpha 12

Additional comments

I am trying to get a CliApplication and WebApplication running on Alpha 12 but unable to. Got the same error as finder_indexer.

avatar skurvish skurvish - open - 6 Nov 2019
avatar joomla-cms-bot joomla-cms-bot - labeled - 6 Nov 2019
avatar skurvish
skurvish - comment - 6 Nov 2019

Also, php 7.2.19 on Ubuntu 18.04.1


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

avatar infograf768 infograf768 - change - 6 Nov 2019
Title
cli/finder_indexer.php fails on Alpha 12
[4.0] cli/finder_indexer.php fails on Alpha 12
avatar infograf768 infograf768 - edited - 6 Nov 2019
avatar infograf768 infograf768 - change - 6 Nov 2019
Labels Added: J4 Issue
avatar infograf768 infograf768 - labeled - 6 Nov 2019
avatar mbabker
mbabker - comment - 6 Nov 2019

That file still needs to be ported to use the CLI Console application. Problem is, there still isn't a good way for components to register commands since Joomla lives in a world where everything is lazy loaded (this is one of those cases where the lazy loading hurts more than helps).

avatar skurvish
skurvish - comment - 6 Nov 2019

Fair enough. I don't actually need console support itself, just want to get both a cli and web application ported over from J3.x They both run fine in J3.9 but not in 4.0. Is there another example I can use?


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

avatar skurvish
skurvish - comment - 6 Nov 2019

In my cli app I am able to get it to run past this error by providing 2 stub functions, one for getName and one for getMenu. I would suspect however that I should be able to startup a cli session without these.

I also note that if I use a constructor in my cli class the parameters from the class call are never passed through to my class or to the parent CliApplication constructor.


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

avatar jwaisner jwaisner - change - 9 Feb 2020
Priority Urgent Medium
Status New Information Required
avatar jwaisner
jwaisner - comment - 9 Feb 2020

@skurvish is this still happen in the latest build?


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

avatar skurvish
skurvish - comment - 10 Feb 2020

I haven't tried it again on J4. Not sure when I can do so as I am heading off on a trip for the next 5 weeks.

avatar skurvish
skurvish - comment - 22 Mar 2020

This issue is still happening under the current J4 git. Two functions missing, Joomla\CMS\Application\CMSApplicationInterface::getInput, Joomla\CMS\Application\CMSApplicationInterface::getName

This is very easy to test, just run /cli/finder_indexer.php from a browser in the local web, or call it via php on a command line.

avatar richard67
richard67 - comment - 29 Mar 2020

@skurvish Please test #28506 .

avatar richard67
richard67 - comment - 29 Mar 2020

Cosing as having Pull Request (PR) #28506 which solves this issue.

avatar richard67 richard67 - change - 29 Mar 2020
Status Information Required Closed
Closed_Date 0000-00-00 00:00:00 2020-03-29 17:07:47
Closed_By richard67
avatar richard67 richard67 - close - 29 Mar 2020

Add a Comment

Login with GitHub to post a comment