Jump to content
  • 0

All Subs In One Npc


mikemaster

Question

Hey i followed a guide here that showed how to make all subs available in one npc i made the following changes but there is a problem :D
When you press Add a subclass it gives the full sub class list but you can add classes that are available retail like in the Npc
For Example Master Jurek where you can add mage classes does not allow you to become a gladiator  and pops up the subclass error ( Mimir's Elixir etc)

Also when i want to change into another subclass the Npc only desplays classes that are available only at that kind of Npc 
I wanna make it like that: When you want to add a class you can add everything from one single NpC except ( dominator and the dwarf class )

and also at the same Npc to be able to change in whatever subclass you like


Changes i made  in L2VillageMasterInstance.java:

/**
		 * If the race of your main class is Elf or Dark Elf, you may not select each class as a subclass to the other class, and you may not select Overlord and Warsmith class as a subclass. You may not select a similar class as the subclass. The occupations classified as similar classes are as
		 * follows: Treasure Hunter, Plainswalker and Abyss Walker Hawkeye, Silver Ranger and Phantom Ranger Paladin, Dark Avenger, Temple Knight and Shillien Knight Warlocks, Elemental Summoner and Phantom Summoner Elder and Shillien Elder Swordsinger and Bladedancer Sorcerer, Spellsinger and
		 * Spellhowler
		 */
		Set<PlayerClass> availSubs = PlayerClass.values()[baseClassId].getAvailableSubclasses(player);
		
		 if (availSubs != null)
	        {
	            for (PlayerClass availSub : availSubs)
	            {
	                for (Iterator<SubClass> subList = iterSubClasses(player); subList.hasNext();)
	                {
	                    SubClass prevSubClass = subList.next();
	                    int subClassId = prevSubClass.getClassId();
	                    if (subClassId >= 88) subClassId = ClassId.values()[subClassId].getParent().getId();

	                    if (availSub.ordinal() == subClassId
	                        || availSub.ordinal() == player.getBaseClass())
	                        availSubs.remove(PlayerClass.values()[availSub.ordinal()]);
	                }
	            }
	        }

	        return availSubs;
	}		
		
	/*
	 * Check new subclass classId for validity (villagemaster race/type is not contains in previous subclasses, but in allowed subclasses) Base class not added into allowed subclasses.
	 */
	private final boolean isValidNewSubClass(L2PcInstance player, int classId)
	{
		if (!checkVillageMaster(classId))
			return false;
		
		final ClassId cid = ClassId.values()[classId];
		for (Iterator<SubClass> subList = iterSubClasses(player); subList.hasNext();)
		{
			SubClass sub = subList.next();
			ClassId subClassId = ClassId.values()[sub.getClassId()];
			
			if (subClassId.equalsOrChildOf(cid))
				return false;
		}
Link to comment
Share on other sites

6 answers to this question

Recommended Posts

  • 0

All you have to change is this.. Like this, I have created a config for myself.

 

 

    /*
     * Returns true if this PlayerClass is allowed for master
     */
    public final boolean checkVillageMaster(PlayerClass pclass)
    {
        if (Config.ALT_GAME_SUBCLASS_EVERYWHERE)
            return true;
        
        return checkVillageMasterRace(pclass) && checkVillageMasterTeachType(pclass);
    }
Link to comment
Share on other sites

  • 0

 

All you have to change is this.. Like this, I have created a config for myself.

    /*
     * Returns true if this PlayerClass is allowed for master
     */
    public final boolean checkVillageMaster(PlayerClass pclass)
    {
        if (Config.ALT_GAME_SUBCLASS_EVERYWHERE)
            return true;
        
        return checkVillageMasterRace(pclass) && checkVillageMasterTeachType(pclass);
    }

so do i have to revert the changes that i already made and simply create a config that suits with this code that you gave me? :D

Link to comment
Share on other sites

Guest
This topic is now closed to further replies.


×
×
  • Create New...