zero-value economy gains/losses are now skipped over ("X gained 0.0 for claiming this land" and similar is a bit odd)

CLeaned up warning given for UnknownFormatConversionException chat exception; the debug info is no longer useful for us, and a distraction for users.
This commit is contained in:
Brettflan 2012-01-31 11:07:48 -06:00
parent b41b3c8d2c
commit 7a38ee42a7
2 changed files with 10 additions and 6 deletions

View File

@ -220,7 +220,14 @@ public class Econ
String acc = ep.getAccountId();
String You = ep.describeTo(ep, true);
if (delta >= 0)
if (delta == 0)
{
// no money actually transferred?
// ep.msg("<h>%s<i> didn't have to pay anything %s.", You, forDoingThis); // might be for gains, might be for losses
return true;
}
if (delta > 0)
{
// The player should gain money
// There is no risk of failure

View File

@ -124,12 +124,9 @@ public class FactionsPlayerListener implements Listener
}
catch (UnknownFormatConversionException ex)
{
P.p.log(Level.SEVERE, "Critical error in chat message formatting! Complete format string: "+yourFormat);
P.p.log(Level.SEVERE, "First half of event.getFormat() string: "+formatStart);
P.p.log(Level.SEVERE, "Second half of event.getFormat() string: "+formatEnd);
P.p.log(Level.SEVERE, "Critical error in chat message formatting!");
P.p.log(Level.SEVERE, "NOTE: To fix this quickly, running this command should work: f config chatTagInsertIndex 0");
P.p.log(Level.SEVERE, "For a more proper fix, please read the chat configuration notes on the configuration page of the Factions user guide.");
ex.printStackTrace();
P.p.log(Level.SEVERE, "For a more proper fix, please read this regarding chat configuration: http://massivecraft.com/plugins/factions/config#Chat_configuration");
return;
}
}