add default implementation for TypeHandler.CanBeVisualized

NeVeSpl · NeVeSpl · commit 013981eaa50d · 2024-05-10T20:26:33.000+02:00
diff --git a/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/Base/TypeHandler.cs b/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/Base/TypeHandler.cs
@@ -47,7 +47,10 @@ IEnumerable<SnoopableObject> ISnoop<T>.Snoop(SnoopableContext context, T value)
         {
             return Snooop(context, value) ?? Enumerable.Empty<SnoopableObject>();
         }
-        protected virtual IEnumerable<SnoopableObject> Snooop(SnoopableContext context, T value) => null;
+        protected virtual IEnumerable<SnoopableObject> Snooop(SnoopableContext context, T value)
+        {
+            yield return new SnoopableObject(context.Document, value);
+        }
 
 
 
diff --git a/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/BoundarySegmentHandler.cs b/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/BoundarySegmentHandler.cs
@@ -13,12 +13,6 @@ internal class BoundarySegmentHandler : TypeHandler<BoundarySegment>
         protected override string ToLabel(SnoopableContext context, BoundarySegment boundarySegment)
         {
             return $"ID: {boundarySegment.ElementId}, {boundarySegment.GetCurve()?.Length} ft"; ;
-        }
-
-        [CodeToString]
-        protected override IEnumerable<SnoopableObject> Snooop(SnoopableContext context, BoundarySegment boundarySegment)
-        {
-            yield return new SnoopableObject(context.Document, boundarySegment);
-        }
+        }        
     }
 }
diff --git a/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/BoundingBoxXYZHandler.cs b/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/BoundingBoxXYZHandler.cs
@@ -17,11 +17,6 @@ protected override string ToLabel(SnoopableContext context, BoundingBoxXYZ box)
             return $"Min({box.Min.X:0.##}, {box.Min.Y:0.##}, {box.Min.Z:0.##}), Max({box.Max.X:0.##}, {box.Max.Y:0.##}, {box.Max.Z:0.##})";
         }
 
-        [CodeToString]
-        protected override IEnumerable<SnoopableObject> Snooop(SnoopableContext context, BoundingBoxXYZ box)
-        {
-            yield return new SnoopableObject(context.Document, box);
-        }
 
         public string GetToolTip(SnoopableContext context, BoundingBoxXYZ value)
         {
diff --git a/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/CategoryHandler.cs b/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/CategoryHandler.cs
@@ -13,12 +13,6 @@ internal sealed class CategoryHandler : TypeHandler<Category>
         protected override string ToLabel(SnoopableContext context, Category category)
         {
             return Labeler.GetLabelForObjectWithId(category.Name, category.Id.Value());
-        }
-
-        [CodeToString]
-        protected override IEnumerable<SnoopableObject> Snooop(SnoopableContext context, Category category)
-        {
-            yield return new SnoopableObject(context.Document, category);
-        }
+        }       
     }
 }
diff --git a/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/CurveHandler.cs b/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/CurveHandler.cs
@@ -14,11 +14,7 @@ internal sealed class CurveHandler : TypeHandler<Curve>
         protected override bool CanBeSnoooped(SnoopableContext context, Curve curve) => true;
         protected override string ToLabel(SnoopableContext context, Curve curve) => curve.GetType()?.GetCSharpName();
 
-        [CodeToString]
-        protected override IEnumerable<SnoopableObject> Snooop(SnoopableContext context, Curve curve)
-        {           
-            yield return new SnoopableObject(context.Document, curve);
-        }
+       
 
         private readonly static Color StartColor = new Color(0, 255, 0);
         private readonly static Color EndColor = new Color(255, 0, 0);
diff --git a/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/EdgeHandler.cs b/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/EdgeHandler.cs
@@ -14,12 +14,7 @@ internal sealed class EdgeHandler : TypeHandler<Edge>
         protected override bool CanBeSnoooped(SnoopableContext context, Edge edge) => true;
         protected override string ToLabel(SnoopableContext context, Edge edge) => edge.GetType()?.GetCSharpName();
 
