[#11335] - PHP Notice on update
- Closed
- 29 Jul 2016
- Medium
- Build: master
- # 11335
PHP Notice: Use of undefined constant JPATH_ADMINISTRATOR - assumed 'JPATH_ADMINISTRATOR' in administrator/components/com_admin/script.php on line 1551
PHP Notice: Use of undefined constant JPATH_MANIFESTS - assumed 'JPATH_MANIFESTS' in administrator/components/com_admin/script.php on line 1552
Update a clean joomla 3.6 to staging and check the php error logs
release blocker tag?
I am going to book a release blocker tag on this - its only a notice but its not good for people to see an error when they update
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11335.
| Labels |
Added:
?
|
||
Why not use JPATH_ROOT instead of JPATH_ADMINISTRATOR and JPATH_MANIFESTS seems to my easier, I will make a PR
/me not a developer - just as long as it is fixed before the release so
people dont see notices and start to panic
On 29 July 2016 at 09:13, Robert Deutz notifications@github.com wrote:
Why not use JPATH_ROOT instead of JPATH_ADMINISTRATOR and JPATH_MANIFESTS
seems to my easier, I will make a PR—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#11335 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8Z_6EnRoVQumPetr9S8MMFTtkgsHks5qabYZgaJpZM4JXTnX
.
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/
PR done please check, thanks for finding the issue
| Status | New | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-07-29 09:46:00 |
| Closed_By | ⇒ | brianteeman |
wilsonge
- | Labels |
Removed:
?
|
||
https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_joomlaupdate/restore_finalisation.php probably needs to ensure the path constants are all loaded up.