Make getProfile optional and implement it

Co-authored-by: Benedikt Engel <uyqhv@students.kit.edu>
Co-authored-by: Julian Tusch <urhrf@student.kit.edu>
Co-authored-by: Peter Albrecht <usnlf@student.kit.edu>

source/RobotAPI/interface/skills/SkillManagerInterface.ice

@@ -247,7 +247,7 @@ module armarx
                     dto::FluxioProviderList
                     getProviderList();
 
-                    dto::FluxioProvider
+                    optional(5) dto::FluxioProvider
                     getProvider(string id);
 
                     dto::FluxioSkillList

source/RobotAPI/libraries/skills/manager/SkillManagerComponentPlugin.cpp

@@ -619,7 +619,7 @@ namespace armarx::plugins
                 p.id = skillId.providerId->providerName;
                 p.name = p.id;
 
-                fluxioDC.providers[p.id] = p;       
+                fluxioDC.providers[p.id] = p;
                 s.skillProviderPtr = &p;
             }
             
@@ -643,7 +643,9 @@ namespace armarx::plugins
         return std::nullopt;
     }
 
-    void SkillManagerComponentPlugin::updateSkill(const skills::FluxioSkill& skill){}
+    void SkillManagerComponentPlugin::updateSkill(const skills::FluxioSkill& skill){
+        
+    }
 
     void SkillManagerComponentPlugin::removeSkill(const std::string& id){}
 
@@ -685,13 +687,18 @@ namespace armarx::plugins
     }
 
     std::vector<skills::FluxioProvider> SkillManagerComponentPlugin::getProviderList(){
-        std::vector<skills::FluxioProvider> ret;
-        return ret;
+        getSkillList(); //TODO: put the refreshProvider functionality in a separate method
+        return convertMapValuesToVector(fluxioDC.providers);
     }
 
-    skills::FluxioProvider SkillManagerComponentPlugin::getProvider(const std::string& id){
-        skills::FluxioProvider ret;
-        return ret;
+    std::optional<skills::FluxioProvider> SkillManagerComponentPlugin::getProvider(const std::string& id){
+        getSkillList(); //TODO: put the refreshProvider functionality in a separate method
+        const auto& providersEntry = fluxioDC.providers.find(id);
+
+        if (providersEntry != fluxioDC.providers.end()) {
+            return providersEntry->second;
+        }
+        return std::nullopt;
     }
 
     std::vector<skills::FluxioSkill> SkillManagerComponentPlugin::getSkillsOfProvider(const std::string& id){

source/RobotAPI/libraries/skills/manager/SkillManagerComponentPlugin.h

@@ -91,7 +91,7 @@ namespace armarx::plugins
 
         std::vector<skills::FluxioProvider> getProviderList();
 
-        skills::FluxioProvider getProvider(const std::string& id);
+        std::optional<skills::FluxioProvider> getProvider(const std::string& id);
 
         std::vector<skills::FluxioSkill> getSkillsOfProvider(const std::string& id);
 

source/RobotAPI/libraries/skills/manager/SkillManagerComponentPluginUser.cpp

@@ -261,9 +261,14 @@ namespace armarx
         return ret;
     }
 
-    skills::manager::dto::FluxioProvider 
+    IceUtil::Optional<skills::manager::dto::FluxioProvider>
     SkillManagerComponentPluginUser::getProvider(const std::string& id,const Ice::Current& current){
-        return this->plugin->getProvider(id).toManagerIce();
+        auto provider = this->plugin->getProvider(id);
+
+        if (provider.has_value()) {
+            return provider->toManagerIce();
+        }
+        return {};
     }
 
     skills::manager::dto::FluxioSkillList

source/RobotAPI/libraries/skills/manager/SkillManagerComponentPluginUser.h

@@ -94,7 +94,7 @@ namespace armarx
         skills::manager::dto::FluxioProviderList
         getProviderList(const Ice::Current& current) override;
 
-        skills::manager::dto::FluxioProvider 
+        IceUtil::Optional<skills::manager::dto::FluxioProvider>
         getProvider(const std::string& id,const Ice::Current& current) override;
 
         skills::manager::dto::FluxioSkillList