ENH: Add datatypes and value classes to Extern C interface

Summary

See https://gitlab.indiscale.com/caosdb/customers/lfpb/management/-/issues/420. Adapts the

Focus

The single get_value functions are now split up into get_<datatype>_value, get_value_list_length, and get_<datatype>_list_value_at functions. Also, importances, roles, and datatypes are now cast from string to enums and back for getting and setting. In the case of datatypes, two additional Booleans for is_reference and is_list.

In addition, the way memory si allocated in case of char pointers has been fixed.

Test Environment

Unit tests and integration tests should be sufficient. Integration testing with deploy in f-grpc and server in f-grpc-f-consolidation, as usual.

Check List for the Author

Please, prepare your MR for a review. Be sure to write a summary and a focus and create gitlab comments for the reviewer. They should guide the reviewer through the changes, explain your changes and also point out open questions. For further good practices have a look at our review guidelines

• All automated tests pass
• Reference related Issues
• Up-to-date CHANGELOG.md
• Annotations in code (Gitlab comments)
  • Intent of new code
  • Problems with old code
  • Why this implementation?

Check List for the Reviewer

• I understand the intent of this MR
• All automated tests pass
• Up-to-date CHANGELOG.md
• The test environment setup works and the intended behavior is reproducible in the test environment
• In-code documentation and comments are up-to-date.
• Check: Are there specifications? Are they satisfied?

For further good practices have a look at our review guidelines.

include/ccaosdb.h

@@ -341,8 +341,23 @@ int caosdb_entity_entity_get_description(caosdb_entity_entity *entity,
                                          char *out);
 int caosdb_entity_entity_get_datatype(caosdb_entity_entity *entity, char *out);
 int caosdb_entity_entity_get_unit(caosdb_entity_entity *entity, char *out);
-// TODO(fspreck)replace by more specific getters
-// int caosdb_entity_entity_get_value(caosdb_entity_entity *entity, char *out);
+
+int caosdb_entity_entity_get_int_value(caosdb_entity_entity *entity, long *out);
+int caosdb_entity_entity_get_double_value(caosdb_entity_entity *entity,
+                                          double *out);
+int caosdb_entity_entity_get_boolean_value(caosdb_entity_entity *entity,
+                                           bool *out);
+int caosdb_entity_entity_get_string_value(caosdb_entity_entity *entity,
+                                          char *out);
+int caosdb_entity_entity_get_int_list_value(caosdb_entity_entity *entity,
+                                            long **out);
+int caosdb_entity_entity_get_double_list_value(caosdb_entity_entity *entity,
+                                               double **out);
+int caosdb_entity_entity_get_boolean_list_value(caosdb_entity_entity *entity,
+                                                bool **out);
+int caosdb_entity_entity_get_string_list_value(caosdb_entity_entity *entity,
+                                               char **out);
+
 int caosdb_entity_entity_get_version_id(caosdb_entity_entity *entity,
                                         char *out);
 int caosdb_entity_entity_get_errors_size(caosdb_entity_entity *entity,
@@ -376,9 +391,23 @@ int caosdb_entity_property_get_datatype(caosdb_entity_property *property,
                                         char *out);
 int caosdb_entity_property_get_unit(caosdb_entity_property *property,
                                     char *out);
-// TODO(fspreck) replace by more specific getter
-// int caosdb_entity_property_get_value(caosdb_entity_property *property,
-//                                char *out);
+
+int caosdb_entity_property_get_int_value(caosdb_entity_property *property,
+                                         long *out);
+int caosdb_entity_property_get_double_value(caosdb_entity_property *property,
+                                            double *out);
+int caosdb_entity_property_get_boolean_value(caosdb_entity_property *property,
+                                             bool *out);
+int caosdb_entity_property_get_string_value(caosdb_entity_property *property,
+                                            char *out);
+int caosdb_entity_property_get_int_list_value(caosdb_entity_property *property,
+                                              long **out);
+int caosdb_entity_property_get_double_list_value(
+  caosdb_entity_property *property, double **out);
+int caosdb_entity_property_get_boolean_list_value(
+  caosdb_entity_property *property, bool **out);
+int caosdb_entity_property_get_string_list_value(
+  caosdb_entity_property *property, char **out);
 
 int caosdb_entity_parent_get_id(caosdb_entity_parent *parent, char *out);
 int caosdb_entity_parent_get_name(caosdb_entity_parent *parent, char *out);
@@ -409,8 +438,27 @@ int caosdb_entity_entity_set_datatype(caosdb_entity_entity *entity,
 int caosdb_entity_entity_set_unit(caosdb_entity_entity *entity,
                                   const char *unit);
 // TODO(fspreck) replace by more specific setters
-// int caosdb_entity_entity_set_value(caosdb_entity_entity *entity,
-//                               const char *value);
+int caosdb_entity_entity_set_int_value(caosdb_entity_entity *entity,
+                                       const long *value);
+int caosdb_entity_entity_set_double_value(caosdb_entity_entity *entity,
+                                          const double *value);
+int caosdb_entity_entity_set_boolean_value(caosdb_entity_entity *entity,
+                                           const bool *value);
+int caosdb_entity_entity_set_string_value(caosdb_entity_entity *entity,
+                                          const char *value);
+int caosdb_entity_entity_set_int_list_value(caosdb_entity_entity *entity,
+                                            const long **value,
+                                            const int length);
+int caosdb_entity_entity_set_double_list_value(caosdb_entity_entity *entity,
+                                               const double **value,
+                                               const int length);
+int caosdb_entity_entity_set_boolean_list_value(caosdb_entity_entity *entity,
+                                                const bool **value,
+                                                const int length);
+int caosdb_entity_entity_set_string_list_value(caosdb_entity_entity *entity,
+                                               const char **value,
+                                               const int length);
+
 int caosdb_entity_entity_append_parent(caosdb_entity_entity *entity,
                                        caosdb_entity_parent *parent);
 int caosdb_entity_entity_remove_parent(caosdb_entity_entity *entity, int index);
@@ -429,9 +477,24 @@ int caosdb_entity_property_set_importance(caosdb_entity_property *property,
                                           const char *importance);
 int caosdb_entity_property_set_unit(caosdb_entity_property *property,
                                     const char *unit);
-// TODO(fspreck) replace by more specific setters
-// int caosdb_entity_property_set_value(caosdb_entity_property *property,
-//                                 const char *value);
+
+int caosdb_entity_property_set_int_value(caosdb_entity_property *property,
+                                         const long *value);
+int caosdb_entity_property_set_double_value(caosdb_entity_property *property,
+                                            const double *value);
+int caosdb_entity_property_set_boolean_value(caosdb_entity_property *property,
+                                             const bool *value);
+int caosdb_entity_property_set_string_value(caosdb_entity_property *property,
+                                            const char *value);
+int caosdb_entity_property_set_int_list_value(caosdb_entity_property *property,
+                                              const long **value,
+                                              const int length);
+int caosdb_entity_property_set_double_list_value(
+  caosdb_entity_property *property, const double **value, const int length);
+int caosdb_entity_property_set_boolean_list_value(
+  caosdb_entity_property *property, const bool **value, const int length);
+int caosdb_entity_property_set_string_list_value(
+  caosdb_entity_property *property, const char **value, const int length);
 
 int caosdb_entity_parent_set_id(caosdb_entity_parent *parent, const char *id);
 int caosdb_entity_parent_set_name(caosdb_entity_parent *parent,