Fix znc#1133: Add MinClients option to simple_away

This adds a new option and command to the module, "MinClients", that
specifies the minimum number of clients that must be connected for the
module to considered the user as available.  If fewer clients are
connected than the minimum, the user will be marked away, even if there
are still clients attached.  The default value is "1", which replicates
the existing behavior of the module.

To modify the MinClients value, `/msg *simple_away MinClients [<value>]`
is available.  Running the command without parameters will display the
current value instead of modifying it.

modules/simple_away.cpp

@@ -39,13 +39,15 @@ class CSimpleAway : public CModule {
 private:
 	CString      m_sReason;
 	unsigned int m_iAwayWait;
+	unsigned int m_iMinClients;
 	bool         m_bClientSetAway;
 	bool         m_bWeSetAway;
 
 public:
 	MODCONSTRUCTOR(CSimpleAway) {
 		m_sReason        = SIMPLE_AWAY_DEFAULT_REASON;
 		m_iAwayWait      = SIMPLE_AWAY_DEFAULT_TIME;
+		m_iMinClients    = 1;
 		m_bClientSetAway = false;
 		m_bWeSetAway     = false;
 
@@ -54,6 +56,7 @@ class CSimpleAway : public CModule {
 		AddCommand("Timer", static_cast<CModCommand::ModCmdFunc>(&CSimpleAway::OnTimerCommand), "", "Prints the current time to wait before setting you away");
 		AddCommand("SetTimer", static_cast<CModCommand::ModCmdFunc>(&CSimpleAway::OnSetTimerCommand), "<seconds>", "Sets the time to wait before setting you away");
 		AddCommand("DisableTimer", static_cast<CModCommand::ModCmdFunc>(&CSimpleAway::OnDisableTimerCommand), "", "Disables the wait time before setting you away");
+		AddCommand("MinClients", static_cast<CModCommand::ModCmdFunc>(&CSimpleAway::OnMinClientsCommand), "", "Get or set the minimum number of clients before going away");
 	}
 
 	virtual ~CSimpleAway() {}
@@ -85,27 +88,33 @@ class CSimpleAway : public CModule {
 				SetReason(sSavedReason, false);
 		}
 
+		// MinClients
+		CString sMinClients = GetNV("minclients");
+		if (!sMinClients.empty())
+			SetMinClients(sMinClients.ToUInt(), false);
+
 		// Set away on load, required if loaded via webadmin
-		if (GetNetwork()->IsIRCConnected() && !GetNetwork()->IsUserAttached())
+		if (GetNetwork()->IsIRCConnected() && MinClientsConnected())
 			SetAway(false);
 
 		return true;
 	}
 
 	void OnIRCConnected() override {
-		if (GetNetwork()->IsUserAttached())
+		if (MinClientsConnected())
 			SetBack();
 		else
 			SetAway(false);
 	}
 
 	void OnClientLogin() override {
-		SetBack();
+		if (MinClientsConnected())
+			SetBack();
 	}
 
 	void OnClientDisconnect() override {
 		/* There might still be other clients */
-		if (!GetNetwork()->IsUserAttached())
+		if (!MinClientsConnected())
 			SetAway();
 	}
 
@@ -141,6 +150,15 @@ class CSimpleAway : public CModule {
 		PutModule("Timer disabled");
 	}
 
+	void OnMinClientsCommand(const CString& sLine) {
+		if (sLine.Token(1).empty()) {
+			PutModule(CString(m_iMinClients));
+		} else {
+			SetMinClients(sLine.Token(1).ToUInt());
+			PutModule("MinClients set");
+		}
+	}
+
 	EModRet OnUserRaw(CString &sLine) override {
 		if (!sLine.Token(0).Equals("AWAY"))
 			return CONTINUE;
@@ -179,6 +197,11 @@ class CSimpleAway : public CModule {
 	}
 
 private:
+	bool MinClientsConnected()
+	{
+		return GetNetwork()->GetClients().size() >= m_iMinClients;
+	}
+
 	CString ExpandReason() {
 		CString sReason = m_sReason;
 		if (sReason.empty())
@@ -204,6 +227,11 @@ class CSimpleAway : public CModule {
 		m_iAwayWait = iAwayWait;
 	}
 
+	void SetMinClients(unsigned int iMinClients, bool bSave = true) {
+		if (bSave)
+			SetNV("minclients", CString(iMinClients));
+		m_iMinClients = iMinClients;
+	}
 };
 
 void CSimpleAwayJob::RunJob() {