-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[5.0] Fix legacy table creation #42180
[5.0] Fix legacy table creation #42180
Conversation
If a legacy table is created by Table::getInstance() twice it fails the second time. Reason for this is that the legacy table name is not used if the table class already exists.
Did you change something in the code? Only intendation is added. |
Add |
Thanks, I was sure that I miss here something :-) |
@georgebara can you please test this pr? |
Thank you for the fast feedback. |
@georgebara Was your approval a real successful test? If so, please mark your test result so it's properly counted. For doing this, go to the PR in the issue tracker here https://issues.joomla.org/tracker/joomla-cms/42180 , then use the blue "Test this" button at the top left corner, then select your test result and finally submit. Thanks in advance. |
I have tested this item ✅ successfully on 6265a1b This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42180. |
I have tested this item ✅ successfully on 6265a1b This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42180. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42180. |
Thx |
Pull Request for Issue #42179 .
Summary of Changes
If a legacy table is created by Table::getInstance() twice it fails the second time. Reason for this is that the legacy table name is not used if the table class already exists.
Testing Instructions
Test joomla core and a random 3rd party extension which uses tables
Actual result BEFORE applying this Pull Request
Could fail if
Table::getInstance()
is called more then 1 time for the same old (JTableXXX) syntax.Expected result AFTER applying this Pull Request
works
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