]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/modules/m_spanningtree/fjoin.cpp
m_spanningtree Replace manual string building of outgoing commands with CmdBuilder...
[user/henk/code/inspircd.git] / src / modules / m_spanningtree / fjoin.cpp
index 0fb4468772bb83ea62bc2d58f3e66d59c4ab1ff4..d8027628fbd2fa0051d9f176c37311bc24338523 100644 (file)
@@ -55,10 +55,12 @@ CmdResult CommandFJoin::Handle(User* srcuser, std::vector<std::string>& params)
         * who succeed at internets. :-)
         *
         * Syntax:
-        * :<sid> FJOIN <chan> <TS> <modes> :[[modes,]<uuid> [[modes,]<uuid> ... ]]
-        * The last parameter is a list consisting of zero or more (modelist, uuid)
-        * pairs (permanent channels may have zero users). The mode list for each
-        * user is a concatenation of the mode letters the user has on the channel
+        * :<sid> FJOIN <chan> <TS> <modes> :[<member> [<member> ...]]
+        * The last parameter is a list consisting of zero or more channel members
+        * (permanent channels may have zero users). Each entry on the list is in the
+        * following format:
+        * [[<modes>,]<uuid>[:<membid>]
+        * <modes> is a concatenation of the mode letters the user has on the channel
         * (e.g.: "ov" if the user is opped and voiced). The order of the mode letters
         * are not important but if a server ecounters an unknown mode letter, it will
         * drop the link to avoid desync.
@@ -67,15 +69,12 @@ CmdResult CommandFJoin::Handle(User* srcuser, std::vector<std::string>& params)
         * had no prefix modes on the channel, InspIRCd 2.2 and later does not require
         * a comma in this case anymore.
         *
+        * <membid> is a positive integer representing the id of the membership.
+        * If not present (in FJOINs coming from pre-1205 servers), 0 is assumed.
+        *
         */
 
-       time_t TS = ConvToInt(params[1]);
-       if (!TS)
-       {
-               ServerInstance->Logs->Log(MODNAME, LOG_DEFAULT, "*** BUG? *** TS of 0 sent to FJOIN. Are some services authors smoking craq, or is it 1970 again?. Dropped.");
-               ServerInstance->SNO->WriteToSnoMask('d', "WARNING: The server %s is sending FJOIN with a TS of zero. Total craq. Command was dropped.", srcuser->server.c_str());
-               return CMD_INVALID;
-       }
+       time_t TS = ServerCommand::ExtractTS(params[1]);
 
        const std::string& channel = params[0];
        Channel* chan = ServerInstance->FindChan(channel);
@@ -99,16 +98,8 @@ CmdResult CommandFJoin::Handle(User* srcuser, std::vector<std::string>& params)
                        }
                        else if (ourTS > TS)
                        {
-                               /* Our TS greater than theirs, clear all our modes from the channel, accept theirs. */
-                               if (Utils->AnnounceTSChange)
-                                       chan->WriteChannelWithServ(ServerInstance->Config->ServerName, "NOTICE %s :TS for %s changed from %lu to %lu", chan->name.c_str(), channel.c_str(), (unsigned long) ourTS, (unsigned long) TS);
-
-                               // while the name is equal in case-insensitive compare, it might differ in case; use the remote version
-                               chan->name = channel;
-                               chan->age = TS;
-                               chan->ClearInvites();
-
-                               CommandFJoin::RemoveStatus(chan);
+                               // Our TS is greater than theirs, remove all modes, extensions, etc. from the channel
+                               LowerTS(chan, TS, channel);
 
                                // XXX: If the channel does not exist in the chan hash at this point, create it so the remote modes can be applied on it.
                                // This happens to 0-user permanent channels on the losing side, because those are removed (from the chan hash, then
@@ -157,16 +148,15 @@ CmdResult CommandFJoin::Handle(User* srcuser, std::vector<std::string>& params)
        }
 
        irc::modestacker modestack(true);
-       TreeSocket* src_socket = Utils->FindServer(srcuser->server)->GetSocket();
+       TreeSocket* src_socket = TreeServer::Get(srcuser)->GetSocket();
 
        /* Now, process every 'modes,uuid' pair */
-       irc::tokenstream users(*params.rbegin());
+       irc::tokenstream users(params.back());
        std::string item;
        irc::modestacker* modestackptr = (apply_other_sides_modes ? &modestack : NULL);
        while (users.GetToken(item))
        {
-               if (!ProcessModeUUIDPair(item, src_socket, chan, modestackptr))
-                       return CMD_INVALID;
+               ProcessModeUUIDPair(item, src_socket, chan, modestackptr);
        }
 
        /* Flush mode stacker if we lost the FJOIN or had equal TS */
@@ -176,24 +166,25 @@ CmdResult CommandFJoin::Handle(User* srcuser, std::vector<std::string>& params)
        return CMD_SUCCESS;
 }
 
