Skip to content

fix: hash checks on duplicate file names#118

Merged
agritheory merged 1 commit intoversion-15from
fix/duplicate-file-hash-check
Mar 11, 2026
Merged

fix: hash checks on duplicate file names#118
agritheory merged 1 commit intoversion-15from
fix/duplicate-file-hash-check

Conversation

@batonac
Copy link
Copy Markdown
Collaborator

@batonac batonac commented Mar 11, 2026

fixes #117

@github-actions
Copy link
Copy Markdown

📝 Draft Changelog Entry

Changelog

Fixed an issue where hash checks were not being performed correctly when files with duplicate names were present in cloud storage.

This changelog entry was automatically generated by the Changelog Generator Action.

@github-actions
Copy link
Copy Markdown

Coverage

Coverage Report
FileStmtsMissCoverMissing
__init__.py11554%13–14, 20–22
commands.py35350%4–6, 9–11, 17–20, 23, 26–27, 49–51, 53–54, 56, 65, 68–72, 75–76, 85–87, 89–90, 92, 94, 97
customize.py25250%4–5, 7, 10–12, 15–22, 27–35, 47–48
hooks.py150100% 
install.py36360%4–7, 10–11, 14–18, 21–27, 30–34, 36–39, 41–49
migration.py19611541%25, 46, 68–69, 88–89, 96, 115, 123–125, 158–159, 161–162, 166–168, 171–172, 187, 196–197, 199–203, 205–210, 212, 219–221, 223, 225–226, 228, 235–236, 238–242, 244–247, 249, 254–257, 259–268, 270–279, 281–282, 284–285, 287–288, 290–294, 298–301, 305, 307–308, 310, 312–314, 317–318, 320–321, 323–329, 331–332, 334, 336
cloud_storage
   __init__.py00100% 
cloud_storage/doctype
   __init__.py00100% 
cloud_storage/doctype/file_association
   __init__.py00100% 
   file_association.py30100% 
cloud_storage/doctype/file_version
   __init__.py00100% 
   file_version.py30100% 
cloud_storage/overrides
   file.py41013467%66, 76, 96, 119, 121, 136, 138, 145–147, 155–156, 171, 176, 183, 198, 231, 241–250, 252–260, 271, 276–277, 285–287, 289, 294, 300, 302, 318, 322, 327, 335–336, 339, 342, 371, 416, 419, 458, 464, 470, 476, 482, 488, 498, 500–502, 506, 514–516, 518, 533, 536, 547–551, 557–558, 560, 583–587, 599–600, 645–646, 649, 652–661, 668–669, 671–673, 676–677, 682–684, 686, 691–694, 699, 701, 713–714, 716, 721–725, 727, 738–744
config
   __init__.py00100% 
   desktop.py330%4, 7–8
   docs.py660%4, 6–8, 12–13
tests
   conftest.py43197%54
   test_file.py173696%89, 174, 208–210, 223
   test_file_preview.py630100% 
TOTAL102236664% 

@agritheory agritheory merged commit b5bf57e into version-15 Mar 11, 2026
6 checks passed
@batonac batonac deleted the fix/duplicate-file-hash-check branch March 11, 2026 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enforce content hash checks on duplicate file names

2 participants