adenaman

omg, // Stop all active skills effects in progress on the L2Character, // if the Character isn't affected by Soul of The Phoenix or Salvation if (this instanceof L2Playable && ((L2Playable)this).isPhoenixBlessed()) { if (((L2Playable)this).getCharmOfLuck()) //remove Lucky Charm if player has SoulOfThePhoenix/Salvation buff ((L2Playable)this).stopCharmOfLuck(null); if (((L2Playable)this).isNoblesseBlessed()) ((L2Playable)this).stopNoblesseBlessing(null); } // Same thing if the Character isn't a Noblesse Blessed L2PlayableInstance else if (this instanceof L2Playable && ((L2Playable)this).isNoblesseBlessed()) { ((L2Playable)this).stopNoblesseBlessing(null); if (((L2Playable)this).getCharmOfLuck()) //remove Lucky Charm if player have Nobless blessing buff ((L2Playable)this).stopCharmOfLuck(null); } else //stopAllEffectsExceptThoseThatLastThroughDeath(); if (this instanceof L2PcInstance && ((L2PcInstance)this).getAgathionId() != 0) ((L2PcInstance)this).setAgathionId(0); calculateRewards(killer); // Send the Server->Client packet StatusUpdate with current HP and MP to all other L2PcInstance to inform broadcastStatusUpdate(); if you delete stopAllEffectsExceptThoseThatLastThroughDeath();, you will have else if (this instanceof L2PcInstance && ((L2PcInstance)this).getAgathionId() != 0) ((L2PcInstance)this).setAgathionId(0); calculateRewards(killer); :O can any1 see where is the problem?

the best working protection is client side, the only server side things that are not useless (atm) is bfdr protection, that is private. Anyway, you need to check client mod section, maybe you can find smth

interesting, but quote my code where i did else if else or else if or smth

yes, yes, plz quote my error in this topic

could you tell me when i tryed to write smth like that? because i dont have any problem when i compile it (didnt tested it ingame)

post here the whole doDie function, or at least where are those errors

// Stop all active skills effects in progress on the L2Character, // if the Character isn't affected by Soul of The Phoenix or Salvation if (this instanceof L2Playable && ((L2Playable)this).isPhoenixBlessed()) { if (((L2Playable)this).getCharmOfLuck()) //remove Lucky Charm if player has SoulOfThePhoenix/Salvation buff ((L2Playable)this).stopCharmOfLuck(null); if (((L2Playable)this).isNoblesseBlessed()) ((L2Playable)this).stopNoblesseBlessing(null); } // Same thing if the Character isn't a Noblesse Blessed L2PlayableInstance else if (this instanceof L2Playable && ((L2Playable)this).isNoblesseBlessed()) { ((L2Playable)this).stopNoblesseBlessing(null); if (((L2Playable)this).getCharmOfLuck()) //remove Lucky Charm if player have Nobless blessing buff ((L2Playable)this).stopCharmOfLuck(null); } else stopAllEffectsExceptThoseThatLastThroughDeath(); this have 3 steps -check if the char has salvation -else check if the char has noblesse -else delete all buffs so you can chose, to a) remove salvation and noblesse (if the char has this buffs), but never remove normall buff then delete else stopAllEffectsExceptThoseThatLastThroughDeath(); b) never remove salvation, noblesse or any other buff then delete all the code quoted in the post (from // Stop all active skills effects in progress on the L2Character, // if the Character isn't affected by Soul of The Phoenix or Salvation if (this instanceof L2Playable && ((L2Playable)this).isPhoenixBlessed()) to else stopAllEffectsExceptThoseThatLastThroughDeath();

ok, when some1 copy the code here, it lose some spaces // Same thing if the Character isn't a Noblesse Blessed L2PlayableInstance else if (this instanceof L2Playable && ((L2Playable)this).isNoblesseBlessed()) { ((L2Playable)this).stopNoblesseBlessing(null); if (((L2Playable)this).getCharmOfLuck()) //remove Lucky Charm if player have Nobless blessing buff ((L2Playable)this).stopCharmOfLuck(null); } else stopAllEffectsExceptThoseThatLastThroughDeath(); all that is ok, you should take a look here (the if before your code) // Stop all active skills effects in progress on the L2Character, // if the Character isn't affected by Soul of The Phoenix or Salvation if (this instanceof L2Playable && ((L2Playable)this).isPhoenixBlessed()) { if (((L2Playable)this).getCharmOfLuck()) //remove Lucky Charm if player has SoulOfThePhoenix/Salvation buff ((L2Playable)this).stopCharmOfLuck(null); if (((L2Playable)this).isNoblesseBlessed()) ((L2Playable)this).stopNoblesseBlessing(null); } without this if, you cant have a else If you want to not lose buffs when die, just delete else stopAllEffectsExceptThoseThatLastThroughDeath(); but take a look at all things that could happen with this. EDIT if you remove that, then you will allways try to ((L2Playable)this).stopCharmOfLuck(null); even if you dont have it

