MDL-68714 clilib: Check set_process_title exists
authorAndrew Nicols <andrew@nicols.co.uk>
Fri, 3 Jul 2020 01:49:08 +0000 (09:49 +0800)
committerAndrew Nicols <andrew@nicols.co.uk>
Fri, 3 Jul 2020 01:49:11 +0000 (09:49 +0800)
The web cron script at /admin/cron.php pretends to be a CLI script which
means that it is not possible to rely upon the CLI_SCRIPT constant.

lib/clilib.php

index 1295e54..6cc767f 100644 (file)
@@ -154,7 +154,7 @@ function cli_get_params(array $longoptions, array $shortmapping=null) {
  * @param string $suffix process suffix
  */
 function cli_set_process_title_suffix(string $suffix) {
  * @param string $suffix process suffix
  */
 function cli_set_process_title_suffix(string $suffix) {
-    if (CLI_SCRIPT && isset($_SERVER['argv'])) {
+    if (CLI_SCRIPT && function_exists('cli_set_process_title') && isset($_SERVER['argv'])) {
         $command = join(' ', $_SERVER['argv']);
         @cli_set_process_title("php $command ($suffix)");
     }
         $command = join(' ', $_SERVER['argv']);
         @cli_set_process_title("php $command ($suffix)");
     }