El 27/02/11 22:33, WanMil escribió:
I have done some development and want you to share my current findings.
1. The MapElement copy constructor seems to have a bug. The attributes map which contains the city, region and country information is not copied. From my point of view this is an important thing that should be fixed in the trunk also.
2. In the patch the Locator is disabled as much as it was possible for me up to now.
3. I am using now separate tags (mkgmap:city, mkgmap:region etc.).
I recommend to run this patch with location-autofill=-1 or 0 to see how the patch works and not how the old Locator works. Some findings running your patch on Spain's map: 1-The list of regions (state/country field) is much better than the one obtained with trunk. All those included are actual regions (some with two different names, e.g. Castilla la Mancha & Castilla-la Mancha). Trunk includes many names that are not actual regions of Spain, but provinces, cities or even villages. 2-The list of countries has grown from ESP (España), España (ESP) to Es, ESP (España), España (ESP), Gribraltar / United Kingdom, Spain, Territorial water of Ibiza and Territorial waters of Mallorca. Could we have some mechanism to unify all forms of a country in a single one, i.e. Es, ESP, España, Spain and Territorial water...->España? Maybe LocatorConfig.xml could/should do that. 3-When the country field is filled in any of the search tabs, State/Province info is not taken into account. 4-When searching for a street, city, region and country information is missing in the results list. Searching for a street+city doesn't work; street+(State/Province | Country) does work.