From 809a2e13f81f23a9158768bafef53a860c599445 Mon Sep 17 00:00:00 2001 From: gka Date: Thu, 1 Oct 2020 12:37:34 +0300 Subject: [PATCH] 0031812: IGES import - NURBS patches lost in the import process - 720 Correction for case invalid color definition to avoid considering an entity that has a number corresponding to the color field as erroneous if entity is not a color entity. --- src/IGESData/IGESData_IGESReaderTool.cxx | 4 +++- tests/bugs/iges/bug31812 | 13 +++++++++++++ 2 files changed, 16 insertions(+), 1 deletion(-) create mode 100644 tests/bugs/iges/bug31812 diff --git a/src/IGESData/IGESData_IGESReaderTool.cxx b/src/IGESData/IGESData_IGESReaderTool.cxx index b5769bfb84..a1792d5d12 100644 --- a/src/IGESData/IGESData_IGESReaderTool.cxx +++ b/src/IGESData/IGESData_IGESReaderTool.cxx @@ -290,13 +290,15 @@ IGESData_IGESReaderTool::IGESData_IGESReaderTool if (v[14] < 0) { fieldent = GetCasted(IGESData_IGESEntity,IR->BoundEntity( (1-v[14])/2 )); Color = GetCasted(IGESData_ColorEntity, fieldent); + //an entity that is not a color entity and has a number corresponding + //to the color field should not be considered erroneous if (Color.IsNull()) { // Sending of message : Incorrect Color Number Message_Msg Msg34 ("XSTEP_34"); Msg34.Arg(thecnum); Msg34.Arg(thectyp.Type()); ach->SendWarning(Msg34); - ent->InitDirFieldEntity(13,fieldent); + ent->InitDirFieldEntity(13,Color); } else ent->InitColor(Color); } diff --git a/tests/bugs/iges/bug31812 b/tests/bugs/iges/bug31812 new file mode 100644 index 0000000000..274550352b --- /dev/null +++ b/tests/bugs/iges/bug31812 @@ -0,0 +1,13 @@ +puts "============" +puts "OCC31812" +puts "============" +puts "" +######################################################################## +# Problem of reading IGES files +######################################################################## + +igesread [locate_data_file OCC31812.igs] a * + +checkshape a +checknbshapes a -face 4 +checkview -display a -2d -path ${imagedir}/${test_image}.png