Skip to content
This repository was archived by the owner on Dec 4, 2023. It is now read-only.

Commit e01aef1

Browse files
authored
[Samples & SDK] Replace Double Brace initialization with Standard Initialization (#1103)
* Fix double brace in botbuilder library * Fix double brace in botbuilder-dialogs library * Fix double brace in bot-schema library * Fix double brace in bot-ai-luis library * Fix double brace in bot-ai-qna library * Fix double brace in bot-azure library * Fix double brace in bot-connector library * Fix double brace in bot-integration library * Fix double brace in sample 3 * Fix double brace in sample 6 * Fix double brace in sample 7 * Fix double brace in sample 8 * Fix double brace in sample 11 * Fix double brace in sample 15 * Fix double brace in sample 17 * Fix double brace in sample 50 * Fix double brace in sample 51 * Fix double brace in sample 52 * Fix double brace in sample 53 * Fix double brace in sample 54 * Fix double brace in sample 55 * Fix double brace in sample 56 * Fix double brace in sample 57 * Fix double brace in sample 58 * Fix double brace in sample 81 * Fix double brace in sample 13 * Replicate change in generator for core bot template
1 parent 4573f00 commit e01aef1

File tree

118 files changed

+3357
-4911
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

118 files changed

+3357
-4911
lines changed

Generator/generator-botbuilder-java/generators/app/templates/core/src/main/java/DateResolverDialog.java

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -75,12 +75,10 @@ private CompletableFuture<DialogTurnResult> initialStep(WaterfallStepContext ste
7575
return stepContext.prompt("DateTimePrompt", promptOptions);
7676
}
7777

78-
DateTimeResolution dateTimeResolution = new DateTimeResolution() {{
79-
setTimex(timex);
80-
}};
81-
List<DateTimeResolution> dateTimeResolutions = new ArrayList<DateTimeResolution>() {{
82-
add(dateTimeResolution);
83-
}};
78+
DateTimeResolution dateTimeResolution = new DateTimeResolution();
79+
dateTimeResolution.setTimex(timex);
80+
List<DateTimeResolution> dateTimeResolutions = new ArrayList<DateTimeResolution>();
81+
dateTimeResolutions.add(dateTimeResolution);
8482
return stepContext.next(dateTimeResolutions);
8583
}
8684