i mean to just put the { anyway, i will take a look at the complete L2character, sec

nvm, i found what is it if (((L2Playable)this).getCharmOfLuck()) //remove Lucky Charm if player have Nobless blessing buff ((L2Playable)this).stopCharmOfLuck(null); } change to if ( ( (L2Playable)this).getCharmOfLuck()) { //remove Lucky Charm if player have Nobless blessing buff

show us more code before that else

http://www.l2jserver.com/forum/viewtopic.php?f=81&p=76689 i dont really know why that could happen

i want to say that you cant have a line final StringBuilder html1 = StringUtil.startAppend(2000, because its not finished. Anyway, im wrong, your problem is that eclipse do not recognizes the import, if its good written try to copy the same line from another file that use the same import, it can sound stupid, but worked for me more that once :)

the client?

http://www.maxcheaters.com/forum/index.php?topic=165826.msg1254681#new take a look at the imports,if i am not wrong, he is trying to sell the free l2jinfinity shared pack, or old svn files, or maybe reselling new svn files (now l2jinfinity is private project)

you can find the sources here http://www.l2jserver.com/svn/trunk/ to change things and compile read this http://www.maxcheaters.com/forum/index.php?topic=92146.0

i dont remember if tvt gives pvp points, so the problem could be the color update when you kill any person

public void setPvpFlag(int pvpFlag) { _pvpFlag = (byte)pvpFlag; } public byte getPvpFlag() { return _pvpFlag; } // // Your code here // public void revalidateZone(boolean force) { // Cannot validate if not in a world region (happens during teleport) if (getWorldRegion() == null) return; // This function is called very often from movement code if (force) _zoneValidateCounter = 4; else { _zoneValidateCounter--; if (_zoneValidateCounter < 0) _zoneValidateCounter = 4; else return; } try to fix the changed l2pcinstance writing public void revalidateZone(boolean force) { before if (getWorldRegion() == null) return; so it will be public void revalidateZone(boolean force) { // Cannot validate if not in a world region (happens during teleport) if (getWorldRegion() == null) return; and np for helping you :)

public void revalidateZone(boolean force) { // Custom PVP Color System - Start public void updatePvPColor(int pvpKillAmount) { that is what i was talking about Find a clean copy of l2pc instance, coz im sure that you deleted some code of revalidateZone, that is another function that you should not touch. anyway, try to delete public void revalidateZone(boolean force) { of your code, and compile it.

[javac] C:\workspace\L2_GameServer_It\java\net\sf\l2j\gameserver\model\actor\instance\L2PcInstance.java:3599: illegal start of expression [javac] public void updatePvPColor(int pvpKillAmount) you are starting to add things in a wrong place that should go public final class L2PcInstance extends L2Playable { ... public byte getPvpFlag() //for example { return _pvpFlag; } ... public smth doSmth() //any function { ... } //your code here ... }

try to use search next time http://www.maxcheaters.com/forum/index.php?topic=132508.0

that should be if (activeChar.getLevel() < 1 || !activeChar.isNewbie()) that getLevel() could be to check if there is an error, its usuall to do things like if (error) activeChar.setLevel() = -1; i could not find any, but maybe in the future some1 will need it Of course the msg used is wrong another thing that i wont to say is the use of the returns and else if, this is not too much important, but will make it a bit better. You can do public boolean useVoicedCommand(String command, L2PcInstance activeChar, String target) { if (command.equalsIgnoreCase("newbiegift")) { if (activeChar.getLevel() >= 1 && activeChar.isNewbie()) { } [b]else[/b] if (activeChar.getInventory().getItemByItemId(3434).getCount() >= 1) { } else { } [b]return true;[/b] } return false; } or public boolean useVoicedCommand(String command, L2PcInstance activeChar, String target) { if (command.equalsIgnoreCase("newbiegift")) { if (activeChar.getLevel() >= 1 && activeChar.isNewbie()) { return true }// if the first if was true, it will execute return, so there is no need to use else if (activeChar.getInventory().getItemByItemId(3434).getCount() >= 1) { return true; } { //just to see that here goes the last code segment return true; } } return false; } i repeat that this is not very important, but, its better to use it this way

« on: March 03, 2009, 06:01:30 PM » i dont think that he still need help

lol, w8, i didnt see that its for interlude, my idea was for epilogue, give me a sec, to find where in interlude is that source EDIT: it should be then clientpackets/RequestEnchantItem.java L2ItemInstance item = activeChar.getInventory().getItemByObjectId( _objectId); L2ItemInstance scroll = activeChar.getActiveEnchantItem(); if (item == null || scroll == null) { activeChar.setActiveEnchantItem(null); return; } // can't enchant rods, hero weapons and shadow items, donator rented // items if (item.getItem().getItemType() == L2WeaponType.ROD || !Config.ENCHANT_HERO_WEAPONS && item.getItemId() >= 6611 && item.getItemId() <= 6621 || item.getItemId() >= 7816 && item.getItemId() <= 7831 || item.isShadowItem() || item.isDonatorRented()) { activeChar.sendPacket(new SystemMessage( SystemMessageId.INAPPROPRIATE_ENCHANT_CONDITION)); activeChar.setActiveEnchantItem(null); return; } the same, some more || or another if

they dont have any limit, b4 it has been fixed (now i think that you can not enchant with scrolls) iv seen a heavy main part +22 anyway clientpackets/AbstractEnchantPacket.java public final boolean isValid(L2ItemInstance enchantItem) { ... switch (type2) { // weapon scrolls can enchant only weapons case L2Item.TYPE2_WEAPON: if (!_isWeapon || (Config.ENCHANT_MAX_WEAPON > 0 && enchantItem.getEnchantLevel() >= Config.ENCHANT_MAX_WEAPON)) return false; break; // armor scrolls can enchant only accessory and armors case L2Item.TYPE2_SHIELD_ARMOR: if (_isWeapon || (Config.ENCHANT_MAX_ARMOR > 0 && enchantItem.getEnchantLevel() >= Config.ENCHANT_MAX_ARMOR)) return false; break; ... there you just have to make some if and return false if enchant >= max