From e4375a1ffab03bf138e6f43e417d342ed9e70cf1 Mon Sep 17 00:00:00 2001
From: Inkfish <Inkfish@54d463be-8e91-2dee-dedb-b68131a5f0ec>
Date: Thu, 7 May 2009 09:19:58 +0000
Subject: * Fixed players can pull water from ME cell (follow up to r13730)

git-svn-id: https://rathena.svn.sourceforge.net/svnroot/rathena/trunk@13733 54d463be-8e91-2dee-dedb-b68131a5f0ec
---
 src/map/skill.c | 36 ++++++++++++------------------------
 1 file changed, 12 insertions(+), 24 deletions(-)

(limited to 'src/map/skill.c')

diff --git a/src/map/skill.c b/src/map/skill.c
index 098f6c327..a5600d74b 100644
--- a/src/map/skill.c
+++ b/src/map/skill.c
@@ -2826,41 +2826,29 @@ int skill_castend_damage_id (struct block_list* src, struct block_list *bl, int
 	case WZ_WATERBALL:
 		skill_attack(BF_MAGIC,src,src,bl,skillid,skilllv,tick,flag);
 		{
-			int range = skilllv/2;
-			int size = 2*range + 1;
+			int range = skilllv / 2;
+			int maxlv = skill_get_max(skillid); // learnable level
 			int count = 0;
 			int x, y;
+			struct skill_unit* unit;
 
-			if( src->type == BL_PC )
-			{// count the number of water cells in range
-				struct skill_unit* unit;
-				int maxlv = skill_get_max(skillid);
+			if( src->type == BL_PC && skilllv > maxlv )
+				range = maxlv / 2;
 
-				if( skilllv > maxlv )
-					range =  maxlv / 2;
-
-				for( y = src->y - range; y <= src->y + range; ++y )
-					for( x = src->x - range; x <= src->x + range; ++x )
+			for( y = src->y - range; y <= src->y + range; ++y )
+				for( x = src->x - range; x <= src->x + range; ++x )
+				{
+					if( !map_find_skill_unit_oncell(src,x,y,SA_LANDPROTECTOR,NULL) )
 					{
-						if( map_getcell(src->m,x,y,CELL_CHKWATER) )
+						if( src->type != BL_PC || map_getcell(src->m,x,y,CELL_CHKWATER) ) // non-players bypass the water requirement
 							count++; // natural water cell
-						else
-						if( (unit = map_find_skill_unit_oncell(src,x,y,SA_DELUGE,NULL)) != NULL
-						||  (unit = map_find_skill_unit_oncell(src,x,y,NJ_SUITON,NULL)) != NULL )
+						else if( (unit = map_find_skill_unit_oncell(src,x,y,SA_DELUGE,NULL)) != NULL || (unit = map_find_skill_unit_oncell(src,x,y,NJ_SUITON,NULL)) != NULL )
 						{
 							count++; // skill-induced water cell
 							skill_delunit(unit); // consume cell
 						}
 					}
-			}
-			else
-			{ // non-players bypass the water requirement
-				count = size*size;
-				for( y = src->y - range; y <= src->y + range; ++y )
-					for( x = src->x - range; x <= src->x + range; ++x )
-						if( map_find_skill_unit_oncell(src,x,y,SA_LANDPROTECTOR,NULL) != NULL )
-							count--;
-			}
+				}
 
 			if( count > 1 ) // queue the remaining count - 1 timerskill Waterballs
 				skill_addtimerskill(src,tick+150,bl->id,0,0,skillid,skilllv,count-1,flag);
-- 
cgit v1.2.3-70-g09d2