Skip to content

Delete wrong check for the no download message#25924

Merged
HLeithner merged 1 commit intojoomla:stagingfrom
zero-24:fixNoDownloadCondition
Aug 19, 2019
Merged

Delete wrong check for the no download message#25924
HLeithner merged 1 commit intojoomla:stagingfrom
zero-24:fixNoDownloadCondition

Conversation

@zero-24
Copy link
Contributor

@zero-24 zero-24 commented Aug 18, 2019

Pull Request for Issue #25827

Summary of Changes

Make sure no download message is displayed in any case when the download URL is not set

Testing Instructions

Expected result

No download message is displayed.

Actual result

PHP Notice:  Undefined property: Joomla\CMS\Updater\Update::$downloadurl in \administrator\components\com_joomlaupdate\views\default\tmpl\default_reinstall.php on line 34
PHP Notice:  Trying to get property '_data' of non-object in \administrator\components\com_joomlaupdate\views\default\tmpl\default_reinstall.php on line 34
PHP Notice:  Undefined property: Joomla\CMS\Updater\Update::$downloadurl in \administrator\components\com_joomlaupdate\views\default\tmpl\default_reinstall.php on line 35
PHP Notice:  Trying to get property '_data' of non-object in \administrator\components\com_joomlaupdate\views\default\tmpl\default_reinstall.php on line 35

Documentation Changes Required

None.

@richard67
Copy link
Member

@zero-24 URL of custom update server missing in description.

@zero-24
Copy link
Contributor Author

zero-24 commented Aug 18, 2019

ha :D Fixed it is: https://www.jah-tz.de/downloads/core/nightlies/next_minor_list.xml

@richard67
Copy link
Member

I have tested this item ✅ successfully on a97466e


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

@richard67
Copy link
Member

Hint for other testers: You can also test with current staging on a cloned GitHub repo, no need do download and install 3.9.12-dev nightly build.

@zero-24
Copy link
Contributor Author

zero-24 commented Aug 18, 2019

Thanks for your test @richard67 and yes staging works too but is not required 👍

@richard67
Copy link
Member

Yes, whatever is easier because available.

@viocassel
Copy link
Contributor

I have tested this item ✅ successfully on a97466e


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

@ghost ghost removed the PR-staging label Aug 19, 2019
@ghost
Copy link

ghost commented Aug 19, 2019

Status "Ready To Commit".

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Aug 19, 2019
@ghost ghost added the PR-staging label Aug 19, 2019
@HLeithner
Copy link
Member

Thank you for bringing some love to the Joomla update system.

@HLeithner HLeithner merged commit 21d65e5 into joomla:staging Aug 19, 2019
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Aug 19, 2019
@HLeithner HLeithner added this to the Joomla! 3.9.12 milestone Aug 19, 2019
@zero-24 zero-24 deleted the fixNoDownloadCondition branch August 20, 2019 00:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants