Closed (fixed)
Project:
User IP Lookup
Version:
8.x-1.3
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
16 Oct 2019 at 21:17 UTC
Updated:
17 Nov 2019 at 23:54 UTC
Jump to comment: Most recent
A client sent me this screenshot.
They are heavy Microsoft users, so they are probably using Edge (waiting for confirmation).
Line 150 of ip_lookup.module encounters a warning because $ub is not defined for many browsers.
There are multiple options here:
Simplest option can look like this, starting at line 147:
else {
$bname = 'Unknown';
$ub = "Unknown";
}
| Comment | File | Size | Author |
|---|---|---|---|
| undefined_variable.png | 114.21 KB | arnaldop |
Comments
Comment #3
jiang_2015 commentedI went to the easiest option and add the Edge to the variable. I will do the third option in the future. Thank you so much for your help.
Comment #4
jiang_2015 commented