PR-6.0-dev PR-5.2-dev Pending

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
24 Apr 2024

Summary of Changes

When adding typehints for these custom fields, I mistakenly used the FileLayout class, while it actually is the respective plugins class.

Testing Instructions

Codereview.

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

avatar Hackwar Hackwar - open - 24 Apr 2024
avatar Hackwar Hackwar - change - 24 Apr 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 24 Apr 2024
Category Front End Plugins
avatar Hackwar Hackwar - change - 25 Apr 2024
Labels Added: PR-5.2-dev
avatar zero-24
zero-24 - comment - 22 May 2024

@HLeithner could it be that we need to merge the 5.2-dev to 6.0-dev first?

avatar HLeithner
HLeithner - comment - 22 May 2024

yes, but didn't head the time yet

avatar zero-24
zero-24 - comment - 22 May 2024

Done: #43515

avatar HLeithner HLeithner - change - 22 May 2024
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-05-22 18:44:04
Closed_By HLeithner
Labels Added: PR-6.0-dev
avatar HLeithner HLeithner - close - 22 May 2024
avatar HLeithner HLeithner - merge - 22 May 2024
avatar HLeithner
HLeithner - comment - 22 May 2024

Merged in 6.0 to prevent override notification system, thanks

avatar HLeithner HLeithner - change - 22 May 2024
Title
[5.2] Custom Fields: Correcting typehint for $this in layout
[6.0] Custom Fields: Correcting typehint for $this in layout
avatar HLeithner HLeithner - edited - 22 May 2024
avatar Hackwar
Hackwar - comment - 22 May 2024

Thank you!

Add a Comment

Login with GitHub to post a comment