The bug is now closed.

Sean


From: Chris McIntosh <cmcintosh@gmail.com>
To: Sean Bridges <sbridges.geo@yahoo.com>
Cc: triplea-developers@lists.sourceforge.net
Sent: Sunday, February 1, 2009 10:32:47 AM
Subject: Re: [Triplea-develop] Please apply patch.

Its only one line I'm not sure why that makes it more crowded/disorganized. But I understand.

For what its worth I found this feature request open for the same thing: https://sourceforge.net/tracker2/index.php?func=detail&aid=1560049&group_id=44492&atid=439740

Should that be closed out if its decided not to do this?

Thanks,
Chris McIntosh

On Sun, Feb 1, 2009 at 9:03 AM, Sean Bridges <sbridges.geo@yahoo.com> wrote:
I'm not sure I like adding another label to the territory panel.  The panel is already pretty crowded and disorganized. 

Sean


From: Chris McIntosh <cmcintosh@gmail.com>
To: triplea-developers@lists.sourceforge.net
Sent: Saturday, January 31, 2009 8:08:17 PM
Subject: Re: [Triplea-develop] Please apply patch.

Oops forgot to include import in patch:

Index: TerritoryDetailPanel.java
===================================================================
--- TerritoryDetailPanel.java    (revision 2293)
+++ TerritoryDetailPanel.java    (working copy)
@@ -15,6 +15,7 @@
 package games.strategy.triplea.ui;
 
 import games.strategy.engine.data.*;
+import games.strategy.triplea.attatchments.TerritoryAttachment;
 import games.strategy.triplea.oddsCalculator.ta.OddsCalculatorDialog;
 import games.strategy.triplea.util.*;
 import games.strategy.ui.OverlayIcon;
@@ -104,6 +105,13 @@
 
         add(new JLabel(territory.getName()));
        
+        // Add Production count for territory to panel.       
+        TerritoryAttachment ta = TerritoryAttachment.get(territory);
+        if (ta != null)
+        {           
+            add(new JLabel("Production: " + ta.getProduction()));
+        }
+
         add(m_showOdds);
        
         Collection<Unit> unitsInTerritory;


On Sat, Jan 31, 2009 at 7:43 PM, Chris McIntosh <cmcintosh@gmail.com> wrote:
I was playing and found it odd that the Territory Detail Panel didn't have the production count in it. I know its in the status bar at the bottom. Please apply this patch to add it.

Thanks,
Chris McIntosh (compavalanche at sourceforge.net)

Index: TerritoryDetailPanel.java
===================================================================
--- TerritoryDetailPanel.java    (revision 2293)
+++ TerritoryDetailPanel.java    (working copy)
@@ -104,6 +104,13 @@
 
         add(new JLabel(territory.getName()));
        
+        // Add Production count for territory to panel.       
+        TerritoryAttachment ta = TerritoryAttachment.get(territory);
+        if (ta != null)
+        {           
+            add(new JLabel("Production: " + ta.getProduction()));
+        }
+
         add(m_showOdds);
        
         Collection<Unit> unitsInTerritory;