Skip to content

Commit f54d1cb

Browse files
authored
Merge pull request #390 from free4murad/comprehension-toasts
No of toasts for same option in Comprehension template fixed.
2 parents bc54a7b + e52f1c6 commit f54d1cb

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

source-code/app/src/main/java/org/buildmlearn/toolkit/templates/ComprehensionAdapter.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -315,6 +315,7 @@ public void onClick(View v) {
315315
if (!options.get(i).getText().toString().trim().isEmpty() && options.get(i).getText().toString().trim().equalsIgnoreCase(options.get(j).getText().toString().trim())) {
316316
Toast.makeText(context, context.getString(R.string.same_options), Toast.LENGTH_SHORT).show();
317317
isValidated = false;
318+
return;
318319
}
319320
}
320321
}

source-code/app/src/main/java/org/buildmlearn/toolkit/templates/ComprehensionTemplate.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -304,7 +304,8 @@ public void onClick(View v) {
304304
for (int j = 0; j < i; j++) {
305305
if (!options.get(i).getText().toString().trim().isEmpty() && options.get(i).getText().toString().trim().equalsIgnoreCase(options.get(j).getText().toString().trim())) {
306306
Toast.makeText(activity.getApplication(), activity.getString(R.string.same_options), Toast.LENGTH_SHORT).show();
307-
isValidated = false;
307+
isValidated=false;
308+
return;
308309
}
309310
}
310311
}

0 commit comments

Comments
 (0)