Merge pull request #79 from cap-js/RenameBug

Rename Attachments Bug
cap-js · Nov 5, 2024 · dd3143d · dd3143d
2 parents d82d3b4 + f6fbbdc
commit dd3143d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,7 +7,9 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/).
 ## Version 1.2.2
 
 ### Fixed
-App crash issue on update of fields when no attachments are added.
+
+ - App crash issue on update of fields when no attachments are added.
+ - Attachment getting renamed even when duplicate attachment of same name exists.
 
 ## Version 1.2.1
 

diff --git a/lib/sdm.js b/lib/sdm.js
@@ -366,7 +366,7 @@ async isFileNameDuplicateInDrafts(data,req) {
           token
         );
 
-        if (response.status ==  undefined && response.response.status !=200) {
+        if (response.status != 200) {
           //modify req.data.attachments
           for(let i = 0; i < req.data.attachments.length; i++) {
             let attachmentUpdate = req.data.attachments[i];