Generator/generator-botbuilder-java/generators/app/templates/core/src/main/java/DialogAndWelcomeBot.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -85,10 +85,10 @@ private Attachment createAdaptiveCardAttachment() {
8585
String adaptiveCardJson = IOUtils
8686
.toString(inputStream, StandardCharsets.UTF_8.toString());
8787

88-
return new Attachment() {{
89-
setContentType("application/vnd.microsoft.card.adaptive");
90-
setContent(Serialization.jsonToTree(adaptiveCardJson));
91-
}};
88+
Attachment attachment = new Attachment();
89+
attachment.setContentType("application/vnd.microsoft.card.adaptive");
90+
attachment.setContent(Serialization.jsonToTree(adaptiveCardJson));
91+
return attachment;
9292

9393
} catch (IOException e) {
9494
e.printStackTrace();

Generator/generator-botbuilder-java/generators/app/templates/core/src/main/java/FlightBookingRecognizer.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -42,12 +42,8 @@ public FlightBookingRecognizer(Configuration configuration) {
4242
// Set the recognizer options depending on which endpoint version you want to use.
4343
// More details can be found in
4444
// https://docs.microsoft.com/en-gb/azure/cognitive-services/luis/luis-migration-api-v3
45-
LuisRecognizerOptionsV3 recognizerOptions = new LuisRecognizerOptionsV3(
46-
luisApplication) {
47-
{
48-
setIncludeInstanceData(true);
49-
}
50-
};
45+
LuisRecognizerOptionsV3 recognizerOptions = new LuisRecognizerOptionsV3(luisApplication);
46+
recognizerOptions.setIncludeInstanceData(true);
5147

5248
this.recognizer = new LuisRecognizer(recognizerOptions);
5349
}

libraries/bot-ai-luis-v3/src/main/java/com/microsoft/bot/ai/luis/LuisRecognizerOptionsV3.java

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -485,11 +485,8 @@ private CompletableFuture<RecognizerResult> recognizeInternal(TurnContext turnCo
485485
ObjectMapper mapper = new ObjectMapper().findAndRegisterModules();
486486

487487
if (utterance == null || utterance.isEmpty()) {
488-
recognizerResult = new RecognizerResult() {
489-
{
490-
setText(utterance);
491-
}
492-
};
488+
recognizerResult = new RecognizerResult();
489+
recognizerResult.setText(utterance);
493490
} else {
494491
try {
495492
Request request = buildRequest(buildRequestBody(utterance));
@@ -541,11 +538,9 @@ private Map<String, IntentScore> getIntents(JsonNode prediction) {
541538
Map.Entry<String, JsonNode> intent = it.next();
542539
double score = intent.getValue().get("score").asDouble();
543540
String intentName = intent.getKey().replace(".", "_").replace(" ", "_");
544-
intents.put(intentName, new IntentScore() {
545-
{
546-
setScore(score);
547-
}
548-
});
541+
IntentScore intentScore = new IntentScore();
542+
intentScore.setScore(score);
543+
intents.put(intentName, intentScore);
549544
}
550545

551546
return intents;

libraries/bot-ai-luis-v3/src/test/java/com/microsoft/bot/ai/luis/LuisRecognizerOptionsV3Tests.java

Lines changed: 23 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -124,13 +124,9 @@ private void shouldParseLuisResponsesCorrectly_TurnContextPassed(String fileName
124124
LuisRecognizerOptionsV3 v3 = buildTestRecognizer(endpoint, testSettings);
125125

126126
// Run test
127-
Activity activity = new Activity() {
128-
{
129-
setText(testData.get("text").asText());
130-
setType(ActivityTypes.MESSAGE);
131-
setChannelId("EmptyContext");
132-
}
133-
};
127+
Activity activity = new Activity(ActivityTypes.MESSAGE);
128+
activity.setText(testData.get("text").asText());
129+
activity.setChannelId("EmptyContext");
134130
doReturn(activity)
135131
.when(turnContext)
136132
.getActivity();
@@ -198,23 +194,21 @@ public void shouldBuildExternalEntities_DialogContextPassed_ExternalRecognizer()
198194
LuisRecognizerOptionsV3 v3 = buildTestRecognizer(endpoint, testSettings);
199195
v3.setExternalEntityRecognizer(recognizer);
200196

201-
Activity activity = new Activity() {
202-
{
203-
setText(testData.get("text").asText());
204-
setType(ActivityTypes.MESSAGE);
205-
setChannelId("EmptyContext");
206-
}
207-
};
197+
Activity activity = new Activity(ActivityTypes.MESSAGE);
198+
activity.setText(testData.get("text").asText());
199+
activity.setChannelId("EmptyContext");
208200

209201
doReturn(CompletableFuture.completedFuture(new ResourceResponse()))
210202
.when(turnContext)
211203
.sendActivity(any(Activity.class));
212204

213205
when(dC.getContext()).thenReturn(turnContext);
214206

215-
doReturn(CompletableFuture.supplyAsync(() -> new RecognizerResult(){{
216-
setEntities(testSettings.get("ExternalRecognizerResult"));
217-
}}))
207+
doReturn(CompletableFuture.supplyAsync(() -> {
208+
RecognizerResult recognizerResult = new RecognizerResult();
209+
recognizerResult.setEntities(testSettings.get("ExternalRecognizerResult"));
210+
return recognizerResult;
211+
}))
218212
.when(recognizer)
219213
.recognize(any(DialogContext.class), any(Activity.class));
220214

@@ -244,13 +238,9 @@ public void shouldBuildExternalEntities_DialogContextPassed_ExternalRecognizer()
244238

245239
public static TurnContext createContext(String message) {
246240

247-
Activity activity = new Activity() {
248-
{
249-
setText(message);
250-
setType(ActivityTypes.MESSAGE);
251-
setChannelId("EmptyContext");
252-
}
253-
};
241+
Activity activity = new Activity(ActivityTypes.MESSAGE);
242+
activity.setText(message);
243+
activity.setChannelId("EmptyContext");
254244

255245
return new TurnContextImpl(new NotImplementedAdapter(), activity);
256246
}
@@ -323,18 +313,18 @@ private LuisRecognizerOptionsV3 buildTestRecognizer (String endpoint, JsonNode t
323313
ObjectMapper mapper = new ObjectMapper().findAndRegisterModules();
324314
ObjectReader readerDynamicList = mapper.readerFor(new TypeReference<List<DynamicList>>() {});
325315
ObjectReader readerExternalentities = mapper.readerFor(new TypeReference<List<ExternalEntity>>() {});
326-
return new LuisRecognizerOptionsV3(
316+
LuisRecognizerOptionsV3 recognizer = new LuisRecognizerOptionsV3(
327317
new LuisApplication(
328318
this.applicationId,
329319
this.subscriptionKey,
330-
endpoint)) {{
331-
setIncludeInstanceData(testSettings.get("IncludeInstanceData").asBoolean());
332-
setIncludeAllIntents(testSettings.get("IncludeAllIntents").asBoolean());
333-
setVersion(testSettings.get("Version") == null ? null : testSettings.get("Version").asText());
334-
setDynamicLists(testSettings.get("DynamicLists") == null ? null : readerDynamicList.readValue(testSettings.get("DynamicLists")));
335-
setExternalEntities(testSettings.get("ExternalEntities") == null ? null : readerExternalentities.readValue(testSettings.get("ExternalEntities")));
336-
setDateTimeReference(testSettings.get("DateTimeReference") == null ? null : testSettings.get("DateTimeReference").asText());
337-
}};
320+
endpoint));
321+
recognizer.setIncludeInstanceData(testSettings.get("IncludeInstanceData").asBoolean());
322+
recognizer.setIncludeAllIntents(testSettings.get("IncludeAllIntents").asBoolean());
323+
recognizer.setVersion(testSettings.get("Version") == null ? null : testSettings.get("Version").asText());
324+
recognizer.setDynamicLists(testSettings.get("DynamicLists") == null ? null : readerDynamicList.readValue(testSettings.get("DynamicLists")));
325+
recognizer.setExternalEntities(testSettings.get("ExternalEntities") == null ? null : readerExternalentities.readValue(testSettings.get("ExternalEntities")));
326+
recognizer.setDateTimeReference(testSettings.get("DateTimeReference") == null ? null : testSettings.get("DateTimeReference").asText());
327+
return recognizer;
338328
}
339329

340330
}

0 commit comments

Comments
 (0)