joomla-cms icon indicating copy to clipboard operation
joomla-cms copied to clipboard

[4.0] Fix package installer not setting new changelogurl

Open Ruud68 opened this issue 4 years ago • 8 comments

Pull Request for Issue # .

Summary of Changes

When you have a package installed that didn't have a changelogurl set in the manifest file and you upgrade that package with a version that does have a changelogurl set in the manifest, this changelogurl is not stored in that database. currently changelogurl is only set (and used) on newly installed packages. This PR fixes this bug and allows for already installed package have a changelogurl when the update package has one

Testing Instructions

install a package that does NOT have a changelogurl in the manifest file upgrade that package with a version that DOES have a changelog URL in the manifest file

Actual result BEFORE applying this Pull Request

Changelogurl not added to the table and thus not used in extension manager

Expected result AFTER applying this Pull Request

Changelogurl correctly added to the extensions table and used in extension manager

Documentation Changes Required

Ruud68 avatar Oct 11 '21 08:10 Ruud68

I have tested this item :white_check_mark: successfully on bfd9b0c9f1c6e7ff7a7e99c370f8dce7e22dc7a5


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

joomdonation avatar Oct 16 '21 12:10 joomdonation

@Ruud68 Could you please check also FileAdapter? I don't have experience with this feature but looks like we have same issue with that adapter. Would be great if it could be fixed in this PR, too.

joomdonation avatar Oct 16 '21 12:10 joomdonation

Hi @joomdonation yes fileadapter and also templateadapter have same bug. No experience with these two and now way to test them, that is why i left them out of the package adapter change.

will update pr to also (hopefully) fix them, but as said that will be a PR that i cannot test. BTW I have very low hopes for this PR as probably nobody is able to test it, let alone test it for package, template and file updates :(

Ruud68 avatar Oct 16 '21 12:10 Ruud68

@Ruud68 Thanks. Actually, TemplateAdapter is OK because it is handled in this line of code https://github.com/joomla/joomla-cms/blob/4.0-dev/libraries/src/Installer/Adapter/TemplateAdapter.php#L592 . So please revert the change which you made to that adapter. We can even remove this line of code https://github.com/joomla/joomla-cms/blob/4.0-dev/libraries/src/Installer/Adapter/TemplateAdapter.php#L583

joomdonation avatar Oct 16 '21 13:10 joomdonation

update to template adapter changed

Ruud68 avatar Oct 18 '21 08:10 Ruud68

Are we good to go for the next release? if not what needs to be done to get this PR in there?

Ruud68 avatar Oct 26 '21 06:10 Ruud68

This pull request has automatically rebased to 4.2-dev.

HLeithner avatar Jun 27 '22 13:06 HLeithner

This pull requests has been automatically converted to the PSR-12 coding standard.

joomla-bot avatar Jun 27 '22 21:06 joomla-bot

I have tested this item :white_check_mark: successfully on 054ae224b8aca74997f3a4c6a87e4a6af36082dc


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

laoneo avatar Oct 22 '22 16:10 laoneo