[#37489] - [4.x] com_templates when using pdo
- Closed
- 6 Apr 2022
- Medium
- Build: 4.1-dev
- # 37489
- Diff
- brianteeman:template_pdo
User tests: Successful: Unsuccessful:
Pull Request for Issue #37487 .
Testing Instructions
on a joomla install using mysql pdo (eg in configuration.php it says public $dbtype = 'mysql'; not mysqli
open the template manager
Actual result BEFORE applying this Pull Request
the path is fubar
Expected result AFTER applying this Pull Request
the path is correct and no error messages when using the template manager
Votes
| Status | New | ⇒ | Pending |
| Labels |
Added:
?
|
||
joomla-cms-bot
- | Category | ⇒ | Administration com_templates |
I have tested this item
It solves my issue.
Now the path is correct, and everything works fine, no more error messages and I can edit files.
Thanks a lot Brian.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37489.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37489.
Thanks for the pr, but would it not be easier to cast to int where the template is fetched form the database? Like that we fix it on the source of the issue and we do not have to patch it all over the core.
| Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-04-06 16:33:15 |
| Closed_By | ⇒ | brianteeman | |
| Labels |
Added:
?
|
||
I have tested this item✅ successfully on 9d3eb69
Tested successfully in Joomla 4.1.3-dev in Wampserver 3.2.7 using PHP 8.0.15
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37489.