Unverified Commit a59442f8 authored by Austin Sanders's avatar Austin Sanders Committed by GitHub
Browse files

Remove restrictive validity check in caminfo (#5553) 

* Removed preemptive validity check for polygon

* Removed unused import

* Updated changelog
parent 1a842603

CHANGELOG.md

+1 −0
Original line number Diff line number Diff line
@@ -41,6 +41,7 @@ release. 




### Fixed

- Fixed a bug in PolygonTools in which the program exited before attempting to fix an invalid Polygon [#5520](https://github.com/DOI-USGS/ISIS3/issues/5520)

- Fixed a bug in QVIEW's Stretch tool where the default min/max type was not an available option [#5289](https://github.com/DOI-USGS/ISIS3/issues/5289)

- Fixed a bug in QVIEW where images would double load if loaded from the commandline [#5505](https://github.com/DOI-USGS/ISIS3/pull/5505)

isis/src/base/objs/PolygonTools/PolygonTools.cpp

+1 −2
Original line number Diff line number Diff line
@@ -104,7 +104,7 @@ namespace Isis { 
        geos::geom::Polygon *newPoly = globalFactory->createPolygon(

                                         globalFactory->createLinearRing(*xycoords), std::move(holes)).release();



        if(newPoly->isValid() && !newPoly->isEmpty() && newPoly->getArea() > 1.0e-14) {

        if(!newPoly->isEmpty() && newPoly->getArea() > 1.0e-14) {

          xyPolys->push_back(newPoly);

        }

        else {
@@ -132,7 +132,6 @@ namespace Isis { 
          throw IException(IException::Programmer, msg, _FILEINFO_);

        }

      }



    } // end else

  }