I have tested this item ✅ successfully on 7b55457

Just a thought: would it be better to use a local $language variable:

$language = $app->getLanguage()

and then use that in the loops. Otherwise, this looks OK to me.

I have tested this item ✅ successfully on 7b55457

@heelc29 Is there a reason why this PR is still draft?

@heelc29 Is there a reason why this PR is still draft?

Since I discovered an use of the variable in the core and updated the testing instructions and also have to update the baseline file.

we need to do better than this. Instead of adding an exception why not fix it. I was taught by @nibra that when we deprecate something we should stop using it in the core at the same time.

we need to do better than this. Instead of adding an exception why not fix it. I was taught by @nibra that when we deprecate something we should stop using it in the core at the same time.

@brianteeman In general that's right, but here we need the exclusion due to the b/c code:

$app   = $this->getApplication() ?: $this->app;

The ?: $this->app part has to be kept for b/c until Joomla 7, and this part causes the PHPstan complaint.

But then it would be better to add a phpstan ignore on that line.

But then it would be better to add a phpstan ignore on that line.

@laoneo I don't know. Maybe yes. As it is a relative new thing that we do not ignore PHPstan in CI, we don't have much practice with that yet. Up to now we were using the baseline file to ignore errors and as a task list.

I think we should discuss that and find a common way in the maintainers team.

Shouldn't there be some documentation explaining why this is being deprecated

Shouldn't there be some documentation explaining why this is being deprecated

In future it is no longer supported by CMSPlugin #38060

that isnt a why? and its not in the accompanying manual entry

RTC as the previous human tests are still valid. There was only a change on the PHPstan baseline file after that and a clean branch update.

To me the joomla/Manual#488 documentation PR looks complete.

✅ Final test before merge, JBT graft full package

• Enabled Log Almost Everything
• Created custom field 'color' and featured article and tested in administrator backend and site frontent
• Created one more article, a field group and fields user, url and usergroup and assigned to the group
  • All three fields are available and working in adminstrator backend
  • User field is missing in site frontend edit view, but is also missing before this PR -> opened #45874 for clarification
• No entries in administrator/logs/*

Thank you @heelc29 for your contribution. Thank you @brianteeman for support. Thank you @ceford and @laoneo for testing.