From 4a9d744c246c2b9b9e2d9c55107cf21ab3fcb793 Mon Sep 17 00:00:00 2001
From: pixhawk <mavteam@student.ethz.ch>
Date: Tue, 15 Jun 2010 17:05:28 +0200
Subject: [PATCH] changed behaviour of the waypoint read button

---
 src/ui/WaypointList.cc | 37 +++++++++++--------------------------
 src/ui/WaypointList.h  |  1 +
 2 files changed, 12 insertions(+), 26 deletions(-)

diff --git a/src/ui/WaypointList.cc b/src/ui/WaypointList.cc
index 9512b77aa..5b608f028 100644
--- a/src/ui/WaypointList.cc
+++ b/src/ui/WaypointList.cc
@@ -66,7 +66,7 @@ WaypointList::WaypointList(QWidget *parent, UASInterface* uas) :
     connect(m_ui->transmitButton, SIGNAL(clicked()), this, SLOT(transmit()));
 
     // REQUEST WAYPOINTS
-    connect(m_ui->readButton, SIGNAL(clicked()), this, SIGNAL(requestWaypoints()));
+    connect(m_ui->readButton, SIGNAL(clicked()), this, SLOT(read()));
 
     // SAVE/LOAD WAYPOINTS
     connect(m_ui->saveButton, SIGNAL(clicked()), this, SLOT(saveWaypoints()));
@@ -105,11 +105,6 @@ void WaypointList::setUAS(UASInterface* uas)
         connect(this, SIGNAL(clearWaypointList()),                          &uas->getWaypointManager(), SLOT(clearWaypointList()));
 
         connect(&uas->getWaypointManager(), SIGNAL(updateStatusString(const QString &)), this, SLOT(updateStatusLabel(const QString &)));
-
-        // This slot is not implemented in UAS: connect(this, SIGNAL(removeWaypointId(int)), uas, SLOT(removeWaypoint(Waypoint*)));
-
-        //qDebug() << "Requesting waypoints";
-        //emit requestWaypoints();
     }
 }
 
@@ -158,29 +153,24 @@ void WaypointList::waypointReached(UASInterface* uas, quint16 waypointId)
     }*/
 }
 
-void WaypointList::transmit()
+void WaypointList::read()
 {
-    transmitDelay->start(1000);
-    m_ui->transmitButton->setEnabled(false);
-    emit clearWaypointList();
-
-    for(int i = 0; i < waypoints.size(); i++)
-    {
-        //Waypoint* wp = waypoints[i];
-        //emit waypointChanged(wp);
-        //if (wp->current)
-        //    emit currentWaypointChanged(wp->id);
-    }
-
-    emit sendWaypoints(waypoints);
-
     while(waypoints.size()>0)
     {
         removeWaypoint(waypoints[0]);
     }
+    
     emit requestWaypoints();
 }
 
+void WaypointList::transmit()
+{
+    transmitDelay->start(1000);
+    m_ui->transmitButton->setEnabled(false);
+
+    emit sendWaypoints(waypoints);
+}
+
 void WaypointList::add()
 {
     // Only add waypoints if UAS is present
@@ -287,11 +277,6 @@ void WaypointList::moveDown(Waypoint* wp)
     }
 }
 
-/*void WaypointList::removeWaypointAndName(Waypoint* wp)
-{
-    removeWaypoint(wp);
-}*/
-
 void WaypointList::removeWaypoint(Waypoint* wp)
 {
     // Delete from list
diff --git a/src/ui/WaypointList.h b/src/ui/WaypointList.h
index 403e71e0c..8aeef71a7 100644
--- a/src/ui/WaypointList.h
+++ b/src/ui/WaypointList.h
@@ -62,6 +62,7 @@ public slots:
     void saveWaypoints();
     void loadWaypoints();    
     void transmit();
+    void read();
     void add();
     void moveUp(Waypoint* wp);
     void moveDown(Waypoint* wp);
-- 
2.22.0