Browse Source

* Fixed @allskill not adding points into advanced job skills

git-svn-id: https://svn.code.sf.net/p/rathena/svn/branches/stable@1165 54d463be-8e91-2dee-dedb-b68131a5f0ec
celest 20 years ago
parent
commit
ef85b4ff38
3 changed files with 7 additions and 5 deletions
  1. 2 0
      Changelog-SVN.txt
  2. 2 2
      Dev/bugs.txt
  3. 3 3
      src/map/pc.c

+ 2 - 0
Changelog-SVN.txt

@@ -1,6 +1,8 @@
 Date	Added
 
 02/23
+        * Fixed another bad typo causing @allskill to not add points into advanced job
+          skills [celest]
         * Fixed a bad typo in status.c's StatusChangeTable [celest]
         * Corrected Spiral Pierce's hits in the skill_db [celest]
         * Moved /common/*.o into a obj folder when compiling [celest]

+ 2 - 2
Dev/bugs.txt

@@ -297,8 +297,8 @@ Problem:	Compilers Bug: Some skills don't work under win32 builds
 		e.g. Whispers can't cloak... Players can't Hide, etc
 		LINUX builds work fine (or maybe 116x fukked them up...)
 		oh yeah, Celest mentoned it
-Assigned:	N/A
-Progress:	0%
+Assigned:	Celest
+Progress:	100% [My bad, fixed ^^; ]
 
 Problem:	Rogues Skill "back stab" works without PREPARING arrows.
 		It consumes arrows, but works even when them aren't prepered (equipped)

+ 3 - 3
src/map/pc.c

@@ -1001,7 +1001,7 @@ int pc_calc_skilltree(struct map_session_data *sd)
 						}
 						if (sd->status.job_level < skill_tree[s][c][i].joblv)
 							f=0;
-						if (id >= 2 && id <= 53 && pc_checkskill(sd, NV_BASIC) < 9)
+						else if (id >= 2 && id <= 53 && pc_checkskill(sd, NV_BASIC) < 9)
 							f=0;						
 					}
 					if(f && sd->status.skill[id].id==0 ){
@@ -4348,9 +4348,9 @@ int pc_allskillup(struct map_session_data *sd)
 		}
 		if(battle_config.enable_upper_class){ //conf‚Å–³?‚łȂ¯‚ê‚Î?‚Ý?‚Þ
 			for(i=355;i<411;i++)
-				sd->status.skill[i].id=i;
+				sd->status.skill[i].lv=skill_get_max(i);
 			for(i=475;i<480;i++)
-				sd->status.skill[i].id=i;
+				sd->status.skill[i].lv=skill_get_max(i);
 		}
 	}
 	else {