fix(rhino): mapping constraints on polylines and facewalls (#2769)

removes polyline assignment and constrains facewall to basic wall types
specklesystems · Jul 13, 2023 · 3ef07cb · 3ef07cb
1 parent a9513f6
commit 3ef07cb
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/ConnectorRhino/ConnectorRhino/ConnectorRhinoShared/UI/MappingBindingsRhino.cs b/ConnectorRhino/ConnectorRhino/ConnectorRhinoShared/UI/MappingBindingsRhino.cs
@@ -117,9 +117,9 @@ private List<Schema> GetObjectSchemas(RhinoObject obj)
               result.Add(new RevitDefaultPipeViewModel());
               result.Add(new RevitDefaultDuctViewModel());
             }
-            else if (c.IsPlanar())
+            else if (c.IsPlanar() && !c.IsPolyline())
             {
-              // If the curve is non-linear, but is planar, it can still be a beam.
+              // If the curve is non-linear, but is planar and is not a polyline, it can still be a beam.
               result.Add(new RevitDefaultBeamViewModel());
               result.Add(new RevitBeamViewModel());
             }

diff --git a/DesktopUI2/DesktopUI2/ViewModels/MappingTool/MappingsViewModel.cs b/DesktopUI2/DesktopUI2/ViewModels/MappingTool/MappingsViewModel.cs
@@ -356,7 +356,7 @@ schema is DirectShapeFreeformViewModel
               break;
 
             case RevitFaceWallViewModel o:
-              var faceWallFamilies = AvailableRevitTypes.Where(x => x.category == "Walls").ToList();
+              var faceWallFamilies = AvailableRevitTypes.Where(x => x.family == "Basic Wall").ToList();
               if (!faceWallFamilies.Any() || !AvailableRevitLevels.Any())
                 break;
               var faceWallFamiliesViewModels = faceWallFamilies