UFO: Alien Invasion Issue Tracker
UFO: Alien Invasion
Go to the previous open issue
Go to the previous issue (open or closed)
star_faded.png
Please log in to bookmark issues
icon_project.png UFO: Alien Invasion / Open Feature request #5256 Better handling of keys.cfg
action_vote_minus_faded.png
0
Votes
action_vote_plus_faded.png
Go to the next issue (open or closed)
Go to the next open issue
Issue basics
  • Type of issue
    Feature request
  • Category
    Not determined
  • Targetted for
    Not determined
  • Status
    Open
  • Priority
    Not determined
User pain
  • Type of bug
    Not triaged
  • Likelihood
    Not triaged
  • Effect
    Not triaged
Affected by this issue (0)
There are no items
People involved
Times and dates
  • Posted at
  • Last updated
Issue details
  • Complexity
    Not determined
Attachments (0)
There is nothing attached to this issue
Duplicate issues (0)
This issue does not have any duplicates
Description
Currently the keybindings file (keys.cfg) is not loaded by the input handler, only saved. The keys file is executed as a script later by the Command Interpreter. The keybinding saving assumes that the configuration was set up correctly and overwrites the old configuration file. Recently a workaround was added that removes the configuration file if no keybinding was saved but it is not the best solution.

Prolems found so far:

1. The keys configuration loaded too late. It is possible that the program ends - and tries to save the key configuration - before it happens.

2. The Command buffer has an upper limit in size. By adding keybindings or tooltips - keys.cfg can contain tooltips for bindings too -, keys.cfg can exceed this limit, the Command Interpreter will truncate the script to its size limit and execute a half/broken configuration.

3. There is no check for a broken key configuration. We should check at least for the most basic bindings (like "+leftmouse") to ensure that the player can navigate in the UI.

4. The saving is immediate, no backups created. On write error or 0 saved bindings the function removes the script - reseting the bindings to the defaults, losing any modifications made by the user. It also doesn't check for the most basic bindings to exists.
Todos (0 / 0)
Issue created
footer_logo.png The Bug Genie 4.3.1 | Support | Feedback spinning_16.gif