J4 Issue ?
avatar astridx
astridx
5 Sep 2018

Steps to reproduce the issue

  1. Set a custom url for joomla update. For testing you can use custom url https://www.jah-tz.de/downloads/core/gsoc18_override_management/gsoc18_update_list.xml
    joomla update options test administration 1

  2. Start a joomlaupdate via https://www.jah-tz.de/downloads/core/gsoc18_override_management/gsoc18_update_list.xml
    joomla update test administration 2

Expected result

All messages are correct.

Actual result

In control pane I see the messages are not correct.
control panel test administration 3

System information (as much as possible)

The current 4.0-dev branch on ubuntu 18.04

Additional comments

I noticed this while I tested an other PR: #21851 (comment)

avatar astridx astridx - open - 5 Sep 2018
avatar joomla-cms-bot joomla-cms-bot - change - 5 Sep 2018
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 5 Sep 2018
avatar brianteeman
brianteeman - comment - 5 Sep 2018

Another issue with the joomla-alert custom element? :(

avatar brianteeman brianteeman - change - 5 Sep 2018
Labels Added: ?
avatar brianteeman brianteeman - labeled - 5 Sep 2018
avatar brianteeman brianteeman - change - 5 Sep 2018
Labels Added: J4 Issue
Removed: ?
avatar brianteeman brianteeman - labeled - 5 Sep 2018
avatar brianteeman brianteeman - unlabeled - 5 Sep 2018
avatar franz-wohlkoenig franz-wohlkoenig - change - 5 Sep 2018
Status New Discussion
avatar franz-wohlkoenig franz-wohlkoenig - change - 5 Sep 2018
Category com_joomlaupdate Language & Strings
avatar infograf768
infograf768 - comment - 6 Sep 2018

Looks like we have an error in the js. Not sure if it is the cause
${Joomla.JText._('PLG_QUICKICON_JOOMLAUPDATE_UPDATEFOUND_BUTOON')}</button>
instead of
PLG_QUICKICON_JOOMLAUPDATE_UPDATEFOUND_BUTTON

avatar brianteeman
brianteeman - comment - 6 Sep 2018

@infograf768 good spot. Will you do the PR for that please

avatar infograf768
infograf768 - comment - 6 Sep 2018

I tested.
Although it is necessary to correct this typo, it does not solve the issue in Control Panel.

The problem is with the js code. After some corrections I can get.
screen shot 2018-09-06 at 12 00 02

Which is not perfect yet (the 0) but may depend from the package offered above as updating with it does not really change the J version.

Will try to make a PR this afternoon.

avatar infograf768 infograf768 - change - 6 Sep 2018
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2018-09-06 10:28:04
Closed_By infograf768
avatar infograf768
infograf768 - comment - 6 Sep 2018

Closing as we have a patch at least for the language string. ;)
#22023

avatar infograf768 infograf768 - close - 6 Sep 2018

Add a Comment

Login with GitHub to post a comment