Skip to content

Commit 1161827

Browse files
committed
FIXES Question's edit input validation in comprehension and quiz template
1 parent 12cf31e commit 1161827

2 files changed

Lines changed: 17 additions & 1 deletion

File tree

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

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -224,6 +224,14 @@ public void onClick(View v) {
224224
isValidated = false;
225225
}
226226

227+
for (int i = 0; i < options.size(); i++) {
228+
for (int j = 0; j < i; j++) {
229+
if (!options.get(i).getText().toString().trim().isEmpty() && options.get(i).getText().toString().trim().equalsIgnoreCase(options.get(j).getText().toString().trim())) {
230+
Toast.makeText(context, context.getString(R.string.same_options), Toast.LENGTH_SHORT).show();
231+
isValidated = false;
232+
}
233+
}
234+
}
227235
if (isValidated) {
228236
dialog.dismiss();
229237
ArrayList<String> answerOptions = new ArrayList<>();

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

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@
1616
import android.widget.RadioButton;
1717
import android.widget.Toast;
1818

19-
2019
import org.buildmlearn.toolkit.R;
2120
import org.buildmlearn.toolkit.activity.TemplateEditor;
2221
import org.buildmlearn.toolkit.views.TextViewPlus;
@@ -222,6 +221,15 @@ public void onClick(View v) {
222221
isValidated = false;
223222
}
224223

224+
for (int i = 0; i < options.size(); i++) {
225+
for (int j = 0; j < i; j++) {
226+
if (!options.get(i).getText().toString().trim().isEmpty() && options.get(i).getText().toString().trim().equalsIgnoreCase(options.get(j).getText().toString().trim())) {
227+
Toast.makeText(context, context.getString(R.string.same_options), Toast.LENGTH_SHORT).show();
228+
isValidated = false;
229+
}
230+
}
231+
}
232+
225233
if (isValidated) {
226234
dialog.dismiss();
227235
ArrayList<String> answerOptions = new ArrayList<>();

0 commit comments

Comments
 (0)