Fix silliness discovered in conversion to JavaScript

So apparently 23 years ago (almost 24), I made a typo in commit
7731d16d that broke the batch transfer menu for RIP users.

Also elminated silly/unnecessary loop-control variable ('done').

I'm working on a port to JS and noticed these oddities (so far).

src/sbbs3/bat_xfer.cpp

@@ -27,7 +27,7 @@
 /****************************************************************************/
 void sbbs_t::batchmenu()
 {
-    char	str[129],tmp2[250],done=0,ch;
+    char	str[129],tmp2[250],ch;
 	char 	tmp[512];
 	char	keys[32];
 	int		sort = -1;
@@ -48,9 +48,9 @@ void sbbs_t::batchmenu()
 		return; 
 	}
 	if(useron.misc&(RIP) && !(useron.misc&EXPERT))
-		menu("batchxfer");
+		menu("batchxfr");
 	lncntr=0;
-	while(online && !done && (cfg.upload_dir!=INVALID_DIR || batdn_total() || batup_total())) {
+	while(online && (cfg.upload_dir!=INVALID_DIR || batdn_total() || batup_total())) {
 		if(!(useron.misc&(EXPERT|RIP))) {
 			sys_status&=~SS_ABORT;
 			if(lncntr) {
@@ -69,7 +69,6 @@ void sbbs_t::batchmenu()
 			logch(ch,0);
 		if(ch==quit_key() || ch=='\r') {	/* Quit */
 			lncntr=0;
-			done=1;
 			break;
 		}
 		switch(ch) {