[mapguide-internals] RE: /branches root directory reserved only for MapGuide?

[Dave Wilson]

Hey Jackie.

I guess it makes sense that Maestro is more likely to lag behind any schema changes so the UI may not always support the latest schema version. I guess about the best you can do is determine the max version supported by Maestro and attempt to save stuff against the server using that version: the server should support it. Of course as you said it looks like the code has some issues in order to be able to support different versions.

This really does sound like an issue for another day.

Dave

-----Original Message-----
From: mapguide-internals-bounces at lists.osgeo.org [mailto:mapguide-internals-bounces at lists.osgeo.org] On Behalf Of Jackie Ng
Sent: Monday, May 03, 2010 6:19 PM
To: mapguide-internals at lists.osgeo.org
Subject: [mapguide-internals] RE: /branches root directory reserved only for MapGuide?


Hi Dave,

I'd like to say that Maestro should be able to edit every known xsd version
of a given resource, but that is not really the case as Maestro still
doesn't have editor support for some features introduced in newer xsd
versions (eg. legend visibility control for layer styles, ping server flag
for web layouts). 

This is because all the xml serialization is handled by auto-generated
classes (generated by xsd.exe or some variant). I believe this was generated
once-off against the mapguide xsd schema set at some point in the past, and
since then hodge-podge hacks have been put in place to be able to save in
different versions of the schema. This isn't surprising given that Maestro
was originally a .net 1.1 project and partial classes didn't exist back
then. Doesn't really allow for clean code. Also it's a bit hard to generate
classes (via xsd.exe) against different versions of the xsd without causing
naming collisions in the generated code (or maybe each particular version
should reside in its own namespace).

This part of the Maestro code base has bothered me the most since taking
over. I'm pretty sure there is a cleaner way to do this, but that's a
different topic of discussion.

- Jackie
-- 
View this message in context: http://osgeo-org.1803224.n2.nabble.com/branches-root-directory-reserved-only-for-MapGuide-tp4996834p5000979.html
Sent from the MapGuide Internals mailing list archive at Nabble.com.
_______________________________________________
mapguide-internals mailing list
mapguide-internals at lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/mapguide-internals