Index: lams_common/src/java/org/lamsfoundation/lams/questions/QuestionParser.java
===================================================================
diff -u -r48c0f070620fb4f585ca97dad722462811e6b72e -r15b63adc56c9ce3121aa02108c25834fe9b41d3e
--- lams_common/src/java/org/lamsfoundation/lams/questions/QuestionParser.java (.../QuestionParser.java) (revision 48c0f070620fb4f585ca97dad722462811e6b72e)
+++ lams_common/src/java/org/lamsfoundation/lams/questions/QuestionParser.java (.../QuestionParser.java) (revision 15b63adc56c9ce3121aa02108c25834fe9b41d3e)
@@ -513,15 +513,8 @@
FileUtils.copyFile(sourceFile, destinationFile);
// ensure that img-responsive class is always added to img tag
- int classAttributeIndex = -1;
- for (int attributeIndex = 0; attributeIndex < imageAttributes
- .size(); attributeIndex++) {
- String attribute = imageAttributes.get(attributeIndex).strip().toLowerCase();
- if (attribute.startsWith("class")) {
- classAttributeIndex = attributeIndex;
- }
- }
-
+ int classAttributeIndex = QuestionParser.getImageAttributeIndex(imageAttributes,
+ "class");
if (classAttributeIndex == -1) {
imageAttributes.add("class=\"img-responsive\"");
} else {
@@ -536,7 +529,13 @@
}
}
- replacement = "
";
} catch (IOException e) {
log.error("Could not store image " + fileName);
@@ -555,6 +554,16 @@
return StringUtils.isBlank(result) ? null : result;
}
+ private static int getImageAttributeIndex(List imageAttributes, String attibuteName) {
+ for (int attributeIndex = 0; attributeIndex < imageAttributes.size(); attributeIndex++) {
+ String attribute = imageAttributes.get(attributeIndex).strip().toLowerCase();
+ if (attribute.startsWith(attibuteName)) {
+ return attributeIndex;
+ }
+ }
+ return -1;
+ }
+
/**
* Find XML file list in IMS QTI package.
*/