Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5.2] Display the correct changelog when updating #44652

Open
wants to merge 2 commits into
base: 5.2-dev
Choose a base branch
from

Conversation

Hackwar
Copy link
Member

@Hackwar Hackwar commented Dec 21, 2024

Pull Request for Issue #43505 .

Summary of Changes

When an extension provides a changelog URL in its update XML, a button is displayed to show the changelog, but the URL only loads the changelog from the current extension, not from the update site.

Testing Instructions

  1. Install an old version of a third party extension, for example a language pack.
  2. Search for an update for that extension.
  3. We now try to emulate a changelog available for that extension: Go to the database and in #__updates look for the entry for your extension update. Edit the changelogurl column and insert a URL to a changelog.xml in your sites root, for example http://localhost/changelog.xml
  4. Create a changelog.xml with the content below. (Adapt the content to the extension you've been using for this)
  5. Go to the "update extensions" view and see the button for the changelog for your extension.

Content for the changelog.xml

<changelogs>
	<changelog>
		<element>pkg_ru-RU</element>
		<type>package</type>
		<version>5.2.2.1</version>
		<security>
			<item>Item A</item>
			<item><![CDATA[<h2>You MUST replace this file</h2>]]></item>
		</security>
		<fix>
			<item>Item A</item>
			<item>Item b</item>
		</fix>
	</changelog>
</changelogs>

Actual result BEFORE applying this Pull Request

A modal opens and is empty.

Expected result AFTER applying this Pull Request

A modal opens and shows a few entries

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

@alikon
Copy link
Contributor

alikon commented Dec 21, 2024

i'm unable to replicate the issue using the current 5.2-dev

image

image

same as #44626 could not replicate the issue

@Hackwar
Copy link
Member Author

Hackwar commented Dec 21, 2024

Did you check, that the URL in the extension table is not set? The problem is, that the code uses the changelog URL of the extension table instead of the updatesite.

@alikon
Copy link
Contributor

alikon commented Dec 21, 2024

ahhh ok removing the changelog url from the extension table i'm able to replicate

@alikon
Copy link
Contributor

alikon commented Dec 21, 2024

I have tested this item ✅ successfully on b9f5436


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

@Hackwar
Copy link
Member Author

Hackwar commented Dec 21, 2024

thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants