From f1623044276120c2b019afb36d8654c45cf68ed7 Mon Sep 17 00:00:00 2001
From: M66B <M66B@users.noreply.github.com>
Date: Thu, 6 Jun 2024 17:24:36 +0200
Subject: [PATCH] Added renaming of attachments

---
 .../eu/faircode/email/AdapterAttachment.java  | 25 ++++++++--
 .../java/eu/faircode/email/DaoAttachment.java |  6 +++
 .../eu/faircode/email/FragmentCompose.java    | 46 ++++++++++++++++++-
 app/src/main/res/values/strings.xml           |  1 +
 4 files changed, 74 insertions(+), 4 deletions(-)

diff --git a/app/src/main/java/eu/faircode/email/AdapterAttachment.java b/app/src/main/java/eu/faircode/email/AdapterAttachment.java
index 3755263770..c0254706fa 100644
--- a/app/src/main/java/eu/faircode/email/AdapterAttachment.java
+++ b/app/src/main/java/eu/faircode/email/AdapterAttachment.java
@@ -273,15 +273,20 @@ public class AdapterAttachment extends RecyclerView.Adapter<AdapterAttachment.Vi
             else {
                 PopupMenuLifecycle popupMenu = new PopupMenuLifecycle(context, powner, view);
 
-                popupMenu.getMenu().add(Menu.NONE, R.string.title_share, 1, R.string.title_share);
-                popupMenu.getMenu().add(Menu.NONE, R.string.title_zip, 2, R.string.title_zip)
+                if (parentFragment instanceof FragmentCompose)
+                    popupMenu.getMenu().add(Menu.NONE, R.string.title_rename, 1, R.string.title_rename);
+
+                popupMenu.getMenu().add(Menu.NONE, R.string.title_share, 2, R.string.title_share);
+                popupMenu.getMenu().add(Menu.NONE, R.string.title_zip, 3, R.string.title_zip)
                         .setEnabled(!attachment.isInline() && !attachment.isCompressed());
 
                 popupMenu.setOnMenuItemClickListener(new PopupMenu.OnMenuItemClickListener() {
                     @Override
                     public boolean onMenuItemClick(MenuItem item) {
                         int itemId = item.getItemId();
-                        if (itemId == R.string.title_share)
+                        if (itemId == R.string.title_rename)
+                            return onRename(attachment);
+                        else if (itemId == R.string.title_share)
                             return onShare(attachment);
                         else if (itemId == R.string.title_zip)
                             return onZip(attachment);
@@ -295,6 +300,20 @@ public class AdapterAttachment extends RecyclerView.Adapter<AdapterAttachment.Vi
             }
         }
 
+        private boolean onRename(EntityAttachment attachment) {
+            Bundle args = new Bundle();
+            args.putLong("id", attachment.id);
+            args.putString("prev", attachment.name);
+            args.putString("name", attachment.name);
+
+            FragmentDialogEditName fragment = new FragmentDialogEditName();
+            fragment.setArguments(args);
+            fragment.setTargetFragment(parentFragment, FragmentCompose.REQUEST_EDIT_ATTACHMENT);
+            fragment.show(parentFragment.getParentFragmentManager(), "attachment:name");
+
+            return true;
+        }
+
         private boolean onShare(final EntityAttachment attachment) {
             try {
                 new ShareCompat.IntentBuilder(context)
diff --git a/app/src/main/java/eu/faircode/email/DaoAttachment.java b/app/src/main/java/eu/faircode/email/DaoAttachment.java
index 39d4cb2896..31e699f1b0 100644
--- a/app/src/main/java/eu/faircode/email/DaoAttachment.java
+++ b/app/src/main/java/eu/faircode/email/DaoAttachment.java
@@ -116,6 +116,12 @@ public interface DaoAttachment {
             " AND NOT (name IS name AND type IS :type AND size IS :size)")
     void setName(long id, String name, String type, Long size);
 
+    @Query("UPDATE attachment" +
+            " SET name = :name" +
+            " WHERE id = :id" +
+            " AND NOT (name IS :name)")
+    void setName(long id, String name);
+
     @Query("UPDATE attachment" +
             " SET type = :type" +
             " WHERE id = :id" +
diff --git a/app/src/main/java/eu/faircode/email/FragmentCompose.java b/app/src/main/java/eu/faircode/email/FragmentCompose.java
index cd0f66e8d8..227247c05b 100644
--- a/app/src/main/java/eu/faircode/email/FragmentCompose.java
+++ b/app/src/main/java/eu/faircode/email/FragmentCompose.java
@@ -348,7 +348,8 @@ public class FragmentCompose extends FragmentBase {
     private static final int REQUEST_LINK = 14;
     private static final int REQUEST_DISCARD = 15;
     private static final int REQUEST_SEND = 16;
-    private static final int REQUEST_REMOVE_ATTACHMENTS = 17;
+    static final int REQUEST_EDIT_ATTACHMENT = 17;
+    private static final int REQUEST_REMOVE_ATTACHMENTS = 18;
 
     ActivityResultLauncher<PickVisualMediaRequest> pickImages;
 
@@ -3373,6 +3374,10 @@ public class FragmentCompose extends FragmentBase {
                         onAction(R.id.action_send, extras, "sendnow");
                     }
                     break;
+                case REQUEST_EDIT_ATTACHMENT:
+                    if (resultCode == RESULT_OK && data != null)
+                        onEditAttachment(data.getBundleExtra("args"));
+                    break;
                 case REQUEST_REMOVE_ATTACHMENTS:
                     if (resultCode == RESULT_OK)
                         onRemoveAttachments();
@@ -4965,6 +4970,45 @@ public class FragmentCompose extends FragmentBase {
         onAction(R.id.action_delete, "delete");
     }
 
+    private void onEditAttachment(Bundle args) {
+        new SimpleTask<Void>() {
+            @Override
+            protected Void onExecute(Context context, Bundle args) throws Throwable {
+                long id = args.getLong("id");
+                String prev = args.getString("prev");
+                String name = args.getString("name");
+
+                if (TextUtils.isEmpty(name))
+                    return null;
+
+                name = Helper.sanitizeFilename(name);
+
+                File source = EntityAttachment.getFile(context, id, prev);
+                File target = EntityAttachment.getFile(context, id, name);
+                if (!source.renameTo(target))
+                    throw new IllegalArgumentException("Could not rename " + source.getName() + " into " + target.getName());
+
+                DB db = DB.getInstance(context);
+                try {
+                    db.beginTransaction();
+
+                    db.attachment().setName(id, name);
+
+                    db.setTransactionSuccessful();
+                } finally {
+                    db.endTransaction();
+                }
+
+                return null;
+            }
+
+            @Override
+            protected void onException(Bundle args, Throwable ex) {
+                Log.unexpectedError(getParentFragmentManager(), ex);
+            }
+        }.execute(this, args, "attachment:edit");
+    }
+
     private void onRemoveAttachments() {
         Bundle args = new Bundle();
         args.putLong("id", working);
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 26cfb919b7..917f4f9755 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -1508,6 +1508,7 @@
     <string name="title_create_rule">Create rule &#8230;</string>
     <string name="title_full_screen">Full screen</string>
     <string name="title_force_light">Force light theme</string>
+    <string name="title_rename">Rename</string>
     <string name="title_share">Share</string>
     <string name="title_share_link" translatable="false">Share link</string>
     <string name="title_share_link_open" translatable="false">Open message</string>