fix(xml): add the missing parsing of min/max subject limits (#8971)

src/others/xml/lv_xml_component.c

@@ -486,10 +486,20 @@ static void process_subject_element(lv_xml_parser_state_t * state, const char *
     }
 
     lv_subject_t * subject = lv_zalloc(sizeof(lv_subject_t));
+    const char * min_value_str = lv_xml_get_value_of(attrs, "min_value");
+    const char * max_value_str = lv_xml_get_value_of(attrs, "max_value");
 
-    if(lv_streq(type, "int")) lv_subject_init_int(subject, lv_xml_atoi(value));
+    if(lv_streq(type, "int")) {
+        lv_subject_init_int(subject, lv_xml_atoi(value));
+        if(min_value_str) lv_subject_set_min_value_int(subject, lv_xml_atoi(min_value_str));
+        if(max_value_str) lv_subject_set_max_value_int(subject, lv_xml_atoi(max_value_str));
+    }
 #if LV_USE_FLOAT
-    else if(lv_streq(type, "float")) lv_subject_init_float(subject, lv_xml_atof(value));
+    else if(lv_streq(type, "float")) {
+        lv_subject_init_float(subject, lv_xml_atof(value));
+        if(min_value_str) lv_subject_set_min_value_float(subject, lv_xml_atof(min_value_str));
+        if(max_value_str) lv_subject_set_max_value_float(subject, lv_xml_atof(max_value_str));
+    }
 #endif
     else if(lv_streq(type, "color")) lv_subject_init_color(subject, lv_xml_to_color(value));
     else if(lv_streq(type, "string")) {

tests/src/test_cases/xml/test_xml_subject.c

@@ -0,0 +1,45 @@
+#if LV_BUILD_TEST
+#include "../lvgl.h"
+
+#include "unity/unity.h"
+
+void setUp(void)
+{
+    /* Function run before every test */
+}
+
+void tearDown(void)
+{
+    /* Function run after every test */
+    lv_obj_clean(lv_screen_active());
+}
+
+void test_xml_subject_int(void)
+{
+    const char * xml = "<globals>"
+                       "	<subjects>"
+                       "		<int name=\"s1\" value=\"10\"/>"
+                       "		<int name=\"s2\" value=\"20\" min_value=\"10\" max_value=\"30\"/>"
+                       "	</subjects>"
+                       "</globals>";
+
+    lv_xml_component_register_from_data("globals", xml);
+
+    lv_subject_t * s1 = lv_xml_get_subject(NULL, "s1");
+    lv_subject_t * s2 = lv_xml_get_subject(NULL, "s2");
+
+    TEST_ASSERT_EQUAL_INT32(10, lv_subject_get_int(s1));
+    TEST_ASSERT_EQUAL_INT32(20, lv_subject_get_int(s2));
+
+    lv_subject_set_int(s1, 35);
+    lv_subject_set_int(s2, 35);
+    TEST_ASSERT_EQUAL_INT32(35, lv_subject_get_int(s1));
+    TEST_ASSERT_EQUAL_INT32(30, lv_subject_get_int(s2));
+
+    lv_subject_set_int(s1, 5);
+    lv_subject_set_int(s2, 5);
+    TEST_ASSERT_EQUAL_INT32(5, lv_subject_get_int(s1));
+    TEST_ASSERT_EQUAL_INT32(10, lv_subject_get_int(s2));
+}
+
+#endif