[4.3] Replace Factory::getUser() by $this->getCurrentUser() in components models

[joomdonation]

Pull Request for Issue # .

Summary of Changes

With #37498 being merged into 4.2, we can now replace deprecated method Factory::getUser() in components model classes with new method $this->getCurrentUser()

Testing Instructions

1. Use Joomla 4.2 nightly build
2. Apply patch
3. Access to random pages (frontend, backend), make sure nothing is broken.

Actual result BEFORE applying this Pull Request

Works, but uses deprecated methods.

Expected result AFTER applying this Pull Request

Works, without using deprecated methods.

[laoneo]

I'm wondering if we should fallback here to Factory::getUser instead of an empty user?

[joomdonation]

Factory::getUser would be more safe, I think (I believe in site, administrator and api applications, the fallback is not needed). I just don't know how it is used in installation and console apps.

[laoneo]

Do you want to do it here or a separate one?

[joomdonation]

Better in the separate PR, please

[laoneo]

Here we go #38106.

[joomla-bot]

This pull requests has been automatically converted to the PSR-12 coding standard.

[laoneo]

Would be good to postpone this for 4.3. What do you think @joomdonation?

[joomdonation]

@laoneo Yes, no rush. Let's give maintainers more time to review to see if there is any flow with this change.

[jwaisner]

I have tested this item ✅ successfully on c19a04a

Tested good on frontend and backend pages in Joomla 4.3 dev

Drone is erroring out but looks like a timeout issue so I do not believe this related.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38090.}

[RickR2H]

I have tested this item ✅ successfully on c19a04a

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38090.}

[RickR2H]

RTC

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38090.}

[laoneo]

Thanks!