RTC bug PR-5.2-dev Pending

User tests: Successful: Unsuccessful:

avatar laoneo
laoneo
27 Oct 2024

Pull Request for Issue #42373.

Summary of Changes

Catches an exception when the user factory is not set in the action log model. This is basically against the principle that we want to have the dependencies injected, but it is good for the transition period till all extensions are up to the new architecture.

Testing Instructions

Happens only on 3rd party extensions, see issue.

Actual result BEFORE applying this Pull Request

Extension produces an error when writing to the action log.

Expected result AFTER applying this Pull Request

Extension works and action log is written to the database.

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

Votes

# of Users Experiencing Issue
1/1
Average Importance Score
5.00

avatar laoneo laoneo - open - 27 Oct 2024
avatar laoneo laoneo - change - 27 Oct 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 27 Oct 2024
Category Administration
d69f89a 27 Oct 2024 avatar laoneo id
avatar laoneo laoneo - change - 27 Oct 2024
Labels Added: PR-5.2-dev
avatar ssnobben
ssnobben - comment - 27 Oct 2024

Yes the solution for seems working ok now after this fix the error message gone. Great! This still an issue wihth the Finder plg that have to be disable to though.. Plugin\Finder\Content\Extension\Content::onFinderAfterSave() error #42617 #42617

avatar laoneo laoneo - change - 28 Oct 2024
Labels Added: bug
avatar laoneo laoneo - change - 28 Oct 2024
The description was changed
avatar laoneo laoneo - edited - 28 Oct 2024
avatar laoneo
laoneo - comment - 28 Oct 2024

The finder issue is not addressed by this pr.

avatar ssnobben
ssnobben - comment - 28 Oct 2024

The finder issue is not addressed by this pr.

Yes ok. Thks Allon . Can you reopen this one for Finder?

avatar laoneo
laoneo - comment - 28 Oct 2024

@ssnobben done

avatar ssnobben
ssnobben - comment - 10 Nov 2024

Can anyone test this one "by the boook" Joomla way so its into next release? or by code review bcs Its working now after this fix. Thks.


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

avatar HLeithner HLeithner - change - 15 Nov 2024
Title
Catch exception to get the user in the action log model
[5.2] Catch exception to get the user in the action log model
avatar HLeithner HLeithner - edited - 15 Nov 2024
avatar Hackwar Hackwar - test_item - 21 Nov 2024 - Tested successfully
avatar Hackwar
Hackwar - comment - 21 Nov 2024

I have tested this item ✅ successfully on e4a6c79


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

avatar Hackwar
Hackwar - comment - 21 Nov 2024

@ssnobben if you were to mark this as successfully tested, then we could merge this into 5.2.3.

avatar viocassel viocassel - test_item - 23 Nov 2024 - Tested successfully
avatar viocassel
viocassel - comment - 23 Nov 2024

I have tested this item ✅ successfully on e4a6c79


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

avatar Quy Quy - change - 23 Nov 2024
Status Pending Ready to Commit
avatar Quy
Quy - comment - 23 Nov 2024

RTC


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

avatar ssnobben
ssnobben - comment - 25 Nov 2024

@ssnobben if you were to mark this as successfully tested, then we could merge this into 5.2.3.

All checks have passed
4 successful checks
@joomla-att01
@joomla-cms-bot
Merging is blocked @Hackwar
The base branch does not allow updates. Learn more about protected branches.

avatar Hackwar
Hackwar - comment - 25 Nov 2024

Every PR for the CMS needs 2 successful manual tests. At the time of my comment, I was test 1 and you could have been test 2. Now @viocassel has thankfully tested it. Everything else is not of concern right now.

avatar ssnobben
ssnobben - comment - 25 Nov 2024

Thanks @viocassel !

avatar ssnobben
ssnobben - comment - 26 Nov 2024

Ok so why was this important issue not merged?? I want to hear an explanation.. I have beew working with Joomla and 3pds that is important for Joomla eccosystem since 2005 and this have been the most waited fix ever...

avatar pe7er
pe7er - comment - 26 Nov 2024

Ok so why was this important issue not merged?? I want to hear an explanation.. I have beew working with Joomla and 3pds that is important for Joomla eccosystem since 2005 and this have been the most waited fix ever...

The second human test for this PR came in too late to include it in Joomla 5.2.2,
which was already scheduled and released earlier today, just 2.5 hours ago.

avatar Hackwar
Hackwar - comment - 27 Nov 2024

@ssnobben Your comment is completely out of line.

avatar ssnobben
ssnobben - comment - 27 Nov 2024

@ssnobben Your comment is completely out of line.

Could be but this have been an issue since 2023 until it finally got an solution. - Thu Oct 26, 2023 10:22 am reported here in Joomla world first. I work with 3pds and people have waiting for a solution for this to release their site forever...

avatar pe7er
pe7er - comment - 27 Nov 2024

Could be but this have been an issue since 2023 until it finally got an solution. - Thu Oct 26, 2023 10:22 am reported here in Joomla world first. I work with 3pds and people have waiting for a solution for this to release their site forever...

I understand your concern. However, every change can affect other parts of Joomla or extensions from third parties. Therefore we need to be careful with adding any code to Joomla. Every PR needs to pass one successful automated test and get two successful human tests before it can be added to the core. One week before a release, a Release Candidate is created for testing that release. At this stage, we do not add new PRs, but only fix issues with the existing code in that package. The second successful test for this PR was reported during this phase so it was too late to add it to 5.2.2.

avatar Hackwar
Hackwar - comment - 27 Nov 2024

We have issues reported as old as 2015 and serious issues for example with MFA from 2020. As unfortunate as it is that it took a year to get it ready, it is not uncommon. And the reason is very simple: We are volunteers and do this in our spare time. You were very welcome to hand in a patch/PR yourself to fix this in all that time. You are not required to first open an issue. If you find a problem, you can directly create a PR and if you then motivate a few other people to test the PR, it can actually merged pretty quickly. If you don't feel that you have the knowledge to do that, you can also hire a developer to do this for you. We don't really discriminate based on this. But if you are unwilling to either work on this yourself or to invest into this and instead want free work from the community, you have to accept that it might sometimes take years until an issue is solved.

avatar laoneo laoneo - change - 28 Nov 2024
Labels Added: RTC
avatar pe7er pe7er - close - 2 Dec 2024
avatar pe7er pe7er - merge - 2 Dec 2024
avatar pe7er pe7er - change - 2 Dec 2024
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-12-02 14:53:35
Closed_By pe7er
avatar pe7er
pe7er - comment - 2 Dec 2024

Thanks @laoneo !

Add a Comment

Login with GitHub to post a comment