From 110397106cb100f3e48675e70559b07eb612064f Mon Sep 17 00:00:00 2001
From: JT Smith <jt@lacunaexpanse.com>
Date: Tue, 28 Dec 2010 10:13:09 -0600
Subject: [PATCH] Fixed a crash bug with fleet movement missions.

---
 lib/Lacuna/DB/Result/Mission.pm | 3 +++
 var/www/public/changes.txt      | 1 +
 2 files changed, 4 insertions(+)

diff --git a/lib/Lacuna/DB/Result/Mission.pm b/lib/Lacuna/DB/Result/Mission.pm
index a7de32a8..14a117c8 100644
--- a/lib/Lacuna/DB/Result/Mission.pm
+++ b/lib/Lacuna/DB/Result/Mission.pm
@@ -276,6 +276,7 @@ sub check_objectives {
                 else {
                     $target = $stars->find($movement->{target_star_id});
                 }
+                next unless defined $target;
                 confess [1013, 'Have not sent '.$ship->type_formatted.' to '.$target->name.' ('.$target->x.','.$target->y.').'];
             }
         }
@@ -356,6 +357,8 @@ sub format_items {
             else {
                 $target = $stars->find($movement->{target_star_id});
             }
+            warn "fleet movement target not found";
+            next unless defined $target;
             push @items, 'Send '.$ship->type_formatted.' to '.$target->name.' ('.$target->x.','.$target->y.').';
         }
     }
diff --git a/var/www/public/changes.txt b/var/www/public/changes.txt
index eb981690..a2acd20a 100644
--- a/var/www/public/changes.txt
+++ b/var/www/public/changes.txt
@@ -1,6 +1,7 @@
 2.4200
  - API now tells you exactly what assignments a spy can be given at the present time and how long the recovery time will be.
  - Added "Security Sweep" defensive assignment.
+ - Fixed a crash bug with fleet movement missions.
 
 2.4102
  - Fixed bleeders and thuds being added to the normal build queue on striking ground.
-- 
2.51.2