Merge pull request #9 from kdeng00/support_new_upload_endpoint

Support new upload endpoint

CMakeLists.txt

@@ -1,27 +1,37 @@
-cmake_minimum_required(VERSION 3.10)
+cmake_minimum_required(VERSION 3.14)
 
 
-include("cmake/HunterGate.cmake")
-HunterGate(
-    URL "https://github.com/ruslo/hunter/archive/v0.23.184.tar.gz"
-    SHA1 "fe3fb05d51c21499c7eebbe7d0e102742a54a9cd"
-)
-
 project(IcarusDownloadManager)
 
 
 
-if(NOT ${CMAKE_VERSION} LESS 3.2)
-    set(CMAKE_CXX_STANDARD 11)
-    set(CMAKE_CXX_STANDARD_REQUIRED ON)
-else()
 message(STATUS "Checking compiler flags for C++17 support.")
 # Set C++17 support flags for various compilers
 include(CheckCXXCompilerFlag)
+
+if(WIN32)
+  message("Windows")
+  set(vs_ver 19.29.30138)
+
+  if (CMAKE_CXX_COMPILER_VERSION VERSION_GREATER_EQUAL ${vs_ver})
+    message("Visual Studio version is at least ${vs_ver}")
+
+    set(CMAKE_CXX_STANDARD_COMPILE_OPTION "-std:c++latest")
+    set(CMAKE_CXX_EXTENSION_COMPILE_OPTION "-std:c++latest")
+
+    set(CMAKE_CXX_STANDARD 17)
+    set(CMAKE_CXX_STANDARD_REQUIRED ON)
+  endif()
+else()
     check_cxx_compiler_flag("-std=c++17" COMPILER_SUPPORTS_CXX17)
     check_cxx_compiler_flag("-std=c++0x" COMPILER_SUPPORTS_CXX0X)
+
     if(COMPILER_SUPPORTS_CXX17)
         message(STATUS "C++17 is supported.")
+
+        set(CMAKE_CXX_STANDARD 17)
+        set(CMAKE_CXX_STANDARD_REQUIRED ON)
+
         if(${CMAKE_SYSTEM_NAME} MATCHES "Darwin")
             set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++17 -stdlib=libc++")
         else()
