GEO IP Code Expert Needed
 

Its a simple GEO IP which is intended to pull the City name from the database according to IP but if it cannot determine the City name, it should show just "Your City". GEO IP module is enabled but this code isn't working.

Can someone please show me what I'm doing wrong here?

You are trying to compare an array as a string on line 4.

You need to address the key in the array to get the string.

Here's a sample from PHP.NET

Do
If you don't see your country listed, it means the module or database for it is not properly configured.

I don't remember the exact variable, but you should be able to access the country with $_SERVER['GEOIP_COUNTRY_NAME'] or something like that - it will be listed in that $_SERVER dump.

But even then

won't work will it ?

I'm not on my dev machine so I can't try it.

It's not "correct" but php is very forgiving, it won't error out and just in that isolated piece of code will work. He should be comparing to $geo['city'] though.

Im pedantic :)

any clues people?

IF your geoip module is properly configured...

Well my code shows me a city.... your code doesnt work for me at all.
Doesn't show any city.

Can you please post the output from the following code.

and

That is because K0nrad did NOT include an echo in there. I think he assumed you knew to echo the output but after reading the thread I don't know that you do. So, without meaning anything derogatory here, let's take this to kindergarten level.

Your code:

$geo=geoip_record_by_name ($_SERVER['REMOTE_ADDR']);
echo $geo['city'];
if ($geo == "") {
$geo['city'] = "Your City";
}

You are echoing the city before you test to see if city is filled. So you see whatever city the geo returned. You might also see nothing come back if the city is blank. You will NEVER see "Your City" come back because you are pushing the output before the if statement. Does that make sense to you?

Also K0nrad, AK and I think you have a mistake in the if statement. You are checking against geo (if geo="") not against geo['city']. You should be comparing against city.

K0nrad fixed the if statement for you but he removed the echo (which was in the wrong place). So let's try K0nrads code with an echo added:

$geo = geoip_record_by_name($_SERVER['REMOTE_ADDR']);
if(!$geo['city']) {
$geo['city'] = "Your City";
}
echo $geo['city'];

That should show the city if geo_ip returned a city and should show "Your City" if geo_ip['city'] comes back null or empty.

See if that works for you or not.

.

For a second there I was ready to jump in and see if I could help, but after RummyBoy's last reply I can see it's a lost cause.

Yeah I said that, read up where I said I was pedantic. :)

I think it was lost on him.

Yep, kindergarten level was needed in fact. I should have told you guys im clueless.

I haven't been able to test the default "Your City" but this version does actually seem to work.

Thanks for all your help people.

Nah, you're not. You were smart enough to ask the question and the only dumb question is one you don't ask.

:thumbsup

Ooops, I read that as "Im pathetic". Sorry about that. My bad ;p

Nothing wrong with Kindergarten code :thumbsup

+1 :thumbsup

.