[#42192] - fix #42191 Unknown column 'id' in 'field list'
- Closed
- 22 Oct 2023
- Medium
- Build: 5.0-dev
- # 42192
- Diff
- svanschu:5.0-dev
User tests: Successful: Unsuccessful:
In this method it tries to load the field 'id' from the extensions table. But there isn't any field called id in this table. The id field for the extenions table is called extension_id
Pull Request for Issue #42191 .
Summary of Changes
Moved the select id statement into the module if clause and create a new select extension_id as id clause for the else branch
Testing Instructions
Actual result BEFORE applying this Pull Request
Expected result AFTER applying this Pull Request
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
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
| Labels |
Added:
PR-5.0-dev
|
||
@svanschu As far as I could see we have the same error also in the 4.4-dev code at the same place. Is that right? If so, the PR should be made for the 4.4-dev branch. When that PR will be merged, maintainers will sooner or later merge it up to 5.0-dev, so the bug will be fixed in 4.4.1 and 5.0.1.
Yes 4.4 ist identical to 5.0 in this code section.
Then I will create a new PR for 4.4 and close this one when it's opened.
@richard67 PR for 4.4-dev is created #42199
Closing this one
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-10-22 14:18:25 |
| Closed_By | ⇒ | svanschu |
@svanschu As far as I could see we have the same error also in the 4.4-dev code at the same place. Is that right? If so, the PR should be made for the 4.4-dev branch. When that PR will be merged, maintainers will sooner or later merge it up to 5.0-dev, so the bug will be fixed in 4.4.1 and 5.0.1.