X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fmodules%2Fm_delayjoin.cpp;h=8b06f060ae3c6600bf1401120fbd0532e16b3d6a;hb=4047a143fc1d16350db70c94b9ea77d79de05714;hp=d157b97bcc90461ebf39d3895aa031919a1d6b7c;hpb=c6e40d36b42a7ebf832c3a57d2816a47ee9c9a76;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/modules/m_delayjoin.cpp b/src/modules/m_delayjoin.cpp
index d157b97bc..8b06f060a 100644
--- a/src/modules/m_delayjoin.cpp
+++ b/src/modules/m_delayjoin.cpp
@@ -24,13 +24,19 @@
 
 class DelayJoinMode : public ModeHandler
 {
+ private:
+	LocalIntExt& unjoined;
+
  public:
-	DelayJoinMode(Module* Parent) : ModeHandler(Parent, "delayjoin", 'D', PARAM_NONE, MODETYPE_CHANNEL)
+	DelayJoinMode(Module* Parent, LocalIntExt& ext)
+		: ModeHandler(Parent, "delayjoin", 'D', PARAM_NONE, MODETYPE_CHANNEL)
+		, unjoined(ext)
 	{
 		ranktoset = ranktounset = OP_VALUE;
 	}
 
 	ModeAction OnModeChange(User* source, User* dest, Channel* channel, std::string& parameter, bool adding) CXX11_OVERRIDE;
+	void RevealUser(User* user, Channel* chan);
 };
 
 
@@ -56,8 +62,9 @@ class JoinHook : public ClientProtocol::EventHook
 	ModResult OnPreEventSend(LocalUser* user, const ClientProtocol::Event& ev, ClientProtocol::MessageList& messagelist) CXX11_OVERRIDE
 	{
 		const ClientProtocol::Events::Join& join = static_cast<const ClientProtocol::Events::Join&>(ev);
-		const User* const u = join.GetMember()->user;
-		if ((unjoined.get(u)) && (u != user))
+		const Membership* const memb = join.GetMember();
+		const User* const u = memb->user;
+		if ((unjoined.get(memb)) && (u != user))
 			return MOD_RES_DENY;
 		return MOD_RES_PASSTHRU;
 	}
@@ -67,16 +74,15 @@ class JoinHook : public ClientProtocol::EventHook
 
 class ModuleDelayJoin : public Module
 {
-	DelayJoinMode djm;
-	void RevealUser(User* user, Channel* chan);
  public:
 	LocalIntExt unjoined;
 	JoinHook joinhook;
+	DelayJoinMode djm;
 
 	ModuleDelayJoin()
-		: djm(this)
-		, unjoined("delayjoin", ExtensionItem::EXT_MEMBERSHIP, this)
+		: unjoined("delayjoin", ExtensionItem::EXT_MEMBERSHIP, this)
 		, joinhook(this, unjoined)
+		, djm(this, unjoined)
 	{
 	}
 
@@ -91,16 +97,6 @@ class ModuleDelayJoin : public Module
 	ModResult OnRawMode(User* user, Channel* channel, ModeHandler* mh, const std::string& param, bool adding) CXX11_OVERRIDE;
 };
 
-// TODO: make RevealUser accessible from DelayJoinMode and get rid of this.
-class DummyMessageDetails : public MessageDetails
-{
-public:
-	DummyMessageDetails() : MessageDetails(MSG_PRIVMSG, "", ClientProtocol::TagMap()) { }
-	bool IsCTCP(std::string& name, std::string& body) const CXX11_OVERRIDE { return false; }
-	bool IsCTCP(std::string&) const CXX11_OVERRIDE { return false; }
-	bool IsCTCP() const CXX11_OVERRIDE { return false; }
-};
-
 ModeAction DelayJoinMode::OnModeChange(User* source, User* dest, Channel* channel, std::string &parameter, bool adding)
 {
 	/* no change */
@@ -113,13 +109,9 @@ ModeAction DelayJoinMode::OnModeChange(User* source, User* dest, Channel* channe
 		 * Make all users visible, as +D is being removed. If we don't do this,
 		 * they remain permanently invisible on this channel!
 		 */
-		MessageTarget msgtarget(channel, 0);
-		DummyMessageDetails msgdetails;
 		const Channel::MemberMap& users = channel->GetUsers();
 		for (Channel::MemberMap::const_iterator n = users.begin(); n != users.end(); ++n)
-		{
-			creator->OnUserMessage(n->first, msgtarget, msgdetails);
-		}
+			RevealUser(n->first, channel);
 	}
 	channel->SetMode(this, adding);
 	return MODEACTION_ALLOW;
@@ -190,10 +182,10 @@ void ModuleDelayJoin::OnUserMessage(User* user, const MessageTarget& target, con
 		return;
 
 	Channel* channel = target.Get<Channel>();
-	RevealUser(user, channel);
+	djm.RevealUser(user, channel);
 }
 
-void ModuleDelayJoin::RevealUser(User* user, Channel* chan)
+void DelayJoinMode::RevealUser(User* user, Channel* chan)
 {
 	Membership* memb = chan->GetUser(user);
 	if (!memb || !unjoined.set(memb, 0))
@@ -225,7 +217,7 @@ ModResult ModuleDelayJoin::OnRawMode(User* user, Channel* channel, ModeHandler*
 	if (!dest)
 		return MOD_RES_PASSTHRU;
 
-	RevealUser(dest, channel);
+	djm.RevealUser(dest, channel);
 	return MOD_RES_PASSTHRU;
 }