Comment 5 for bug 746204

Scott,

I have to agree with kiwi.
Using your code or not makes no difference, with the exception of when a place has been imported from the GEDCOM as kiwi says.
This is also where I believe the code should be fixed. (and I will endeavor to do that after consultation with other developers)

However I did find that when a non defined place is found on the "place check" admin window (item is in red with red x's)
that if you click this red location to define the place, then a red marker did appear at location (0, 0)
I have now fixed this, and the marker will be postioned at the parent level location.
All that remains to be done now (as before), is to either:
a) do a "search on this level" to provide any possibilities with yellow markers which can be clicked and the click "Use this value" in the info-window.
OR
b) Drag the red marker to the position you want
OR
c) Click the map to position the red marker where you want.

The only other thing that happens that is misleading, is on the place hierarchy page.
If a location is not defined, and there are no other places at this level below the parent, then the parent map, (which normally shows the "child locations"), is blank, with no markers shown, indicating that a place check should be run to verify this.
(This is logical)
However the parent map is centered at (0, 0)
(This is not logical. it should be centered at the parent location.)
This I am in the process of fixing now.

Both fixes I will post to svn shortly when done.

Brian