I showed you 2 posts before.
else if (event.equals("levelUpClan"))
{
if (player.getClan() == null)
return "ClanLevelUp-NoClan.htm";
else if (!player.isClanLeader())
return "ClanLevelUp-NoLeader.htm";
else if (player.getClan().getLevel() == 8)
return "ClanLevelUp-Limit.htm"; // CREATE THIS FILE, IF YOU DON'T HAVE IT
else if (st.getQuestItemsCount(clanLevelItemsId[2]) < clanLevelItemsCount[2])
return "ClanLevelUp-NoItems.htm";
else
{
st.takeItems(clanLevelItemsId[2], clanLevelItemsCount[2]);
player.getClan().changeLevel(8);
player.getClan().addReputationScore(100000);
for (L2Skill cs : SkillTable.getClanSkills())
player.getClan().addNewSkill(cs);
player.getClan().broadcastClanStatus();
player.sendMessage("Your clan is now level " + player.getClan().getLevel() + ".");
player.sendMessage("Your reputation score is now " + player.getClan().getReputationScore() + ".");
player.sendMessage("Your clan has learned all clan skills.");
player.broadcastPacket(new MagicSkillUse(player, player, 5103, 1, 1000, 0));
return "ClanLevelUp.htm";
}
}
Your code, if you compare, is messy, don't follow the order. Notably the for loop :D