Storage usage seems to be shown incorrectly

Open
#9,353 26 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
35/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Stale
Tech stack
android, kotlin
Domain
mobile

Research direction

Start by reproducing the Storage screen using the reported Samsung SM-J111F, Android 5.1.1, and Signal 4.53.7, then compare the main storage chart with the detailed Media, Audio, Files, and Videos totals. Done means the displayed category totals reconcile with the actual media files, or the remaining discrepancy is explained and covered by a regression test.

Written by the indexing model from the issue text.

Description

acknowledged
Bug description

I was recently exploring the "Storage" feature which shows the statistics of the storage used by media messages. That's when I found that the actual storage taken up by all the media files doesn't seem to be adding up to the usage shown in the main screen the space taken by all the media. Let me explain that in a little detail.

The total usage shown in the main screen is 25 MB with the following split-up (pie-chart):
image

When I tap on “REVIEW STORAGE” to see the more detailed view, I found that there was nothing listed in the “Audio” and “Files” section. So, I’m not sure why “Files” occupies such a large portion of the pie-chart when I don’t have any. The case is same for “Videos”. I went through the files shown in the “Media” section and could not find any videos.
image image

To find the total space taken up by the files in the “Media”, I selected all the files shown in the “Media” section (using the “Select All” option). The size it shows for the 54 items there is 7.9 MB.
image

So, there seems to be a gap of 17.1 MB between the actual storage space used by the media files (7.9 MB) and the one shown in the main screen (25 MB). This is the reason why I think the storage usage is shown incorrectly.

Device info

Device: Samsung SM-J111F
Android version: 5.1.1
Signal version: 4.53.7

Note: Initially discussed in the linked community forum post

Dominant language
Kotlin
Stars
29.4k
Forks
6.9k
No merged PRs in 30d

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from signalapp/Signal-Android

All issues in signalapp/Signal-Android

Similar issues

More Kotlin issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.