order by last post

[ndwebbie.git] / editRaid.pl

diff --git a/editRaid.pl b/editRaid.pl
index fd109320880459d6edab93e7740f5b1c3be9bae0..94ed68da85685c964d3ef05d55b15759c0bd5e07 100644 (file)
--- 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";
        }