diff --git a/service/src/pfr_mgr.cpp b/service/src/pfr_mgr.cpp
index ed97fd2..b773916 100644
--- a/service/src/pfr_mgr.cpp
+++ b/service/src/pfr_mgr.cpp
@@ -59,12 +59,20 @@
      * xyz.openbmc_project.Software.Version. since these versions are already
      * active, so we should set "activation" to Active and
      * "RequestedActivation" to None. */
-    std::string activation =
-        (imgType == ImageType::bmcRecovery ||
-         imgType == ImageType::biosRecovery ||
-         imgType == ImageType::cpldRecovery)
-            ? "xyz.openbmc_project.Software.Activation.Activations.StandbySpare"
-            : "xyz.openbmc_project.Software.Activation.Activations.Active";
+    std::string activation;
+    if (imgType == ImageType::bmcRecovery ||
+        imgType == ImageType::biosRecovery ||
+        imgType == ImageType::cpldRecovery)
+    {
+        activation =
+            "xyz.openbmc_project.Software.Activation.Activations.StandbySpare";
+    }
+    else
+    {
+        activation =
+            "xyz.openbmc_project.Software.Activation.Activations.Active";
+    }
+
     std::string reqActNone =
         "xyz.openbmc_project.Software.Activation.RequestedActivations.None";
     auto activationIface = server.add_interface(