From 0f3597bdcc7261009529c7d0b7ef4ded799a0228 Mon Sep 17 00:00:00 2001
From: Reinder Feenstra <reinderfeenstra@gmail.com>
Date: Thu, 26 May 2022 09:57:54 +0200
Subject: [PATCH] fix: server:  internal properties/methods could be set/called

---
 server/src/network/session.cpp | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/server/src/network/session.cpp b/server/src/network/session.cpp
index d62f8b88..b19f6427 100644
--- a/server/src/network/session.cpp
+++ b/server/src/network/session.cpp
@@ -157,7 +157,7 @@ bool Session::processMessage(const Message& message)
       {
         if(ObjectPtr object = m_handles.getItem(message.read<Handle>()))
         {
-          if(AbstractProperty* property = object->getProperty(message.read<std::string>()))
+          if(AbstractProperty* property = object->getProperty(message.read<std::string>()); property && !property->isInternal())
           {
             try
             {
@@ -206,7 +206,7 @@ bool Session::processMessage(const Message& message)
     {
       if(ObjectPtr object = m_handles.getItem(message.read<Handle>()))
       {
-        if(AbstractUnitProperty* property = dynamic_cast<AbstractUnitProperty*>(object->getProperty(message.read<std::string>())))
+        if(AbstractUnitProperty* property = dynamic_cast<AbstractUnitProperty*>(object->getProperty(message.read<std::string>())); property && !property->isInternal())
         {
           try
           {
@@ -225,7 +225,7 @@ bool Session::processMessage(const Message& message)
     {
       if(ObjectPtr object = m_handles.getItem(message.read<Handle>()))
       {
-        if(AbstractObjectProperty* property = dynamic_cast<AbstractObjectProperty*>(object->getProperty(message.read<std::string>())))
+        if(AbstractObjectProperty* property = dynamic_cast<AbstractObjectProperty*>(object->getProperty(message.read<std::string>())); property && !property->isInternal())
         {
           try
           {
@@ -253,7 +253,7 @@ bool Session::processMessage(const Message& message)
     {
       if(ObjectPtr object = m_handles.getItem(message.read<Handle>()))
       {
-        if(AbstractMethod* method = object->getMethod(message.read<std::string>()))
+        if(AbstractMethod* method = object->getMethod(message.read<std::string>()); method && !method->isInternal())
         {
           const ValueType resultType = message.read<ValueType>();
           const uint8_t argumentCount = message.read<uint8_t>();