Bug: Copyzone command checks the wrong set of areas by default
Danger: Low - The command will simply fail if the wrong area names are checked.
Discovered in: AFKMud 2.03
Found by: Samson
Fixed by: Samson

---

shell.cpp, do_copyzone

Locate:

      if( !str_cmp( argument, "gods.are" ) )
         fname2 = "gods.are";
      if( !str_cmp( argument, "bywater.are" ) )
         fname2 = "bywater.are";
      if( !str_cmp( argument, "entry.are" ) )
         fname2 = "entry.are";
      if( !str_cmp( argument, "astral.are" ) )
         fname2 = "astral.are";
      if( !str_cmp( argument, "void.are" ) )
         fname2 = "void.are";
      if( !str_cmp( argument, "alsherok.are" ) )
         fname2 = "alsherok.are";
      if( !str_cmp( argument, "alatia.are" ) )
         fname2 = "alatia.are";
      if( !str_cmp( argument, "eletar.are" ) )
         fname2 = "eletar.are";
      if( !str_cmp( argument, "varsis.are" ) )
         fname2 = "varsis.are";
      if( !str_cmp( argument, "gwyn.are" ) )
         fname2 = "gwyn.are";
      if( !str_cmp( argument, "sindhae.are" ) )
         fname2 = "sindhae.are";

Change to:

      if( !str_cmp( argument, "entry.are" ) )
         fname2 = "entry.are";
      if( !str_cmp( argument, "astral.are" ) )
         fname2 = "astral.are";
      if( !str_cmp( argument, "void.are" ) )
         fname2 = "void.are";
      if( !str_cmp( argument, "immtrain.are" ) )
         fname2 = "immtrain.are";
      if( !str_cmp( argument, "one.are" ) )
         fname2 = "one.are";

Locate:

   if( fname2 == "gods.are" || fname2 == "void.are" || fname2 == "astral.are"
       || fname2 == "bywater.are" || fname2 == "entry.are" || fname2 == "alsherok.are" || fname2 == "alatia.are"
       || fname2 == "eletar.are" || fname2 == "varsis.are" || fname2 == "gwyn.are" || fname2 == "sindhae.are" )
   {
      ch->print( "&GArea file(s) updated to code port.\r\n" );
      funcf( ch, do_mudexec, "cp %s%s %s", BUILDZONEDIR, fname2, CODEZONEDIR );
   }

Change to:

   if( fname2 == "entry.are" || fname2 == "void.are" || fname2 == "astral.are" || fname2 == "one.are" || fname2 == "immtrain.are" )
   {
      ch->print( "&GArea file(s) updated to code port.\r\n" );
      funcf( ch, do_mudexec, "cp %s%s %s", BUILDZONEDIR, fname2, CODEZONEDIR );
   }

The list of areas specified by this command are not distributed with the codebase, so checking for them is a bit silly.