Commit 86d484e5 authored by acpaquette's avatar acpaquette Committed by Jesse Mapel
Browse files

Updated FindFeatures Logging (#4399) 

* Fixed logging for findfeatures

* Corrected log write check in findfeatures application

* Updated the change log
parent e0c63ecc

CHANGELOG.md

+1 −0
Original line number Diff line number Diff line
@@ -61,6 +61,7 @@ release. 
- Fixed hardcoded file naming in the hijitter app dealing with output from pipeline. [#4372](https://github.com/USGS-Astrogeology/ISIS3/pull/4372)

- Fixed "About Qview" to point to website documentation. [4333](https://github.com/USGS-Astrogeology/ISIS3/issues/4333)

- Fixed bug where the time bias was not being added to the ephemeris times in ckwriter. [4129](https://github.com/USGS-Astrogeology/ISIS3/issues/4129)

- Fixed logging in FindFeatures where we were trying to get a non-existent Pvl group from the Pvl log. [#4375](https://github.com/USGS-Astrogeology/ISIS3/issues/4375)



### Changed

isis/src/control/apps/findfeatures/findfeatures.cpp

+1 −2
Original line number Diff line number Diff line
@@ -81,8 +81,7 @@ namespace Isis{ 
        }

        else {

          if (log){

            PvlGroup results = data.findGroup("Results");

            log->addGroup(results);

            log->addObject(data);

          }

        }

      }

isis/src/control/apps/findfeatures/main.cpp

+3 −4
Original line number Diff line number Diff line
@@ -33,10 +33,9 @@ void IsisMain() { 
    Application::Log(*grpIt);

  }



  PvlGroup results = appLog.findGroup("Results");

  if( ui.WasEntered("TO") && ui.IsInteractive() ) {

  Pvl results;

  if( (ui.GetBoolean("LISTALL") || ui.GetBoolean("LISTSPEC")) && ui.IsInteractive() ) {

    results.addObject(appLog.findObject("FeatureAlgorithms"));

    Application::GuiLog(results);

  }



  SessionLog::TheLog().AddResults(results);

}