[#15753] - Correcting Notices when saving article in frontend
- Fixed in Code Base
- 3 May 2017
- Medium
- Build: staging
- # 15753
- Diff
- infograf768:undefined_article_save
User tests: Successful: Unsuccessful:
When saving an article in frontend one gets 2 notices:
PHP Notice: Undefined index: created_by in /administrator/components/com_content/models/article.php on line 470
PHP Notice: Undefined index: modified_by in /administrator/components/com_content/models/article.php on line 470
Summary of Changes
The validate method is OK in backend as it prevents users without users manager permissions to modify these fields input, but it is unnecessary in frontend as these fields are already Unset in the xml.
Therefore this PR checks for client instead.
Testing Instructions
Before patch, save an article in frontend. The Notices will display in the PHP logs.
After patch no more Notices.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_content |
I have tested this item
Patch ok for me.
Similar problem also in save "Site Settings" in Frontend by super user:
PHP Notice: Undefined index: force_ssl in /opt/lampp/htdocs/Joomla_3.7.1-dev-Development-Full_Package/administrator/components/com_config/model/application.php on line 123
PHP Notice: Undefined index: cache_path in /opt/lampp/htdocs/Joomla_3.7.1-dev-Development-Full_Package/administrator/components/com_config/model/application.php on line 286
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15753.
| Status | Pending | ⇒ | Ready to Commit |
RTC after two successful tests.
rdeutz
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-05-03 08:38:48 |
| Closed_By | ⇒ | rdeutz | |
| Labels |
Added:
?
|
||
PHP Notice: Undefined index: force_ssl in /opt/lampp/htdocs/Joomla_3.7.1-dev-Development-Full_Package/administrator/components/com_config/model/application.php on line 123
PHP Notice: Undefined index: cache_path in /opt/lampp/htdocs/Joomla_3.7.1-dev-Development-Full_Package/administrator/components/com_config/model/application.php on line 286
looking into these
I have tested this item✅ successfully on 645c59c
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15753.