#11:

 - Add extra namespace filters to SHFB project
 - Fix a whole lot of documentation warnings as reported by SHFB

Doc/MaestroAPI.shfbproj

@@ -69,6 +69,7 @@
       <Filter entryType="Namespace" fullName="OSGeo.MapGuide.ObjectModels.MapDefinition.v2_4_0" isExposed="False" />
       <Filter entryType="Namespace" fullName="OSGeo.MapGuide.ObjectModels.PrintLayout.v1_0_0" isExposed="False" />
       <Filter entryType="Namespace" fullName="OSGeo.MapGuide.ObjectModels.RuntimeMap.v2_6_0" isExposed="False" />
+      <Filter entryType="Namespace" fullName="OSGeo.MapGuide.ObjectModels.RuntimeMap.v3_0_0" isExposed="False" />
       <Filter entryType="Namespace" fullName="OSGeo.MapGuide.ObjectModels.SymbolDefinition.v1_0_0" isExposed="False" />
       <Filter entryType="Namespace" fullName="OSGeo.MapGuide.ObjectModels.SymbolDefinition.v1_1_0" isExposed="False" />
       <Filter entryType="Namespace" fullName="OSGeo.MapGuide.ObjectModels.SymbolDefinition.v2_4_0" isExposed="False" />
@@ -79,6 +80,7 @@
       <Filter entryType="Namespace" fullName="OSGeo.MapGuide.ObjectModels.WebLayout.v1_1_0" isExposed="False" />
       <Filter entryType="Namespace" fullName="OSGeo.MapGuide.ObjectModels.WebLayout.v2_4_0" isExposed="False" />
       <Filter entryType="Namespace" fullName="OSGeo.MapGuide.ObjectModels.WebLayout.v2_6_0" isExposed="False" />
+      <Filter entryType="Namespace" fullName="OSGeo.MapGuide.ObjectModels.TileSetDefinition.v3_0_0" isExposed="False" />
     </ApiFilter>
     <SdkLinkType>None</SdkLinkType>
     <FeedbackEMailAddress>jumpinjackie%40gmail.com</FeedbackEMailAddress>

Maestro.Base/Editor/DrawingSourceEditor.cs

