From 4f04fd358d84f7ea8a9a26bf4c2ef4bef553c732 Mon Sep 17 00:00:00 2001 From: Daichi Hirono Date: Fri, 18 Mar 2016 18:29:56 +0900 Subject: [PATCH] Fix the logic to add suffix when copying. Previously the logic add '.' even if the extension is empty. BUG=27729309 Change-Id: I3fa4b09fea977b5c716ec5cbbbc7d9f4ab96db08 --- .../src/com/android/mtp/MtpDocumentsProvider.java | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/packages/MtpDocumentsProvider/src/com/android/mtp/MtpDocumentsProvider.java b/packages/MtpDocumentsProvider/src/com/android/mtp/MtpDocumentsProvider.java index 68c19923ccb56..cf5bee5d727d8 100644 --- a/packages/MtpDocumentsProvider/src/com/android/mtp/MtpDocumentsProvider.java +++ b/packages/MtpDocumentsProvider/src/com/android/mtp/MtpDocumentsProvider.java @@ -360,8 +360,12 @@ public class MtpDocumentsProvider extends DocumentsProvider { if (i == 0) { infoUniqueName = info; } else { - infoUniqueName = new MtpObjectInfo.Builder(info).setName( - baseName + " (" + i + ")." + extension).build(); + String suffixedName = baseName + " (" + i + " )"; + if (!extension.isEmpty()) { + suffixedName += "." + extension; + } + infoUniqueName = + new MtpObjectInfo.Builder(info).setName(suffixedName).build(); } try { objectHandle = mMtpManager.createDocument(