PHP Classes

Undefined index

Recommend this page to a friend!

      EPub  >  All threads  >  Undefined index  >  (Un) Subscribe thread alerts  
Subject:Undefined index
Summary:Notice: Undefined index: HTTPS
Messages:7
Author:Max
Date:2012-09-03 18:38:44
Update:2012-09-03 19:50:00
 

  1. Undefined index   Reply   Report abuse  
Picture of Max Max - 2012-09-03 18:38:44
Hi,
Tested on last version (2.06)
On server without HTTPS and error reporting max level I have:

Notice: Undefined index: HTTPS
EPub.php on line 1334
EPub.php on line 1353

if ($_SERVER["HTTPS"] == "on") {
$pageURL .= "s";
}

maybe just fix it :)

if (@$_SERVER["HTTPS"] == "on") {
$pageURL .= "s";
}

Thanks for great work!

Regards
Max

  2. Re: Undefined index   Reply   Report abuse  
Picture of Asbjorn Grandt Asbjorn Grandt - 2012-09-03 19:03:11 - In reply to message 1 from Max
So you are saying that because HTTPS is missing it might throw an error, and adding the @ should silence this?

  3. Re: Undefined index   Reply   Report abuse  
Picture of Max Max - 2012-09-03 19:20:45 - In reply to message 2 from Asbjorn Grandt
Yes.
Alredy try this on my server.

  4. Re: Undefined index   Reply   Report abuse  
Picture of Asbjorn Grandt Asbjorn Grandt - 2012-09-03 19:27:31 - In reply to message 1 from Max
Hi

Adding the @ is enough, but I prefer something else really.

Replace the two if's with
if (isset($_SERVER["HTTPS"]) && $_SERVER["HTTPS"] == "on") {


I'll be uploading 2.08 shortly

  5. Re: Undefined index   Reply   Report abuse  
Picture of Max Max - 2012-09-03 19:32:45 - In reply to message 4 from Asbjorn Grandt
Sure! I`am agree with better solution.

TXH

  6. Re: Undefined index   Reply   Report abuse  
Picture of Asbjorn Grandt Asbjorn Grandt - 2012-09-03 19:38:14 - In reply to message 5 from Max
2.08 updated :)

Thanks for noticing the bug.

  7. Re: Undefined index   Reply   Report abuse  
Picture of Max Max - 2012-09-03 19:50:00 - In reply to message 6 from Asbjorn Grandt
Awesome! Downloaded :)
THX!