update code arch a little

ZedSpawner/Classes/ZedSpawner.uc

@@ -64,7 +64,7 @@ var private Array<class<KFPawn_Monster> > SpawnAtPlayerStartZeds;
 var private bool   SpawnActive;
 var private String SpawnListsComment;
 
-var private Array<ZedSpawnerRepLink> RepLinks;
+var private Array<ZedSpawnerRepInfo> RepInfos;
 
 public simulated function bool SafeDestroy()
 {
@@ -83,20 +83,11 @@ public event PreBeginPlay()
 	}
 	
 	Super.PreBeginPlay();
+	
+	PreInit();
 }
 
-public event PostBeginPlay()
+private function PreInit()
-{
-	`Log_Trace();
-	
-	if (bPendingDelete || bDeleteMe) return;
-	
-	Super.PostBeginPlay();
-	
-	Init();
-}
-
-private function InitConfig()
 {
 	if (Version == `NO_CONFIG)
 	{
@@ -141,9 +132,50 @@ private function InitConfig()
 		Version = LatestVersion;
 		SaveConfig();
 	}
+	
+	if (LogLevel == LL_WrongLevel)
+	{
+		LogLevel = LL_Info;
+		`Log_Warn("Wrong 'LogLevel', return to default value");
+		SaveConfig();
+	}
+	`Log_Base("LogLevel:" @ LogLevel);
+	
+	if (Tickrate <= 0)
+	{
+		`Log_Error("Spawner tickrate must be positive (current value:" @ Tickrate $ ")");
+		`Log_Fatal("Wrong settings, Destroy...");
+		SafeDestroy();
+		return;
 	}
 	
-private function Init()
+	dt = 1 / Tickrate;
+	`Log_Info("Spawner tickrate:" @ Tickrate @ "(update every" @ dt $ "s)");
+	
+	if (!CfgSpawn.static.Load(LogLevel))
+	{
+		`Log_Fatal("Wrong settings, Destroy...");
+		SafeDestroy();
+		return;
+	}
+
+	SpawnListRW = CfgSpawnListRW.static.Load(LogLevel);
+	SpawnListBW = CfgSpawnListBW.static.Load(LogLevel);
+	SpawnAtPlayerStartZeds = CfgSpawnAtPlayerStart.static.Load(LogLevel);
+}
+
+public event PostBeginPlay()
+{
+	`Log_Trace();
+	
+	if (bPendingDelete || bDeleteMe) return;
+	
+	Super.PostBeginPlay();
+	
+	PostInit();
+}
+
+private function PostInit()
 {
 	local S_SpawnEntry SE;
 	local String CurrentMap;
@@ -159,37 +191,16 @@ private function Init()
 	}
 	
 	KFGIA = new(KFGIS) class'KFGI_Access';
-	KFGIE = KFGameInfo_Endless(KFGIS);
+	if (KFGIA == None)
-	
-	InitConfig();
-
-	if (LogLevel == LL_WrongLevel)
 	{
-		LogLevel = LL_Info;
+		`Log_Fatal("Can't create KFGI_Access object");
-		`Log_Warn("Wrong 'LogLevel', return to default value");
-		SaveConfig();
-	}
-	`Log_Base("LogLevel:" @ LogLevel);
-	
-	if (Tickrate <= 0)
-	{
-		`Log_Error("Spawner tickrate must be positive (current value:" @ Tickrate $ ")");
-	}
-	
-	if (!CfgSpawn.static.Load(LogLevel) || Tickrate <= 0)
-	{
-		`Log_Fatal("Wrong settings, Destroy...");
 		SafeDestroy();
 		return;
 	}
 	
-	dt = 1 / Tickrate;
+	KFGIE = KFGameInfo_Endless(KFGIS);
-	`Log_Info("Spawner tickrate:" @ Tickrate @ "(update every" @ dt $ "s)");
 	
-	SpawnListRW = CfgSpawnListRW.static.Load(LogLevel);
-	SpawnListBW = CfgSpawnListBW.static.Load(LogLevel);
 	SpawnListSW = CfgSpawnListSW.static.Load(KFGIE, LogLevel);
-	SpawnAtPlayerStartZeds = CfgSpawnAtPlayerStart.static.Load(LogLevel);
 	
 	CurrentMap = String(WorldInfo.GetPackageName());
 	GlobalSpawnAtPlayerStart = (CfgSpawnAtPlayerStart.default.Map.Find(CurrentMap) != INDEX_NONE);
@@ -756,53 +767,59 @@ public function NotifyLogin(Controller C)
 {
 	`Log_Trace();
 
-	CreateRepLink(C);
+	if (!CreateRepInfo(C))
+	{
+		`Log_Error("Can't create RepInfo for:" @ C);
+	}
 }
 
 public function NotifyLogout(Controller C)
 {
 	`Log_Trace();
 
-	DestroyRepLink(C);
+	if (!DestroyRepInfo(C))
+	{
+		`Log_Error("Can't destroy RepInfo of:" @ C);
+	}
 }
 
-public function bool CreateRepLink(Controller C)
+public function bool CreateRepInfo(Controller C)
 {
-	local ZedSpawnerRepLink RepLink;
+	local ZedSpawnerRepInfo RepInfo;
 	
 	`Log_Trace();
 	
 	if (C == None) return false;
 	
-	RepLink = Spawn(class'ZedSpawnerRepLink', C);
+	RepInfo = Spawn(class'ZedSpawnerRepInfo', C);
 	
-	if (RepLink == None) return false;
+	if (RepInfo == None) return false;
 	
-	RepLink.LogLevel = LogLevel;
+	RepInfo.LogLevel = LogLevel;
-	RepLink.CustomZeds = CustomZeds;
+	RepInfo.CustomZeds = CustomZeds;
-	RepLink.ZS = Self;
+	RepInfo.ZS = Self;
 	
-	RepLinks.AddItem(RepLink);
+	RepInfos.AddItem(RepInfo);
 	
-	RepLink.ServerSync();
+	RepInfo.ServerSync();
 	
 	return true;
 }
 
-public function bool DestroyRepLink(Controller C)
+public function bool DestroyRepInfo(Controller C)
 {
-	local ZedSpawnerRepLink RepLink;
+	local ZedSpawnerRepInfo RepInfo;
 	
 	`Log_Trace();
 	
 	if (C == None) return false;
 	
-	foreach RepLinks(RepLink)
+	foreach RepInfos(RepInfo)
 	{
-		if (RepLink.Owner == C)
+		if (RepInfo.Owner == C)
 		{
-			RepLink.SafeDestroy();
+			RepInfo.SafeDestroy();
-			RepLinks.RemoveItem(RepLink);
+			RepInfos.RemoveItem(RepInfo);
 			return true;
 		}
 	}

ZedSpawner/Classes/ZedSpawnerMut.uc

@@ -38,16 +38,16 @@ public function AddMutator(Mutator Mut)
 
 public function NotifyLogin(Controller C)
 {
-	Super.NotifyLogin(C);
-	
 	ZS.NotifyLogin(C);
+	
+	Super.NotifyLogin(C);
 }
 
 public function NotifyLogout(Controller C)
 {
-	Super.NotifyLogout(C);
-	
 	ZS.NotifyLogout(C);
+	
+	Super.NotifyLogout(C);
 }
 
 DefaultProperties

ZedSpawner/Classes/ZedSpawnerRepInfo.uc

@@ -1,4 +1,4 @@
-class ZedSpawnerRepLink extends ReplicationInfo;
+class ZedSpawnerRepInfo extends ReplicationInfo;
 
 var public  ZedSpawner ZS;
 var public  E_LogLevel LogLevel;
@@ -50,7 +50,7 @@ public reliable server function ServerSync()
 	{
 		`Log_Debug("Sync finished");
 		SyncFinished();
-		if (!ZS.DestroyRepLink(Controller(Owner)))
+		if (!ZS.DestroyRepInfo(Controller(Owner)))
 		{
 			SafeDestroy();
 		}