MantisBT - Doomseeker
View Issue Details
0004019DoomseekerUIpublic2022-07-30 11:082023-02-19 14:14
auratoostronk 
Zalewa 
nonefeatureN/A
resolvedfixed 
1.3.3 
1.4.01.4.0 
0004019: When trying to join a server without a game executable path specified, Doomseeker should try to find it automatically
I can't believe that until after 5 years of using doomseeker I didn't know that there was a little magnifying glass by the game executable path that tries to find it for you (see attached image). Why not make Doomseeker use this function when joining a server in instances where the game executable can't be found?
No tags attached.
png THERE IS A FIND BUTTON.png (32,700) 2022-07-30 11:08
https://zandronum.com/tracker/file_download.php?file_id=2760&type=bug
png

png 2022-07-30-132432_maim.png (11,090) 2022-07-30 13:25
https://zandronum.com/tracker/file_download.php?file_id=2761&type=bug
png

png 2022-07-30-132721_maim.png (13,725) 2022-07-30 13:27
https://zandronum.com/tracker/file_download.php?file_id=2762&type=bug
png

jpg ds-20230115-configure-game-when-joining.jpg (12,395) 2023-01-15 14:58
https://zandronum.com/tracker/file_download.php?file_id=2832&type=bug
jpg

jpg ds-20230115-configure-game-when-showing-cmd.jpg (16,705) 2023-01-15 14:58
https://zandronum.com/tracker/file_download.php?file_id=2833&type=bug
jpg
Issue History
2022-07-30 11:08auratoostronkNew Issue
2022-07-30 11:08auratoostronkFile Added: THERE IS A FIND BUTTON.png
2022-07-30 13:21WubTheCaptainNote Added: 0022296
2022-07-30 13:25WubTheCaptainFile Added: 2022-07-30-132432_maim.png
2022-07-30 13:25WubTheCaptainNote Added: 0022297
2022-07-30 13:26WubTheCaptainNote Edited: 0022297bug_revision_view_page.php?rev_id=13657
2022-07-30 13:27WubTheCaptainFile Added: 2022-07-30-132721_maim.png
2022-07-30 13:29WubTheCaptainNote Edited: 0022297bug_revision_view_page.php?rev_id=13658
2022-07-30 13:32WubTheCaptainNote Edited: 0022297bug_revision_view_page.php?rev_id=13659
2022-07-30 13:36WubTheCaptainNote Edited: 0022297bug_revision_view_page.php?rev_id=13660
2022-07-30 13:38WubTheCaptainNote Edited: 0022297bug_revision_view_page.php?rev_id=13661
2022-07-30 13:39WubTheCaptainNote Deleted: 0022297
2022-07-30 13:39WubTheCaptainNote Added: 0022298
2022-07-30 13:55WubTheCaptainNote Added: 0022299
2022-07-30 13:58WubTheCaptainNote Added: 0022300
2022-07-30 17:42WubTheCaptainPrioritylow => none
2022-07-31 20:35auratoostronkNote Added: 0022301
2022-07-31 20:40auratoostronkNote Edited: 0022301bug_revision_view_page.php?bugnote_id=22301#r13663
2022-08-01 19:08ZalewaStatusnew => acknowledged
2022-09-10 14:37ZalewaTarget Version => 1.4.0
2022-12-14 17:46ZalewaAssigned To => Zalewa
2022-12-14 17:46ZalewaStatusacknowledged => assigned
2022-12-17 20:39ZalewaNote Added: 0022527
2022-12-17 20:39ZalewaStatusassigned => needs review
2022-12-17 20:40ZalewaNote Edited: 0022527bug_revision_view_page.php?bugnote_id=22527#r13777
2023-01-05 12:15ZalewaNote Added: 0022645
2023-01-05 12:15ZalewaStatusneeds review => needs testing
2023-01-15 14:58ZalewaFile Added: ds-20230115-configure-game-when-joining.jpg
2023-01-15 14:58ZalewaFile Added: ds-20230115-configure-game-when-showing-cmd.jpg
2023-01-15 14:59ZalewaNote Added: 0022700
2023-01-15 14:59ZalewaStatusneeds testing => resolved
2023-01-15 14:59ZalewaFixed in Version => 1.4.0
2023-01-15 14:59ZalewaResolutionopen => fixed
2023-01-15 22:26ZalewaNote Added: 0022702
2023-01-15 22:26ZalewaStatusresolved => needs review
2023-01-16 20:29ZalewaSummaryWhen trying to join a server without a game executeable path specified, Doomseeker should try to find it automatically => When trying to join a server without a game executable path specified, Doomseeker should try to find it automatically
2023-02-19 14:14ZalewaStatusneeds review => resolved

