backward compatibilty

Author: Bajusz15

providers/os/resources/journald.go

@@ -137,6 +137,49 @@ func (s *mqlJournaldConfig) sections(file *mqlFile) ([]any, error) {
 	return s.Sections.Data, s.Sections.Error
 }
 
+// params is deprecated, use sections instead
+func (s *mqlJournaldConfig) params(file *mqlFile) (map[string]any, error) {
+	if err := s.parse(file); err != nil {
+		return nil, err
+	}
+
+	// For backward compatibility, return the [Journal] section's params as a map
+	// Find the Journal section
+	for _, sectionAny := range s.Sections.Data {
+		section := sectionAny.(*mqlJournaldConfigSection)
+		name := section.GetName()
+		if name.Error != nil {
+			continue
+		}
+
+		// Only return params from the Journal section for backward compatibility
+		if name.Data != "Journal" {
+			continue
+		}
+
+		params := section.GetParams()
+		if params.Error != nil {
+			return nil, params.Error
+		}
+
+		// Convert params array to map[string]any
+		// If there are duplicate keys, the last one wins
+		result := make(map[string]any, len(params.Data))
+		for _, paramAny := range params.Data {
+			param := paramAny.(*mqlJournaldConfigSectionParam)
+			paramName := param.GetName()
+			paramValue := param.GetValue()
+			if paramName.Error != nil || paramValue.Error != nil {
+				continue
+			}
+			result[paramName.Data] = paramValue.Data
+		}
+		return result, nil
+	}
+
+	return map[string]any{}, nil
+}
+
 func (s *mqlJournaldConfigSection) id() (string, error) {
 	name := s.GetName()
 	if name.Error != nil {

providers/os/resources/journald_test.go

@@ -37,13 +37,27 @@ func TestResource_JournaldConfig(t *testing.T) {
 		assert.NoError(t, res[0].Data.Error)
 	})
 
+	t.Run("backwards compatibility: journald is downcasing relevant params", func(t *testing.T) {
+		res := x.TestQuery(t, "journald.config.params.Compress")
+		assert.NotEmpty(t, res)
+		assert.NoError(t, res[0].Data.Error)
+		assert.Equal(t, "yes", res[0].Data.Value)
+	})
+
 	t.Run("journald is downcasing relevant params", func(t *testing.T) {
 		res := x.TestQuery(t, "journald.config.sections.where(name == 'Journal')[0].params.where(name == 'Compress')[0].value")
 		assert.NotEmpty(t, res)
 		assert.NoError(t, res[0].Data.Error)
 		assert.Equal(t, "yes", res[0].Data.Value)
 	})
 
+	t.Run("backwards compatibility: journald is NOT downcasing other params", func(t *testing.T) {
+		res := x.TestQuery(t, "journald.config.params.Storage")
+		assert.NotEmpty(t, res)
+		assert.NoError(t, res[0].Data.Error)
+		assert.Equal(t, "persistent", res[0].Data.Value)
+	})
+
 	t.Run("journald is NOT downcasing other params", func(t *testing.T) {
 		res := x.TestQuery(t, "journald.config.sections.where(name == 'Journal')[0].params.where(name == 'Storage')[0].value")
 		assert.NotEmpty(t, res)

providers/os/resources/os.lr

@@ -825,6 +825,8 @@ journald.config {
   init(path? string)
   // File of this journald configuration
   file() file
+  // deprecated, use sections instead
+  params(file) map[string]string
   // All sections in this journald configuration
   sections(file) []journald.config.section
 }