Skip to content

Commit

Permalink
Moved ISO Data to a location where it's more likely that we'll have w…
Browse files Browse the repository at this point in the history 
…rite access

Moved ISO data to the user's app data directory, since that is often easier to get write permissions for.
Fixed compilation issue in ASCIILogFile when using updated AgIsoStack.
  • Loading branch information
@ad3154
ad3154 committed Apr 24, 2024
1 parent 6465e2b commit 569ae47
Show file tree
Hide file tree
Showing 2 changed files with 25 additions and 20 deletions.
4 changes: 2 additions & 2 deletions include/ASCIILogFile.hpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,8 @@ class ASCIILogFile

private:
File logFile;
std::shared_ptr<void> canFrameReceivedListener;
std::shared_ptr<void> canFrameSentListener;
isobus::EventCallbackHandle canFrameReceivedListener;
isobus::EventCallbackHandle canFrameSentListener;
Time initialTimestamp;
};

Expand Down
41 changes: 23 additions & 18 deletions src/ServerMainComponent.cpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -180,7 +180,7 @@ std::vector<std::array<std::uint8_t, 7>> ServerMainComponent::get_versions(isobu
std::ostringstream nameString;
std::vector<std::array<std::uint8_t, 7>> retVal;
nameString << std::hex << std::setfill('0') << std::setw(16) << clientNAME.get_full_name();
File isoDirectory(std::filesystem::current_path().string() + File::getSeparatorString() + ISO_DATA_PATH + File::getSeparatorString() + nameString.str());
File isoDirectory(File::getSpecialLocation(File::userApplicationDataDirectory).getFullPathName().toStdString() + File::getSeparatorString() + ISO_DATA_PATH + File::getSeparatorString() + nameString.str());

if (isoDirectory.exists() && isoDirectory.isDirectory())
{
Expand Down Expand Up @@ -232,13 +232,14 @@ std::vector<std::uint8_t> ServerMainComponent::load_version(const std::vector<st
std::ostringstream nameString;
std::vector<std::uint8_t> loadedIOPData;
std::vector<std::uint8_t> loadedVersionLabel(7);
auto path = (File::getSpecialLocation(File::userApplicationDataDirectory).getFullPathName().toStdString() + ISO_DATA_PATH + File::getSeparatorString()).toStdString();
nameString << std::hex << std::setfill('0') << std::setw(16) << clientNAME.get_full_name();

if ((std::filesystem::is_directory(ISO_DATA_PATH + "/" + nameString.str()) ||
std::filesystem::exists(ISO_DATA_PATH + "/" + nameString.str())) &&
if ((std::filesystem::is_directory(path + nameString.str()) ||
std::filesystem::exists(path + nameString.str())) &&
(7 == versionLabel.size()))
{
for (const auto &entry : std::filesystem::directory_iterator(ISO_DATA_PATH + "/" + nameString.str()))
for (const auto &entry : std::filesystem::directory_iterator(path + nameString.str()))
{
if (entry.path().has_extension() && entry.path().extension() == ".iopx")
{
Expand Down Expand Up @@ -277,24 +278,26 @@ std::vector<std::uint8_t> ServerMainComponent::load_version(const std::vector<st
bool ServerMainComponent::save_version(const std::vector<std::uint8_t> &objectPool, const std::vector<std::uint8_t> &versionLabel, isobus::NAME clientNAME)
{
bool retVal = false;
auto userAppData = File::getSpecialLocation(File::userApplicationDataDirectory);
std::string path = (userAppData.getFullPathName() + File::getSeparatorString() + String(ISO_DATA_PATH)).toStdString();

// Main saved data folder
if (!std::filesystem::is_directory(ISO_DATA_PATH) || !std::filesystem::exists(ISO_DATA_PATH))
{ // Check if src folder exists
std::filesystem::create_directory(ISO_DATA_PATH); // create src folder
if (!std::filesystem::is_directory(path) || !std::filesystem::exists(path))
{
std::filesystem::create_directory(path);
}

// NAME specific folder
std::ostringstream nameString;
nameString << std::hex << std::setfill('0') << std::setw(16) << clientNAME.get_full_name();

if (!std::filesystem::is_directory(ISO_DATA_PATH + "/" + nameString.str()) || !std::filesystem::exists(ISO_DATA_PATH + "/" + nameString.str()))
if (!std::filesystem::is_directory(path + "/" + nameString.str()) || !std::filesystem::exists(path + "/" + nameString.str()))
{ // Check if src folder exists
std::filesystem::create_directory(ISO_DATA_PATH + "/" + nameString.str()); // create src folder
std::filesystem::create_directory(path + "/" + nameString.str()); // create src folder
}

std::ofstream iopxFile(ISO_DATA_PATH + "/" + nameString.str() + "/object_pool_with_label_" + std::to_string(number_of_iop_files_in_directory(ISO_DATA_PATH + "/" + nameString.str())) + ".iopx", std::ios::trunc | std::ios::binary);
std::ofstream iopFile(ISO_DATA_PATH + "/" + nameString.str() + "/object_pool_" + std::to_string(number_of_iop_files_in_directory(ISO_DATA_PATH + "/" + nameString.str())) + ".iop", std::ios::trunc | std::ios::binary);
std::ofstream iopxFile(path + "/" + nameString.str() + "/object_pool_with_label_" + std::to_string(number_of_iop_files_in_directory(path + "/" + nameString.str())) + ".iopx", std::ios::trunc | std::ios::binary);
std::ofstream iopFile(path + "/" + nameString.str() + "/object_pool_" + std::to_string(number_of_iop_files_in_directory(path + "/" + nameString.str())) + ".iop", std::ios::trunc | std::ios::binary);

if (iopxFile.is_open())
{
Expand All @@ -317,13 +320,14 @@ bool ServerMainComponent::delete_version(const std::vector<std::uint8_t> &versio
std::ostringstream nameString;
std::vector<std::uint8_t> loadedVersionLabel(7);
std::vector<std::filesystem::directory_entry> filesToRemove;
auto path = (File::getSpecialLocation(File::userApplicationDataDirectory).getFullPathName().toStdString() + ISO_DATA_PATH + File::getSeparatorString()).toStdString();
nameString << std::hex << std::setfill('0') << std::setw(16) << clientNAME.get_full_name();

if ((std::filesystem::is_directory(ISO_DATA_PATH + "/" + nameString.str()) ||
std::filesystem::exists(ISO_DATA_PATH + "/" + nameString.str())) &&
if ((std::filesystem::is_directory(path + nameString.str()) ||
std::filesystem::exists(path + nameString.str())) &&
(7 == versionLabel.size()))
{
for (const auto &entry : std::filesystem::directory_iterator(ISO_DATA_PATH + "/" + nameString.str()))
for (const auto &entry : std::filesystem::directory_iterator(path + nameString.str()))
{
if (entry.path().has_extension() && entry.path().extension() == ".iopx")
{
Expand Down Expand Up @@ -371,12 +375,13 @@ bool ServerMainComponent::delete_all_versions(isobus::NAME clientNAME)
std::ostringstream nameString;
std::vector<std::uint8_t> loadedVersionLabel(7);
std::vector<std::filesystem::directory_entry> filesToRemove;
auto path = (File::getSpecialLocation(File::userApplicationDataDirectory).getFullPathName().toStdString() + ISO_DATA_PATH + File::getSeparatorString()).toStdString();
nameString << std::hex << std::setfill('0') << std::setw(16) << clientNAME.get_full_name();

if ((std::filesystem::is_directory(ISO_DATA_PATH + "/" + nameString.str()) ||
std::filesystem::exists(ISO_DATA_PATH + "/" + nameString.str())))
if ((std::filesystem::is_directory(path + nameString.str()) ||
std::filesystem::exists(path + nameString.str())))
{
for (const auto &entry : std::filesystem::directory_iterator(ISO_DATA_PATH + "/" + nameString.str()))
for (const auto &entry : std::filesystem::directory_iterator(path + nameString.str()))
{
if (entry.path().has_extension() && entry.path().extension() == ".iopx")
{
Expand Down Expand Up @@ -1488,7 +1493,7 @@ void ServerMainComponent::remove_working_set(std::shared_ptr<isobus::VirtualTerm

void ServerMainComponent::clear_iso_data()
{
File isoDirectory(std::filesystem::current_path().string() + File::getSeparatorString() + ISO_DATA_PATH + File::getSeparatorString());
File isoDirectory(File::getSpecialLocation(File::userApplicationDataDirectory).getFullPathName().toStdString() + File::getSeparatorString() + ISO_DATA_PATH + File::getSeparatorString());

if (isoDirectory.exists() && isoDirectory.isDirectory())
{
Expand Down

0 comments on commit 569ae47

Please sign in to comment.