Merge pull request #182 from SOFAgh/FixIssue153

Fix Issue #153
FriendsOfCADability · Sep 12, 2024 · 0ff903a · 0ff903a
2 parents c6f6171 + 4890263
commit 0ff903a
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/CADability/ImportStep.cs b/CADability/ImportStep.cs
@@ -4816,11 +4816,19 @@ private void GetContext(Item item)
                                 context.factor = (double)(vc.val as List<Item>)[0].val; // why * 1000 ? definitely wrong for 83855_elp11b.stp
                                 if (it.parameter.TryGetValue("name", out Item name))
                                 {
-                                    if (name.type == Item.ItemType.stringval && name.sval == "METRE") context.factor *= 1000; // added because of "PROBLEM ELE NULLPUNKT.stp"
+                                    //REVIEW: Is *= 1000 really right? Shouldn't it be = 1000?
+                                    if (name.type == Item.ItemType.stringval && name.sval == "METRE") 
+                                        context.factor *= 1000; // added because of "PROBLEM ELE NULLPUNKT.stp"
                                 }
                             }
                         }
                     }
+                    else if (it.parameter.TryGetValue("name", out Item name))
+                    {
+                        //TODO: Add other units
+                        if (name.type == Item.ItemType.keyword && name.sval == "METRE") 
+                            context.factor = 1000; // added because of "issue153.stp"
+                    }
                 }
             }
         }