WIP: Repository class refactoring

include/cloysterhpc/services/repo.h

@@ -58,25 +58,27 @@ class repository_exception : public std::runtime_error {
     }
 };
 
+
+
 /**
  * Generic file class
  *
- * This class should only read and write,
+ * This class should only read and write, 
  */
 class GenericFile {
 protected:
     std::filesystem::path m_path;
-
 public:
     GenericFile(const std::filesystem::path& path)
         : m_path(path)
-    {
-    }
+    {}
 
-    virtual void read() { }
-    virtual void write() { }
+    virtual void read() {}
+    virtual void write() {}
 };
 
+
+
 struct ELRepo {
     std::string group;
     std::string name;
@@ -88,28 +90,33 @@ struct ELRepo {
 
     // (P/ todos os repositórios)
     // std::string release;
+
 };
 
+
 /**
  * Repository file class
  *
  * This class should parse the repository data
  */
 class ELRepoFile : GenericFile {
-private:
+private:    
     Glib::RefPtr<Glib::KeyFile> m_file;
 
+    std::vector<ELRepo> parseData();
+    void unparseData(const std::vector<ELRepo>&);
 public:
     ELRepoFile(const std::filesystem::path& path)
         : GenericFile(path)
-    {
-    }
+    {}
 
     virtual void read() override;
     virtual void write() override;
 
     std::vector<ELRepo> parse();
-    void unparse(const std::vector<ELRepo>& repositories);
+    std::vector<ELRepo> parse(const std::stringstream& ss);
+    void unparse( const std::vector<ELRepo>& repositories );
+    void unparse( const std::vector<ELRepo>& repositories, std::stringstream& ss );
 };
 
 #endif // CLOYSTERHPC_REPO_H_

src/services/repo.cpp

@@ -75,6 +75,19 @@ std::vector<ELRepo> ELRepoFile::parse()
 {
     this->read();
 
+    return this->parseData();
+}
+
+std::vector<ELRepo> ELRepoFile::parse(const std::stringstream& ss)
+{
+    m_file = Glib::KeyFile::create();
+    m_file->load_from_data(ss.str().c_str());
+    return this->parseData();
+}
+
+
+std::vector<ELRepo>  ELRepoFile::parseData()
+{
     auto reponames = m_file->get_groups();
 
     std::vector<ELRepo> repositories;
@@ -117,15 +130,26 @@ std::vector<ELRepo> ELRepoFile::parse()
     return repositories;
 }
 
-void ELRepoFile::unparse(const std::vector<ELRepo>& repositories)
-{
-
+void ELRepoFile::unparseData(const std::vector<ELRepo>& repositories)
+{    
     for (const auto& repo : repositories) {
         m_file->set_string(repo.group, "name", repo.name);
         m_file->set_boolean(repo.group, "enabled", repo.enabled);
         m_file->set_boolean(repo.group, "gpgcheck", repo.gpgcheck);
         m_file->set_string(repo.group, "gpgkey", repo.gpgkey);
-    }
-
+    }    
+}
+
+void ELRepoFile::unparse(const std::vector<ELRepo>& repositories)
+{
+    this->unparseData(repositories);
     this->write();
 }
+
+
+void ELRepoFile::unparse(const std::vector<ELRepo>& repositories, std::stringstream& ss)
+{
+    this->unparseData(repositories);
+    ss.seekp(0);
+    ss << m_file->to_data();
+}