PR-4.3-dev Pending

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
25 Oct 2022

Summary of Changes

Phan requires function names to be in the interface so migrates our functions from the alias's to the functions that are inside the interfaces without changing functionality

The only change beyond function names is removing the AS functionality in this single place https://github.com/joomla/joomla-cms/compare/4.3-dev...wilsonge:phan_analysis?expand=1#diff-617530abeea53181b5e5f641ba3906dcbf758238861e8a5281422722533d8532R1487

Testing Instructions

Code review and review the phan analysis

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 wilsonge wilsonge - open - 25 Oct 2022
avatar wilsonge wilsonge - change - 25 Oct 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 25 Oct 2022
Category Administration com_installer com_joomlaupdate Libraries Front End Plugins
avatar wilsonge wilsonge - change - 25 Oct 2022
Labels Added: PR-4.3-dev
avatar richard67
richard67 - comment - 26 Oct 2022

Testing Instructions

Code review and review the phan analysis

Would be good to know for which error message to check in the phan log in drone.

avatar wilsonge
wilsonge - comment - 26 Oct 2022

PhanUndeclaredMethod Call to undeclared method \Joomla\Database\DatabaseInterface::qn

is the error we’re looking to fix here

avatar rdeutz rdeutz - change - 27 Oct 2022
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-10-27 11:13:58
Closed_By rdeutz
avatar rdeutz rdeutz - close - 27 Oct 2022
avatar rdeutz rdeutz - merge - 27 Oct 2022

Add a Comment

Login with GitHub to post a comment