X-Git-Url: https://ruin.nu/git/?a=blobdiff_plain;f=editRaid.pl;h=94ed68da85685c964d3ef05d55b15759c0bd5e07;hb=03830799201db0b0f28e9c494fdd1b5b5143749c;hp=fd109320880459d6edab93e7740f5b1c3be9bae0;hpb=aa6a1cd834d42beaf03b7b9ddb5e5686ea3b132e;p=ndwebbie.git

diff --git a/editRaid.pl b/editRaid.pl
index fd10932..94ed68d 100644
--- a/editRaid.pl
+++ b/editRaid.pl
@@ -18,10 +18,10 @@
 #**************************************************************************/
 
 use strict;
-use POSIX;
+use warnings FATAL => 'all';
+use ND::Include;
 our $BODY;
 our $DBH;
-our $LOG;
 my $error;
 
 $ND::TEMPLATE->param(TITLE => 'Create/Edit Raids');
@@ -32,11 +32,11 @@ my @alliances = alliances();
 $BODY->param(Alliances => \@alliances);
 
 my $raid;
-if (param('raid') =~ /^(\d+)$/){
+if (defined param 'raid' and param('raid') =~ /^(\d+)$/){
 	my $query = $DBH->prepare(q{SELECT id,tick,waves,message,released_coords,open FROM raids WHERE id = ?});
 	$raid = $DBH->selectrow_hashref($query,undef,$1);
 }
-if (param('cmd') eq 'submit'){
+if (defined param('cmd') && param('cmd') eq 'submit'){
 	my $query = $DBH->prepare(q{INSERT INTO raids (tick,waves,message) VALUES(?,?,'')});
 	if ($query->execute(param('tick'),param('waves'))){
 		$raid = $DBH->last_insert_id(undef,undef,undef,undef,"raids_id_seq");
@@ -47,7 +47,7 @@ if (param('cmd') eq 'submit'){
 	}
 }
 
-if ($raid){
+if ($raid && defined param('cmd')){
 	if (param('cmd') eq 'remove'){
 		$DBH->do(q{UPDATE raids SET open = FALSE, removed = TRUE WHERE id = ?},undef,$raid->{id});
 	}elsif (param('cmd') eq 'Open'){
@@ -67,6 +67,11 @@ if ($raid){
 		if($DBH->do(q{UPDATE raids SET released_coords = FALSE WHERE id = ?},undef,$raid->{id})){
 			$raid->{released_coords} = 0;
 		}
+	}elsif (param('cmd') eq 'comment'){
+		$DBH->do(q{UPDATE raid_targets SET comment = ? WHERE id = ?}
+			,undef,escapeHTML(param('comment')),param('target'))
+			or $error .= p($DBH->errstr);
+
 	}elsif (param('cmd') eq 'change'){
 		$DBH->begin_work;
 		my $message = escapeHTML(param('message'));
@@ -94,7 +99,7 @@ if ($raid){
 			}
 		}
 		if (param('alliance') =~ /^(\d+)$/ && $1 != 1){
-			$LOG->execute($ND::UID,"BC adding alliance $1 to raid");
+			log_message $ND::UID,"BC adding alliance $1 to raid";
 			my $addtarget = $DBH->prepare(qq{INSERT INTO raid_targets(raid,planet) (
 	SELECT ?,id FROM current_planet_stats p WHERE alliance_id = ? $sizelimit)});
 			unless ($addtarget->execute($raid->{id},$1)){
@@ -107,6 +112,7 @@ if ($raid){
 		$groups->execute();
 		while (my $group = $groups->fetchrow_hashref){
 			my $query;
+			next unless defined param $group->{gid};
 			if (param($group->{gid}) eq 'remove'){
 				$query = $delgroup;
 			}elsif(param($group->{gid}) eq 'add'){
@@ -122,12 +128,12 @@ if ($raid){
 			$error .= "<p> Something went wrong: ".$DBH->errstr."</p>";
 		}
 	}
-	if (param('removeTarget')){
-		$error .= "test";
-		unless($DBH->do(q{DELETE FROM raid_targets WHERE raid = ? AND id = ?}
-				,undef,$raid->{id},param('removeTarget'))){
-			$error .= "<p> Something went wrong: ".$DBH->errstr."</p>";
-		}
+}
+if ($raid && param('removeTarget')){
+	$error .= "test";
+	unless($DBH->do(q{DELETE FROM raid_targets WHERE raid = ? AND id = ?}
+			,undef,$raid->{id},param('removeTarget'))){
+		$error .= "<p> Something went wrong: ".$DBH->errstr."</p>";
 	}
 }
 
@@ -167,7 +173,7 @@ if ($raid){
 	$BODY->param(Message => $raid->{message});
 	
 	my $order = "p.x,p.y,p.z";
-	if (param('order') =~ /^(score|size|value|xp|race)$/){
+	if (param('order') && param('order') =~ /^(score|size|value|xp|race)$/){
 		$order = "$1 DESC";
 	}