Update config_provider.py

config_provider.py

@@ -5,7 +5,7 @@ import threading
 import os
 import json
 import commentjson
-from typing import Optional, Dict, Any, List
+from typing import Optional, Dict, List, Any
 from datetime import datetime
 from pathlib import Path
 import tempfile
@@ -258,7 +258,7 @@ class ConfigProvider:
             
             return project
     
-@classmethod
+    @classmethod
     def update_project(cls, project_id: int, update_data: dict, username: str, expected_last_update: Optional[str] = None) -> ProjectConfig:
         """Update project with optimistic locking"""
         with cls._lock:
@@ -425,6 +425,66 @@ class ConfigProvider:
             )
             
             return version
+
+    @classmethod
+    def update_version(cls, project_id: int, version_no: int, update_data: dict, username: str, expected_last_update: Optional[str] = None) -> VersionConfig:
+        """Update version with optimistic locking"""
+        with cls._lock:
+            config = cls.get()
+            project = cls.get_project(project_id)
+            
+            if not project:
+                raise ResourceNotFoundError("project", project_id)
+            
+            version = next((v for v in project.versions if v.no == version_no), None)
+            if not version:
+                raise ResourceNotFoundError("version", version_no)
+            
+            # Published versions cannot be edited
+            if version.published:
+                raise ValidationError("Published versions cannot be modified")
+            
+            # Check race condition
+            if expected_last_update is not None:
+                if version.last_update_date != expected_last_update:
+                    raise RaceConditionError(
+                        f"Version {version_no} was modified by another user",
+                        current_user=username,
+                        last_update_user=version.last_update_user,
+                        last_update_date=version.last_update_date,
+                        entity_type="version",
+                        entity_id=f"{project_id}:{version_no}"
+                    )
+            
+            # Update fields
+            for key, value in update_data.items():
+                if hasattr(version, key) and key not in ['no', 'created_date', 'created_by', 'published', 'last_update_date']:
+                    setattr(version, key, value)
+            
+            version.last_update_date = datetime.utcnow().isoformat()
+            version.last_update_user = username
+            
+            # Update project last update
+            project.last_update_date = datetime.utcnow().isoformat()
+            project.last_update_user = username
+            
+            # Log activity
+            cls._add_activity(
+                config, username, "UPDATE_VERSION",
+                "version", f"{project.id}:{version.no}", f"{project.name} v{version.no}"
+            )
+            
+            # Save
+            cls.save(config, username)
+            
+            log_info(
+                "Version updated",
+                project_id=project.id,
+                version_no=version.no,
+                user=username
+            )
+            
+            return version
     
     # ===================== API Methods =====================
     
@@ -469,8 +529,8 @@ class ConfigProvider:
             return api
     
     @classmethod
-    def update_api(cls, api_name: str, update_data: dict, username: str) -> APIConfig:
-        """Update API configuration"""
+    def update_api(cls, api_name: str, update_data: dict, username: str, expected_last_update: Optional[str] = None) -> APIConfig:
+        """Update API with optimistic locking"""
         with cls._lock:
             config = cls.get()
             api = config.get_api(api_name)
@@ -479,20 +539,20 @@ class ConfigProvider:
                 raise ResourceNotFoundError("api", api_name)
             
             # Check race condition
-            if 'last_update_date' in update_data:
-                if api.last_update_date != update_data['last_update_date']:
+            if expected_last_update is not None:
+                if api.last_update_date != expected_last_update:
                     raise RaceConditionError(
-                        f"API '{api_name}' was modified by another user",
+                        f"API '{api.name}' was modified by another user",
                         current_user=username,
                         last_update_user=api.last_update_user,
                         last_update_date=api.last_update_date,
                         entity_type="api",
-                        entity_id=api_name
+                        entity_id=api.name
                     )
             
             # Update fields
             for key, value in update_data.items():
-                if hasattr(api, key) and key not in ['name', 'created_date', 'created_by']:
+                if hasattr(api, key) and key not in ['name', 'created_date', 'created_by', 'last_update_date']:
                     setattr(api, key, value)
             
             api.last_update_date = datetime.utcnow().isoformat()