@@ -37,6 +37,9 @@ namespace Maestro.Base.Editor
     /// </remarks>
     public partial class DrawingSourceEditor : EditorContentBase
     {
+        /// <summary>
+        /// Constructor
+        /// </summary>
         public DrawingSourceEditor()
         {
             InitializeComponent();
@@ -46,6 +49,16 @@ public DrawingSourceEditor()
 
         private bool _init = false;
 
+        /// <summary>
+        /// Binds the specified resource to this control. This effectively initializes
+        /// all the fields in this control and sets up databinding on all fields. All
+        /// subclasses *must* override this method.
+        ///
+        /// Also note that this method may be called more than once (e.g. Returning from
+        /// and XML edit of this resource). Thus subclasses must take this scenario into
+        /// account when implementing
+        /// </summary>
+        /// <param name="service">The editor service</param>
         protected override void Bind(IEditorService service)
         {
             if (!_init)

Maestro.Base/Editor/EditorContentBase.cs

@@ -132,7 +132,7 @@ private void OnSessionIdChanged(object sender, EventArgs e)
         /// <summary>
         /// Gets the XML content of the edited resource
         /// </summary>
-        /// <returns></returns>
+        /// <returns>The XML content</returns>
         public virtual string GetXmlContent()
         {
             using (var sr = new System.IO.StreamReader(ObjectFactory.Serialize(this.Resource)))
@@ -288,7 +288,7 @@ private void OnDirtyStateChanged(object sender, EventArgs e)
         /// and XML edit of this resource). Thus subclasses must take this scenario into
         /// account when implementing
         /// </summary>
-        /// <param name="service"></param>
+        /// <param name="service">The editor service</param>
         protected virtual void Bind(IEditorService service)
         {
             throw new NotImplementedException();
@@ -342,7 +342,7 @@ public bool IsNew
         }
 
         /// <summary>
-        /// Gets whether this resource can be previewed
+        /// Gets whether this resource can be profiled
         /// </summary>
         public virtual bool CanProfile
         {

Maestro.Base/Editor/FeatureSourceEditor.cs

@@ -43,6 +43,9 @@ namespace Maestro.Base.Editor
     /// </remarks>
     public partial class FeatureSourceEditor : EditorContentBase
     {
+        /// <summary>
+        /// Constructor
+        /// </summary>
         public FeatureSourceEditor()
         {
             InitializeComponent();
@@ -52,6 +55,16 @@ public FeatureSourceEditor()
         private IEditorService _edsvc;
         private bool _init = false;
 
+        /// <summary>
+        /// Binds the specified resource to this control. This effectively initializes
+        /// all the fields in this control and sets up databinding on all fields. All
+        /// subclasses *must* override this method.
+        ///
+        /// Also note that this method may be called more than once (e.g. Returning from
+        /// and XML edit of this resource). Thus subclasses must take this scenario into
+        /// account when implementing
+        /// </summary>
+        /// <param name="service">The editor service</param>
         protected override void Bind(IEditorService service)
         {
             if (!_init)
@@ -121,6 +134,9 @@ protected override void OnBeforeSave(object sender, CancelEventArgs e)
             base.OnBeforeSave(sender, e);
         }
 
+        /// <summary>
+        /// Gets whether this resource can be profiled
+        /// </summary>
         public override bool CanProfile
         {
             get

Maestro.Base/Editor/FusionEditor.cs

@@ -38,6 +38,9 @@ namespace Maestro.Base.Editor
     /// </remarks>
     public partial class FusionEditor : EditorContentBase
     {
+        /// <summary>
+        /// Constructor
+        /// </summary>
         public FusionEditor()
         {
             InitializeComponent();
@@ -47,6 +50,16 @@ public FusionEditor()
         private IEditorService _edsvc;
         private bool _init = false;
 
+        /// <summary>
+        /// Binds the specified resource to this control. This effectively initializes
+        /// all the fields in this control and sets up databinding on all fields. All
+        /// subclasses *must* override this method.
+        ///
+        /// Also note that this method may be called more than once (e.g. Returning from
+        /// and XML edit of this resource). Thus subclasses must take this scenario into
+        /// account when implementing
+        /// </summary>
+        /// <param name="service">The editor service</param>
         protected override void Bind(IEditorService service)
         {
             if (!_init)

Maestro.Base/Editor/IEditorFactory.cs

@@ -42,7 +42,7 @@ public interface IEditorFactory
         /// <summary>
         /// Creates an instance of this resource editor
         /// </summary>
-        /// <returns></returns>
+        /// <returns>The editor instance</returns>
         IEditorViewContent Create();
     }
 

Maestro.Base/Editor/IEditorViewContent.cs

@@ -57,7 +57,7 @@ public interface IEditorViewContent : IViewContent
         /// <summary>
         /// Gets the XML content of the edited resource
         /// </summary>
-        /// <returns></returns>
+        /// <returns>The XML content</returns>
         string GetXmlContent();
 
         /// <summary>

Maestro.Base/Editor/LayerDefinitionEditor.cs

@@ -45,6 +45,9 @@ namespace Maestro.Base.Editor
     /// </remarks>
     public partial class LayerDefinitionEditor : EditorContentBase
     {
+        /// <summary>
+        /// Constructor
+        /// </summary>
         public LayerDefinitionEditor()
         {
             InitializeComponent();
@@ -54,6 +57,16 @@ public LayerDefinitionEditor()
         private IEditorService _edsvc;
         private bool _init = false;
 
+        /// <summary>
+        /// Binds the specified resource to this control. This effectively initializes
+        /// all the fields in this control and sets up databinding on all fields. All
+        /// subclasses *must* override this method.
+        ///
+        /// Also note that this method may be called more than once (e.g. Returning from
+        /// and XML edit of this resource). Thus subclasses must take this scenario into
+        /// account when implementing
+        /// </summary>
+        /// <param name="service">The editor service</param>
         protected override void Bind(Maestro.Editors.IEditorService service)
         {
             if (!_init)
@@ -141,6 +154,9 @@ protected override void OnBeforeSave(object sender, CancelEventArgs e)
             base.OnBeforeSave(sender, e);
         }
 
+        /// <summary>
+        /// Gets whether this resource can be profiled
+        /// </summary>
         public override bool CanProfile
         {
             get

Maestro.Base/Editor/LoadProcedureEditor.cs

@@ -37,11 +37,24 @@ namespace Maestro.Base.Editor
     /// </remarks>
     public partial class LoadProcedureEditor : EditorContentBase
     {
+        /// <summary>
+        /// Constructor
+        /// </summary>
         public LoadProcedureEditor()
         {
             InitializeComponent();
         }
 
+        /// <summary>
+        /// Binds the specified resource to this control. This effectively initializes
+        /// all the fields in this control and sets up databinding on all fields. All
+        /// subclasses *must* override this method.
+        ///
+        /// Also note that this method may be called more than once (e.g. Returning from
+        /// and XML edit of this resource). Thus subclasses must take this scenario into
+        /// account when implementing
+        /// </summary>
+        /// <param name="service">The editor service</param>
         protected override void Bind(Maestro.Editors.IEditorService service)
         {
             panelBody.Controls.Clear();

Maestro.Base/Editor/MapDefinitionEditor.cs

@@ -42,6 +42,9 @@ namespace Maestro.Base.Editor
     /// </remarks>
     public partial class MapDefinitionEditor : EditorContentBase
     {
+        /// <summary>
+        /// Constructor
+        /// </summary>
         public MapDefinitionEditor()
         {
             InitializeComponent();
@@ -51,6 +54,16 @@ public MapDefinitionEditor()
         private IEditorService _edsvc;
         private bool _init = false;
 
+        /// <summary>
+        /// Binds the specified resource to this control. This effectively initializes
+        /// all the fields in this control and sets up databinding on all fields. All
+        /// subclasses *must* override this method.
+        ///
+        /// Also note that this method may be called more than once (e.g. Returning from
+        /// and XML edit of this resource). Thus subclasses must take this scenario into
+        /// account when implementing
+        /// </summary>
+        /// <param name="service">The editor service</param>
         protected override void Bind(IEditorService service)
         {
             if (!_init)
@@ -106,6 +119,9 @@ protected override void OnBeforeSave(object sender, CancelEventArgs e)
             base.OnBeforeSave(sender, e);
         }
 
+        /// <summary>
+        /// Gets whether this resource can be profiled
+        /// </summary>
         public override bool CanProfile
         {
             get

Maestro.Base/Editor/PrintLayoutEditor.cs

@@ -36,11 +36,24 @@ namespace Maestro.Base.Editor
     /// </remarks>
     public partial class PrintLayoutEditor : EditorContentBase
     {
+        /// <summary>
+        /// Constructor
+        /// </summary>
         public PrintLayoutEditor()
         {
             InitializeComponent();
         }
 
+        /// <summary>
+        /// Binds the specified resource to this control. This effectively initializes
+        /// all the fields in this control and sets up databinding on all fields. All
+        /// subclasses *must* override this method.
+        ///
+        /// Also note that this method may be called more than once (e.g. Returning from
+        /// and XML edit of this resource). Thus subclasses must take this scenario into
+        /// account when implementing
+        /// </summary>
+        /// <param name="service">The editor service</param>
         protected override void Bind(Maestro.Editors.IEditorService service)
         {
             panelBody.Controls.Clear();

Maestro.Base/Editor/SymbolDefinitionEditor.cs

@@ -37,11 +37,24 @@ namespace Maestro.Base.Editor
     /// </remarks>
     public partial class SymbolDefinitionEditor : EditorContentBase
     {
+        /// <summary>
+        /// Constructor
+        /// </summary>
         public SymbolDefinitionEditor()
         {
             InitializeComponent();
         }
 
+        /// <summary>
+        /// Binds the specified resource to this control. This effectively initializes
+        /// all the fields in this control and sets up databinding on all fields. All
+        /// subclasses *must* override this method.
+        ///
+        /// Also note that this method may be called more than once (e.g. Returning from
+        /// and XML edit of this resource). Thus subclasses must take this scenario into
+        /// account when implementing
+        /// </summary>
+        /// <param name="service">The editor service</param>
         protected override void Bind(Maestro.Editors.IEditorService service)
         {
             panelBody.Controls.Clear();

Maestro.Base/Editor/TileSetEditor.cs

@@ -44,6 +44,16 @@ public TileSetEditor()
             InitializeComponent();
         }
 
+        /// <summary>
+        /// Binds the specified resource to this control. This effectively initializes
+        /// all the fields in this control and sets up databinding on all fields. All
+        /// subclasses *must* override this method.
+        ///
+        /// Also note that this method may be called more than once (e.g. Returning from
+        /// and XML edit of this resource). Thus subclasses must take this scenario into
+        /// account when implementing
+        /// </summary>
+        /// <param name="service">The editor service</param>
         protected override void Bind(Editors.IEditorService service)
         {
             panelBody.Controls.Clear();

Maestro.Base/Editor/WatermarkDefinitionEditor.cs

@@ -38,6 +38,9 @@ namespace Maestro.Base.Editor
     /// </remarks>
     public partial class WatermarkDefinitionEditor : EditorContentBase
     {
+        /// <summary>
+        /// Constructor
+        /// </summary>
         public WatermarkDefinitionEditor()
         {
             InitializeComponent();
@@ -47,6 +50,16 @@ public WatermarkDefinitionEditor()
         private IEditorService _edsvc;
         private bool _init = false;
 
+        /// <summary>
+        /// Binds the specified resource to this control. This effectively initializes
+        /// all the fields in this control and sets up databinding on all fields. All
+        /// subclasses *must* override this method.
+        ///
+        /// Also note that this method may be called more than once (e.g. Returning from
+        /// and XML edit of this resource). Thus subclasses must take this scenario into
+        /// account when implementing
+        /// </summary>
+        /// <param name="service">The editor service</param>
         protected override void Bind(Maestro.Editors.IEditorService service)
         {
             if (!_init)

Maestro.Base/Editor/WebLayoutEditor.cs

@@ -36,11 +36,24 @@ namespace Maestro.Base.Editor
     /// </remarks>
     public partial class WebLayoutEditor : EditorContentBase
     {
+        /// <summary>
+        /// Constructor
+        /// </summary>
         public WebLayoutEditor()
         {
             InitializeComponent();
         }
 
+        /// <summary>
+        /// Binds the specified resource to this control. This effectively initializes
+        /// all the fields in this control and sets up databinding on all fields. All
+        /// subclasses *must* override this method.
+        ///
+        /// Also note that this method may be called more than once (e.g. Returning from
+        /// and XML edit of this resource). Thus subclasses must take this scenario into
+        /// account when implementing
+        /// </summary>
+        /// <param name="service">The editor service</param>
         protected override void Bind(Maestro.Editors.IEditorService service)
         {
             panelBody.Controls.Clear();

Maestro.Base/Editor/XmlEditor.cs

@@ -48,6 +48,9 @@ namespace Maestro.Base.Editor
     /// </remarks>
     public partial class XmlEditor : EditorContentBase
     {
+        /// <summary>
+        /// Constructor
+        /// </summary>
         public XmlEditor()
         {
             InitializeComponent();
@@ -68,6 +71,16 @@ private void ValidateXml(out string[] errors, out string[] warnings)
 
         private IEditorService _edSvc;
 
+        /// <summary>
+        /// Binds the specified resource to this control. This effectively initializes
+        /// all the fields in this control and sets up databinding on all fields. All
+        /// subclasses *must* override this method.
+        ///
+        /// Also note that this method may be called more than once (e.g. Returning from
+        /// and XML edit of this resource). Thus subclasses must take this scenario into
+        /// account when implementing
+        /// </summary>
+        /// <param name="service">The editor service</param>
         protected override void Bind(IEditorService service)
         {
             //NOTE: This is exempt from #1656 requirements because this will never be called when returing