Skip to content

Commit f89b09b

Browse files
authored
Assign ConfigContainer Builder return values. (#4194)
1 parent 79fcab2 commit f89b09b

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

firebase-config/src/main/java/com/google/firebase/remoteconfig/internal/ConfigFetchHttpClient.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -413,7 +413,7 @@ private static ConfigContainer extractConfigs(JSONObject fetchResponse, Date fet
413413
// Do nothing if entries do not exist.
414414
}
415415
if (entries != null) {
416-
containerBuilder.replaceConfigsWith(entries);
416+
containerBuilder = containerBuilder.replaceConfigsWith(entries);
417417
}
418418

419419
JSONArray experimentDescriptions = null;
@@ -423,7 +423,7 @@ private static ConfigContainer extractConfigs(JSONObject fetchResponse, Date fet
423423
// Do nothing if entries do not exist.
424424
}
425425
if (experimentDescriptions != null) {
426-
containerBuilder.withAbtExperiments(experimentDescriptions);
426+
containerBuilder = containerBuilder.withAbtExperiments(experimentDescriptions);
427427
}
428428

429429
JSONObject personalizationMetadata = null;
@@ -433,7 +433,7 @@ private static ConfigContainer extractConfigs(JSONObject fetchResponse, Date fet
433433
// Do nothing if personalizationMetadata does not exist.
434434
}
435435
if (personalizationMetadata != null) {
436-
containerBuilder.withPersonalizationMetadata(personalizationMetadata);
436+
containerBuilder = containerBuilder.withPersonalizationMetadata(personalizationMetadata);
437437
}
438438

439439
return containerBuilder.build();

firebase-config/src/test/java/com/google/firebase/remoteconfig/internal/ConfigFetchHandlerTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1033,10 +1033,10 @@ private static ConfigContainer extractConfigs(JSONObject fetchResponse, Date fet
10331033
ConfigContainer.newBuilder().withFetchTime(fetchTime);
10341034

10351035
JSONObject entries = fetchResponse.getJSONObject(ENTRIES);
1036-
containerBuilder.replaceConfigsWith(entries);
1036+
containerBuilder = containerBuilder.replaceConfigsWith(entries);
10371037

10381038
JSONArray experimentDescriptions = fetchResponse.getJSONArray(EXPERIMENT_DESCRIPTIONS);
1039-
containerBuilder.withAbtExperiments(experimentDescriptions);
1039+
containerBuilder = containerBuilder.withAbtExperiments(experimentDescriptions);
10401040

10411041
return containerBuilder.build();
10421042
}

0 commit comments

Comments
 (0)