Feeling Lucky
?
[#8650] - new ConfigModelApplication::storePermissions should use existing API
- Closed
- 23 Dec 2015
- Medium
- Build: master
- # 8650
Referenced as
Pull Request for:
# 8651
New ConfigModelApplication::storePermissions reads and writes assets database table directly.
https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_config/model/application.php#L348
https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_config/model/application.php#L416
Those queries should be replaced by usage of existing API from JTableAsset , also we should avoid direct coupling with db structure here.
E.g. first query can be replaced by JTableAsset::loadByName
| Title |
|
||||||
| Labels |
Added:
?
|
||
| Status | New | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-12-23 11:06:46 |
| Closed_By | ⇒ | brianteeman |
Closed as we have a PR #8651
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8650.