Skip to content

Commit 551f74a

Browse files
fix logging replacing %s with {} (#137)
1 parent 35af591 commit 551f74a

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

core-api/src/main/java/com/optimizely/ab/config/ProjectConfig.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -311,7 +311,7 @@ public boolean setForcedVariation(@Nonnull String experimentKey,
311311
if (removedVariationId != null) {
312312
Variation removedVariation = experiment.getVariationIdToVariationMap().get(removedVariationId);
313313
if (removedVariation != null) {
314-
logger.debug("Variation mapped to experiment \"%s\" has been removed for user \"%s\"", experiment.getKey(), userId);
314+
logger.debug("Variation mapped to experiment \"{}\" has been removed for user \"{}\"", experiment.getKey(), userId);
315315
}
316316
else {
317317
logger.debug("Removed forced variation that did not exist in experiment");
@@ -324,7 +324,7 @@ public boolean setForcedVariation(@Nonnull String experimentKey,
324324
}
325325
else {
326326
String previous = experimentToVariation.put(experiment.getId(), variation.getId());
327-
logger.debug("Set variation \"%s\" for experiment \"%s\" and user \"%s\" in the forced variation map.",
327+
logger.debug("Set variation \"{}\" for experiment \"{}\" and user \"{}\" in the forced variation map.",
328328
variation.getKey(), experiment.getKey(), userId);
329329
if (previous != null) {
330330
Variation previousVariation = experiment.getVariationIdToVariationMap().get(previous);
@@ -365,20 +365,20 @@ public boolean setForcedVariation(@Nonnull String experimentKey,
365365
if (experimentToVariation != null) {
366366
Experiment experiment = getExperimentKeyMapping().get(experimentKey);
367367
if (experiment == null) {
368-
logger.debug("No experiment \"%s\" mapped to user \"%s\" in the forced variation map ", experimentKey, userId);
368+
logger.debug("No experiment \"{}\" mapped to user \"{}\" in the forced variation map ", experimentKey, userId);
369369
return null;
370370
}
371371
String variationId = experimentToVariation.get(experiment.getId());
372372
if (variationId != null) {
373373
Variation variation = experiment.getVariationIdToVariationMap().get(variationId);
374374
if (variation != null) {
375-
logger.debug("Variation \"%s\" is mapped to experiment \"%s\" and user \"%s\" in the forced variation map",
375+
logger.debug("Variation \"{}\" is mapped to experiment \"{}\" and user \"{}\" in the forced variation map",
376376
variation.getKey(), experimentKey, userId);
377377
return variation;
378378
}
379379
}
380380
else {
381-
logger.debug("No variation for experiment \"%s\" mapped to user \"%s\" in the forced variation map ", experimentKey, userId);
381+
logger.debug("No variation for experiment \"{}\" mapped to user \"{}\" in the forced variation map ", experimentKey, userId);
382382
}
383383
}
384384
return null;

0 commit comments

Comments
 (0)