Opened 12 years ago

Closed 12 years ago

#2218 closed defect (fixed)

UpdateFeatures throws wrongly intialyzed MgException

Reported by: bscott Owned by:
Priority: high Milestone: 2.5
Component: Feature Service Version: 2.4.0
Severity: blocker Keywords:
Cc: External ID:

Description

When using UpdateFeatures, If the database returns an error. an MgException stays in memory and further call to the service throws wrong exception

Attachments (4)

2218_TestCase.patch​ (7.5 KB ) - added by jng 12 years ago.
Test Case (draft)
OpUpdateFeatures.cpp.patch​ (1.3 KB ) - added by bscott 12 years ago.
2218.patch​ (2.2 KB ) - added by jng 12 years ago.
Revised patch
2218.php​ (6.4 KB ) - added by jng 12 years ago.
PHP test case

Download all attachments as: .zip

Change History (9)

by jng, 12 years ago

Attachment: 2218_TestCase.patch​ added

Test Case (draft)

by bscott, 12 years ago

Attachment: OpUpdateFeatures.cpp.patch​ added

comment:1 by bscott, 12 years ago

I think i have found it this time. With UpdateFeatures in Non-Transactional mode the response was written twice. And the second one with m_opCompleted flag set to true. this was messing up the operation handler.

Patch is updated, someone could check that piece of Code. Walt?

comment:2 by jng, 12 years ago

Resolution: → fixed
Status: new → closed

Fixed

2.5: r7370, r7371 and r7372 trunk: r7373

comment:3 by jng, 12 years ago

Resolution: fixed
Status: closed → reopened

by jng, 12 years ago

Attachment: 2218.patch​ added

Revised patch

by jng, 12 years ago

Attachment: 2218.php​ added

PHP test case

comment:4 by jng, 12 years ago

Note that the PHP test case has one test disabled due to #2252

comment:5 by jng, 12 years ago

Resolution: → fixed
Status: reopened → closed

Fixed trunk (r7452) and 2.5 (r7448)

Note: See TracTickets for help on using tickets.