-        [CodeToString]
-        protected override IEnumerable<SnoopableObject> Snooop(SnoopableContext context, Edge edge)
-        {
-            yield return new SnoopableObject(context.Document, edge);
-        }
-
+   
 
         private readonly static Color StartColor = new Color(0, 255, 0);
         private readonly static Color EndColor = new Color(255, 0, 0);
diff --git a/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/FaceHandler.cs b/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/FaceHandler.cs
@@ -14,11 +14,7 @@ internal sealed class FaceHandler : TypeHandler<Face>
         protected override bool CanBeSnoooped(SnoopableContext context, Face face) => true;
         protected override string ToLabel(SnoopableContext context, Face face) => face.GetType()?.GetCSharpName();
 
-        protected override IEnumerable<SnoopableObject> Snooop(SnoopableContext context, Face face)
-        {
-            yield return new SnoopableObject(context.Document, face);
-        }
-
+        
 
         private readonly static Color FaceColor = new Color(80, 175, 228);
 
diff --git a/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/LocationHandler.cs b/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/LocationHandler.cs
@@ -9,7 +9,7 @@ namespace RevitDBExplorer.Domain.DataModel.ValueContainers
 {
     internal class LocationHandler : TypeHandler<Location>
     {
-        protected override bool CanBeSnoooped(SnoopableContext context, Location location) => location is not null;
+        protected override bool CanBeSnoooped(SnoopableContext context, Location location) => true;
 
         protected override string ToLabel(SnoopableContext context, Location location)
         {
@@ -34,11 +34,6 @@ protected override string ToLabel(SnoopableContext context, Location location)
 
 
             return $"{typeName} : {details}";
-        }
-
-        protected override IEnumerable<SnoopableObject> Snooop(SnoopableContext context, Location location)
-        {
-            yield return new SnoopableObject(context.Document, location);
-        }
+        }        
     }
 }
diff --git a/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/ObjectHandler.cs b/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/ObjectHandler.cs
@@ -8,7 +8,7 @@ namespace RevitDBExplorer.Domain.DataModel.ValueContainers
 {
     internal sealed class ObjectHandler : TypeHandler<object>
     {
-        protected override bool CanBeSnoooped(SnoopableContext context, object @object) => @object is not null;
+        protected override bool CanBeSnoooped(SnoopableContext context, object @object) => true;
         protected override string ToLabel(SnoopableContext context, object @object)
         {
             string name = @object.TryGetPropertyValue(propertyThatContainsName);
@@ -20,10 +20,7 @@ protected override string ToLabel(SnoopableContext context, object @object)
             }
             return $"{typeName}";
         }
-        protected override IEnumerable<SnoopableObject> Snooop(SnoopableContext context, object @object)
-        {
-            yield return new SnoopableObject(context.Document, @object);
-        }
+       
 
 
         private static readonly string[] propertyThatContainsName = new[]  { "Name", "Title", "SchemaName", "FieldName" };       
diff --git a/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/PointHandler.cs b/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/PointHandler.cs
@@ -10,12 +10,9 @@ namespace RevitDBExplorer.Domain.DataModel.ValueContainers
 {
     class PointHandler : TypeHandler<Point>
     {
-        protected override bool CanBeSnoooped(SnoopableContext context, Point point) => point is not null;
+        protected override bool CanBeSnoooped(SnoopableContext context, Point point) => true;
         protected override string ToLabel(SnoopableContext context, Point point) => point.GetType()?.GetCSharpName();
-        protected override IEnumerable<SnoopableObject> Snooop(SnoopableContext context, Point point)
-        {
-            yield return new SnoopableObject(context.Document, point);
-        }
+      
 
 
      
diff --git a/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/SolidHandler.cs b/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/SolidHandler.cs
@@ -10,12 +10,9 @@ namespace RevitDBExplorer.Domain.DataModel.ValueContainers
 {
     internal sealed class SolidHandler : TypeHandler<Solid>
     {
-        protected override bool CanBeSnoooped(SnoopableContext context, Solid solid) => solid is not null;
+        protected override bool CanBeSnoooped(SnoopableContext context, Solid solid) => true;
         protected override string ToLabel(SnoopableContext context, Solid solid) => solid.GetType()?.GetCSharpName();
-        protected override IEnumerable<SnoopableObject> Snooop(SnoopableContext context, Solid solid)
-        {
-            yield return new SnoopableObject(context.Document, solid);
-        }
+     
 
 
 
diff --git a/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/StructuralSectionHandler.cs b/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/StructuralSectionHandler.cs
@@ -18,9 +18,5 @@ protected override string ToLabel(SnoopableContext context, StructuralSection va
             return value.ToString();
         }
 
-        protected override IEnumerable<SnoopableObject> Snooop(SnoopableContext context, StructuralSection value)
-        {
-            yield return new SnoopableObject(context.Document, value);
-        }
     }
 }
diff --git a/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/TransformHandler.cs b/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/TransformHandler.cs
@@ -25,9 +25,6 @@ protected override string ToLabel(SnoopableContext context, Transform transform)
             return $"Transform: {id}";
         }
 
-        protected override IEnumerable<SnoopableObject> Snooop(SnoopableContext context, Transform transform)
-        {
-            yield return new SnoopableObject(context.Document, transform);
-        }
+       
     }
 }
