Commit 2dd7fd87 authored by Tracie Sucharski's avatar Tracie Sucharski
Browse files

PROG: Fixed documentation in header and isExecutable method.

parent 0bb02c82

isis/src/qisis/objs/Directory/ExportControlNetWorkOrder.cpp

+1 −1
Original line number Diff line number Diff line
@@ -48,6 +48,7 @@ namespace Isis { 
    m_isSynchronous = false;

    m_isUndoable = false;

    QAction::setText(tr("&Export Control Network..."));

    QUndoCommand::setText(tr("Export Control Network..."));

  }
@@ -97,7 +98,6 @@ namespace Isis { 
   * true indicates that there is one control list in the project.

   */

  bool ExportControlNetWorkOrder::isExecutable(ControlList *controls) {

    // TODO: This shouldn't be executable (in the menu) if there are no imported control networks?



    if (controls) {

      return (controls->count() == 1);

isis/src/qisis/objs/Directory/ExportControlNetWorkOrder.h

+3 −1
Original line number Diff line number Diff line
@@ -26,7 +26,9 @@ namespace Isis { 
   *                           it no longer causes a segmentation fault when the user attempts

   *                           to export a control network from the file menu.  Fixes #4760.

   *   @history 2017-11-02 Tyler Wilson - Added a null pointer check on the controls variable in

   *                           isExecutable to prevent potential seg faults.  References #4492.

   *                           isExecutable to prevent potential seg faults.  References #4760.

   *   @history 2018-03-13 Tracie Sucharski - Added Undo text to prevent runtime warning. Also

   *                           correct redmine ticket number in previous history entry.

   */

  class ExportControlNetWorkOrder : public WorkOrder {

      Q_OBJECT