MDL-28484 Libraries: Update comments to suggest using new is_https()
authorDamyon Wiese <damyon@moodle.com>
Wed, 8 Oct 2014 01:52:17 +0000 (09:52 +0800)
committerDamyon Wiese <damyon@moodle.com>
Wed, 8 Oct 2014 01:54:05 +0000 (09:54 +0800)
lib/pagelib.php
lib/setup.php

index d39b5f2..8f66ba5 100644 (file)
@@ -1432,7 +1432,7 @@ class moodle_page {
 
         // Now the real test and redirect!
         // NOTE: do NOT use this test for detection of https on current page because this code is not compatible with SSL proxies,
-        //       instead use (strpos($CFG->httpswwwroot, 'https:') === 0).
+        //       instead use is_https().
         if (strpos($FULLME, 'https:') !== 0) {
             // This may lead to infinite redirect on an incorrectly configured site.
             // In that case set $CFG->loginhttps=0; within /config.php.
index ea59015..5ba6be0 100644 (file)
@@ -491,7 +491,7 @@ global $OUTPUT;
  * Full script path including all params, slash arguments, scheme and host.
  *
  * Note: Do NOT use for getting of current page URL or detection of https,
- * instead use $PAGE->url or strpos($CFG->httpswwwroot, 'https:') === 0
+ * instead use $PAGE->url or is_https().
  *
  * @global string $FULLME
  * @name $FULLME