Failed to set map name using MgMap.Open

[liuar]

• Assume we have a MapDefinition called “Library://Test/Default.MapDefinition”
• We create the map using the following code:

    MgMap map = new MgMap();
    map.Create(resourceService, new MgResourceIdentifier(“Library://Test/Default.MapDefinition”), “Default1234”);
    map.Save();

• In other code, we want to use this map, so we use this code:

    MgMap map = new MgMap();
    map.Open(resourceService, “Default1234”);

• Now the Name property for map in step c) is supposed to be “Default1234”. However, you’ll find it’s still “Default”, the name of map definition, not the name of map.

If you open the code of “
trunk\MgDev\Common\MapGuideCommon\MapLayer\Map.cpp”, you’ll find the line 504 and 505 is like this:

    m_name = mapName;
    MgResource::Open(m_resourceService, resId);

However, MgResource::Open() will use the name of map definition, not the name of map to replace the value of m_name. So we should put “m_name = mapName;” behind “MgResource::Open()”.

[christinebao]

Hi Spark,

You already have fixing for this ticket. Would you please attach it for review/submission?

Thanks & regards,
Christine

[sparkliu]

The fix for the Ticket 1175

[christinebao]

The patch looks good.

In MgResource::Open(m_resourceService, resId); m_name is deserialized from stream, thus m_name = mapName; is override. To keep m_name is mapName as expected, the code should be moved down.

The code is submitted on behalf of Spark Liu to fix this ticket.

[brucedechant]

Fixed in trunk. Submission r4461