X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fmodules%2Fm_cycle.cpp;h=57882b2912e5f08dadaad0e41cbfa58ebe0b76bb;hb=8e8445d1c0744130f232d3c6f56d5fe1079773ad;hp=295e7320889b071245f1ab341922aa8803ee1eb0;hpb=2b3394855d5adddb16285b905503d9ffe5a1d963;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/modules/m_cycle.cpp b/src/modules/m_cycle.cpp
index 295e73208..57882b291 100644
--- a/src/modules/m_cycle.cpp
+++ b/src/modules/m_cycle.cpp
@@ -2,7 +2,7 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  InspIRCd: (C) 2002-2007 InspIRCd Development Team
+ *  InspIRCd: (C) 2002-2008 InspIRCd Development Team
  * See: http://www.inspircd.org/wiki/index.php/Credits
  *
  * This program is free but copyrighted software; see
@@ -17,19 +17,19 @@
 
 /** Handle /CYCLE
  */
-class cmd_cycle : public command_t
+class CommandCycle : public Command
 {
  public:
-	cmd_cycle (InspIRCd* Instance) : command_t(Instance,"CYCLE", 0, 1)
+	CommandCycle (InspIRCd* Instance) : Command(Instance,"CYCLE", 0, 1, false, 3)
 	{
 		this->source = "m_cycle.so";
 		syntax = "<channel> :[reason]";
 		TRANSLATE3(TR_TEXT, TR_TEXT, TR_END);
 	}
 
-	CmdResult Handle (const char** parameters, int pcnt, userrec *user)
+	CmdResult Handle (const char* const* parameters, int pcnt, User *user)
 	{
-		chanrec* channel = ServerInstance->FindChan(parameters[0]);
+		Channel* channel = ServerInstance->FindChan(parameters[0]);
 		std::string reason = ConvToStr("Cycling");
 			
 		if (pcnt > 1)
@@ -38,7 +38,13 @@ class cmd_cycle : public command_t
 			reason = reason + ": " + parameters[1];
 		}
 		
-		if (channel)
+		if (!channel)
+		{
+			user->WriteNumeric(403, "%s %s :No such channel", user->nick, parameters[0]);
+			return CMD_FAILURE;
+		}
+		
+		if (channel->HasUser(user))
 		{
 			/*
 			 * technically, this is only ever sent locally, but pays to be safe ;p
@@ -52,18 +58,18 @@ class cmd_cycle : public command_t
 					return CMD_FAILURE;
 				}
 				
-				/* XXX in the future, this may move to a static chanrec method (the delete.) -- w00t */
+				/* XXX in the future, this may move to a static Channel method (the delete.) -- w00t */
 				if (!channel->PartUser(user, reason.c_str()))
 					delete channel;
 				
-				chanrec::JoinUser(ServerInstance, user, parameters[0], true, "", ServerInstance->Time(true));
+				Channel::JoinUser(ServerInstance, user, parameters[0], true, "", false, ServerInstance->Time());
 			}
 
 			return CMD_LOCALONLY;
 		}
 		else
 		{
-			user->WriteServ("NOTICE %s :*** You are not on this channel", user->nick);
+			user->WriteNumeric(442, "%s %s :You're not on that channel", user->nick, channel->name);
 		}
 
 		return CMD_FAILURE;
@@ -73,14 +79,15 @@ class cmd_cycle : public command_t
 
 class ModuleCycle : public Module
 {
-	cmd_cycle*	mycommand;
+	CommandCycle*	mycommand;
  public:
 	ModuleCycle(InspIRCd* Me)
 		: Module(Me)
 	{
 		
-		mycommand = new cmd_cycle(ServerInstance);
+		mycommand = new CommandCycle(ServerInstance);
 		ServerInstance->AddCommand(mycommand);
+
 	}
 	
 	virtual ~ModuleCycle()
@@ -89,7 +96,7 @@ class ModuleCycle : public Module
 	
 	virtual Version GetVersion()
 	{
-		return Version(1, 1, 0, 1, VF_COMMON | VF_VENDOR, API_VERSION);
+		return Version(1, 2, 0, 1, VF_COMMON | VF_VENDOR, API_VERSION);
 	}
 	
 };