OPS-6558 Fix bug where files in fields of a 1PW item never get updated

onepwd/plugins/action/item.py

@@ -49,6 +49,8 @@ def run(self, tmp=None, task_vars=None, **kwargs):
 
     def run_present(self, op:onepwd.OnePwd, category, name, vault, fields, generate_password, overwrite, check):
         assignment_statements = ""
+        # Dry-run doesn't recognize changes in files so we always update items containing files
+        overwrite_file_fields = False
         for field in fields:
             if 'overwrite' in field and field['overwrite'] is False:
                 labels = op.get('item', item_name=name, vault=vault)
@@ -59,14 +61,16 @@ def run_present(self, op:onepwd.OnePwd, category, name, vault, fields, generate_
                 if not label_existing:
                     assignment_statements += " " + onepwd.build_assignment_statement(field)
             else:
+                if lowercase(field['type']) == 'file':
+                    overwrite_file_fields = True
                 assignment_statements += " " + onepwd.build_assignment_statement(field)
 
         result = {}
         diff = {}
         try:
             get_result = op.get('item', item_name=name, vault=vault)
             edit_result = op.edit_item(name, assignment_statements, vault=vault, generate_password=generate_password, dry_run=True)
-            changed = not items_equal(get_result, edit_result) and overwrite
+            changed = (not items_equal(get_result, edit_result) or overwrite_file_fields) and overwrite
             if changed:
                 if not check:
                     edit_result = op.edit_item(name, assignment_statements, vault=vault, generate_password=generate_password)