From b9019cdfef4fff5b297f98d8d729938fac31d570 Mon Sep 17 00:00:00 2001 From: Jonathan Gamba Date: Wed, 23 Oct 2024 14:38:16 -0600 Subject: [PATCH] #29498 Add private constructors and default constructor for CDI Added a private constructor in Predicates to prevent instantiation and a default constructor in RetryPolicyProcessor for CDI proxy creation. This change improves the design by enforcing non-instantiability where necessary and ensuring the proper creation of CDI proxies. --- .../jobs/business/api/events/RealTimeJobMonitor.java | 4 ++++ .../dotcms/jobs/business/error/RetryPolicyProcessor.java | 9 ++++++++- 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/dotCMS/src/main/java/com/dotcms/jobs/business/api/events/RealTimeJobMonitor.java b/dotCMS/src/main/java/com/dotcms/jobs/business/api/events/RealTimeJobMonitor.java index 43d5f93f4e5e..29c9eab2e0b7 100644 --- a/dotCMS/src/main/java/com/dotcms/jobs/business/api/events/RealTimeJobMonitor.java +++ b/dotCMS/src/main/java/com/dotcms/jobs/business/api/events/RealTimeJobMonitor.java @@ -237,6 +237,10 @@ public void onJobProgressUpdated(@Observes JobProgressUpdatedEvent event) { */ public static class Predicates { + private Predicates() { + // Prevent instantiation + } + /** * Creates a predicate that matches jobs with any of the specified states. * diff --git a/dotCMS/src/main/java/com/dotcms/jobs/business/error/RetryPolicyProcessor.java b/dotCMS/src/main/java/com/dotcms/jobs/business/error/RetryPolicyProcessor.java index 687354659ad2..1775684c8f33 100644 --- a/dotCMS/src/main/java/com/dotcms/jobs/business/error/RetryPolicyProcessor.java +++ b/dotCMS/src/main/java/com/dotcms/jobs/business/error/RetryPolicyProcessor.java @@ -16,7 +16,14 @@ @ApplicationScoped public class RetryPolicyProcessor { - private final NoRetryStrategy noRetryStrategy; + private NoRetryStrategy noRetryStrategy; + + /** + * Default constructor required for CDI proxy creation. + */ + public RetryPolicyProcessor() { + // Default constructor for CDI + } @Inject public RetryPolicyProcessor(NoRetryStrategy noRetryStrategy) {