fix(servers): fix issue of space in server name

Replace the space in a server name with a hyphen (-), also noticed some
interesting behaviour with the 'count up naming', so tweaked that.

Resolves danielyxie/bitburner#1999
This commit is contained in:
Dan 2021-12-19 04:03:14 +00:00
parent 790ffeb8a1
commit 6d79561859
No known key found for this signature in database
GPG Key ID: CD0F515DA7B576B5
2 changed files with 8 additions and 5 deletions

@ -16,22 +16,25 @@ import { isValidNumber } from "../utils/helpers/isValidNumber";
* does not have a duplicate hostname/ip.
*/
export function safetlyCreateUniqueServer(params: IConstructorParams): Server {
let hostname: string = params.hostname.replace(/ /g, `-`);
if (params.ip != null && ipExists(params.ip)) {
params.ip = createUniqueRandomIp();
}
if (GetServer(params.hostname) != null) {
if (GetServer(hostname) != null) {
hostname = `${hostname}-0`;
// Use a for loop to ensure that we don't get suck in an infinite loop somehow
let hostname: string = params.hostname;
for (let i = 0; i < 200; ++i) {
hostname = `${params.hostname}-${i}`;
hostname = hostname.replace(/-[0-9]+$/, `-${i}`);
if (GetServer(hostname) == null) {
break;
}
}
params.hostname = hostname;
}
params.hostname = hostname;
return new Server(params);
}

@ -96,7 +96,7 @@ export function purchaseServer(hostname: string, ram: number, cost: number, p: I
p.loseMoney(cost, "servers");
dialogBoxCreate("Server successfully purchased with hostname " + hostname);
dialogBoxCreate("Server successfully purchased with hostname " + newServ.hostname);
}
// Manually upgrade RAM on home computer (NOT through Netscript)