Bug 455007 - Ark should zero-pad hashes in the "CRC checksum" column
Summary: Ark should zero-pad hashes in the "CRC checksum" column
Status: RESOLVED FIXED
Alias: None
Product: ark
Classification: Applications
Component: general (show other bugs)
Version: 22.04.1
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Elvis Angelaccio
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-06-08 03:55 UTC by Nagy Tibor
Modified: 2022-06-09 17:04 UTC (History)
2 users (show)

See Also:
Latest Commit: https://invent.kde.org/utilities/ark/commit/0b47588458f8b499c7c78fe3235217c9fe1888cc
Version Fixed In:
Sentry Crash Report:

Attachments
Test ZIP archive (496 bytes, application/zip)
2022-06-08 03:55 UTC, Nagy Tibor 		Details
Screenshot (57.33 KB, image/png)
2022-06-08 03:56 UTC, Nagy Tibor 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Nagy Tibor 2022-06-08 03:55:50 UTC 
Created attachment 149545 [details]
Test ZIP archive

SUMMARY
When opening ZIP archives Ark should zero-pad the CRC32 hashes in the main list view to a fixed 8-digit width. Currently this checksum column is unpadded. This makes the column look jank.

STEPS TO REPRODUCE
Open the attached test ZIP archive.

OBSERVED RESULT

| Name                 | ... | CRC checksum |
| 0_leading_zeroes.bin | ... | 3488DF43     |
| 2_leading_zeroes.bin | ... | 6D18CB       |
| 4_leading_zeroes.bin | ... | 8D96         |
| 8_leading_zeroes.bin | ... | 0            |

EXPECTED RESULT

| Name                 | ... | CRC checksum |
| 0_leading_zeroes.bin | ... | 3488DF43     |
| 2_leading_zeroes.bin | ... | 006D18CB     |
| 4_leading_zeroes.bin | ... | 00008D96     |
| 8_leading_zeroes.bin | ... | 00000000     |

SOFTWARE/OS VERSIONS
Operating System: KDE neon 5.24
KDE Plasma Version: 5.24.5
KDE Frameworks Version: 5.94.0
Qt Version: 5.15.4
Graphics Platform: X11
Comment 1 Nagy Tibor 2022-06-08 03:56:13 UTC 
Created attachment 149546 [details]
Screenshot
Comment 2 Bug Janitor Service 2022-06-08 21:33:27 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/utilities/ark/-/merge_requests/123
Comment 3 Albert Astals Cid 2022-06-09 17:04:57 UTC 
Git commit 0b47588458f8b499c7c78fe3235217c9fe1888cc by Albert Astals Cid.
Committed on 09/06/2022 at 17:03.
Pushed by aacid into branch 'master'.

Zip: pad CRC numbers to 8 characters

M  +1    -1    plugins/libzipplugin/libzipplugin.cpp

https://invent.kde.org/utilities/ark/commit/0b47588458f8b499c7c78fe3235217c9fe1888cc