diff --git a/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/XYZHandler.cs b/sources/RevitDBExplorer/Domain/DataModel/ValueContainers/XYZHandler.cs
@@ -10,7 +10,7 @@ namespace RevitDBExplorer.Domain.DataModel.ValueContainers
 {
     internal sealed class XYZHandler : TypeHandler<XYZ>, IHaveToolTip<XYZ>
     {
-        protected override bool CanBeSnoooped(SnoopableContext context, XYZ xyz) => false;
+        protected override bool CanBeSnoooped(SnoopableContext context, XYZ xyz) => true;
         protected override string ToLabel(SnoopableContext context, XYZ xyz)
         {
             return $"({xyz.X}, {xyz.Y}, {xyz.Z})";

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,10 @@ IEnumerable<SnoopableObject> ISnoop<T>.Snoop(SnoopableContext context, T value)`
`47`	`47`	`{`
`48`	`48`	`return Snooop(context, value) ?? Enumerable.Empty<SnoopableObject>();`
`49`	`49`	`}`
`50`		`- protected virtual IEnumerable<SnoopableObject> Snooop(SnoopableContext context, T value) => null;`
	`50`	`+ protected virtual IEnumerable<SnoopableObject> Snooop(SnoopableContext context, T value)`
	`51`	`+ {`
	`52`	`+ yield return new SnoopableObject(context.Document, value);`
	`53`	`+ }`
`51`	`54`
`52`	`55`
`53`	`56`
Original file line number	Diff line number	Diff line change
`@@ -13,12 +13,6 @@ internal class BoundarySegmentHandler : TypeHandler<BoundarySegment>`
`13`	`13`	`protected override string ToLabel(SnoopableContext context, BoundarySegment boundarySegment)`
`14`	`14`	`{`
`15`	`15`	`return $"ID: {boundarySegment.ElementId}, {boundarySegment.GetCurve()?.Length} ft"; ;`
`16`		`- }`
`17`		`-`
`18`		`- [CodeToString]`
`19`		`- protected override IEnumerable<SnoopableObject> Snooop(SnoopableContext context, BoundarySegment boundarySegment)`
`20`		`- {`
`21`		`- yield return new SnoopableObject(context.Document, boundarySegment);`
`22`		`- }`
	`16`	`+ }`
`23`	`17`	`}`
`24`	`18`	`}`
Original file line number	Diff line number	Diff line change
`@@ -17,11 +17,6 @@ protected override string ToLabel(SnoopableContext context, BoundingBoxXYZ box)`
`17`	`17`	`return $"Min({box.Min.X:0.##}, {box.Min.Y:0.##}, {box.Min.Z:0.##}), Max({box.Max.X:0.##}, {box.Max.Y:0.##}, {box.Max.Z:0.##})";`
`18`	`18`	`}`
`19`	`19`
`20`		`- [CodeToString]`
`21`		`- protected override IEnumerable<SnoopableObject> Snooop(SnoopableContext context, BoundingBoxXYZ box)`
`22`		`- {`
`23`		`- yield return new SnoopableObject(context.Document, box);`
`24`		`- }`
`25`	`20`
`26`	`21`	`public string GetToolTip(SnoopableContext context, BoundingBoxXYZ value)`
`27`	`22`	`{`
Original file line number	Diff line number	Diff line change
`@@ -13,12 +13,6 @@ internal sealed class CategoryHandler : TypeHandler<Category>`
`13`	`13`	`protected override string ToLabel(SnoopableContext context, Category category)`
`14`	`14`	`{`
`15`	`15`	`return Labeler.GetLabelForObjectWithId(category.Name, category.Id.Value());`
`16`		`- }`
`17`		`-`
`18`		`- [CodeToString]`
`19`		`- protected override IEnumerable<SnoopableObject> Snooop(SnoopableContext context, Category category)`
`20`		`- {`
`21`		`- yield return new SnoopableObject(context.Document, category);`
`22`		`- }`
	`16`	`+ }`
`23`	`17`	`}`
`24`	`18`	`}`
Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@ namespace RevitDBExplorer.Domain.DataModel.ValueContainers`
`9`	`9`	`{`
`10`	`10`	`internal class LocationHandler : TypeHandler<Location>`
`11`	`11`	`{`
`12`		`- protected override bool CanBeSnoooped(SnoopableContext context, Location location) => location is not null;`
	`12`	`+ protected override bool CanBeSnoooped(SnoopableContext context, Location location) => true;`
