Mhoska Posted October 7, 2012 Share Posted October 7, 2012 This is wrong here ->https://www.assembla.com/code/l2jesios/subversion/changesets/35 newChar.setTitle(""); if (Config.ALLOW_CHAR_TITLE) newChar.setTitle(Config.CHAR_TITLE); You must Do it like this: if (Config.ALLOW_CHAR_TITLE) newChar.setTitle(Config.CHAR_TITLE); else newChar.setTitle(""); ps:also what are you doing? you follow the frozen? adding custom shits? at least learn how the if -else if -else working because you made a mistake here :) hahaha that rlly show your skills absolute. +1 on messy dumb codding 101. Link to comment Share on other sites More sharing options...
Lucky Dice Posted October 7, 2012 Share Posted October 7, 2012 DIS Link to comment Share on other sites More sharing options...
vmlinuz Posted October 7, 2012 Share Posted October 7, 2012 Reached rev 45. Added several features. Features New character custom title Welcome screen message Entering messages Last visit message Casino Npc Antiheavy Classes Npc Buffer Instance Siege Informer Npc Random Reward Box Pvp Chat Webservered Html Npc Siege Reward Automatic Restard Petition Logger Nice Link to comment Share on other sites More sharing options...
vmlinuz Posted October 7, 2012 Share Posted October 7, 2012 hahaha that rlly show your skills absolute. +1 on messy dumb codding 101. newChar.addAdena("Init", Config.STARTING_ADENA, null, false); newChar.setXYZInvisible(template.spawnX, template.spawnY, template.spawnZ); - newChar.setTitle(""); + if(Config.ALLOW_START_TITLE) + newChar.setTitle(Config.START_TITLE); Link to comment Share on other sites More sharing options...
Mhoska Posted October 7, 2012 Share Posted October 7, 2012 newChar.addAdena("Init", Config.STARTING_ADENA, null, false); newChar.setXYZInvisible(template.spawnX, template.spawnY, template.spawnZ); - newChar.setTitle(""); + if(Config.ALLOW_START_TITLE) + newChar.setTitle(Config.START_TITLE); ?? read the whole code before you post something please. Link to comment Share on other sites More sharing options...
Zoey76 Posted October 7, 2012 Share Posted October 7, 2012 I had a good laugh about the custom title. I'd do it like this: newChar.setTitle(Config.START_TITLE); Where Config.START_TITLE is empty String as default and you could drop the useless boolean Config.ALLOW_START_TITLE Although, not sure, but if you don't set something it may be null and you would get NPEs around the code, if you keep the Config.ALLOW_START_TITLE then you should set an empty String before, not in an else clause. Have fun! Link to comment Share on other sites More sharing options...
Tryskell Posted October 7, 2012 Share Posted October 7, 2012 As said Zoey you don't need like half the time boolean configs, as you can make a check based on value (> 0 or != 0 depending what you aim) if other config implies an int, or !isEmpty for String. Link to comment Share on other sites More sharing options...
Matim Posted October 7, 2012 Share Posted October 7, 2012 So big drama about few lines of the code, and bunch of as licker`s, come one - enough. Link to comment Share on other sites More sharing options...
IKnowWtf Posted October 7, 2012 Share Posted October 7, 2012 you try correct Elfo omg noob??? Elfocrash has been creating codes since you didn't exist pffff idiots Can't stand quiet while seeing suck 'hyperlick' tho, especially by such pitiful person. Link to comment Share on other sites More sharing options...
paulos13 Posted October 8, 2012 Share Posted October 8, 2012 can u give us info with npcs ids commands and custom item id? Link to comment Share on other sites More sharing options...
vampir Posted October 8, 2012 Share Posted October 8, 2012 I had a good laugh about the custom title. I'd do it like this: newChar.setTitle(Config.START_TITLE); Where Config.START_TITLE is empty String as default and you could drop the useless boolean Config.ALLOW_START_TITLE Although, not sure, but if you don't set something it may be null and you would get NPEs around the code, if you keep the Config.ALLOW_START_TITLE then you should set an empty String before, not in an else clause. Have fun! yeah, thats best solution. Anyway this is really serious problem, we need more ideas :D:D Link to comment Share on other sites More sharing options...
Destorion Posted October 8, 2012 Share Posted October 8, 2012 nice idea from you elfo but there are alot interlude projects like aCis L2J and so on.... maybe you could try to use Hi5. That's my opinion. Anyways good luck and wish you da best Link to comment Share on other sites More sharing options...
An4rchy Posted October 8, 2012 Share Posted October 8, 2012 there are alot interlude projects like aCis L2J Could you name one please? Link to comment Share on other sites More sharing options...
Guest Elfocrash Posted October 8, 2012 Share Posted October 8, 2012 Could you name one please? yeah this is what i was about to ask too Link to comment Share on other sites More sharing options...
GOld3x Posted October 8, 2012 Share Posted October 8, 2012 You must Update your topic ! Link to comment Share on other sites More sharing options...
Recommended Posts