Feeling Lucky
?
bug
PR-4.3-dev
[#41135] - [4.3] check of invalid json 'joomla.asset.json'
- Fixed in Code Base
- 14 Jul 2023
- Medium
- Build: 4.3-dev
- # 41135
- Diff
- heelc29:4.3/templates/joomla-asset
User tests: Successful: Unsuccessful:
Summary of Changes
Validation of joomla.asset.json (JSON) is broken since J4.1 #35998 (child templates)
str_ends_with should covered by polyfill:
Line 84 in c944137
Testing Instructions
try to edit joomla.asset.json and save with an invalid json (eg. double comma)
Actual result BEFORE applying this Pull Request
file saved
site broken (here frontend cassiopeia)
Expected result AFTER applying this Pull Request
file not saved
Link to documentations
Please select:
- No documentation changes for docs.joomla.org needed
- No documentation changes for manual.joomla.org needed
joomla-cms-bot
- | Category | ⇒ | Administration com_templates |
| Status | New | ⇒ | Pending |
| Labels |
Added:
bug
PR-4.3-dev
|
||
alikon
- comment
- 13 Jul 2023
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41135.
| Status | Pending | ⇒ | Ready to Commit |
alikon
- comment
- 13 Jul 2023
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41135.
| Labels |
Added:
?
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-07-14 20:26:40 |
| Closed_By | ⇒ | obuisard |
I have tested this item✅ successfully on 5bf1e08
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41135.