Opened 5 years ago

Closed 5 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 5 years ago.
Test Case (draft)
OpUpdateFeatures.cpp.patch (1.3 KB) - added by bscott 5 years ago.
2218.patch (2.2 KB) - added by jng 5 years ago.
Revised patch
2218.php (6.4 KB) - added by jng 5 years ago.
PHP test case

Download all attachments as: .zip

Change History (9)

Changed 5 years ago by jng

Attachment: 2218_TestCase.patch added

Test Case (draft)

Changed 5 years ago by bscott

Attachment: OpUpdateFeatures.cpp.patch added

comment:1 Changed 5 years ago by bscott

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 Changed 5 years ago by jng

Resolution: fixed
Status: newclosed

Fixed

2.5: r7370, r7371 and r7372 trunk: r7373

comment:3 Changed 5 years ago by jng

Resolution: fixed
Status: closedreopened

Changed 5 years ago by jng

Attachment: 2218.patch added

Revised patch

Changed 5 years ago by jng

Attachment: 2218.php added

PHP test case

comment:4 Changed 5 years ago by jng

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

comment:5 Changed 5 years ago by jng

Resolution: fixed
Status: reopenedclosed

Fixed trunk (r7452) and 2.5 (r7448)

Note: See TracTickets for help on using tickets.