From aa0112659b72d40b9ce0c3ea4e8f4c2979aa1b4e Mon Sep 17 00:00:00 2001
From: David Morris <gm3dmo@github.com>
Date: Sun, 17 Dec 2023 15:37:29 +0000
Subject: [PATCH 1/3] Adding decorations picker which seems to work.

---
 templates/cmp/soldier.html | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/templates/cmp/soldier.html b/templates/cmp/soldier.html
index ef11a1f..429d0e3 100644
--- a/templates/cmp/soldier.html
+++ b/templates/cmp/soldier.html
@@ -15,9 +15,16 @@ <h1>Soldier Record</h1>
       <tr><td>Army Number:</td><td>{{ soldier.army_number }}</td></tr>
       <tr><td>Notes:</td><td>{{ soldier.notes }}</td></tr>
 
-      <tr><td colspan="2"><h4>Decorations Record:</h4></td></tr>
+      {% if soldier.soldierdecoration_set.all %}
+          <tr><td colspan="2"><h4>Decorations Record:</h4></td></tr>
+          {% for decoration in soldier.soldierdecoration_set.all %}
+             <tr><td>decoration: </td><td>{{ decoration }}</td></tr>
+          {% endfor %}
+      {% endif %}
+
       <tr><td colspan="2"><h4>Prisoner of War Record:</h4></td></tr>
 
+
       <tr><td colspan="2"><h4>Casualty Record:</h4></td></tr>
       <tr><td>Date of Death:</td><td>{{ soldier.soldierdeath.date }}</td></tr>
       <tr><td>Company:</td><td>{{ soldier.soldierdeath.company }}</td></tr>

From 433385bf602c68024728236da22308bcaee5d96f Mon Sep 17 00:00:00 2001
From: David Morris <gm3dmo@github.com>
Date: Sun, 17 Dec 2023 15:42:05 +0000
Subject: [PATCH 2/3] Adding soldierdeath picker which also seems to work

---
 templates/cmp/soldier.html | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

diff --git a/templates/cmp/soldier.html b/templates/cmp/soldier.html
index 429d0e3..ca33d87 100644
--- a/templates/cmp/soldier.html
+++ b/templates/cmp/soldier.html
@@ -25,12 +25,16 @@ <h1>Soldier Record</h1>
       <tr><td colspan="2"><h4>Prisoner of War Record:</h4></td></tr>
 
 
-      <tr><td colspan="2"><h4>Casualty Record:</h4></td></tr>
-      <tr><td>Date of Death:</td><td>{{ soldier.soldierdeath.date }}</td></tr>
-      <tr><td>Company:</td><td>{{ soldier.soldierdeath.company }}</td></tr>
-      <tr><td>Cemetery:</td><td>{{ soldier.soldierdeath.cemetery }}</td></tr>
-      <tr><td colspan="2">Map</td></tr>
-      <tr><td colspan="2">A map goes here</tr>
+      {% if soldier.soldierdeath %}
+         <tr><td colspan="2"><h4>Casualty Record:</h4></td></tr>
+         <tr><td>Date of Death:</td><td>{{ soldier.soldierdeath.date }}</td></tr>
+         <tr><td>Company:</td><td>{{ soldier.soldierdeath.company }}</td></tr>
+         <tr><td>Cemetery:</td><td>{{ soldier.soldierdeath.cemetery }}</td></tr>
+         <tr><td colspan="2">Map</td></tr>
+         <tr><td colspan="2">A map goes here</tr>
+      {% endif %}
+
+
   
 
   </tbody>

From d2ad127a18b06c8ce02576e8316bef470910739e Mon Sep 17 00:00:00 2001
From: David Morris <gm3dmo@github.com>
Date: Sun, 17 Dec 2023 15:56:20 +0000
Subject: [PATCH 3/3] Adding pow country picker which finally seems to be
 working.

---
 cmp/models.py              | 6 +++++-
 templates/cmp/soldier.html | 8 +++++++-
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/cmp/models.py b/cmp/models.py
index c36060e..73afa9c 100644
--- a/cmp/models.py
+++ b/cmp/models.py
@@ -107,7 +107,8 @@ class SoldierDeath(models.Model):
     company  = models.ForeignKey(Company, blank=True, null=True, default="UNKNOWN", on_delete=models.CASCADE, related_name='companies')
     cemetery = models.ForeignKey(Cemetery, blank=True, null=True, default=110, on_delete=models.CASCADE, related_name='cemeteries')
     cwgc_id  = models.IntegerField(blank=True, null=True, unique=False, verbose_name="War Graves ID")
-    def __unicode__(self):
+
+    def __str__(self):
         return '%s %s %s' % (self.Soldier, self.date, self.cemetery)
 
     def cwgc_url(self):
@@ -152,6 +153,8 @@ class SoldierImprisonment(models.Model):
     legacy_date_from = models.CharField(max_length=255, unique=False, default='')
     legacy_date_to = models.CharField(max_length=255, unique=False, default='')
     notes = models.CharField(max_length=255, unique=False, default='')
+    def __str__(self):
+        return self.pow_camp.name
 
 
 class SoldierDecoration(models.Model):
@@ -165,6 +168,7 @@ class SoldierDecoration(models.Model):
     country  = models.ForeignKey(Country, blank=True, null=True, on_delete=models.CASCADE)
     citation = models.TextField(max_length=50000, blank=True)
     notes = models.TextField(max_length=50000, blank=True)
+
     def __str__(self):
         return self.decoration.name
 
diff --git a/templates/cmp/soldier.html b/templates/cmp/soldier.html
index ca33d87..9ea18b3 100644
--- a/templates/cmp/soldier.html
+++ b/templates/cmp/soldier.html
@@ -22,8 +22,14 @@ <h1>Soldier Record</h1>
           {% endfor %}
       {% endif %}
 
-      <tr><td colspan="2"><h4>Prisoner of War Record:</h4></td></tr>
+      {% if soldier.soldierimprisonment_set.exists %}
+       <tr><td colspan="2"><h4>Prisoner of War Record:</h4></td></tr>
+       {% for imprisonment in soldier.soldierimprisonment_set.all %}
+          <tr><td>Imprisonment: </td><td>{{ imprisonment }} {{imprisonment.pow_camp.country}} {{imprisonment.pow_camp.country.flag}}</td></tr>
+       {% endfor %}
 
+      {% endif %}
+      
 
       {% if soldier.soldierdeath %}
          <tr><td colspan="2"><h4>Casualty Record:</h4></td></tr>