Notes
(0022296)
WubTheCaptain   
2022-07-30 13:21   
I don't want the configuration and its executable paths to be changed automatically unprompted without approval from the user. Consider unmounted filesystems / disks and such.
(0022298)
WubTheCaptain   
2022-07-30 13:39   
But the copy of the relevant error text could be improved, and direct the user directly to that plugin's Configuration with a "Configure" button. As a more graceful and friendly alternative, it could go through a proposed new configuration wizard for that game to set its executable before connecting (to reduce the amount of steps required to connect), instead of representing an error and refusing to solve it for you.
(0022299)
WubTheCaptain   
2022-07-30 13:55   
There's also an extra consideration for UNIX-like operating systems: I don't think the current system searches in $PATH environment, but instead looks for the paths hard-configured in PathFinder::genericPathFinder function (src/core/pathfinder/pathfinder.cpp), so the game client search function is not always be precise. Ideally it should utilize $PATH to be more helpful.
(0022300)
WubTheCaptain   
2022-07-30 13:58   
Quote from WubTheCaptain
There's also an extra consideration for UNIX-like operating systems

Microsoft Windows too, but I expect hardly any user to deviate from the default Path environment value on that operating system, really. Except developers and server administrators, who may use command-line interface tools.
(0022301)
auratoostronk   
2022-07-31 20:35   
(edited on: 2022-07-31 20:40)
I agree with prompting the user before changing their path for their exe. Anything that makes it easier for players to join is fine in my book.

(0022527)
Zalewa   
2022-12-17 20:39   
(edited on: 2022-12-17 20:40)
The fun thing is that Doomseeker was already opening the config box if you couldn't join the game due to it not being configured. However, due to a bug, the config page for the game wasn't selected properly. This should be fixed here.

Now, as far as the auto-config goes, I agree with Wub here that Doomseeker shouldn't meddle with the configuration like this or try to launch auto-guessed executables on its own. But, to help with the configuration issue, I decided to make the UI more explicit for the user.

So, the config box won't auto-open anymore, and instead a configure game button is displayed in the error message box. This should help with leading the user towards the proper solution of the "lack of game setup" problem.

I also made an attempt at making the error messages themselves sound more human, but I don't know if that was successful.

(0022645)
Zalewa   
2023-01-05 12:15   
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 if the "Configure game" button appears in the error box whenever user tries to join a game (or obtain a command line) when the game isn't configured or is configured improperly.

(0022700)
Zalewa   
2023-01-15 14:59   
When the client executable is misconfigured and the user tries to join a server or obtain its command line, they will get one a message box with the "Configure game" button as shown on:
- ds-20230115-configure-game-when-joining.jpg
- ds-20230115-configure-game-when-showing-cmd.jpg
(0022702)
Zalewa   
2023-01-15 22:26   
As pointed out by Wub in mail, saying "file" instead of "executable" (screenshots from 0004019:0022700) was inconsistent and imprecise.

Adjusted that here:'https://bitbucket.org/Doomseeker/doomseeker/commits/06240249c7135935311679563800c7e289d922dd [^]'