@@ -47,6 +57,7 @@ set(SOURCES
     src/Managers/FileManager.cpp
     src/Managers/TokenManager.cpp
     src/Managers/UserManager.cpp
+    src/Models/Song.cpp
     src/Parsers/APIParser.cpp
     src/Syncers/Delete.cpp
     src/Syncers/Download.cpp
@@ -54,36 +65,23 @@ set(SOURCES
     src/Syncers/Upload.cpp
     src/Utilities/Conversions.cpp
 )
-set(HEADERS
-    include/Managers/ActionManager.h
-    include/Managers/CommitManager.h
-    include/Managers/FileManager.h
-    include/Managers/TokenManager.h
-    include/Managers/UserManager.h
-    include/Models/API.h
-    include/Models/Flags.h
-    include/Models/IcarusAction.h
-    include/Models/Song.h
-    include/Models/Token.h
-    include/Models/UploadForm.h
-    include/Models/User.h
-    include/Parsers/APIParser.h
-    include/Syncers/Delete.h
-    include/Syncers/Download.h
-    include/Syncers/RetrieveRecords.h
-    include/Syncers/SyncerBase.h
-    include/Syncers/Upload.h
-    include/Utilities/Conversions.h
-)
+
+set(IDM_INCLUDE_DIR
+    "${CMAKE_CURRENT_SOURCE_DIR}/include")
+
+# conan
+set(CONAN_BUILDINFO
+${CMAKE_BINARY_DIR}/conanbuildinfo.cmake)
+
+message("conan build info ${CONAN_BUILDINFO}")
+include(${CONAN_BUILDINFO})
+conan_basic_setup()
+
+include_directories(${CPR_INCLUDE_DIRS} ${IDM_INCLUDE_DIR})
+
+set(USE_SYSTEM_CURL OFF)
+set(BUILD_CPR_TESTS OFF)
 
 
-hunter_add_package(nlohmann_json)
-find_package(nlohmann_json CONFIG REQUIRED)
-
-hunter_add_package(cpr)
-find_package(cpr CONFIG REQUIRED)
-
-
-add_executable(icd ${SOURCES} ${HEADERS})
-target_link_libraries(icd PUBLIC nlohmann_json::nlohmann_json cpr::cpr)
-include_directories(include/)
+add_executable(icd ${SOURCES})
+target_link_libraries(icd PUBLIC ${CONAN_LIBS})

README.md

@@ -1,6 +1,18 @@
 # IcarusDownloadManager
 
-IcarusDownloadManager is a Linux UI software client application that has the feature of uploading and downloading songs from the [Icarus](https://github.com/amazing-username/Icarus) Music Server. 
+IcarusDownloadManager is a Linux CLI software client application that has the feature of uploading and downloading songs from the [Icarus](https://github.com/kdeng00/Icarus) Music Server. 
+
+
+## Built With
+
+* C++ with C++17 features
+* CMake
+* GCC >= 9 or Visual Studio >= 16 [2019]
+* [conan](https://github.com/conan-io/conan)
+* [json](https://github.com/nlohmann/json)
+* [openssl](https://github.com/openssl/openssl)
+* [curl](https://github.com/curl/curl)
+* [cpr](https://github.com/libcpr/cpr)
 
 
 ### Getting Started
@@ -8,26 +20,57 @@ IcarusDownloadManager is a Linux UI software client application that has the fea
 Build the project:
 
 ```
-export HUNTER_ROOT=/path/to/download/hunter/files/for/dependencies
-mkdir _build
-cd _build
-cmake -H. -B_builds -DHUNTER_STATUS_DEBUG=ON -DCMAKE_BUILD_TYPE=DEBUG
-cmake --build _builds --config Debug
-make
+git clone --recursive https://github.com/kdeng00/IcarusDownloadManager
+
+
+mkdir build
+cd build
+
+conan install .. --build
+
+cmake ..
+cmake --build . --config release -j
 ```
 
-The program has been built and can be executed by the binary file *icd*
+The program has been built and can be executed by the binary file *icd*. For information on how to use icd, merely execute the program without any command line arguments.
+
+### Downloading Song
+
+```BASH
+icd download -u spacecadet -p stellar40 -h https://icarus.com -b 15
+```
+
+### Uploading Song
+
+```BASH
+icd upload -u spacecadet -p stellar40 -h https://icarus.com -s /path/of/song.mp3
+```
+
+### Uploading Song with metadata
+
+```BASH
+icd upload-meta -u spacecadet -p stellar40 -h https://icarus.com -s /path/of/song.mp3 -t 1 -m /path/to/metadata/config/collection.json -ca /path/to/cover/art/image.png
+```
+
+### Uploading Song with metadata from directory
+
+```BASH
+icd upload-meta -u spacecadet -p stellar40 -h https://icarus.com -smca /path/where/songs/and/metadata/exists/
+```
 
 
-## Built With
+### Retrieving Song in json
+
+```Bash
+icd retrieve -u spacecadet -p stellar40 -h https://icarus.com -rt songs
+```
+
+### Deleting Song
+
+```BASH
+icd delete -u spacecadet -p stellar40 -h https://icarus.com -D 15
+```
 
-* C++
-* CMake
-* GCC
-* [Hunter](https://github.com/ruslo/hunter)
-* libCurl
-* [json](https://github.com/nlohmann/json)
-* [cpr](http://whoshuu.github.io/cpr/)
 
 ## Contributing
 
@@ -35,13 +78,15 @@ Please read [CONTRIBUTING.md](CONTRIBUTING.md) for details on the code of conduc
 
 ## Versioning
 
-[v0.1.0](https://github.com/amazing-username/IcarusDownloadManager/releases/tag/0.1.0)
+[v0.2.0](https://github.com/kdeng00/IcarusDownloadManager/releases/tag/v0.2.0)  
+[v0.1.1](https://github.com/kdeng00/IcarusDownloadManager/releases/tag/v0.1.1)  
+[v0.1.0](https://github.com/kdeng00/IcarusDownloadManager/releases/tag/0.1.0)
 
 ## Authors
 
-* **Kun Deng** - [amazing-username](https://github.com/amazing-username)
+* **Kun Deng** - [kdeng00](https://github.com/kdeng00)
 
-See also the list of [contributors](https://github.com/amazing-username/Icarus/graphs/contributors) who participated in this project.
+See also the list of [contributors](https://github.com/kdeng00/Icarus/graphs/contributors) who participated in this project.
 
 ## License
 

cmake/HunterGate.cmake

@@ -1,528 +0,0 @@
-# Copyright (c) 2013-2019, Ruslan Baratov
-# All rights reserved.
-#
-# Redistribution and use in source and binary forms, with or without
-# modification, are permitted provided that the following conditions are met:
-#
-# * Redistributions of source code must retain the above copyright notice, this
-#   list of conditions and the following disclaimer.
-#
-# * Redistributions in binary form must reproduce the above copyright notice,
-#   this list of conditions and the following disclaimer in the documentation
-#   and/or other materials provided with the distribution.
-#
-# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
-# AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
-# IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
-# DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE
-# FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
-# DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
-# SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
-# CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
-# OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
-# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-
-# This is a gate file to Hunter package manager.
-# Include this file using `include` command and add package you need, example:
-#
-#     cmake_minimum_required(VERSION 3.2)
-#
-#     include("cmake/HunterGate.cmake")
-#     HunterGate(
-#         URL "https://github.com/path/to/hunter/archive.tar.gz"
-#         SHA1 "798501e983f14b28b10cda16afa4de69eee1da1d"
-#     )
-#
-#     project(MyProject)
-#
-#     hunter_add_package(Foo)
-#     hunter_add_package(Boo COMPONENTS Bar Baz)
-#
-# Projects:
-#     * https://github.com/hunter-packages/gate/
-#     * https://github.com/ruslo/hunter
-
-option(HUNTER_ENABLED "Enable Hunter package manager support" ON)
-
-if(HUNTER_ENABLED)
-  if(CMAKE_VERSION VERSION_LESS "3.2")
-    message(
-        FATAL_ERROR
-        "At least CMake version 3.2 required for Hunter dependency management."
-        " Update CMake or set HUNTER_ENABLED to OFF."
-    )
-  endif()
-endif()
-
-include(CMakeParseArguments) # cmake_parse_arguments
-
-option(HUNTER_STATUS_PRINT "Print working status" ON)
-option(HUNTER_STATUS_DEBUG "Print a lot info" OFF)
-option(HUNTER_TLS_VERIFY "Enable/disable TLS certificate checking on downloads" ON)
-
-set(HUNTER_ERROR_PAGE "https://docs.hunter.sh/en/latest/reference/errors")
-
-function(hunter_gate_status_print)
-  if(HUNTER_STATUS_PRINT OR HUNTER_STATUS_DEBUG)
-    foreach(print_message ${ARGV})
-      message(STATUS "[hunter] ${print_message}")
-    endforeach()
-  endif()
-endfunction()
-
-function(hunter_gate_status_debug)
-  if(HUNTER_STATUS_DEBUG)
-    foreach(print_message ${ARGV})
-      string(TIMESTAMP timestamp)
-      message(STATUS "[hunter *** DEBUG *** ${timestamp}] ${print_message}")
-    endforeach()
-  endif()
-endfunction()
-
-function(hunter_gate_error_page error_page)
-  message("------------------------------ ERROR ------------------------------")
-  message("    ${HUNTER_ERROR_PAGE}/${error_page}.html")
-  message("-------------------------------------------------------------------")
-  message("")
-  message(FATAL_ERROR "")
-endfunction()
-
-function(hunter_gate_internal_error)
-  message("")
-  foreach(print_message ${ARGV})
-    message("[hunter ** INTERNAL **] ${print_message}")
-  endforeach()
-  message("[hunter ** INTERNAL **] [Directory:${CMAKE_CURRENT_LIST_DIR}]")
-  message("")
-  hunter_gate_error_page("error.internal")
-endfunction()
-
-function(hunter_gate_fatal_error)
-  cmake_parse_arguments(hunter "" "ERROR_PAGE" "" "${ARGV}")
-  if("${hunter_ERROR_PAGE}" STREQUAL "")
-    hunter_gate_internal_error("Expected ERROR_PAGE")
-  endif()
-  message("")
-  foreach(x ${hunter_UNPARSED_ARGUMENTS})
-    message("[hunter ** FATAL ERROR **] ${x}")
-  endforeach()
-  message("[hunter ** FATAL ERROR **] [Directory:${CMAKE_CURRENT_LIST_DIR}]")
-  message("")
-  hunter_gate_error_page("${hunter_ERROR_PAGE}")
-endfunction()
-
-function(hunter_gate_user_error)
-  hunter_gate_fatal_error(${ARGV} ERROR_PAGE "error.incorrect.input.data")
-endfunction()
-
-function(hunter_gate_self root version sha1 result)
-  string(COMPARE EQUAL "${root}" "" is_bad)
-  if(is_bad)
-    hunter_gate_internal_error("root is empty")
-  endif()
-
-  string(COMPARE EQUAL "${version}" "" is_bad)
-  if(is_bad)
-    hunter_gate_internal_error("version is empty")
-  endif()
-
-  string(COMPARE EQUAL "${sha1}" "" is_bad)
-  if(is_bad)
-    hunter_gate_internal_error("sha1 is empty")
-  endif()
-
-  string(SUBSTRING "${sha1}" 0 7 archive_id)
-
-  set(
-      hunter_self
-      "${root}/_Base/Download/Hunter/${version}/${archive_id}/Unpacked"
-  )
-
-  set("${result}" "${hunter_self}" PARENT_SCOPE)
-endfunction()
-
-# Set HUNTER_GATE_ROOT cmake variable to suitable value.
-function(hunter_gate_detect_root)
-  # Check CMake variable
-  string(COMPARE NOTEQUAL "${HUNTER_ROOT}" "" not_empty)
-  if(not_empty)
-    set(HUNTER_GATE_ROOT "${HUNTER_ROOT}" PARENT_SCOPE)
-    hunter_gate_status_debug("HUNTER_ROOT detected by cmake variable")
-    return()
-  endif()
-
-  # Check environment variable
-  string(COMPARE NOTEQUAL "$ENV{HUNTER_ROOT}" "" not_empty)
-  if(not_empty)
-    set(HUNTER_GATE_ROOT "$ENV{HUNTER_ROOT}" PARENT_SCOPE)
-    hunter_gate_status_debug("HUNTER_ROOT detected by environment variable")
-    return()
-  endif()
-
-  # Check HOME environment variable
-  string(COMPARE NOTEQUAL "$ENV{HOME}" "" result)
-  if(result)
-    set(HUNTER_GATE_ROOT "$ENV{HOME}/.hunter" PARENT_SCOPE)
-    hunter_gate_status_debug("HUNTER_ROOT set using HOME environment variable")
-    return()
-  endif()
-
-  # Check SYSTEMDRIVE and USERPROFILE environment variable (windows only)
-  if(WIN32)
-    string(COMPARE NOTEQUAL "$ENV{SYSTEMDRIVE}" "" result)
-    if(result)
-      set(HUNTER_GATE_ROOT "$ENV{SYSTEMDRIVE}/.hunter" PARENT_SCOPE)
-      hunter_gate_status_debug(
-          "HUNTER_ROOT set using SYSTEMDRIVE environment variable"
-      )
-      return()
-    endif()
-
-    string(COMPARE NOTEQUAL "$ENV{USERPROFILE}" "" result)
-    if(result)
-      set(HUNTER_GATE_ROOT "$ENV{USERPROFILE}/.hunter" PARENT_SCOPE)
-      hunter_gate_status_debug(
-          "HUNTER_ROOT set using USERPROFILE environment variable"
-      )
-      return()
-    endif()
-  endif()
-
-  hunter_gate_fatal_error(
-      "Can't detect HUNTER_ROOT"
-      ERROR_PAGE "error.detect.hunter.root"
-  )
-endfunction()
-
-function(hunter_gate_download dir)
-  string(
-      COMPARE
-      NOTEQUAL
-      "$ENV{HUNTER_DISABLE_AUTOINSTALL}"
-      ""
-      disable_autoinstall
-  )
-  if(disable_autoinstall AND NOT HUNTER_RUN_INSTALL)
-    hunter_gate_fatal_error(
-        "Hunter not found in '${dir}'"
-        "Set HUNTER_RUN_INSTALL=ON to auto-install it from '${HUNTER_GATE_URL}'"
-        "Settings:"
-        "  HUNTER_ROOT: ${HUNTER_GATE_ROOT}"
-        "  HUNTER_SHA1: ${HUNTER_GATE_SHA1}"
-        ERROR_PAGE "error.run.install"
-    )
-  endif()
-  string(COMPARE EQUAL "${dir}" "" is_bad)
-  if(is_bad)
-    hunter_gate_internal_error("Empty 'dir' argument")
-  endif()
-
-  string(COMPARE EQUAL "${HUNTER_GATE_SHA1}" "" is_bad)
-  if(is_bad)
-    hunter_gate_internal_error("HUNTER_GATE_SHA1 empty")
-  endif()
-
-  string(COMPARE EQUAL "${HUNTER_GATE_URL}" "" is_bad)
-  if(is_bad)
-    hunter_gate_internal_error("HUNTER_GATE_URL empty")
-  endif()
-
-  set(done_location "${dir}/DONE")
-  set(sha1_location "${dir}/SHA1")
-
-  set(build_dir "${dir}/Build")
-  set(cmakelists "${dir}/CMakeLists.txt")
-
-  hunter_gate_status_debug("Locking directory: ${dir}")
-  file(LOCK "${dir}" DIRECTORY GUARD FUNCTION)
-  hunter_gate_status_debug("Lock done")
-
-  if(EXISTS "${done_location}")
-    # while waiting for lock other instance can do all the job
-    hunter_gate_status_debug("File '${done_location}' found, skip install")
-    return()
-  endif()
-
-  file(REMOVE_RECURSE "${build_dir}")
-  file(REMOVE_RECURSE "${cmakelists}")
-
-  file(MAKE_DIRECTORY "${build_dir}") # check directory permissions
-
-  # Disabling languages speeds up a little bit, reduces noise in the output
-  # and avoids path too long windows error
-  file(
-      WRITE
-      "${cmakelists}"
-      "cmake_minimum_required(VERSION 3.2)\n"
-      "project(HunterDownload LANGUAGES NONE)\n"
-      "include(ExternalProject)\n"
-      "ExternalProject_Add(\n"
-      "    Hunter\n"
-      "    URL\n"
-      "    \"${HUNTER_GATE_URL}\"\n"
-      "    URL_HASH\n"
-      "    SHA1=${HUNTER_GATE_SHA1}\n"
-      "    DOWNLOAD_DIR\n"
-      "    \"${dir}\"\n"
-      "    TLS_VERIFY\n"
-      "    ${HUNTER_TLS_VERIFY}\n"
-      "    SOURCE_DIR\n"
-      "    \"${dir}/Unpacked\"\n"
-      "    CONFIGURE_COMMAND\n"
-      "    \"\"\n"
-      "    BUILD_COMMAND\n"
-      "    \"\"\n"
-      "    INSTALL_COMMAND\n"
-      "    \"\"\n"
-      ")\n"
-  )
-
-  if(HUNTER_STATUS_DEBUG)
-    set(logging_params "")
-  else()
-    set(logging_params OUTPUT_QUIET)
-  endif()
-
-  hunter_gate_status_debug("Run generate")
-
-  # Need to add toolchain file too.
-  # Otherwise on Visual Studio + MDD this will fail with error:
-  # "Could not find an appropriate version of the Windows 10 SDK installed on this machine"
-  if(EXISTS "${CMAKE_TOOLCHAIN_FILE}")
-    get_filename_component(absolute_CMAKE_TOOLCHAIN_FILE "${CMAKE_TOOLCHAIN_FILE}" ABSOLUTE)
-    set(toolchain_arg "-DCMAKE_TOOLCHAIN_FILE=${absolute_CMAKE_TOOLCHAIN_FILE}")
-  else()
-    # 'toolchain_arg' can't be empty
-    set(toolchain_arg "-DCMAKE_TOOLCHAIN_FILE=")
-  endif()
-
-  string(COMPARE EQUAL "${CMAKE_MAKE_PROGRAM}" "" no_make)
-  if(no_make)
-    set(make_arg "")
-  else()
-    # Test case: remove Ninja from PATH but set it via CMAKE_MAKE_PROGRAM
-    set(make_arg "-DCMAKE_MAKE_PROGRAM=${CMAKE_MAKE_PROGRAM}")
-  endif()
-
-  execute_process(
-      COMMAND
-      "${CMAKE_COMMAND}"
-      "-H${dir}"
-      "-B${build_dir}"
-      "-G${CMAKE_GENERATOR}"
-      "${toolchain_arg}"
-      ${make_arg}
-      WORKING_DIRECTORY "${dir}"
-      RESULT_VARIABLE download_result
-      ${logging_params}
-  )
-
-  if(NOT download_result EQUAL 0)
-    hunter_gate_internal_error(
-        "Configure project failed."
-        "To reproduce the error run: ${CMAKE_COMMAND} -H${dir} -B${build_dir} -G${CMAKE_GENERATOR} ${toolchain_arg} ${make_arg}"
-        "In directory ${dir}"
-    )
-  endif()
-
-  hunter_gate_status_print(
-      "Initializing Hunter workspace (${HUNTER_GATE_SHA1})"
-      "  ${HUNTER_GATE_URL}"
-      "  -> ${dir}"
-  )
-  execute_process(
-      COMMAND "${CMAKE_COMMAND}" --build "${build_dir}"
-      WORKING_DIRECTORY "${dir}"
-      RESULT_VARIABLE download_result
-      ${logging_params}
-  )
-
-  if(NOT download_result EQUAL 0)
-    hunter_gate_internal_error("Build project failed")
-  endif()
-
-  file(REMOVE_RECURSE "${build_dir}")
-  file(REMOVE_RECURSE "${cmakelists}")
-
-  file(WRITE "${sha1_location}" "${HUNTER_GATE_SHA1}")
-  file(WRITE "${done_location}" "DONE")
-
-  hunter_gate_status_debug("Finished")
-endfunction()
-
-# Must be a macro so master file 'cmake/Hunter' can
-# apply all variables easily just by 'include' command
-# (otherwise PARENT_SCOPE magic needed)
-macro(HunterGate)
-  if(HUNTER_GATE_DONE)
-    # variable HUNTER_GATE_DONE set explicitly for external project
-    # (see `hunter_download`)
-    set_property(GLOBAL PROPERTY HUNTER_GATE_DONE YES)
-  endif()
-
-  # First HunterGate command will init Hunter, others will be ignored
-  get_property(_hunter_gate_done GLOBAL PROPERTY HUNTER_GATE_DONE SET)
-
-  if(NOT HUNTER_ENABLED)
-    # Empty function to avoid error "unknown function"
-    function(hunter_add_package)
-    endfunction()
-
-    set(
-        _hunter_gate_disabled_mode_dir
-        "${CMAKE_CURRENT_LIST_DIR}/cmake/Hunter/disabled-mode"
-    )
-    if(EXISTS "${_hunter_gate_disabled_mode_dir}")
-      hunter_gate_status_debug(
-          "Adding \"disabled-mode\" modules: ${_hunter_gate_disabled_mode_dir}"
-      )
-      list(APPEND CMAKE_PREFIX_PATH "${_hunter_gate_disabled_mode_dir}")
-    endif()
-  elseif(_hunter_gate_done)
-    hunter_gate_status_debug("Secondary HunterGate (use old settings)")
-    hunter_gate_self(
-        "${HUNTER_CACHED_ROOT}"
-        "${HUNTER_VERSION}"
-        "${HUNTER_SHA1}"
-        _hunter_self
-    )
-    include("${_hunter_self}/cmake/Hunter")
-  else()
-    set(HUNTER_GATE_LOCATION "${CMAKE_CURRENT_SOURCE_DIR}")
-
-    string(COMPARE NOTEQUAL "${PROJECT_NAME}" "" _have_project_name)
-    if(_have_project_name)
-      hunter_gate_fatal_error(
-          "Please set HunterGate *before* 'project' command. "
-          "Detected project: ${PROJECT_NAME}"
-          ERROR_PAGE "error.huntergate.before.project"
-      )
-    endif()
-
-    cmake_parse_arguments(
-        HUNTER_GATE "LOCAL" "URL;SHA1;GLOBAL;FILEPATH" "" ${ARGV}
-    )
-
-    string(COMPARE EQUAL "${HUNTER_GATE_SHA1}" "" _empty_sha1)
-    string(COMPARE EQUAL "${HUNTER_GATE_URL}" "" _empty_url)
-    string(
-        COMPARE
-        NOTEQUAL
-        "${HUNTER_GATE_UNPARSED_ARGUMENTS}"
-        ""
-        _have_unparsed
-    )
-    string(COMPARE NOTEQUAL "${HUNTER_GATE_GLOBAL}" "" _have_global)
-    string(COMPARE NOTEQUAL "${HUNTER_GATE_FILEPATH}" "" _have_filepath)
-
-    if(_have_unparsed)
-      hunter_gate_user_error(
-          "HunterGate unparsed arguments: ${HUNTER_GATE_UNPARSED_ARGUMENTS}"
-      )
-    endif()
-    if(_empty_sha1)
-      hunter_gate_user_error("SHA1 suboption of HunterGate is mandatory")
-    endif()
-    if(_empty_url)
-      hunter_gate_user_error("URL suboption of HunterGate is mandatory")
-    endif()
-    if(_have_global)
-      if(HUNTER_GATE_LOCAL)
-        hunter_gate_user_error("Unexpected LOCAL (already has GLOBAL)")
-      endif()
-      if(_have_filepath)
-        hunter_gate_user_error("Unexpected FILEPATH (already has GLOBAL)")
-      endif()
-    endif()
-    if(HUNTER_GATE_LOCAL)
-      if(_have_global)
-        hunter_gate_user_error("Unexpected GLOBAL (already has LOCAL)")
-      endif()
-      if(_have_filepath)
-        hunter_gate_user_error("Unexpected FILEPATH (already has LOCAL)")
-      endif()
-    endif()
-    if(_have_filepath)
-      if(_have_global)
-        hunter_gate_user_error("Unexpected GLOBAL (already has FILEPATH)")
-      endif()
-      if(HUNTER_GATE_LOCAL)
-        hunter_gate_user_error("Unexpected LOCAL (already has FILEPATH)")
-      endif()
-    endif()
-
-    hunter_gate_detect_root() # set HUNTER_GATE_ROOT
-
-    # Beautify path, fix probable problems with windows path slashes
-    get_filename_component(
-        HUNTER_GATE_ROOT "${HUNTER_GATE_ROOT}" ABSOLUTE
-    )
-    hunter_gate_status_debug("HUNTER_ROOT: ${HUNTER_GATE_ROOT}")
-    if(NOT HUNTER_ALLOW_SPACES_IN_PATH)
-      string(FIND "${HUNTER_GATE_ROOT}" " " _contain_spaces)
-      if(NOT _contain_spaces EQUAL -1)
-        hunter_gate_fatal_error(
-            "HUNTER_ROOT (${HUNTER_GATE_ROOT}) contains spaces."
-            "Set HUNTER_ALLOW_SPACES_IN_PATH=ON to skip this error"
-            "(Use at your own risk!)"
-            ERROR_PAGE "error.spaces.in.hunter.root"
-        )
-      endif()
-    endif()
-
-    string(
-        REGEX
-        MATCH
-        "[0-9]+\\.[0-9]+\\.[0-9]+[-_a-z0-9]*"
-        HUNTER_GATE_VERSION
-        "${HUNTER_GATE_URL}"
-    )
-    string(COMPARE EQUAL "${HUNTER_GATE_VERSION}" "" _is_empty)
-    if(_is_empty)
-      set(HUNTER_GATE_VERSION "unknown")
-    endif()
-
-    hunter_gate_self(
-        "${HUNTER_GATE_ROOT}"
-        "${HUNTER_GATE_VERSION}"
-        "${HUNTER_GATE_SHA1}"
-        _hunter_self
-    )
-
-    set(_master_location "${_hunter_self}/cmake/Hunter")
-    get_filename_component(_archive_id_location "${_hunter_self}/.." ABSOLUTE)
-    set(_done_location "${_archive_id_location}/DONE")
-    set(_sha1_location "${_archive_id_location}/SHA1")
-
-    # Check Hunter already downloaded by HunterGate
-    if(NOT EXISTS "${_done_location}")
-      hunter_gate_download("${_archive_id_location}")
-    endif()
-
-    if(NOT EXISTS "${_done_location}")
-      hunter_gate_internal_error("hunter_gate_download failed")
-    endif()
-
-    if(NOT EXISTS "${_sha1_location}")
-      hunter_gate_internal_error("${_sha1_location} not found")
-    endif()
-    file(READ "${_sha1_location}" _sha1_value)
-    string(COMPARE EQUAL "${_sha1_value}" "${HUNTER_GATE_SHA1}" _is_equal)
-    if(NOT _is_equal)
-      hunter_gate_internal_error(
-          "Short SHA1 collision:"
-          "  ${_sha1_value} (from ${_sha1_location})"
-          "  ${HUNTER_GATE_SHA1} (HunterGate)"
-      )
-    endif()
-    if(NOT EXISTS "${_master_location}")
-      hunter_gate_user_error(
-          "Master file not found:"
-          "  ${_master_location}"
-          "try to update Hunter/HunterGate"
-      )
-    endif()
-    include("${_master_location}")
-    set_property(GLOBAL PROPERTY HUNTER_GATE_DONE YES)
-  endif()
-endmacro()

conanfile.txt

@@ -0,0 +1,20 @@
+[requires]
+nlohmann_json/3.10.4
+openssl/1.1.1l
+libcurl/7.80.0
+cpr/1.7.2
+
+[generators]
+cmake
+
+[options]
+openssl:shared=False
+libcurl:shared=False
+cpr:shared=False
+libcurl:with_ssl=openssl
+libcurl:with_ftp=False
+libcurl:with_gopher=False
+libcurl:with_imap=False
+libcurl:with_pop3=False
+libcurl:with_smb=False
+libcurl:with_smtp=False

include/Managers/ActionManager.h

@@ -1,7 +1,9 @@
 #ifndef ACTIONMANAGER_H_
 #define ACTIONMANAGER_H_
 
+#include<algorithm>
 #include<string>
+#include<string_view>
 #include<array>
 #include<vector>
 
@@ -10,6 +12,9 @@
 
 namespace Managers
 {
+
+
+
 class ActionManager
 {
 public:
@@ -17,28 +22,72 @@ namespace Managers
 
     Models::IcarusAction retrieveIcarusAction() const;
 private:
-        bool isNumber(std::string);
+    constexpr std::array<const char*, 16> supportedFlags() noexcept
+    {
+        constexpr std::array<const char*, 16> allFlags{"-u", "-p", "-t", "-h", "-s",
+            "-sd", "-sr", "-d", "-D", "-b", "-rt", "-nc",
+            "-m", "-ca", "-smca", "-t"};
+
+        return allFlags;
+    }
+    constexpr std::array<const char*, 4> supportedActions() noexcept;
 
     void initialize();
     void validateFlags();
+    // Checks to see if the flag is valid
+    template<typename Str>
+    bool isValidFlag(const Str flag)
+    {
+        const auto flags = supportedFlags();
+        const auto i = std::find_if(flags.begin(), flags.end(), [&](const Str &f)
+        {
+            return f.compare(flag) == 0 ? true : false;
+        });
+
+        auto result = i != flags.end() ? true : false;
+
+        return result;
+    }
+
+    template<typename Str>
+    bool doesFlagHaveValue(const Str flag)
+    {
+        const auto flags = parsedFlags();
+        auto i = std::find_if(flags.begin(), flags.end(), [&](const Str &f)
+        {
+            return f.compare(flag) == 0 ? true : false;
+        });
+
+        if (i != flags.end())
+        {
+            if (++i != flags.end() && !isValidFlag<Str>(*i))
+            {
+                return true;
+            }
+            else
+            {
+                return false;
+            }
+        }
+        else
+        {
+            return false;
+        }
+    }
+
+    void printAction() noexcept;
+    void printFlags() noexcept;
 
     std::vector<std::string> parsedFlags();
 
-        void printAction();
-        void printFlags();
-
     std::string action;
-        std::array<std::string, 4> supportedActions{
-            "download", "upload", "retrieve", "delete"
-        };
-        std::array<std::string, 11> supportedFlags{
-            "-u", "-p", "-t", "-h", "-s", "-sd",
-            "-sr", "-d", "-D", "-b", "-rt"
-        };
+    
     std::vector<Models::Flags> flags;
+
     char **params;
     int paramCount;
 };
+
 }
 
 #endif

include/Managers/CommitManager.h

@@ -2,14 +2,19 @@
 #define COMMITMANAGER_H_
 
 #include<map>
+#include<iostream>
 #include<string>
+#include<string_view>
 
 #include"Models/API.h"
-#include"Models/Token.h"
 #include"Models/IcarusAction.h"
+#include"Models/Song.h"
+#include"Models/Token.h"
+#include"Utilities/Checks.h"
 
 namespace Managers
 {
+
 class CommitManager
 {
 public:
@@ -17,35 +22,194 @@ namespace Managers
 
     void commitAction();
 
-        enum RetrieveTypes
+
+    enum class RetrieveTypes
     {
         songs
     };
     
+        // Used for parsing songs from the metadata file
+    class Album
+    {
+    public:
+        Album() = default;
+
+        void printInfo();
+
+
+        std::string album;
+        std::string albumArtist;
+        std::string genre;
+        int year;
+        int trackCount;
+        int discCount;
+        std::vector<Models::Song> songs;
+    };
+
 private:
-        Models::Token parseToken(Models::API);
+    enum class ActionValues;
+    
+    std::map<std::string, ActionValues> mapActions() noexcept;
 
     void deleteSong();
     void downloadSong();
     void retrieveObjects();
     void uploadSong();
 
-        enum ActionValues
+    Models::Token parseToken(Models::API);
+
+    // Uploads a single song. The song is constructed from a metadata file that contains
+    // information about the album the song is from. Also, the cover art of the song must
+    // be present.
+    //
+    // Expects
+    // * Song - mp3 file path
+    // * TrackID - track number to chose from when retrieving metadata. "1" and "1:1" are similar
+    // * Metadata - Source file containing metadata of the song
+    // * Cover art - path to image cover art
+    void uploadSongWithMetadata();
+
+    // Expects the song path, trackID, metadata file path, and cover path
+    void singTargetUpload(const std::string &songPath, const std::string &trackID, 
+        const std::string &metaPath, const std::string &coverPath);
+    // Expects the source directory that contains songs, a metadata file, and cover image
+    // Disc and Track is retrieved from the filename if the filename conforms to a standard.
+    // If not, then the disc and track will default to 1
+    //
+    // Standards
+    // * track01.mp3 - Disc 1, Track 1
+    // * track05d02.mp3 - Disc 2, Track 5
+    void multiTargetUpload(const std::string &sourcePath);
+
+    // Standards
+    // * track01.mp3 - Disc 1, Track 1
+    // * track05d02.mp3 - Disc 2, Track 5
+    template<typename Song, typename Str>
+    void initializeDiscAndTrack(Song &song)
+    {
+        auto disc = 1;
+        auto track = 1;
+        // If 1 go with first standard, if 2 go with the second, if 0 then will default to 1 for disc and track
+        auto mode = 0;
+        const Str &songPath = song.songPath;
+
+        auto trd = song.songPath.find("trackd");
+        auto tr = song.songPath.find("track");
+
+        if (tr != Str::npos)
+        {
+            mode = 1;
+        }
+
+        if (trd != Str::npos)
+        {
+            mode = 2;
+        }
+
+        auto dl = [](char c, char t){ return c == t; };
+        auto d = Utilities::Checks::itemIterInContainer<char, Str>(songPath, 'd', dl);
+        auto k = Utilities::Checks::itemIterInContainer<char, Str>(songPath, 'k', dl);
+        auto dot = Utilities::Checks::itemIterInContainer<char, Str>(songPath, '.', dl);
+
+        switch(mode)
+        {
+        case 1:
+        {
+            if (k != songPath.end() && dot != songPath.end())
+            {
+                auto tStr = std::string(++k, dot);
+                std::cout << "TStr: " << tStr<<"\n";
+
+                if (Utilities::Checks::isNumber(tStr))
+                {
+                    track = std::atoi(tStr.c_str());
+                }
+            }
+            break;
+        }
+        case 2:
+        {
+            if (k != songPath.end() && dot != songPath.end() && d != songPath.end())
+            {
+                auto tStr = std::string(++k, d);
+                auto dStr = std::string(++d, dot);
+                std::cout<<"DStr: "<<dStr<<" TStr: " << tStr<<"\n";
+
+                if (Utilities::Checks::isNumber(tStr))
+                {
+                    track = std::atoi(tStr.c_str());
+                }
+                else if (Utilities::Checks::isNumber(dStr))
+                {
+                    disc = std::atoi(dStr.c_str());
+                }
+            }
+            break;
+        }
+        }
+
+
+        song.disc = disc;
+        song.track = track;
+    }
+
+    template<typename Song, typename Str>
+    void parseDiscAndTrack(Song &song, const Str &trackID)
+    {
+        auto sep = [](char c, char t) { return c == t; };
+        auto separator = Utilities::Checks::itemIterInContainer<char, Str>(trackID, ':', sep);
+
+        if (separator != trackID.end())
+        {
+            auto dStr = Str(trackID.begin(), separator);
+            auto tStr = Str(++separator, trackID.end());
+
+            song.disc = std::atoi(dStr.c_str());
+            song.track = std::atoi(tStr.c_str());
+        }
+        else
+        {
+            auto isNumber = Utilities::Checks::isNumber(trackID);
+            if (isNumber)
+            {
+                song.track = std::atoi(trackID.c_str());
+            }
+        }
+    }
+
+
+    // Checks for the no confirm flag. Used when uploading songs from a directory
+    bool checkForNoConfirm()
+    {
+        for (const auto &arg: this->icaAction.flags)
+        {
+            if (arg.flag.compare("-nc") == 0)
+            {
+                return true;
+            }
+        }
+
+        return false;
+    }
+
+
+    Album retrieveMetadata(const std::string_view path);
+    std::string retrieveFileContent(const std::string_view path);
+
+    enum class ActionValues
     {
         deleteAct,
         downloadAct,
         retrieveAct,
-            uploadAct
+        uploadAct,
+        UPLOAD_SONG_WITH_METADATA // Uploads the song with metadata, including cover art
     };
 
-        std::map<std::string, ActionValues> mapActions{
-			{"delete", deleteAct}, {"download", downloadAct},
-			{"retrieve", retrieveAct},
-			{"upload", uploadAct}
-		};
 
     Models::IcarusAction icaAction;
+
 };
+
 }
 
 #endif

include/Managers/FileManager.h

@@ -5,6 +5,7 @@
 
 namespace Managers
 {
+
 class FileManager
 {
 public:
@@ -24,6 +25,7 @@ namespace Managers
     bool fileRead;
     int fileBufferLength;
 };
+
 }
 
 #endif

include/Managers/TokenManager.h

@@ -7,6 +7,7 @@
 
 namespace Managers
 {
+
 class TokenManager
 {
 public:
@@ -18,6 +19,7 @@ namespace Managers
     Models::API api;
     Models::User user;
 };
+
 }
 
 #endif

include/Managers/UserManager.h

@@ -8,6 +8,7 @@
 
 namespace Managers
 {
+
 class UserManager
 {
 public:
@@ -21,6 +22,7 @@ namespace Managers
     Models::User user;
     Models::IcarusAction icaAction;
 };
+
 }
 
 #endif

include/Models/API.h

@@ -5,12 +5,15 @@
 
 namespace Models
 {
-    struct API
+
+class API
 {
+public:
     std::string url;
     std::string endpoint;
     std::string version;
 };
+
 }
 
 #endif

include/Models/Flags.h

@@ -5,11 +5,14 @@
 
 namespace Models
 {
-    struct Flags
+
+class Flags
 {
+public:
     std::string flag;
     std::string value;
 };
+
 }
 
 #endif

include/Models/IcarusAction.h

@@ -2,17 +2,52 @@
 #define ICARUSACTION_H_
 
 #include<string>
+#include<algorithm>
+#include<string_view>
 #include<vector>
+#include<iostream>
 
 #include"Flags.h"
 
 namespace Models
 {
-    struct IcarusAction
+
+class IcarusAction
 {
+public:
+    std::string retrieveFlagValue(const std::string_view flag)
+    {
+        std::string value;
+
+        const auto fg = std::find_if(flags.begin(), flags.end(), [&](Flags f)
+        {
+            return f.flag.compare(flag) == 0 ? true : false;
+        });
+
+        if (fg != flags.end())
+        {
+            value.assign(fg->value);
+        }
+
+        return value;
+    }
+    void print_action_and_flags() noexcept
+    {
+        std::cout<<"Action: "<<this->action<<"\n";
+        std::cout<<"Flag count: "<<this->flags.size()<<"\n";
+
+        for (const auto &flag : this->flags)
+        {
+            std::cout<<"flag "<<flag.flag<<" value "<<flag.value<<"\n";
+        }
+
+        std::cout<<"\n";
+    }
+
     std::string action;
     std::vector<Flags> flags;
 };
+
 }
 
 #endif

include/Models/Song.h

@@ -2,12 +2,65 @@
 #define SONG_H_
 
 #include <string>
+#include <iostream>
+#include <sstream>
 
 
 namespace Models
 {
-    struct Song
+
+class Song
 {
+public:
+    Song() = default;
+
+    void printInfo()
+    {
+        std::cout<<"Title: "<<this->title<<"\n";
+        std::cout<<"\n";
+    }
+
+    std::string song_path() noexcept
+    {
+        std::stringstream buffer;
+
+        buffer << this->directory;
+
+        const auto count = this->directory.size();
+
+        if (this->directory.at(count - 1) != '/' || this->directory.at(count - 1) != '\\')
+        {
+            buffer << "/";
+        }
+
+        buffer << this->filename;
+
+        return buffer.str();
+    }
+
+    int generate_filename_from_track()
+    {
+        auto result = 0;
+        std::stringstream buffer;
+        buffer << "track";
+
+        // NOTE: Multiple discs in one directory is not being addressed
+        if (this->track < 10)
+        {
+            buffer << "0";
+        }
+
+        buffer << this->track << ".mp3";
+
+
+        this->filename.assign(buffer.str());
+
+        return result;
+    }
+
+    std::string toMetadataJson();
+
+
     int id;
     std::string title;
     std::string artist;
@@ -16,9 +69,22 @@ namespace Models
     int year;
     int duration;
     int track;
+    int disc;
     std::string data;
+    [[deprecated("Use song_path() function instead")]]
     std::string songPath;
+    std::string filename;
+    std::string directory;
 };
+
+class CoverArt
+{
+public:
+    int id;
+    std::string title;
+    std::string path;
+};
+
 } 
 
 #endif

include/Models/Token.h

@@ -5,12 +5,14 @@
 
 namespace Models
 {
+
 struct Token
 {
     std::string accessToken;
     std::string tokenType;
     int expiration;
 };
+
 }
 
 #endif

include/Models/UploadForm.h

@@ -6,11 +6,13 @@
 
 namespace Models
 {
+
 struct UploadForm
 {
     std::string url;
     std::string filePath;
 };
+
 } 
 
 #endif

include/Models/User.h

@@ -5,11 +5,13 @@
 
 namespace Models
 {
+
 struct User
 {
     std::string username;
     std::string password;
 };
+
 }
 
 #endif

include/Parsers/APIParser.h

@@ -6,6 +6,7 @@
 
 namespace Parsers
 {
+
 class APIParser
 {
 public:
@@ -18,6 +19,7 @@ namespace Parsers
     Models::API api;
     Models::IcarusAction icaAct;
 };
+
 }
 
 #endif

include/Syncers/Delete.h

@@ -9,6 +9,7 @@
 
 namespace Syncers
 {
+
 class Delete : SyncerBase
 {
 public:
@@ -18,6 +19,7 @@ namespace Syncers
 private:
     std::string retrieveUrl(Models::Song);
 };
+
 }
 
 #endif

include/Syncers/Download.h

@@ -12,6 +12,7 @@
 
 namespace Syncers
 {
+
 class Download : SyncerBase
 {
 public:
@@ -26,6 +27,7 @@ namespace Syncers
     std::string downloadFilePath;
     void saveSong(Models::Song&);
 };
+
 }
 
 #endif

include/Syncers/RetrieveRecords.h

@@ -8,6 +8,7 @@
 
 namespace Syncers
 {
+
 class RetrieveRecords: public SyncerBase
 {
 public:
@@ -21,6 +22,7 @@ namespace Syncers
     Models::API api;
     Models::Token token;
 };
+
 }
 
 #endif

include/Syncers/SyncerBase.h

@@ -7,6 +7,7 @@
 
 namespace Syncers
 {
+
 class SyncerBase
 {
 protected:
@@ -14,7 +15,15 @@ namespace Syncers
     const int OK = 200;
     const int UNAUTHORIZED = 401;
     const int NOTFOUND = 404;
+
+    enum class Result 
+    {
+        OK = 200,
+        UNAUTHORIZED = 401,
+        NOTFOUND = 404
     };
+};
+
 }
 
 #endif

include/Syncers/Upload.h

@@ -7,6 +7,7 @@
 
 #include <nlohmann/json.hpp>
 
+#include "Managers/CommitManager.h"
 #include "Managers/FileManager.h"
 #include "Models/API.h"
 #include "Models/Song.h"
@@ -18,18 +19,24 @@ namespace fs = std::filesystem;
 
 namespace Syncers
 {
+
 class Upload
 {
 public:
-            Upload();
-            Upload(Models::API);
+    Upload() = default;
+    Upload(Models::API api, Models::Token token) : m_token(token), api(api)
+    {
+        this->api.endpoint = "song/data";
+    }
 
-            Models::Song uploadSong(const Models::Token&, Models::Song&);
-            void uploadSongsFromDirectory(const Models::Token&, const std::string&, bool);
+    Models::Song uploadSong(Models::Song&);
+    void uploadSongsFromDirectory(const std::string&, const bool, bool);
+    void uploadSongWithMetadata(Managers::CommitManager::Album&, Models::Song&, Models::CoverArt&);
 private:
     Managers::FileManager fMgr;
     Models::API api;
     Models::Song song;
+    Models::Token m_token;
 
     std::vector<Models::Song> retrieveAllSongsFromDirectory(const std::string&,
         bool);
@@ -40,8 +47,9 @@ namespace Syncers
 
     void printSongDetails();
     void printSongDetails(std::vector<Models::Song>&);
-            void printJsonData(nlohmann::json);
+    void printJsonData(const nlohmann::json&);
 };
+
 }
 
 #endif

include/UI/AboutWindow.h

@@ -13,6 +13,7 @@
 
 namespace UI
 {
+
 class AboutWindow: public QDialog, public CommonWindow
 {
     Q_OBJECT
@@ -29,6 +30,7 @@ namespace UI
 private slots:
     void closeWindow();
 };
+
 }
 
 #endif

include/UI/CommonWindow.h

@@ -10,6 +10,9 @@
 #include<QPushButton>
 
 
+namespace UI
+{
+
 class CommonWindow
 {
 public:
@@ -24,4 +27,7 @@ protected:
     std::unique_ptr<QVBoxLayout> subLayoutTwoQt;
     int windowHeight, windowWidth;
 };
+
+}
+
 #endif

include/UI/MainWindow.h

@@ -22,6 +22,7 @@
 
 namespace UI
 {
+
 class MainWindow: public QMainWindow, public CommonWindow
 {
     Q_OBJECT
@@ -77,6 +78,7 @@ namespace UI
     void displaySoftwareInformation();
     void setCurrentIndex(int);
 };
+
 }
 
 #endif

include/Utilities/Checks.h

@@ -0,0 +1,68 @@
+#ifndef CHECKS_H_
+#define CHECKS_H_
+
+#include<algorithm>
+#include<cstdlib>
+#include<ctype.h>
+
+namespace Utilities
+{
+    class Checks
+    {
+    public:
+        Checks() = delete;
+
+        static auto isNumber(const std::string &val)
+        {
+            return !val.empty() && std::find_if(val.begin(), 
+                val.end(), [](char c)
+                {
+                    return !std::isdigit(c);
+                    }) == val.end();
+        }
+
+        // Note: Not implemented
+        template<typename Item, typename Container, typename Func>
+        static auto itemInContainer(const Container container, const Item item, Func func)
+        {
+            auto result = false;
+            auto i = std::find_if(container.begin(), container.end(), [&](Item i)
+            {
+                return func(item, i);
+            });
+
+            if (i != container.end())
+            {
+                result = true;
+            }
+
+            return result;
+        }
+
+        template<typename Item, typename Container, typename Func>
+        static auto itemIterInContainer(const Container &container, const Item &item, Func func)
+        {
+            auto result = false;
+            // std::cout<<container<<"\n";
+            auto ii = std::find_if(container.begin(), container.end(), [&](Item i)
+            {
+                // std::cout<<"iter "<<i<<" target "<<item<<"\n";
+                return func(i, item);
+            });
+
+            if (ii == container.end())
+            {
+                // std::cout<<item<<" not found in container\n";
+                ii = container.end();
+            }
+
+            return ii;
+        }
+    private:
+    };
+}
+
+#endif
+
+
+

include/Utilities/Conversions.h

@@ -6,17 +6,27 @@
 
 namespace Utilities
 {
+
 class Conversions
 {
 public:
     Conversions();
 
+    static void toLowerChar(char &c)
+    {
+        if (std::isalpha(c))
+        {
+            c = std::tolower(c);
+        }
+    }
+
     void initializeValues();
 
     template<typename T>
-        void printValue(T);
+    void printValue(T val);
 private:
 };
+
 }
 
 #endif

scripts/bld.sh

@@ -1 +0,0 @@
-cmake -H. -B_builds -DHUNTER_STATUS_DEBUG=ON -DCMAKE_BUILD_TYPE=DEBUG

scripts/deb.sh

@@ -1 +0,0 @@
-cmake --build _builds --config Debug

scripts/rel.sh

@@ -1 +0,0 @@
-cmake --build _builds --config Release

src/Main.cpp

@@ -12,6 +12,11 @@ using std::string;
 using Managers::ActionManager;
 using Managers::CommitManager;
 
+constexpr static auto IcarusDownloadManager_version()
+{
+    return "v0.2.0";
+}
+
 void printHelp()
 {
     cout<<"icd [Action] [flag]\n\n";
@@ -31,7 +36,8 @@ void printHelp()
     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<<"-sr directory where to recursively search for songs to upload (Optional)\n";
+    cout<<"-nc will not prompt the user when uploading from a directory\n\n";
 
     cout<<"Required for download\n";
     cout<<"-b song id\n";
@@ -50,13 +56,15 @@ int main(int argc, char** argv)
     if (argc < 2)
     {
         printHelp();
-        return 1;
+        return -1;
     }
 
-    ActionManager actMgr{argv, argc};
+    ActionManager actMgr(argv, argc);
     auto chosenAction = actMgr.retrieveIcarusAction();
 
-    CommitManager commitMgr{chosenAction};
+    chosenAction.print_action_and_flags();
+
+    CommitManager commitMgr(chosenAction);
     commitMgr.commitAction();
 
     return 0;

src/Managers/ActionManager.cpp

@@ -8,6 +8,7 @@
 using std::cout;
 using std::endl;
 using std::string;
+using std::string_view;
 using std::vector;
 
 using Models::Flags;
@@ -15,33 +16,27 @@ using Models::IcarusAction;
 
 namespace Managers
 {
-    #pragma
+
+#pragma region Constructors
 ActionManager::ActionManager(char **param, int paramCount) : 
     params(std::move(param)), paramCount(paramCount)
 {
     initialize();
 }
-    #pragma Constructors
+#pragma endregion
 
 
-    #pragma
+#pragma region Functions
 IcarusAction ActionManager::retrieveIcarusAction() const
 {
-        auto icarusAction = IcarusAction{};
+    IcarusAction icarusAction;
     icarusAction.flags = flags;
     icarusAction.action = action;
 
     return icarusAction;
 }
 
-    bool ActionManager::isNumber(string val)
-    {
-        return !val.empty() && std::find_if(val.begin(), 
-            val.end(), [](char c)
-            {
-                return !std::isdigit(c);
-                }) == val.end();
-    }
+
 
 void ActionManager::initialize()
 {
@@ -53,54 +48,51 @@ namespace Managers
 }
 void ActionManager::validateFlags()
 {
-        cout<<"Validating flags"<<endl;
+        cout<<"Validating flags\n";
 
-        auto flagVals = parsedFlags();
+    const auto flagVals = parsedFlags();
 
-        Flags flg{};
-
-        for (auto flag : flagVals)
+    for (auto flag = flagVals.begin(); flag != flagVals.end(); ++flag)
     {
-            if (flag.size() > 3 || isNumber(flag))
+        Flags flg;
+        cout<<"Value: "<<*flag<<"\n";
+
+        if (isValidFlag<string>(*flag) && doesFlagHaveValue<string>(*flag))
         {
-                flg.value = flag;
-                //cout<<"flag value "<<flg.value<<endl;
-                flags.push_back(flg);
-                flg = Flags{};
-                continue;
+            cout<<"Flag has value\n";
+            flg.flag = *flag;
+            flg.value = *(++flag);
         }
-
-            if (std::any_of(supportedFlags.begin(), supportedFlags.end(), 
-                [&](string& val)
+        else if (isValidFlag<string>(*flag))
         {
-                    return !val.compare(flag);
-                }))
-            {
-                //cout<<"flag "<<flag<<endl;
-                flg.flag = flag;
+            cout<<"Flag does not have a value\n";
+            flg.flag = *flag;
         }
         else
         {
-                cout<<"Action is not valid"<<endl;
+            cout<<"Flag "<<*flag<<" is not valid"<<endl;
             exit(1);
         }
+
+        flags.emplace_back(std::move(flg));
     }
 }
 
 vector<string> ActionManager::parsedFlags()
 {
-        auto parsed = vector<string>{};
+    auto parsed = vector<string>();
     
     for (auto i = 2; i < paramCount; ++i)
     {
-            parsed.push_back(std::move(*(params + i)));
+        const std::string flag(std::move(*(params + i)));
+        parsed.push_back(std::move(flag));
     }
 
     return parsed;
 }
 
-    #pragma
-    void ActionManager::printAction()
+#pragma region Testing
+void ActionManager::printAction() noexcept
 {
     if (action.empty())
     {
@@ -111,7 +103,7 @@ namespace Managers
         cout<<"Action is "<<action<<endl;
     }
 }
-    void ActionManager::printFlags()
+void ActionManager::printFlags() noexcept
 {
     cout<<"\nPrinting flags..."<<endl;
     for (auto flag: flags)
@@ -120,6 +112,7 @@ namespace Managers
         cout<<"value "<<flag.value<<endl;
     }
 }
-    #pragma Testing
-    #pragma Functions
+#pragma endregion
+#pragma endregion
+
 }

src/Managers/CommitManager.cpp

@@ -1,6 +1,11 @@
 #include"Managers/CommitManager.h"
 
 #include <iostream>
+#include <fstream>
+#include <sstream>
+#include <filesystem>
+
+#include "nlohmann/json.hpp"
 
 #include "Models/API.h"
 #include "Models/Song.h"
@@ -31,39 +36,70 @@ using Syncers::Download;
 using Syncers::RetrieveRecords;
 using Syncers::Upload;
 
+namespace filesystem = std::filesystem;
+
 namespace Managers
 {
-	#pragma
+
+#pragma region Constructors
 CommitManager::CommitManager(IcarusAction& icaAct) : icaAction(std::move(icaAct))
 { }
-	#pragma Constructors;
+#pragma endregion
 
 
-	#pragma
+#pragma region Functions
 void CommitManager::commitAction()
 {
     auto action = icaAction.action;
     cout<<"Commiting "<<action<<" action"<<endl;
-		switch (mapActions[action])
+
+    switch (mapActions()[action])
     {
-			case deleteAct:
+        case ActionValues::deleteAct:
             deleteSong();
             break;
-			case downloadAct:
+        case ActionValues::downloadAct:
             downloadSong();
             break;
-			case retrieveAct:
+        case ActionValues::retrieveAct:
             retrieveObjects();
             break;
-			case uploadAct:
+        case ActionValues::uploadAct:
             uploadSong();
             break;
+        case ActionValues::UPLOAD_SONG_WITH_METADATA:
+            uploadSongWithMetadata();
+            break;
+        default:
+            break;
     }
 }
 
+
+
+
+
+
+std::map<std::string, CommitManager::ActionValues> 
+    CommitManager::mapActions() noexcept
+{
+    const std::map<std::string, ActionValues> actions{
+        {"delete", ActionValues::deleteAct},
+        {"download", ActionValues::downloadAct},
+        {"retrieve", ActionValues::retrieveAct},
+        {"upload", ActionValues::uploadAct},
+        {"upload-meta", ActionValues::UPLOAD_SONG_WITH_METADATA}
+    };
+
+    return actions;
+}
+
+
+
+
 Token CommitManager::parseToken(API api)
 {
-        cout<<"fetching token"<<endl;
+    cout<<"fetching token\n";
     UserManager usrMgr{icaAction};
     auto user = usrMgr.retrieveUser();
 
@@ -155,10 +191,12 @@ namespace Managers
     songs.retrieve(retrieveType);
 
 }
+
 void CommitManager::uploadSong()
 {
-        bool uploadSingleSong = true;
-        bool recursiveDirectory = false;
+    auto uploadSingleSong = true;
+    auto recursiveDirectory = false;
+    const auto noConfirm = checkForNoConfirm();
     string songDirectory;
     APIParser apiPrs{icaAction};
     auto api = apiPrs.retrieveAPI();
@@ -189,17 +227,219 @@ namespace Managers
         }
     }
 
-		Upload upld{api};
+    Upload upld{api, token};
     if (uploadSingleSong)
     {
         cout<<"Uploading song..."<<endl;
-		    upld.uploadSong(token, song);
+        upld.uploadSong(song);
     }
     else
     {
         cout<<"Uploading songs from " << songDirectory << endl;
-            upld.uploadSongsFromDirectory(token, songDirectory, recursiveDirectory);
+        upld.uploadSongsFromDirectory(songDirectory, noConfirm, recursiveDirectory);
     }
 }
-	#pragma Functions
+
+
+
+void CommitManager::uploadSongWithMetadata()
+{
+    cout<<"Uploading single song with metadata\n\n";
+
+    // Either the set of "-s", "-m", "-ca", "-t" flags or "-smca" must exist with values
+    // in order to be valid but not both
+    const auto songPath = this->icaAction.retrieveFlagValue("-s");
+    const auto metadataPath = this->icaAction.retrieveFlagValue("-m");
+    const auto coverPath = this->icaAction.retrieveFlagValue("-ca");
+    const auto trackID = this->icaAction.retrieveFlagValue("-t");
+    const auto singleTarget = !songPath.empty() && !metadataPath.empty() && 
+        !coverPath.empty() && !trackID.empty() ? true : false;
+
+    const auto uni = this->icaAction.retrieveFlagValue("-smca");
+    const auto multiTarget = !uni.empty() ? true : false;
+
+    if (singleTarget && multiTarget)
+    {
+        cout<<"Cannot upload from source and directory\n";
+        return;
+    }
+
+    cout<<"Song path: "<<songPath<<"\n";
+    cout<<"TrackID: "<<trackID<<"\n";
+    cout<<"Metadata path: "<<metadataPath<<"\n";
+    cout<<"Cover Art path: "<<coverPath<<"\n";
+
+    if (singleTarget)
+    {
+        singTargetUpload(songPath, trackID, metadataPath, coverPath);
+    }
+    else if (multiTarget)
+    {
+        multiTargetUpload(uni);
+    }
+}
+
+
+void CommitManager::singTargetUpload(const std::string &songPath, const std::string &trackID, 
+    const std::string &metaPath, const std::string &coverPath)
+{
+    APIParser apiPrs(icaAction);
+    auto api = apiPrs.retrieveAPI();
+    const auto token = parseToken(api);
+
+    auto album = retrieveMetadata(metaPath);
+    album.printInfo();
+
+    Song song;
+    song.track = 1;
+    song.disc = 1;
+
+    cout<<"TrackID: "<<trackID<<"\n";
+
+    parseDiscAndTrack<Song, std::string>(song, trackID);
+
+    auto c = [](const Song &songA, const Song &songB) { return songA.track == songB.track && songA.disc == songB.disc; };
+    auto sng = Utilities::Checks::itemIterInContainer<Song, std::vector<Song>>(album.songs, song, c);
+
+    if (sng == album.songs.end())
+    {
+        cout<<"Not found with disc "<<song.disc<<" track "<<song.track<<"\n";
+        std::exit(-1);
+    }
+
+    song = *sng;
+    const auto p = fs::path(songPath);
+    song.directory = p.parent_path.string();
+    song.generate_filename_from_track();
+
+    Models::CoverArt cover;
+    cover.title = song.title;
+    cover.path = coverPath;
+
+    Upload up(api, token);
+    up.uploadSongWithMetadata(album, song, cover);
+}
+
+void CommitManager::multiTargetUpload(const std::string &sourcePath)
+{
+    APIParser apiPrs(icaAction);
+    auto api = apiPrs.retrieveAPI();
+    const auto token = parseToken(api);
+
+    if (!fs::is_directory(sourcePath))
+    {
+        cout<<sourcePath<<" is not a directory\n";
+        std::exit(-1);
+    }
+
+    std::vector<Song> songs;
+    Models::CoverArt cover;
+    string metadataPath;
+
+    for (auto &p: fs::directory_iterator(sourcePath))
+    {
+        const auto &pp = p.path();
+        const auto stem = pp.stem();
+        const auto file = pp.filename();
+        const auto extension = pp.extension();
+
+        cout<<"Stem "<<stem<<" Extension "<<extension<<"\n";
+
+        if (extension.compare(".mp3") == 0)
+        {
+            Song song;
+            song.songPath = pp.string();
+
+            initializeDiscAndTrack<Song, std::string>(song);
+
+            songs.emplace_back(std::move(song));
+        }
+        else if (extension.compare(".jpg") == 0 || extension.compare(".png") == 0)
+        {
+            cover.path.assign(pp.string());
+        }
+        else if (extension.compare(".json") == 0)
+        {
+            metadataPath.assign(pp.string());
+        }
+    }
+
+    auto album = retrieveMetadata(metadataPath);
+    songs.clear();
+    songs.assign(album.songs.begin(), album.songs.end());
+
+    Upload up(api, token);
+
+    for (auto &song : songs)
+    {
+        up.uploadSongWithMetadata(album, song, cover);
+    }
+}
+
+#pragma region private
+CommitManager::Album CommitManager::retrieveMetadata(const std::string_view path)
+{
+    CommitManager::Album album;
+    const auto fileContent = retrieveFileContent(path);
+    cout<<"Parsing...\n";
+    auto serialized = nlohmann::json::parse(fileContent);
+    cout<<"Parsed\n";
+
+    album.album = serialized["album"].get<std::string>();
+    album.albumArtist = serialized["album_artist"].get<std::string>();
+    album.genre = serialized["genre"].get<std::string>();
+    album.year = serialized["year"].get<int>();
+    album.trackCount = serialized["track_count"].get<int>();
+    album.discCount = serialized["disc_count"].get<int>();
+    album.songs.reserve(album.trackCount);
+
+    for (auto &j : serialized["tracks"])
+    {
+        Song song;
+        song.title = j["title"].get<std::string>();
+        song.track = j["track"].get<int>();
+        song.disc = j["disc"].get<int>();
+        song.artist = j["artist"].get<std::string>();
+        song.album = album.album;
+        song.year = album.year;
+        song.genre = album.genre;
+        song.generate_filename_from_track();
+        const auto p = fs::path(path);
+        song.directory = p.parent_path().string();
+
+        album.songs.push_back(song);
+    }
+
+    return album;
+}
+
+string CommitManager::retrieveFileContent(const std::string_view path)
+{
+    string path_str(path);
+    string value;
+
+    std::stringstream buffer;
+    std::fstream file(path_str, std::ios::in);
+    buffer<<file.rdbuf();
+    file.close();
+
+    value.assign(buffer.str());
+
+    return value;
+}
+#pragma endregion
+
+void CommitManager::Album::printInfo()
+{
+    std::cout<<"Album: "<<this->album<<"\n";
+    std::cout<<"Album Artist: "<<this->albumArtist<<"\n";
+    std::cout<<"Genre: "<<this->genre<<"\n";
+    std::cout<<"Year: "<<this->year<<"\n";
+    std::cout<<"Track count: "<<this->trackCount<<"\n";
+    std::cout<<"Disc count: "<<this->discCount<<"\n";
+    std::cout<<"\n";
+}
+
+#pragma region Functions
+
 }

src/Managers/FileManager.cpp

@@ -11,14 +11,18 @@ using std::string;
 
 namespace Managers
 {
+
+#pragma region Constructors
 FileManager::FileManager() {}
 FileManager::FileManager(string filePath)
 {
     this->filePath = filePath;
     readFile();
 }   
+#pragma endregion
 
 
+#pragma region Functions
 void FileManager::saveFile(string newFilePath)
 {
     if (!fileRead)
@@ -63,4 +67,7 @@ namespace Managers
 }
 
 int FileManager::retrieveFileBufferLength() const { return fileBufferLength; }
+
+#pragma endregion
+
 }

src/Managers/TokenManager.cpp

@@ -17,7 +17,8 @@ using Models::User;
 
 namespace Managers
 {
-    #pragma
+
+#pragma region Constructors
 TokenManager::TokenManager(const User& user)
 {
     this->user = user;
@@ -29,10 +30,10 @@ namespace Managers
     this->api.endpoint = "api/" + api.version 
         + "/login";
 }
-    #pragma Constructors
+#pragma endregion
 
 
-    #pragma
+#pragma region Functions
 Token TokenManager::requestToken()
 {
     Token token{};
@@ -56,5 +57,6 @@ namespace Managers
 
     return token;
 }
-    #pragma Functions
+#pragma endregion
+
 }

src/Managers/UserManager.cpp

@@ -12,7 +12,8 @@ using Models::User;
 
 namespace Managers
 {
-    #pragma
+
+#pragma region Constructors
 UserManager::UserManager(User user)
 {
     this->user = user;
@@ -23,10 +24,10 @@ namespace Managers
     this->user = User{};
     parseUserFromActions();
 }
-    #pragma Constructors
+#pragma endregion
 
 
-    #pragma
+#pragma region Functions
 User UserManager::retrieveUser() const
 {
     return user;
@@ -49,5 +50,6 @@ namespace Managers
         }
     }
 }
-    #pragma Functions
+#pragma endregion
+
 }

src/Models/Song.cpp

@@ -0,0 +1,22 @@
+#include "Models/Song.h"
+
+#include "nlohmann/json.hpp"
+
+using std::string;
+
+namespace Models
+{
+    string Song::toMetadataJson()
+    {
+        nlohmann::json s;
+        s["title"] = this->title;
+        s["artist"] = this->artist;
+        s["album"] = this->album;
+        s["genre"] = this->genre;
+        s["year"] = this->year;
+        s["track"] = this->track;
+        s["disc"] = this->disc;
+
+        return s.dump();
+    }
+}

src/Parsers/APIParser.cpp

@@ -10,7 +10,8 @@ using Models::IcarusAction;
 
 namespace Parsers
 {
-    #pragma
+
+#pragma region Constructors
 APIParser::APIParser(IcarusAction icaAct) : icaAct(icaAct)
 {
     api = API{};
@@ -19,7 +20,7 @@ namespace Parsers
 #pragma endregion
 
 
-    #pragma
+#pragma region Functions
 API APIParser::retrieveAPI() const
 {
     return api;
@@ -40,7 +41,6 @@ namespace Parsers
             api.url = (value[value.size()-1] == '/') ? value : value + "/";
             break;
         }
-
     }
 
     // TODO: For now I will hard code
@@ -48,5 +48,6 @@ namespace Parsers
     // on version 1
     api.version = "v1";
 }
-    #pragma functions
+#pragma endregion
+
 }

src/Syncers/Delete.cpp

@@ -16,16 +16,17 @@ using Models::Token;
 
 namespace Syncers
 {
-    #pragma
+
+#pragma region Constructors
 Delete::Delete(API api)
 {
     this->api = api;
     this->api.endpoint = "song/data";
 }
-    #pragma Constructors
+#pragma endregion
 
 
-    #pragma
+#pragma region Functions
 void Delete::deleteSong(const Token token, Song song)
 {
     try
@@ -58,5 +59,6 @@ namespace Syncers
 
     return url;
 }
-    #pragma Functions
+#pragma endregion
+
 }

src/Syncers/Download.cpp

@@ -18,7 +18,8 @@ using Models::Token;
 
 namespace Syncers
 {
-    #pragma
+
+#pragma region Constructors
 Download::Download() { }
 Download::Download(API api)
 {
@@ -29,10 +30,10 @@ namespace Syncers
 {
     downloadFilePath = filePath;
 }
-    #pragma Constructors
+#pragma endregion
 
 
-    #pragma
+#pragma region Functions
 void Download::downloadSong(const Token token, Song song)
 {
     try
@@ -87,5 +88,6 @@ namespace Syncers
     saveSong.write(data, bufferLength);
     saveSong.close();
 }
-    #pragma Functions
+#pragma endregion
+
 }

src/Syncers/RetrieveRecords.cpp

@@ -20,10 +20,13 @@ using Utilities::Conversions;
 namespace Syncers
 {
 
+#pragma region Constructors
 RetrieveRecords::RetrieveRecords() { }
 RetrieveRecords::RetrieveRecords(API api, Token token) 
     : token(token), api(api) { }
+#pragma endregion
 
+#pragma region Functions
 void RetrieveRecords::retrieve(CommitManager::RetrieveTypes type)
 {
     switch (type)
@@ -59,4 +62,6 @@ namespace Syncers
     writeData<<songData.dump(4);
     writeData.close();
 }
+#pragma endregion
+
 }

src/Syncers/Upload.cpp

@@ -2,12 +2,13 @@
 #include <filesystem>
 #include <exception>
 
-#include<cpr/cpr.h>
-#include<nlohmann/json.hpp>
+#include "cpr/cpr.h"
 
 #include "Syncers/Upload.h"
+#include "Utilities/Conversions.h"
 
 using std::cout;
+using std::cin;
 using std::endl;
 using std::exception;
 using std::string;
@@ -23,22 +24,21 @@ using namespace cpr;
 
 namespace Syncers
 {
-    Upload::Upload() { }
-    Upload::Upload(API api) : api(api)
-    {
-        this->api.endpoint = "song/data";
-    }
+
+#pragma region Constructors
+#pragma endregion
 
 
-    Song Upload::uploadSong(const Models::Token& token, Song& song)
+#pragma region Functions
+Song Upload::uploadSong(Song& song)
 {
     try
     {
         auto url = retrieveUrl();
 
         cout<<"url "<<url<<endl;
-            string auth{token.tokenType};
-            auth.append(" " + token.accessToken);
+        string auth{this->m_token.tokenType};
+        auth.append(" " + this->m_token.accessToken);
         auto r = cpr::Post(cpr::Url{url},
                     cpr::Multipart{{"key", "small value"},
                             {"file", cpr::File{song.songPath}}},
@@ -46,6 +46,7 @@ namespace Syncers
             );
 
         cout << "status code: " << r.status_code<< std::endl;
+        cout << r.text << endl;
         auto result = nlohmann::json::parse(r.text);
         cout<<"Finished"<<endl;
         song.id = result["id"].get<int>();
@@ -59,7 +60,7 @@ namespace Syncers
 
         return song;
     }
-        catch (exception e)
+    catch (exception& e)
     {
         auto msg = e.what();
         cout<<msg<<endl;
@@ -68,16 +69,32 @@ namespace Syncers
     return song;
 }
 
-    void Upload::uploadSongsFromDirectory(const Models::Token& token, 
-        const std::string& directory, 
-        bool recursive = false)
+void Upload::uploadSongsFromDirectory(const std::string& directory, 
+    const bool noConfirm, bool recursive = false)
 {
     try
     {
         auto songs = retrieveAllSongsFromDirectory(directory, recursive);
+        auto confirmUpload = true;
+
+        while (!noConfirm) 
+        {
+            auto answer = 'a';
+            cout << "are you sure you want to upload " << songs.size() << " songs? [y/n]";
+            cin >> answer;
+            Utilities::Conversions::toLowerChar(answer);
+
+            if (answer == 'y' || answer == 'Y') 
+            {
+                confirmUpload = true;
+                break;
+            } 
+        }
+
+        cout << "uploading songs\n";
         for (auto& song: songs)
         {
-                song = uploadSong(token, song);
+            song = uploadSong(song);
         }
     }
     catch (exception& e)
@@ -87,10 +104,57 @@ namespace Syncers
 }
 
 
+void Upload::uploadSongWithMetadata(Managers::CommitManager::Album &album, Models::Song& song, Models::CoverArt &cover)
+{
+    this->api.endpoint.assign("song/data/upload/with/data");
+
+    try
+    {
+        auto url = retrieveUrl();
+
+        cout << "url " << url << "\n";
+        string auth(this->m_token.tokenType);
+        auth.append(" " + this->m_token.accessToken);
+
+        nlohmann::json s;
+        s["title"] = song.title;
+        s["album"] = album.album;
+        s["album_artist"] = album.albumArtist;
+        s["artist"] = song.artist;
+        s["year"] = album.year;
+        s["genre"] = album.genre;
+        s["disc"] = song.disc;
+        s["track"] = song.track;
+        s["disc_count"] = album.discCount;
+        s["track_count"] = album.trackCount;
+
+        const auto meta = s.dump();
+
+        cout<<"\n\nMeta:\n"<<meta<<"\n";
+        cout << "Filepath: " << song.song_path() << "\n";
+
+        auto multipart = cpr::Multipart{{"cover", cpr::File{cover.path}},
+            {"metadata", meta},
+            {"file", cpr::File{song.song_path()}}};
+
+        auto r = cpr::Post(cpr::Url{url}, multipart,
+            cpr::Header{{"authorization", auth}}
+        );
+
+        cout << "status code: " << r.status_code<< std::endl;
+        cout << r.text << endl;
+    }
+    catch (exception &e)
+    {
+        auto msg = e.what();
+        cout<<"Error: "<<msg<<"\n";
+    }
+}
 std::vector<Song> Upload::retrieveAllSongsFromDirectory(const std::string& directory,
     bool recursive)
 {
     std::vector<Song> allSongs;
+
     if (recursive)
     {
         for (auto p: fs::recursive_directory_iterator(directory))
@@ -141,18 +205,8 @@ namespace Syncers
 }
 
 
-    #pragma
-    void Upload::printSongDetails()
-    {
-        cout<<"Song details: "<<endl;
-        cout<<"Id: "<<song.id<<endl;
-        cout<<"Title: "<<song.title<<endl;
-        cout<<"Artist: "<<song.artist<<endl;
-        cout<<"Album: "<<song.album<<endl;
-        cout<<"Genre: "<<song.genre<<endl;
-        cout<<"Year: "<<song.year<<endl;
-        cout<<"Duration: "<<song.duration<<endl;
-    }
+#pragma region Testing
+
 void Upload::printSongDetails(std::vector<Song>& songs)
 {
     for (auto& song: songs)
@@ -168,7 +222,8 @@ namespace Syncers
         cout<<"Path: "<<song.songPath<<endl;
     }
 }
-    void Upload::printJsonData(json obj)
+
+void Upload::printJsonData(const json& obj)
 {
     cout<<endl<<endl<<"JSon data: "<<endl;
     cout<<"id: "<<obj["id"]<<endl;
@@ -180,8 +235,9 @@ namespace Syncers
     cout<<"duration: "<<obj["duration"]<<endl;
     cout<<"song_data: "<<obj["song_data"]<<endl;
 
-        cout<<endl<<endl;;
+    cout<<endl<<endl;
 }
-    #pragma Testing
-    #pragma Functions
+#pragma endregion
+#pragma endregion
+
 }

src/UI/AboutWindow.cpp

@@ -4,12 +4,15 @@ using std::unique_ptr;
 
 namespace UI
 {
+#pragma region Constructors
 AboutWindow::AboutWindow(QWidget* parent): QDialog(parent)
 {
     setupWindow();
 }
+#pragma endregion
 
 
+#pragma region Functions
 void AboutWindow::setupWindow()
 {
     windowWidth = 250;
@@ -44,4 +47,6 @@ namespace UI
 {
     this->hide();   
 }
+#pragma endregion
+
 }

src/UI/MainWindow.cpp

@@ -17,13 +17,17 @@ using Syncers::Upload;
 
 namespace UI
 {
+
+#pragma region Constructors
 MainWindow::MainWindow() 
 {
     setupMainWindow();
     aboutWindow = unique_ptr<AboutWindow>{new AboutWindow};
 }
+#pragma endregion
 
 
+#pragma region Functions
 void MainWindow::configureDownloadSection()
 {
 }
@@ -171,4 +175,6 @@ namespace UI
 
     uploadSongQt->setEnabled(true);
 }
+#pragma endregion
+
 }

src/Utilities/Conversions.cpp

@@ -7,6 +7,7 @@ using std::unique_ptr;
 
 namespace Utilities
 {
+
 Conversions::Conversions()
 {
     initializeValues();
@@ -18,7 +19,8 @@ namespace Utilities
 template <typename T>
 void Conversions::printValue(T val)
 {
-        std::cout<<"going to print value"<<std::endl;
-        std::cout<<val<<std::endl;
+    std::cout<<"going to print value\n";
+    std::cout<<val<< "\n";
 }
+
 }