Skip to content

Commit 8a187ea

Browse files
committed
improve readability
1 parent a50b96e commit 8a187ea

File tree

1 file changed

+21
-10
lines changed
  • modules/simple-java-mail/src/main/java/org/simplejavamail/converter/internal/mimemessage

1 file changed

+21
-10
lines changed

modules/simple-java-mail/src/main/java/org/simplejavamail/converter/internal/mimemessage/MimeMessageHelper.java

Lines changed: 21 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -260,18 +260,29 @@ static String determineResourceName(final AttachmentResource attachmentResource,
260260
resourceName = "resource" + UUID.randomUUID();
261261
}
262262
if (includeExtension && !valueNullOrEmpty(datasourceName)) {
263-
@SuppressWarnings("UnnecessaryLocalVariable")
264-
final String possibleFilename = datasourceName;
265-
if (!resourceName.contains(".") && possibleFilename.contains(".")) {
266-
final String extension = possibleFilename.substring(possibleFilename.lastIndexOf("."));
267-
if (!resourceName.endsWith(extension)) {
268-
resourceName += extension;
269-
}
270-
}
263+
resourceName = possiblyAddExtension(datasourceName, resourceName);
271264
} else if (!includeExtension && resourceName.contains(".") && resourceName.equals(datasourceName)) {
272-
final String extension = resourceName.substring(resourceName.lastIndexOf("."));
273-
resourceName = resourceName.replace(extension, "");
265+
resourceName = removeExtension(resourceName);
274266
}
275267
return MiscUtil.encodeText(resourceName);
276268
}
269+
270+
@NotNull
271+
private static String removeExtension(String resourceName) {
272+
final String extension = resourceName.substring(resourceName.lastIndexOf("."));
273+
return resourceName.replace(extension, "");
274+
}
275+
276+
@NotNull
277+
private static String possiblyAddExtension(final String datasourceName, String resourceName) {
278+
@SuppressWarnings("UnnecessaryLocalVariable")
279+
final String possibleFilename = datasourceName;
280+
if (!resourceName.contains(".") && possibleFilename.contains(".")) {
281+
final String extension = possibleFilename.substring(possibleFilename.lastIndexOf("."));
282+
if (!resourceName.endsWith(extension)) {
283+
resourceName += extension;
284+
}
285+
}
286+
return resourceName;
287+
}
277288
}

0 commit comments

Comments
 (0)