Best Option

if ( %guestkickbanonjoin. [ $+ [ # ] ] == $true ) { if (> isin $nick) { sockwrite -n dc.chat. $+ $cid $fchan $dc.fullnick($schan,$nick) :No Guests Allowed | access # add deny >*!* } 

What would be the best option or shortest code for this?

Thanks

personally i set an auto access check that keeps guest ban on all the time.. there is no reason to allow guests.

alias banallguests {
 var %i = 1
 while (%i <= $chan(0)) {
   if ($me isowner $chan(%i)) {
     access $chan(%i) clear
     access $chan(%i) add deny >* 0 :Guest Ban
     access $chan(%i) add deny *!*@GateKeeper$* 0 :Guest Ban
     access $chan(%i) add owner $$ial($me).addr 0 :
   }
   inc %i
 }
}

this is a bit tricky, i don't remember anymore, but i think > changes to Guest_ when your using DC. This however should do what you want it to.

on *:SIGNAL:dc.chat.*: {
 if ($me isop $schan) && (:>* JOIN iswm $1-2) ss sockwrite -n $signal access $right($4,-1) add deny *!*@gatekeeper 0 :Guest Ban $crlf kick $right($4,-1) $right($gettok($1,1,33),-1) :Guest Kick
}

well.. since guests will be specs, then why not just use it in a spec kicker. would be less coding

true, but you might want to ban guests instead... so you'd need the code anyways.