So like a previous poster said, using the original order.php does not seem to affect the function of this module in any way. I decided to use the original order.php and now this mod seems to be working great!
My understanding of code is pretty limited, but like the other poster mentioned, I am not sure why this code is necessary?
$_SERVER['REMOTE_ADDR'] = $server_remote_addr;
if (zen_not_null($server_remote_addr)) { $server_remote_address = ' - ' . $server_remote_addr; }
....
'ip_address' => $_SESSION['customers_ip_address'] . $server_remote_address
This conditional statement and reassignment of the 'ip_address' field in the array appears to lead to the same outcome as the original unmodified order.php. The only thing I can see these lines of code adding is to make sure that $_SERVER['REMOTE_ADDR'] does not return a null value before assigning it to the $server_remote_address variable...
Also, after sitting down and looking at this code and trying to understand it, it actually seems like there really is no reason why a simple extra conditional check would cause an internal server issue...
The only thing I can think of (and I think also seems to make sense given the structure of the conditional statement) is that the code should actually read like...
$server_remote_addr = $_SERVER['REMOTE_ADDR'];
if (zen_not_null($server_remote_addr)) { $server_remote_address = ' - ' . $server_remote_addr; }
Because isn't $server_remote_addr an undefined value that you are assigning the customer's IP by calling the $_SERVER['REMOTE_ADDR'] function?? And then using the newly defined $server_remote_addr in the conditional statement?
It doesn't seem to make sense to have it as... $_SERVER['REMOTE_ADDR'] = $server_remote_addr;
Because $_SERVER['REMOTE_ADDR'] isn't a variable, it's a function call, and having the code this way leaves $server_remote_addr with a null value...
Sorry, my understanding of php is limited, but this is how I see it... any clarification you can offer would be immensely helpful, thanks SES!!!