`13`	`13`
`14`	`14`	`protected override string ToLabel(SnoopableContext context, Location location)`
`15`	`15`	`{`
`@@ -34,11 +34,6 @@ protected override string ToLabel(SnoopableContext context, Location location)`
`34`	`34`
`35`	`35`
`36`	`36`	`return $"{typeName} : {details}";`
`37`		`- }`
`38`		`-`
`39`		`- protected override IEnumerable<SnoopableObject> Snooop(SnoopableContext context, Location location)`
`40`		`- {`
`41`		`- yield return new SnoopableObject(context.Document, location);`
`42`		`- }`
	`37`	`+ }`
`43`	`38`	`}`
`44`	`39`	`}`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@ namespace RevitDBExplorer.Domain.DataModel.ValueContainers`
`8`	`8`	`{`
`9`	`9`	`internal sealed class ObjectHandler : TypeHandler<object>`
`10`	`10`	`{`
`11`		`- protected override bool CanBeSnoooped(SnoopableContext context, object @object) => @object is not null;`
	`11`	`+ protected override bool CanBeSnoooped(SnoopableContext context, object @object) => true;`
`12`	`12`	`protected override string ToLabel(SnoopableContext context, object @object)`
`13`	`13`	`{`
`14`	`14`	`string name = @object.TryGetPropertyValue(propertyThatContainsName);`
`@@ -20,10 +20,7 @@ protected override string ToLabel(SnoopableContext context, object @object)`
`20`	`20`	`}`
`21`	`21`	`return $"{typeName}";`
`22`	`22`	`}`
`23`		`- protected override IEnumerable<SnoopableObject> Snooop(SnoopableContext context, object @object)`
`24`		`- {`
`25`		`- yield return new SnoopableObject(context.Document, @object);`
`26`		`- }`
	`23`	`+`
`27`	`24`
`28`	`25`
`29`	`26`	`private static readonly string[] propertyThatContainsName = new[] { "Name", "Title", "SchemaName", "FieldName" };`
Original file line number	Diff line number	Diff line change
`@@ -10,12 +10,9 @@ namespace RevitDBExplorer.Domain.DataModel.ValueContainers`
`10`	`10`	`{`
`11`	`11`	`class PointHandler : TypeHandler<Point>`
`12`	`12`	`{`
`13`		`- protected override bool CanBeSnoooped(SnoopableContext context, Point point) => point is not null;`
	`13`	`+ protected override bool CanBeSnoooped(SnoopableContext context, Point point) => true;`
`14`	`14`	`protected override string ToLabel(SnoopableContext context, Point point) => point.GetType()?.GetCSharpName();`
`15`		`- protected override IEnumerable<SnoopableObject> Snooop(SnoopableContext context, Point point)`
`16`		`- {`
`17`		`- yield return new SnoopableObject(context.Document, point);`
`18`		`- }`
	`15`	`+`
`19`	`16`
`20`	`17`
`21`	`18`