From: sam_marshall <sam_marshall>
Date: Fri, 7 Dec 2007 10:24:00 +0000 (+0000)
Subject: MDL-11845 Updated the three file downloads from Snoopy to download_file_content ... 
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=ae8c48f726d9a4320cf778035b71a88348caf48b;p=moodle.git

MDL-11845 Updated the three file downloads from Snoopy to download_file_content (which I didn't know about before :)
---

diff --git a/admin/timezoneimport.php b/admin/timezoneimport.php
index 086d95525e..b38ecdbbed 100644
--- a/admin/timezoneimport.php
+++ b/admin/timezoneimport.php
@@ -6,7 +6,6 @@
     require_once($CFG->libdir.'/adminlib.php');
     require_once($CFG->libdir.'/filelib.php');
     require_once($CFG->libdir.'/olson.php');
-    require_once($CFG->libdir.'/snoopy/Snoopy.class.inc');
     
     admin_externalpage_setup('timezoneimport');
 
@@ -63,14 +62,10 @@
     }
 
 /// Otherwise, let's try moodle.org's copy
-    $snoopy = new Snoopy;
-    $snoopy->proxy_host = $CFG->proxyhost;
-    $snoopy->proxy_port = $CFG->proxyport;
-
     $source = 'http://download.moodle.org/timezones/';
-    if (!$importdone && $snoopy->fetch($source)) {
+    if (!$importdone && ($content=download_file_content($source))) {
         if ($file = fopen($CFG->dataroot.'/temp/timezones.txt', 'w')) {            // Make local copy
-            fwrite($file, $snoopy->results);
+            fwrite($file, $content);
             fclose($file);
             if ($timezones = get_records_csv($CFG->dataroot.'/temp/timezones.txt', 'timezone')) {  // Parse it
                 update_timezone_records($timezones);
@@ -82,7 +77,6 @@
 
 
 /// Final resort, use the copy included in Moodle
-
     $source = $CFG->dirroot.'/lib/timezones.txt';
     if (!$importdone and is_readable($source)) {  // Distribution file
         if ($timezones = get_records_csv($source, 'timezone')) {
diff --git a/lib/componentlib.class.php b/lib/componentlib.class.php
index c899a7596d..b79214798f 100644
--- a/lib/componentlib.class.php
+++ b/lib/componentlib.class.php
@@ -124,8 +124,6 @@
 // error to be retrieved by one standard get_string() call against the error.php lang file.
 //
 // That's all!
-global $CFG;
-require_once($CFG->libdir.'/snoopy/Snoopy.class.inc');
 
 // Some needed constants
 define('ERROR',           0);
@@ -274,13 +272,7 @@ class component_installer {
         $source = $this->sourcebase.'/'.$this->zippath.'/'.$this->zipfilename;
         $zipfile= $CFG->dataroot.'/temp/'.$this->zipfilename;
 
-    /// Prepare Snoopy client and set up proxy info
-        $snoopy = new Snoopy;
-        global $CFG;
-        $snoopy->proxy_host = $CFG->proxyhost;
-        $snoopy->proxy_port = $CFG->proxyport;
-        if($snoopy->fetch($source)) {
-            $contents = $snoopy->results;
+        if($contents = download_file_content($source)) {
             if ($file = fopen($zipfile, 'w')) {
                 if (!fwrite($file, $contents)) {
                     fclose($file);
@@ -468,15 +460,9 @@ class component_installer {
         /// Not downloaded, let's do it now
             $availablecomponents = array();
 
-            /// Prepare Snoopy client and set up proxy info
-            $snoopy = new Snoopy;
-            global $CFG;
-            $snoopy->proxy_host = $CFG->proxyhost;
-            $snoopy->proxy_port = $CFG->proxyport;
-
-            if ($snoopy->fetch($source)) {
+            if ($contents = download_file_content($source)) {
             /// Split text into lines
-                $lines=preg_split('/\r?\n/',$snoopy->results);
+                $lines=preg_split('/\r?\n/',$contents);
             /// Each line will be one component
                 foreach($lines as $line) {
                     $availablecomponents[] = split(',', $line);