Skip to content

[5.0] Fix changelog parsing#41675

Merged
HLeithner merged 1 commit intojoomla:5.0-devfrom
Fedik:changelog-parse-fix
Sep 12, 2023
Merged

[5.0] Fix changelog parsing#41675
HLeithner merged 1 commit intojoomla:5.0-devfrom
Fedik:changelog-parse-fix

Conversation

@Fedik
Copy link
Member

@Fedik Fedik commented Sep 8, 2023

Pull Request for Issue #41635 and #41636.

Summary of Changes

Fix changelog parsing, that tries to create a non existing properties.

Testing Instructions

Please follow #41635

Actual result BEFORE applying this Pull Request

A deprecation warning

Expected result AFTER applying this Pull Request

No deprecation warning

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

Reference:

@conseilgouz
Copy link
Contributor

I have tested this item ✅ successfully on baca35d

Also tested it with Olivier's extension : https://extensions.joomla.org/extension/latest-news-enhanced/


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

@HLeithner HLeithner merged commit a937e06 into joomla:5.0-dev Sep 12, 2023
@HLeithner
Copy link
Member

thanks @conseilgouz can you test this please (tomorrows nightly or latest commit build)?

@Fedik Fedik deleted the changelog-parse-fix branch September 12, 2023 15:14
@conseilgouz
Copy link
Contributor

conseilgouz commented Sep 13, 2023

Just tested it : J 5 beta-2-dev nightly build 13 september 2023 00:27:40
Everything ok.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments