Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#1443 closed defect (fixed)

MgProxyFeatureService::GetCapabilities is using a fixed api version 1.0.0

Reported by: wuma Owned by: Mars Wu
Priority: high Milestone: 2.3
Component: General Version: 2.1.0
Severity: critical Keywords:
Cc: brucedechant External ID: 1342879

Description (last modified by brucedechant)

Steps:

  • Run WebTier tests:Feature/GetCapabilities. It will get the capabilites for a certain FDO provider.
  • Keep the initial values for the test page except for changing the version to be 2.0.0
  • Submit the request

You will find the response is always for 1.0.0 instead of 2.0.0.

The reasons is like following:

MgByteReader* MgProxyFeatureService::GetCapabilities(CREFSTRING providerName, CREFSTRING connectionString)
{
    MgCommand cmd;
    cmd.ExecuteCommand(m_connProp,                                  // Connection
                       MgCommand::knObject,                         // Return type expected
                       MgFeatureServiceOpId::GetCapabilities_Id,    // Command Code
                       2,                                           // No of arguments
                       Feature_Service,                             // Service Id
                       BUILD_VERSION(1,0,0),                        // Operation version
                       MgCommand::knString, &providerName,          // Argument#1
                       MgCommand::knString, &connectionString,      // Argument#2
                       MgCommand::knNone);                          // End of argument

    SetWarning(cmd.GetWarningObject());

    return (MgByteReader*)cmd.GetReturnValue().val.m_obj;
}

The version is a fixed one: BUILD_VERSION(1,0,0) instead of the requested version.

Attachments (1)

ticket_1443.patch (1.2 KB ) - added by brucedechant 14 years ago.
Changed the fix BUILD_VERSION(1,0,0) to be m_connProp->GetUserInfo()->GetApiVersion()

Download all attachments as: .zip

Change History (4)

comment:1 by brucedechant, 14 years ago

Cc: brucedechant added
Description: modified (diff)

by brucedechant, 14 years ago

Attachment: ticket_1443.patch added

Changed the fix BUILD_VERSION(1,0,0) to be m_connProp->GetUserInfo()->GetApiVersion()

comment:2 by brucedechant, 14 years ago

Milestone: 2.3
Resolution: fixed
Status: newclosed

Original patch submission r5152

Memory leak fix for original patch r5161

comment:3 by brucedechant, 14 years ago

Fixed in sandbox/adsk/2.2gp r5309

Note: See TracTickets for help on using tickets.