Please review patch for ticket #1052

2 messages Options
Embed this post
Permalink
Klain Qin

Please review patch for ticket #1052

Reply Threaded More More options
Print post
Permalink
Hi All,

Please review the patch for ticket #1052: http://trac.osgeo.org/mapguide/ticket/1052

Thanks,
Klain

_______________________________________________
mapguide-internals mailing list
[hidden email]
http://lists.osgeo.org/mailman/listinfo/mapguide-internals
Haris Kurtagic

RE: Please review patch for ticket #1052

Reply Threaded More More options
Print post
Permalink
Hi,

It looks good to me. lot of code for one quick look but..:)

I am not sure how the case when provider doesn't support transaction is
handled ? Didn't saw any special code for that. Is it supposed just to
pass trough exception from provider ?

Haris

-----Original Message-----
From: [hidden email]
[mailto:[hidden email]] On Behalf Of Klain
Qin
Sent: Wednesday, August 05, 2009 7:55 AM
To: MapGuide Internals Mail List
Subject: [mapguide-internals] Please review patch for ticket #1052

Hi All,

Please review the patch for ticket #1052:
http://trac.osgeo.org/mapguide/ticket/1052

Thanks,
Klain

_______________________________________________
mapguide-internals mailing list
[hidden email]
http://lists.osgeo.org/mailman/listinfo/mapguide-internals
_______________________________________________
mapguide-internals mailing list
[hidden email]
http://lists.osgeo.org/mailman/listinfo/mapguide-internals