tweeper.php: factor out an html_to_xml() function from the tweep() method

[tweeper.git] / tweeper.php

diff --git a/tweeper.php b/tweeper.php
index 17fff8b..524928a 100644 (file)
--- a/tweeper.php
+++ b/tweeper.php
@@ -164,6 +164,21 @@ class Tweeper {
     return $xsltProcessor;
   }
 
     return $xsltProcessor;
   }
 

+  private function html_to_xml($html) {
+    $xmlDoc = new DOMDocument();
+
+    // Handle warnings and errors when loading invalid HTML.
+    $xml_errors_value = libxml_use_internal_errors(true);
+    $xmlDoc->loadHTML($html);
+    foreach (libxml_get_errors() as $xml_error) {
+      $this->log_xml_error($xml_error);
+    }
+    libxml_clear_errors();
+    libxml_use_internal_errors($xml_errors_value);
+
+    return $xmlDoc;
+  }
+

   public function tweep($src_url) {
     $url = parse_url($src_url);
     if (FALSE === $url || empty($url["host"])) {
   public function tweep($src_url) {
     $url = parse_url($src_url);
     if (FALSE === $url || empty($url["host"])) {


@@ -176,18 +191,15 @@ class Tweeper {
       return NULL;
     }
 
       return NULL;
     }
 

-    $html = Tweeper::get_contents($src_url);
-
-    $xmlDoc = new DOMDocument();
+    $html = $this->get_contents($src_url);
+    if (FALSE === $html) {
+      return NULL;
+    }

 
 

-    // Handle warnings and errors when loading invalid HTML.
-    $xml_errors_value = libxml_use_internal_errors(true);
-    $xmlDoc->loadHTML($html);
-    foreach (libxml_get_errors() as $xml_error) {
-      $this->log_xml_error($xml_error);
+    $xmlDoc = $this->html_to_xml($html);
+    if (NULL === $xmlDoc) {
+      return NULL;

     }
     }

-    libxml_clear_errors();
-    libxml_use_internal_errors($xml_errors_value);

 
     $output = $xsltProcessor->transformToXML($xmlDoc);
 
 
     $output = $xsltProcessor->transformToXML($xmlDoc);