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

@@ -334,13 +334,13 @@ typedef struct {
 } caosdb_entity_message;
 
 // GETTERS FOR EVERYTHING
-int caosdb_entity_entity_get_id(caosdb_entity_entity *entity, char *out);
-int caosdb_entity_entity_get_role(caosdb_entity_entity *entity, char *out);
-int caosdb_entity_entity_get_name(caosdb_entity_entity *entity, char *out);
+int caosdb_entity_entity_get_id(caosdb_entity_entity *entity, char **out);
+int caosdb_entity_entity_get_role(caosdb_entity_entity *entity, char **out);
+int caosdb_entity_entity_get_name(caosdb_entity_entity *entity, char **out);
 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);
+                                         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);
 
 int caosdb_entity_entity_get_int_value(caosdb_entity_entity *entity, long *out);
 int caosdb_entity_entity_get_double_value(caosdb_entity_entity *entity,
@@ -348,18 +348,20 @@ int caosdb_entity_entity_get_double_value(caosdb_entity_entity *entity,
 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);
+                                          char **out);
 int caosdb_entity_entity_get_int_list_value(caosdb_entity_entity *entity,
-                                            long **out);
+                                            long *out);
 int caosdb_entity_entity_get_double_list_value(caosdb_entity_entity *entity,
-                                               double **out);
+                                               double *out);
 int caosdb_entity_entity_get_boolean_list_value(caosdb_entity_entity *entity,
-                                                bool **out);
+                                                bool *out);
 int caosdb_entity_entity_get_string_list_value(caosdb_entity_entity *entity,
                                                char **out);
+int caosdb_entity_entity_get_value_list_length(caosdb_entity_entity *entity,
+                                               int *out);
 
 int caosdb_entity_entity_get_version_id(caosdb_entity_entity *entity,
-                                        char *out);
+                                        char **out);
 int caosdb_entity_entity_get_errors_size(caosdb_entity_entity *entity,
                                          int *out);
 int caosdb_entity_entity_get_error(caosdb_entity_entity *entity,
@@ -380,17 +382,17 @@ int caosdb_entity_entity_get_parents_size(caosdb_entity_entity *entity,
 int caosdb_entity_entity_get_parent(caosdb_entity_entity *entity,
                                     caosdb_entity_parent *out, int index);
 
-int caosdb_entity_property_get_id(caosdb_entity_property *property, char *out);
+int caosdb_entity_property_get_id(caosdb_entity_property *property, char **out);
 int caosdb_entity_property_get_name(caosdb_entity_property *property,
-                                    char *out);
+                                    char **out);
 int caosdb_entity_property_get_description(caosdb_entity_property *property,
-                                           char *out);
+                                           char **out);
 int caosdb_entity_property_get_importance(caosdb_entity_property *property,
-                                          char *out);
+                                          char **out);
 int caosdb_entity_property_get_datatype(caosdb_entity_property *property,
-                                        char *out);
+                                        char **out);
 int caosdb_entity_property_get_unit(caosdb_entity_property *property,
-                                    char *out);
+                                    char **out);
 
 int caosdb_entity_property_get_int_value(caosdb_entity_property *property,
                                          long *out);
@@ -399,24 +401,26 @@ int caosdb_entity_property_get_double_value(caosdb_entity_property *property,
 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);
+                                            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);
+  caosdb_entity_property *property, double *out);
 int caosdb_entity_property_get_boolean_list_value(
-  caosdb_entity_property *property, bool **out);
+  caosdb_entity_property *property, bool *out);
 int caosdb_entity_property_get_string_list_value(
-  caosdb_entity_property *property, char **out);
+  caosdb_entity_property *property, char *out[]);
+int caosdb_entity_property_get_value_list_length(
+  caosdb_entity_property *property, int *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);
+int caosdb_entity_parent_get_id(caosdb_entity_parent *parent, char **out);
+int caosdb_entity_parent_get_name(caosdb_entity_parent *parent, char **out);
 int caosdb_entity_parent_get_description(caosdb_entity_parent *parent,
-                                         char *out);
+                                         char **out);
 
 int caosdb_entity_message_get_code(caosdb_entity_message *message, int *out);
 int caosdb_entity_message_get_description(caosdb_entity_message *message,
-                                          char *out);
+                                          char **out);
 
 // CONSTRUCTORS AND DESTRUCTORS
 int caosdb_entity_create_entity(caosdb_entity_entity *out);