kyeme - Aug 29, 2013 5:53
Whenever I use hiding skill (from smokie card) on my Knight, or Paladin w/ a Pecopeco, my skill tree will be lost. It doesn't happen if I dont have a pecopeco.
Please confirm thank you :)
[url="http://www.youtube.com/watch?v=Q_T3ivEIBLE&feature=youtu.be"]http://www.youtube.com/watch?v=Q_T3ivEIBLE&feature=youtu.be[/url]
Frost - Aug 29, 2013 7:23
Even in official servers also happens this?
hemagx - Aug 29, 2013 14:58
@Frost
No :)
@Kyeme
i have no problem with it -.-
Ind - Sep 9, 2013 18:44
i can't reproduce it either, whats your client version?
kyeme - Sep 9, 2013 18:46
Im using 2012-04-17a ragexe client, pre-renewal
Ind - Sep 9, 2013 18:50
hmm o-o I'll check with other devs, I can't see how this can be done -- the hiding status doesn't force the server to re-send nor re-calculate the skill tree
bgamez23 - Nov 30, 2013 4:25
up for this.
kyeme - Jul 13, 2014 9:12
Up~
kyeme - Sep 21, 2014 2:24
Up~ :D
This post has been edited by
kyeme
on Sep 21, 2014 2:25
Ind - Sep 21, 2014 2:33
Question is this reproduceable nowadays? I think I might have fixed it on a commit earlier (I tried to reproduce it just now and I couldn't thus why I'm asking if you still can reproduce)
kyeme - Sep 21, 2014 2:41
Yes
[img]http://i.imgur.com/9QW8Q7d.jpg[/img]
Frost - Sep 21, 2014 2:47
[quote name="kyeme" timestamp="1411267317"]
Yes
[img]http://i.imgur.com/9QW8Q7d.jpg[/img]
[/quote]
Hmm.. i can't reproduce it now.. -_- maybe a client problem?
This post has been edited by
Frost
on Sep 21, 2014 2:48
Frost - Sep 21, 2014 2:50
I tried on 3 clients, 2010 client can't reproduce this problem, 2011 can, 2012 can, dont know if 2013 also have this problem.
Ind - Sep 21, 2014 3:09
Kyeme I have a question, if possible, could you test this?
in src/map/status.c you'll find this:[code=auto:0]
case SC_HIDING:
sc->option &= ~OPTION_HIDE;
opt_flag|= 2|4; //Check for warp trigger + AoE trigger
break;
[/code]Change to[code=auto:0]
case SC_HIDING:
sc->option &= ~OPTION_HIDE;
opt_flag|= 2;
break;
[/code]and let me know if it still happens (that is, if you can test). Thanks!
kyeme - Sep 21, 2014 3:14
[quote name="Ind" timestamp="1411268988"]
Kyeme I have a question, if possible, could you test this?
in src/map/status.c you'll find this:[code=auto:0]
case SC_HIDING:
sc->option &= ~OPTION_HIDE;
opt_flag|= 2|4; //Check for warp trigger + AoE trigger
break;
[/code]Change to[code=auto:0]
case SC_HIDING:
sc->option &= ~OPTION_HIDE;
opt_flag|= 2;
break;
[/code]and let me know if it still happens (that is, if you can test). Thanks![/quote]
^ Fixed! Thanks sir :D
Ind - Sep 21, 2014 3:17
Okay I see what the problem is, can't fix straight away because the variable is being shared for another functionality.[code=auto:0]
if(opt_flag&4) //Out of hiding, invoke on place.
skill->unit_move(bl,timer->gettick(),1);
if(opt_flag&2 && sd && map->getcell(bl->m,bl->x,bl->y,CELL_CHKNPC))
npc->touch_areanpc(sd,bl->m,bl->x,bl->y); //Trigger on-touch event.
[/code]
kyeme - Dec 13, 2015 5:06
Up~
bgamez23 - Dec 23, 2015 15:29
up for this