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. */