From d7d980a4fd0f49eeea213ee1158cc1c18ba06770 Mon Sep 17 00:00:00 2001
From: kdeng00 <kundeng94@gmail.com>
Date: Sat, 26 Oct 2019 10:09:22 -0400
Subject: [PATCH] Added information on actions and flags to use the program
 when no arguments are supplied

---
 include/Syncers/RetrieveRecords.h |  3 ++-
 src/Main.cpp                      | 35 ++++++++++++++++++++++++++++++-
 src/Syncers/RetrieveRecords.cpp   |  6 ++++++
 src/Syncers/Upload.cpp            |  2 --
 4 files changed, 42 insertions(+), 4 deletions(-)

diff --git a/include/Syncers/RetrieveRecords.h b/include/Syncers/RetrieveRecords.h
index c3b824f..6f1709a 100644
--- a/include/Syncers/RetrieveRecords.h
+++ b/include/Syncers/RetrieveRecords.h
@@ -4,10 +4,11 @@
 #include "Managers/CommitManager.h"
 #include "Models/API.h"
 #include "Models/Token.h"
+#include "Syncers/SyncerBase.h"
 
 namespace Syncers
 {
-    class RetrieveRecords
+    class RetrieveRecords: public SyncerBase
     {
     public:
         RetrieveRecords();
diff --git a/src/Main.cpp b/src/Main.cpp
index fe29a57..ea577cd 100644
--- a/src/Main.cpp
+++ b/src/Main.cpp
@@ -12,11 +12,44 @@ using std::string;
 using Managers::ActionManager;
 using Managers::CommitManager;
 
+void printHelp()
+{
+    cout<<"icd [Action] [flag]\n\n";
+
+    cout<<"Actions\n";
+    cout<<"download\n";
+    cout<<"upload\n";
+    cout<<"retrieve\n";
+    cout<<"delete\n\n";
+
+    cout<<"Flags\n";
+    cout<<"Required for all actions\n";
+    cout<<"-u username\n";
+    cout<<"-p password\n";
+    cout<<"-h host\n\n";
+
+    cout<<"Required for upload\n";
+    cout<<"-s path of song\n";
+    cout<<"-sd directory where to search for songs to upload (Optional)\n";
+    cout<<"-sr directory where to recursively search for songs to upload (Optional)\n\n";
+
+    cout<<"Required for download\n";
+    cout<<"-b song id\n";
+    cout<<"-d path to download song (Optional)\n\n";
+
+    cout<<"Required for retrieving records\n";
+    cout<<"-rt retrieve type (songs is only accepted)\n\n";
+
+    cout<<"Required for deleting a song\n";
+    cout<<"-D song id\n\n";
+}
+
+
 int main(int argc, char** argv)
 {
     if (argc < 2)
     {
-        cout<<"No actions provided"<<endl;
+        printHelp();
         return 1;
     }
 
diff --git a/src/Syncers/RetrieveRecords.cpp b/src/Syncers/RetrieveRecords.cpp
index 72008ec..b4ebfc9 100644
--- a/src/Syncers/RetrieveRecords.cpp
+++ b/src/Syncers/RetrieveRecords.cpp
@@ -46,6 +46,12 @@ namespace Syncers
              cpr::Header{{"authorization", auth},
                  });
 
+        if (r.status_code != (int)Result::OK) {
+            cout<<"something went wrong\n";
+            cout<<"status code: "<<r.status_code<<endl;
+            cout<<"message: "<<r.text<<endl;
+            return;
+        }
         auto songData = nlohmann::json::parse(r.text);
 
         ofstream writeData{};
diff --git a/src/Syncers/Upload.cpp b/src/Syncers/Upload.cpp
index 0f5923d..ed40d0e 100644
--- a/src/Syncers/Upload.cpp
+++ b/src/Syncers/Upload.cpp
@@ -79,8 +79,6 @@ namespace Syncers
             {
                 song = uploadSong(token, song);
             }
-            printSongDetails(songs);
-
         }
         catch (exception& e)
         {