J4 Issue ?
avatar hgh-esn
hgh-esn
23 Jun 2019

Steps to reproduce the issue

Install "German Language Pack 3.9.6.1

Expected result

While there is no <changelogurl>-tag in the xml-file, for me, the Changelog-Button hasn't to appear. Instead a N/A, just like with JCE?

Actual result

The Changelog-Button appears and the modal shows an empty value.
2019-06-23_16h57_34

System information (as much as possible)

J4-Alpha-10-dev

Additional comments

avatar hgh-esn hgh-esn - open - 23 Jun 2019
avatar joomla-cms-bot joomla-cms-bot - change - 23 Jun 2019
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 23 Jun 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 23 Jun 2019
Labels Added: J4 Issue
avatar franz-wohlkoenig franz-wohlkoenig - labeled - 23 Jun 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 23 Jun 2019
Status New Discussion
avatar franz-wohlkoenig franz-wohlkoenig - edited - 23 Jun 2019
avatar infograf768
infograf768 - comment - 24 Jun 2019

Confirmed with 3.9.8.1 French pack

avatar brianteeman
brianteeman - comment - 24 Jun 2019

The code is only checking if there is a changelog url - it doesnt check if there is any content at that url

avatar roland-d
roland-d - comment - 24 Jun 2019

I have noticed this as well. There is a problem in the code somewhere as it executes the wrong query. I haven't been able to put my finger on it yet.

avatar infograf768
infograf768 - comment - 24 Jun 2019

The code is only checking if there is a changelog url - it doesnt check if there is any content at that url

The code is wrong imho
The button should not display if no changelog url.

I tested
<?php if ($item->changelogurl != null) : ?>
instead of
<?php if ($item->changelogurl !== null) : ?>
and it will display N/A instead of the button on which one can click and get the empty modal.

Screen Shot 2019-06-24 at 11 02 39

avatar infograf768
infograf768 - comment - 24 Jun 2019

or more simply
<?php if ($item->changelogurl !== '') : ?>

here works ok when there is a changelog url

avatar infograf768
infograf768 - comment - 24 Jun 2019

Please test #25311
Closing as we have a patch.

avatar infograf768 infograf768 - close - 24 Jun 2019
avatar infograf768 infograf768 - change - 24 Jun 2019
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2019-06-24 10:13:17
Closed_By infograf768

Add a Comment

Login with GitHub to post a comment