Merge pull request #5 from fatihorhan/master

Some fixes and improvements

3 files changed, 13 insertions, 5 deletions

diff --git a/inc/Readability.php b/inc/Readability.php
index 52a37d50..c50bf2ef 100755
--- a/inc/Readability.php
+++ b/inc/Readability.php
@@ -80,7 +80,7 @@ class Readability
         public $debug = false;
         protected $body = null; // 
         protected $bodyCache = null; // Cache the body HTML in case we need to re-use it later
-        protected $flags = FLAG_CLEAN_CONDITIONALLY; // 1 | 2 | 4;   // Start with all flags set.
+        protected $flags = self::FLAG_CLEAN_CONDITIONALLY; // 1 | 2 | 4;   // Start with all flags set.
         protected $success = false; // indicates whether we were able to extract or not
         
         /**
diff --git a/index.php b/index.php
index 9e6e2256..d89fa41c 100755
--- a/index.php
+++ b/index.php
@@ -53,7 +53,11 @@ switch ($action) {
 }
 
 function url(){
-  $protocol = ($_SERVER['HTTPS'] && $_SERVER['HTTPS'] != "off") ? "https" : "http";
+  $protocol = "http";
+  if(isset($_SERVER['HTTPS']))
+    if($_SERVER['HTTPS'] != "off")
+      $protocol = "https";
+  
   return $protocol . "://" . $_SERVER['HTTP_HOST'] . $_SERVER['REQUEST_URI'];
 }
 ?>
@@ -80,7 +84,7 @@ function url(){
                                 <li><a href="index.php">home</a></li>
                                 <li><a href="#">favorites</a></li>
                                 <li><a href="#">archive</a></li>
-                <li><a href="javascript:(function(){var%20url%20=%20location.href;var%20title%20=%20document.title%20||%20url;window.open('<? echo url()?>index.php?action=add&url='%20+%20encodeURIComponent(url),'_self');})();">poche it !</a></li>
+                <li><a href="javascript:(function(){var%20url%20=%20location.href;var%20title%20=%20document.title%20||%20url;window.open('<?php echo url()?>?action=add&url='%20+%20encodeURIComponent(url),'_self');})();">poche it !</a></li>
                         </ul>
                 <?php
                         $query = $db_handle->prepare("SELECT * FROM entries WHERE read=?");
diff --git a/readityourself.php b/readityourself.php
index 4a4661c2..c59bb9d1 100755
--- a/readityourself.php
+++ b/readityourself.php
@@ -26,7 +26,11 @@ require_once dirname(__FILE__).'/inc/rain.tpl.class.php';
 
 
 function url(){
-  $protocol = ($_SERVER['HTTPS'] && $_SERVER['HTTPS'] != "off") ? "https" : "http";
+  $protocol = "http";
+  if(isset($_SERVER['HTTPS']))
+    if($_SERVER['HTTPS'] != "off")
+      $protocol = "https";
+  
   return $protocol . "://" . $_SERVER['HTTP_HOST'] . $_SERVER['REQUEST_URI'];
 }
 
@@ -92,7 +96,7 @@ function get_external_file($url, $timeout) {
         }
 
         // if response is not empty and response is OK
-        if (isset($data) and isset($httpcodeOK) and httpcodeOK ) {
+        if (isset($data) and isset($httpcodeOK) and $httpcodeOK ) {
 
                 // take charset of page and get it
                 preg_match('#<meta .*charset=.*>#Usi', $data, $meta);