There are two pretty minor bugs that I'd like to point out with solution changes in the GitHub code. They are both purely experimental, but in theory and because of the results of the tests I ran, they will probably work.
The first bug is the one where the disband message is sent across each rack server, and the number of times that message is sent is based on the number of rack servers. So right now, since there are 4 rack servers, on each rack server the message would be sent 4 times. When more rack servers are added in and a faction disbands, chat could be flooded pretty easily, and here is what I would add to the code in the /src/com/massivecraft/factions/event/FactionsEventDisband.java. From this:
To this:
The changes made were:
1. A Private Final String universe was set up in the Fields section
2. In the Construct, in the FactionsEventDisband, this.universe was added in
3. Also, in the FactionsEventDisband, the String universe must be passed by reference to the function
And for the second bug, it really isn't a bug, just something I noticed that was changed. The descriptions, when changed, did not send a global message. If you add in the exact same changes that were used in the disband, I believe the description should be "fixed" as well. I will still keep looking through it for other small errors, and look for other ways to solve this if the problem isn't.
The first bug is the one where the disband message is sent across each rack server, and the number of times that message is sent is based on the number of rack servers. So right now, since there are 4 rack servers, on each rack server the message would be sent 4 times. When more rack servers are added in and a faction disbands, chat could be flooded pretty easily, and here is what I would add to the code in the /src/com/massivecraft/factions/event/FactionsEventDisband.java. From this:
Code:
package com.massivecraft.factions.event;
import org.bukkit.command.CommandSender;
import org.bukkit.event.HandlerList;
import com.massivecraft.factions.entity.Faction;
public class FactionsEventDisband extends FactionsEventAbstractSender
{
// -------------------------------------------- //
// REQUIRED EVENT CODE
// -------------------------------------------- //
private static final HandlerList handlers = new HandlerList();
@Override public HandlerList getHandlers() { return handlers; }
public static HandlerList getHandlerList() { return handlers; }
// -------------------------------------------- //
// FIELDS
// -------------------------------------------- //
private final Faction faction;
public Faction getFaction() { return this.faction; }
private final String factionId;
public String getFactionId() { return this.factionId; }
// -------------------------------------------- //
// CONSTRUCT
// -------------------------------------------- //
public FactionsEventDisband(CommandSender sender, Faction faction)
{
super(sender);
this.faction = faction;
this.factionId = faction.getId();
}
}
Code:
package com.massivecraft.factions.event;
import org.bukkit.command.CommandSender;
import org.bukkit.event.HandlerList;
import com.massivecraft.factions.entity.Faction;
public class FactionsEventDisband extends FactionsEventAbstractSender
{
// -------------------------------------------- //
// REQUIRED EVENT CODE
// -------------------------------------------- //
private static final HandlerList handlers = new HandlerList();
@Override public HandlerList getHandlers() { return handlers; }
public static HandlerList getHandlerList() { return handlers; }
// -------------------------------------------- //
// FIELDS
// -------------------------------------------- //
private final String universe;
public final String getUniverse() { return this.universe; }
private final Faction faction;
public Faction getFaction() { return this.faction; }
private final String factionId;
public String getFactionId() { return this.factionId; }
// -------------------------------------------- //
// CONSTRUCT
// -------------------------------------------- //
public FactionsEventDisband(CommandSender sender, Faction faction)
{
super(sender);
this.universe = universe;
this.faction = faction;
this.factionId = faction.getId();
}
}The changes made were:
1. A Private Final String universe was set up in the Fields section
2. In the Construct, in the FactionsEventDisband, this.universe was added in
3. Also, in the FactionsEventDisband, the String universe must be passed by reference to the function
And for the second bug, it really isn't a bug, just something I noticed that was changed. The descriptions, when changed, did not send a global message. If you add in the exact same changes that were used in the disband, I believe the description should be "fixed" as well. I will still keep looking through it for other small errors, and look for other ways to solve this if the problem isn't.
