Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#1778 closed enhancement (fixed)

[PATCH] Remember last name and IP at "Join Game" dialog

Reported by: Aurélio A. Heckert Owned by: leper
Priority: Should Have Milestone: Alpha 13
Component: Core engine Keywords: patch
Cc: Patch:

Description

In the future a hole new full featured dialog will be nice, but, by now, remembering the last used "player name" and "server IP" will be a great help.

Attachments (2)

0ad-adding-GetDefaultMPServer.diff (2.9 KB ) - added by Aurélio A. Heckert 11 years ago.
50% of the feature request, adding the ability to remember the server.
0ad-adding-RecordLastMPServerAndPlayer.diff (2.4 KB ) - added by Aurélio A. Heckert 11 years ago.
The last 50% to close the ticket.

Download all attachments as: .zip

Change History (10)

by Aurélio A. Heckert, 11 years ago

50% of the feature request, adding the ability to remember the server.

comment:1 by Aurélio A. Heckert, 11 years ago

Milestone: BacklogAlpha 13
Summary: Remember last name and IP at "Join Game" dialog[PATCH] Remember last name and IP at "Join Game" dialog

comment:2 by leper, 11 years ago

Keywords: patch review added

comment:3 by leper, 11 years ago

In 13003:

Read default multiplayer server name/IP address from config. Patch by Aurium. Refs #1778.

comment:4 by leper, 11 years ago

Keywords: review removed

I applied your change with a little modification (There is no default to 127.0.0.1 in the code as there is for "anonymous", so I removed the .empty() check and added it as the default to system.conf).

Regarding your question on IRC (as I'm not sure that you are reading the logs):
Check SetSplashScreenEnabled in source/gui/scripting/ScriptFunctions.cpp

by Aurélio A. Heckert, 11 years ago

The last 50% to close the ticket.

comment:5 by Aurélio A. Heckert, 11 years ago

Keywords: review added

comment:6 by leper, 11 years ago

Owner: set to leper

As already noted on IRC I found a bug in the CreateValue function and I want to fix this in one go, so I'm assigning this to myself. Thanks for the patch!

comment:7 by leper, 11 years ago

Resolution: fixed
Status: newclosed

In 13021:

Save last multiplayer server. Based on patch by Aurium. Fixes #1778.
Fix bug in ConfigDB: Creating a value could result in changing a value in a different namespace.

comment:8 by leper, 11 years ago

Keywords: review removed

Thanks for the patch.

Note: See TracTickets for help on using tickets.