CRM-21203 Fixes for Joomla 3.8+ CRON/cli and directories #11062
Conversation
I've tested this on Joomla 3.8.1 and J3.7.5 with CiviCRM 4.7.25 and confirm crons are running as expected, with no adverse behaviour. |
@totten Thoughts on merging? Without it civi on Joomla is a bit broken... and it doesn't seem to have any adverse effects for those who have installed. As it only touches one file it can't impact anything other than Joomla sites... |
I think the code makes sense and is limited to Joomla integration only. I have seen supportive comments in JIRA and here that this fixes the issues reported. I think this is good to merge @totten @eileenmcnaughton |
cdb0a39
into
civicrm:master
I have tried this on a couple of joomla 3.8.1 sites running under PHP 7.0.24 and 7.1.16 - although CRON runs I get a list of warnings: PHP Notice: Constant JPATH_ISWIN already defined in /var/www/vhosts/nlcbanham.org.uk/httpdocs/libraries/import.legacy.php on line 64 |
Work in progress fixes for Joomla 3.8 to fix cli.php and cron.php