MantisBT - Doomseeker
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0003501Doomseeker[All Projects] Suggestionpublic2018-09-20 17:302024-11-03 19:10
ReporterZalewa 
Assigned ToZalewa 
PrioritynormalSeverityminorReproducibilityN/A
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version1.1 
Target Version1.4.0Fixed in Version1.4.0 
Summary0003501: Inform user when demo cannot be recorded due to missing 'write' privilege to the demos directory
DescriptionThere's a silent data loss when user has demo recording enabled, the demos storage directory doesn't exist or has no write permissions and the user starts the game. Doomseeker could check the storage directory before starting the game, try to amend some erroreneous situations or ultimately fail and display an error message.

Case: directory is missing
Approach: Try to create the directory, if that fails then fail to start the game with appropriate error message notifying the user that they want to record demos but the demos cannot be saved as the storage directory is missing.

Case: directory exists but has no write permissions
Approach: Immediately fail to start the game and display relevant error message like above.
Steps To Reproduce
Additional InformationTrigger vectors:

1. Join a server from the server list.
2. Launch offline game from Create Game box
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2018-09-20 17:30ZalewaNew Issue
2018-09-21 12:57WubTheCaptainNote Added: 0019588
2018-09-21 12:57WubTheCaptainStatusnew => acknowledged
2022-09-10 15:01ZalewaTarget Version => 1.4.0
2022-12-14 17:46ZalewaAssigned To => Zalewa
2022-12-14 17:46ZalewaStatusacknowledged => assigned
2022-12-28 17:24ZalewaNote Added: 0022565
2022-12-28 17:24ZalewaStatusassigned => needs review
2023-01-05 12:19ZalewaNote Added: 0022649
2023-01-05 12:19ZalewaStatusneeds review => needs testing
2023-02-19 14:11ZalewaStatusneeds testing => resolved
2023-02-19 14:11ZalewaFixed in Version => 1.4.0
2023-02-19 14:11ZalewaResolutionopen => fixed
2024-11-03 19:10ZalewaStatusresolved => closed

Notes
(0019588)
WubTheCaptain   
2018-09-21 12:57   
As mentioned in 0003453.
(0022565)
Zalewa   
2022-12-28 17:24   
Should be done:
1.'https://bitbucket.org/Doomseeker/doomseeker/commits/f3713f9aa30f98c6fd1a564f1bca314a32bbbcb0 [^]'
2. Plus try to provide more detailed info on why the directory couldn't be created:'https://bitbucket.org/Doomseeker/doomseeker/commits/39ddefe81f92d07c0d6b03fcbe04a8b05162e945 [^]'
(0022649)
Zalewa   
2023-01-05 12:19   
Beta package for Windows available at the beta auto-update channel and at:
'https://devbuilds.drdteam.org/doomseeker/doomseeker-1.3.3~beta-230105-1140_windows.zip [^]'

Please test what happens if the privileges are taken away from the demo storage directory and demo recording is enabled.

The demos storage directory can be found in the startup logs, look for entry:


 Data directory: C:/Users/<user>/AppData/Local/doomseeker

And there should be `demos` directory inside.