Skip to content
Snippets Groups Projects
Commit 2af71aa3 authored by Julian Tusch's avatar Julian Tusch :no_entry_sign:
Browse files

added null pointer check in skill conversion

parent 987175b8
No related branches found
No related tags found
3 merge requests!460Draft: fluxio/dev-skill-timeout,!449Fluxio preliminary release,!446Draft: Fluxio related changes
Pipeline #18697 failed
#include "FluxioSkill.h"
#include <vector>
#include <ArmarXCore/core/logging/Logging.h>
#include "FluxioParameterNode.h"
#include "FluxioNode.h"
#include "FluxioEdge.h"
......@@ -13,6 +14,12 @@ namespace armarx
manager::dto::FluxioSkill
FluxioSkill::toManagerIce() const
{
if(skillProviderPtr == nullptr)
{
ARMARX_WARNING << "SkillProvider for Skill with id " << id << " not set"; // TODO: create general error message structure
return {};
}
manager::dto::FluxioSkill ret;
ret.id = id;
......
......@@ -149,7 +149,7 @@ namespace armarx
}
//** Fluxio related methods
skills::manager::dto::FluxioSkillList
SkillManagerComponentPluginUser::getSkillList(const Ice::Current& current)
{
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment