LPS-141305 work even if 'enterprise.product.commerce.enabled' is set …

…to false
liferay · Oct 22, 2021 · 0dc4a52 · 0dc4a52
1 parent f6ba000
commit 0dc4a52
Show file tree

Hide file tree

Showing 2 changed files with 58 additions and 25 deletions.
diff --git a/...r/src/main/java/com/liferay/site/initializer/extender/internal/BundleSiteInitializer.java b/...r/src/main/java/com/liferay/site/initializer/extender/internal/BundleSiteInitializer.java
@@ -109,6 +109,7 @@
 import com.liferay.portal.kernel.util.MimeTypesUtil;
 import com.liferay.portal.kernel.util.NaturalOrderStringComparator;
 import com.liferay.portal.kernel.util.Portal;
+import com.liferay.portal.kernel.util.PropsUtil;
 import com.liferay.portal.kernel.util.SetUtil;
 import com.liferay.portal.kernel.util.StringUtil;
 import com.liferay.portal.kernel.util.UnicodeProperties;
@@ -644,9 +645,14 @@ private void _addCPDefinitions(
 	private void _addCPDefinitions(ServiceContext serviceContext)
 		throws Exception {
 
-		_addCommerceCatalogs(
-			_addCommerceChannel(serviceContext),
-			_addCommerceInventoryWarehouses(serviceContext), serviceContext);
+		if (GetterUtil.getBoolean(
+				PropsUtil.get("enterprise.product.commerce.enabled"))) {
+
+			_addCommerceCatalogs(
+				_addCommerceChannel(serviceContext),
+				_addCommerceInventoryWarehouses(serviceContext),
+				serviceContext);
+		}
 	}
 
 	private void _addDDMStructures(ServiceContext serviceContext)

diff --git a/...src/main/java/com/liferay/site/initializer/extender/internal/SiteInitializerExtender.java b/...src/main/java/com/liferay/site/initializer/extender/internal/SiteInitializerExtender.java
@@ -69,6 +69,8 @@
 import org.osgi.service.component.annotations.Component;
 import org.osgi.service.component.annotations.Deactivate;
 import org.osgi.service.component.annotations.Reference;
+import org.osgi.service.component.annotations.ReferenceCardinality;
+import org.osgi.service.component.annotations.ReferencePolicyOption;
 import org.osgi.util.tracker.BundleTracker;
 import org.osgi.util.tracker.BundleTrackerCustomizer;
 
@@ -159,30 +161,55 @@ protected void deactivate() {
 	private BundleContext _bundleContext;
 	private BundleTracker<?> _bundleTracker;
 
-	@Reference
-	private CatalogResource.Factory _catalogResourceFactory;
-
-	@Reference
-	private ChannelResource.Factory _channelResourceFactory;
-
-	@Reference
-	private CommerceAccountRoleHelper _commerceAccountRoleHelper;
-
-	@Reference
-	private CommerceCurrencyLocalService _commerceCurrencyLocalService;
-
-	@Reference
-	private CommerceInventoryWarehousesImporter
+	@Reference(
+		cardinality = ReferenceCardinality.OPTIONAL,
+		policyOption = ReferencePolicyOption.GREEDY
+	)
+	private volatile CatalogResource.Factory _catalogResourceFactory;
+
+	@Reference(
+		cardinality = ReferenceCardinality.OPTIONAL,
+		policyOption = ReferencePolicyOption.GREEDY
+	)
+	private volatile ChannelResource.Factory _channelResourceFactory;
+
+	@Reference(
+		cardinality = ReferenceCardinality.OPTIONAL,
+		policyOption = ReferencePolicyOption.GREEDY
+	)
+	private volatile CommerceAccountRoleHelper _commerceAccountRoleHelper;
+
+	@Reference(
+		cardinality = ReferenceCardinality.OPTIONAL,
+		policyOption = ReferencePolicyOption.GREEDY
+	)
+	private volatile CommerceCurrencyLocalService _commerceCurrencyLocalService;
+
+	@Reference(
+		cardinality = ReferenceCardinality.OPTIONAL,
+		policyOption = ReferencePolicyOption.GREEDY
+	)
+	private volatile CommerceInventoryWarehousesImporter
 		_commerceInventoryWarehousesImporter;
 
-	@Reference
-	private CPDefinitionsImporter _cpDefinitionsImporter;
-
-	@Reference
-	private CPFileImporter _cpFileImporter;
-
-	@Reference
-	private CPMeasurementUnitLocalService _cpMeasurementUnitLocalService;
+	@Reference(
+		cardinality = ReferenceCardinality.OPTIONAL,
+		policyOption = ReferencePolicyOption.GREEDY
+	)
+	private volatile CPDefinitionsImporter _cpDefinitionsImporter;
+
+	@Reference(
+		cardinality = ReferenceCardinality.OPTIONAL,
+		policyOption = ReferencePolicyOption.GREEDY
+	)
+	private volatile CPFileImporter _cpFileImporter;
+
+	@Reference(
+		cardinality = ReferenceCardinality.OPTIONAL,
+		policyOption = ReferencePolicyOption.GREEDY
+	)
+	private volatile CPMeasurementUnitLocalService
+		_cpMeasurementUnitLocalService;
 
 	@Reference
 	private DDMStructureLocalService _ddmStructureLocalService;