-bool CommandFJoin::ProcessModeUUIDPair(const std::string& item, TreeSocket* src_socket, Channel* chan, irc::modestacker* modestack)
+void CommandFJoin::ProcessModeUUIDPair(const std::string& item, TreeSocket* src_socket, Channel* chan, irc::modestacker* modestack)
 {
        std::string::size_type comma = item.find(',');
 
        // Comma not required anymore if the user has no modes
-       std::string uuid = ((comma == std::string::npos) ? item : item.substr(comma+1));
+       const std::string::size_type ubegin = (comma == std::string::npos ? 0 : comma+1);
+       std::string uuid(item, ubegin, UIDGenerator::UUID_LENGTH);
        User* who = ServerInstance->FindUUID(uuid);
        if (!who)
        {
                // Probably KILLed, ignore
-               return true;
+               return;
        }
 
        /* Check that the user's 'direction' is correct */
-       TreeServer* route_back_again = Utils->BestRouteTo(who->server);
-       if ((!route_back_again) || (route_back_again->GetSocket() != src_socket))
+       TreeServer* route_back_again = TreeServer::Get(who);
+       if (route_back_again->GetSocket() != src_socket)
        {
-               return true;
+               return;
        }
 
        /* Check if the user received at least one mode */
@@ -204,34 +195,39 @@ bool CommandFJoin::ProcessModeUUIDPair(const std::string& item, TreeSocket* src_
                for (std::string::const_iterator i = item.begin(); i != commait; ++i)
                {
                        if (!ServerInstance->Modes->FindMode(*i, MODETYPE_CHANNEL))
-                       {
-                               ServerInstance->SNO->WriteToSnoMask('d', "Unrecognised mode '%c' for a user in FJOIN, dropping link", *i);
-                               return false;
-                       }
+                               throw ProtocolException("Unrecognised mode '" + std::string(1, *i) + "'");
 
                        /* Add any modes this user had to the mode stack */
                        modestack->Push(*i, who->nick);
                }
        }
 
-       chan->ForceJoin(who, NULL, route_back_again->bursting);
-       return true;
+       Membership* memb = chan->ForceJoin(who, NULL, route_back_again->bursting);
+       if (!memb)
+               return;
+
+       // Assign the id to the new Membership
+       Membership::Id membid = 0;
+       const std::string::size_type colon = item.rfind(':');
+       if (colon != std::string::npos)
+               membid = Membership::IdFromString(item.substr(colon+1));
+       memb->id = membid;
 }
 
 void CommandFJoin::RemoveStatus(Channel* c)
 {
        irc::modestacker stack(false);
 
-       for (char modeletter = 'A'; modeletter <= 'z'; ++modeletter)
+       const ModeParser::ModeHandlerMap& mhs = ServerInstance->Modes->GetModes(MODETYPE_CHANNEL);
+       for (ModeParser::ModeHandlerMap::const_iterator i = mhs.begin(); i != mhs.end(); ++i)
        {
-               ModeHandler* mh = ServerInstance->Modes->FindMode(modeletter, MODETYPE_CHANNEL);
+               ModeHandler* mh = i->second;
 
                /* Passing a pointer to a modestacker here causes the mode to be put onto the mode stack,
                 * rather than applied immediately. Module unloads require this to be done immediately,
                 * for this function we require tidyness instead. Fixes bug #493
                 */
-               if (mh)
-                       mh->RemoveMode(c, stack);
+               mh->RemoveMode(c, stack);
        }
 
        ApplyModeStack(ServerInstance->FakeClient, c, stack);
@@ -248,3 +244,64 @@ void CommandFJoin::ApplyModeStack(User* srcuser, Channel* c, irc::modestacker& s
                stackresult.erase(stackresult.begin() + 1, stackresult.end());
        }
 }
+
+void CommandFJoin::LowerTS(Channel* chan, time_t TS, const std::string& newname)
+{
+       if (Utils->AnnounceTSChange)
+               chan->WriteChannelWithServ(ServerInstance->Config->ServerName, "NOTICE %s :TS for %s changed from %lu to %lu", chan->name.c_str(), newname.c_str(), (unsigned long) chan->age, (unsigned long) TS);
+
+       // While the name is equal in case-insensitive compare, it might differ in case; use the remote version
+       chan->name = newname;
+       chan->age = TS;
+
+       // Remove all pending invites
+       chan->ClearInvites();
+
+       // Clear all modes
+       CommandFJoin::RemoveStatus(chan);
+
+       // Unset all extensions
+       chan->FreeAllExtItems();
+
+       // Clear the topic, if it isn't empty then send a topic change message to local users
+       if (!chan->topic.empty())
+       {
+               chan->topic.clear();
+               chan->WriteChannelWithServ(ServerInstance->Config->ServerName, "TOPIC %s :", chan->name.c_str());
+       }
+       chan->setby.clear();
+       chan->topicset = 0;
+}
+
+CommandFJoin::Builder::Builder(Channel* chan)
+       : CmdBuilder("FJOIN")
+{
+       push(chan->name).push_int(chan->age).push_raw(" +");
+       pos = str().size();
+       push_raw(chan->ChanModes(true)).push_raw(" :");
+}
+
+void CommandFJoin::Builder::add(Membership* memb)
+{
+       push_raw(memb->modes).push_raw(',').push_raw(memb->user->uuid);
+       push_raw(':').push_raw_int(memb->id);
+       push_raw(' ');
+}
+
+bool CommandFJoin::Builder::has_room(Membership* memb) const
+{
+       return ((str().size() + memb->modes.size() + UIDGenerator::UUID_LENGTH + 2 + membid_max_digits + 1) <= maxline);
+}
+
+void CommandFJoin::Builder::clear()
+{
+       content.erase(pos);
+       push_raw(" :");
+}
+
+const std::string& CommandFJoin::Builder::finalize()
+{
+       if (*content.rbegin() == ' ')
+               content.erase(content.size()-1);
+       return str();
+}