set default retention period to 1 year

Author: sanjacornelius

ProcessMaker/Console/Commands/EvaluateCaseRetention.php

@@ -30,7 +30,7 @@ public function handle()
         $this->info('Evaluating and deleting cases past their retention period');
 
         // Process all processes when retention policy is enabled
-        // Processes without retention_period will default to 6 months
+        // Processes without retention_period will default to 1_year
         Process::chunkById(100, function ($processes) {
             foreach ($processes as $process) {
                 dispatch(new EvaluateProcessRetentionJob($process->id));

ProcessMaker/Jobs/EvaluateProcessRetentionJob.php

@@ -27,9 +27,8 @@ public function __construct(public int $processId)
     public function handle(): void
     {
         // Only run if case retention policy is enabled
-        // Use getenv() to read directly from environment (works better in tests)
-        $enabled = getenv('CASE_RETENTION_POLICY_ENABLED');
-        if ($enabled === false || $enabled === 'false' || $enabled === '0' || $enabled === '') {
+        $enabled = config('app.case_retention_policy_enabled');
+        if (!$enabled) {
             return;
         }
 
@@ -40,14 +39,14 @@ public function handle(): void
             return;
         }
 
-        // Default to 6 months if retention_period is not set
-        $retentionPeriod = $process->properties['retention_period'] ?? '6_months';
+        // Default to 1_year if retention_period is not set
+        $retentionPeriod = $process->properties['retention_period'] ?? '1_year';
         $retentionMonths = match ($retentionPeriod) {
             '6_months' => 6,
             '1_year' => 12,
             '3_years' => 36,
             '5_years' => 60,
-            default => 6, // Default to 6 months
+            default => 12, // Default to 1_year
         };
 
         // Default retention_updated_at to now if not set