Issue information

Issue ID
#5658
Status
Fixed
Severity
High
Started
Hercules Elf Bot
Apr 28, 2012 11:08
Last Post
Hercules Elf Bot
Jun 9, 2012 21:20
Confirmation
Yes (4)
No (0)

Hercules Elf Bot - Apr 28, 2012 11:08

Originally posted by [b]emong[/b]
Attached in this report is the error in the quest..

How to solve it? Thanks!

Hercules Elf Bot - Apr 28, 2012 11:16

Originally posted by [b]emong[/b]
[img]http://a2.sphotos.ak.fbcdn.net/hphotos-ak-ash3/s720x720/552077_178004672322807_100003398264579_269971_420337930_n.jpg[/img]

Here's is the message when the error happened.. How to fix this? I'm noob at scripting. Thank you..

This post has been edited by emong on Apr 29, 2012 1:12

Hercules Elf Bot - May 1, 2012 0:57

Originally posted by [b]emong[/b]
Help me about this issue :(

Hercules Elf Bot - May 1, 2012 2:02

Originally posted by [b]xRaisen[/b]
Same here. I also having that error related

Hercules Elf Bot - May 2, 2012 13:24

Originally posted by [b]emong[/b]
Good day.. how to solve this? Thanks

Hercules Elf Bot - May 2, 2012 16:14

Originally posted by [b]xRaisen[/b]
I dont know. it seems some other instances like this link http://rathena.org/board/topic/56456-300-monster-challenge/ is also broken.

Hercules Elf Bot - May 3, 2012 8:48

Originally posted by [b]emong[/b]
do you have any temporary solution for that error?

Hercules Elf Bot - May 3, 2012 10:24

Originally posted by [b]xRaisen[/b]
hmmm yes.. revert your server to [b]r15996 [/b]or [b]r15995[/b].

This post has been edited by xRaisen on May 3, 2012 10:32

Hercules Elf Bot - May 4, 2012 11:45

Originally posted by [b]emong[/b]
are you sure with that?

Hercules Elf Bot - May 4, 2012 19:14

Originally posted by [b]xRaisen[/b]
im sure. thats what i did. im waiting for the fix before ill update it to the latest. silly, but we need that to get it running for now.

Hercules Elf Bot - May 5, 2012 9:18

Originally posted by [b]emong[/b]
it didn't work for me..

Hercules Elf Bot - May 8, 2012 10:39

Originally posted by [b]emong[/b]
Bump..

Hercules Elf Bot - May 9, 2012 0:32

Originally posted by [b]xRaisen[/b]
bump*

it work for me. you might doing it wrong. use [url="http://tortoisesvn.net/downloads.html"][i]TortoiseSVN.[/i][/url]

[spoiler][img]http://i48.tinypic.com/2572czm.jpg[/img][/spoiler]
---------------------------------
[spoiler][img]http://i45.tinypic.com/2z4ckfp.jpg[/img][/spoiler]

This post has been edited by xRaisen on May 9, 2012 0:33

Hercules Elf Bot - May 9, 2012 7:27

Originally posted by [b]emong[/b]
It's really not working for me. :(

Hercules Elf Bot - May 9, 2012 8:12

Originally posted by [b]xRaisen[/b]
It works for me. that's all i can help :(

Hercules Elf Bot - May 11, 2012 21:53

Originally posted by [b]sizenine[/b]
epoque needs to look at this already..

Hercules Elf Bot - May 12, 2012 2:31

Originally posted by [b]xRaisen[/b]
im sure he knew it already. hes figuring it out by now i think since hes responsible for changing it to cause the instances to broke.

Hercules Elf Bot - May 12, 2012 13:09

Originally posted by [b]sizenine[/b]
I don't think so, nobody put the status to confirmed...

i just checked out a fresh copy of the latest svn to test it again (without any mods), confirmed to happen with any instance as long as you run it twice. the second run fails to duplicate:

[IMG]http://i.minus.com/jyj9XKvvdAml9.png[/IMG]

This post has been edited by sizenine on May 12, 2012 14:02

Hercules Elf Bot - May 14, 2012 8:18

Originally posted by [b]emong[/b]
Yes. I also encountered this error. I'm glad someone confirmed it already :) Hope they'll notice this.

Hercules Elf Bot - May 14, 2012 10:58

Originally posted by [b]xRaisen[/b]
Instances are broken. im planning to instance all of my events. but due to this, i cant make that happen.

Hercules Elf Bot - May 15, 2012 13:08

Originally posted by [b]emong[/b]
Do you have any solution for this xRaisen?

Hercules Elf Bot - May 17, 2012 1:08

Originally posted by [b]Ind[/b]
this is odd at first i was able to reproduce but then i recompiled and it didn't happen again o.o

Hercules Elf Bot - May 17, 2012 1:13

Originally posted by [b]sizenine[/b]
What procedures did you take? Try running the instance a third time maybe?

Hercules Elf Bot - May 17, 2012 3:51

Originally posted by [b]xRaisen[/b]
instances shows duplicate npc's when run it twice as for my case. i wonder when i reverted it back to the old revision [b]r15996 [/b]it runs perfect!

Hercules Elf Bot - May 17, 2012 4:23

Originally posted by [b]sizenine[/b]
[quote name='xRaisen' timestamp='1337226704' post='9296']
instances shows duplicate npc's when run it twice as for my case. i wonder when i reverted it back to the old revision [b]r15996 [/b]it runs perfect!
[/quote]

actually, reverting to 15996 doesn't solve the problem for me. it seems to not occur as often but it still happens sometimes.

Hercules Elf Bot - May 17, 2012 4:33

Originally posted by [b]xRaisen[/b]
Ok i'll try it again later. for what i am doing before is I go one step backwards at a time starting around from r16005 until i reach a revision that works which is r15996

and i presumed that the last revision made at that time by epoque broke it, maybe. and i was able to trace it which was the last revision at that time that the instances work.

This post has been edited by xRaisen on May 17, 2012 4:40

Hercules Elf Bot - May 17, 2012 9:24

Originally posted by [b]emong[/b]
I tried also to revert my rev to 15996 but it didn't work. It is still the same..

Hercules Elf Bot - May 23, 2012 3:59

Originally posted by [b]Ind[/b]
still not able to reproduce, would like more information.

Hercules Elf Bot - May 24, 2012 22:43

Originally posted by [b]sizenine[/b]
well its definitely there if all of us produced it and even Masao (confirmed) and you produced it before. I'll try again with the current latest checkout and see if it's still there.

Hercules Elf Bot - May 24, 2012 23:30

Originally posted by [b]Masao[/b]
Well i only confirmed it and moved it to core section because you annoyed the hell out of me with this bug report.

I never tried to reproduce it so i couldn't reproduce it.

Hercules Elf Bot - May 25, 2012 5:11

Originally posted by [b]xRaisen[/b]
im sorry if its annoying, but i always getting this kind of duplicate npc error. i suspect that the clearing of the npc variable is not clearing it correctly. maybe it does but it skips the npc on db_destroy? im sure the [CODE]instance[instance_id].vars = NULL;[/CODE] is working well for it is issuing the same map when the map is available but the npc cant be duplicated. please correct me if im pointing it wrong. heres my thread [url="http://rathena.org/board/tracker/issue-5678-instances-not-working/"]http://rathena.org/board/tracker/issue-5678-instances-not-working/[/url]

This post has been edited by xRaisen on May 25, 2012 5:17

Hercules Elf Bot - May 25, 2012 6:53

Originally posted by [b]Ind[/b]
[quote name='xRaisen' timestamp='1337922705' post='9545']
im sorry if its annoying, but i always getting this kind of duplicate npc error. i suspect that the clearing of the npc variable is not clearing it correctly. maybe it does but it skips the npc on db_destroy? im sure the [CODE]instance[instance_id].vars = NULL;[/CODE] is working well for it is issuing the same map when the map is available but the npc cant be duplicated. please correct me if im pointing it wrong. heres my thread [url="http://rathena.org/board/tracker/issue-5678-instances-not-working/"]http://rathena.org/board/tracker/issue-5678-instances-not-working/[/url]
[/quote]
not related at all, these vars stored in the instance array are for the \'\ npc variable type which is exclusive to instances and yet not used in any of the instances rathena provides.

Hercules Elf Bot - May 25, 2012 8:12

Originally posted by [b]xRaisen[/b]
thanks Ind for the light. im kinda desperate to find whats causing it as part of my rathena source exploration.

This post has been edited by xRaisen on May 25, 2012 8:36

Hercules Elf Bot - May 26, 2012 10:44

Originally posted by [b]Pig321[/b]
Solution, modify the map.c

int cleanup_sub(struct block_list *bl, va_list ap)
{
nullpo_ret(bl);
switch(bl->type) {
case BL_PC:
map_quit((struct map_session_data *) bl);
break;
case BL_NPC:
[color=#008000]//npc_unload((struct npc_data *)bl,false);[/color]
[color=#ff0000]npc_unload((struct npc_data *)bl,true);[/color]
break;
case BL_MOB:
unit_free(bl,CLR_OUTSIGHT);
break;
case BL_PET:
//There is no need for this, the pet is removed together with the player. [Skotlex]
break;
case BL_ITEM:
map_clearflooritem(bl->id);
break;
case BL_SKILL:
skill_delunit((struct skill_unit *) bl);
break;
}
return 1;
}

Hercules Elf Bot - May 26, 2012 21:38

Originally posted by [b]sizenine[/b]
[quote name='Masao' timestamp='1337902206' post='9544']
Well i only confirmed it and moved it to core section because you annoyed the hell out of me with this bug report.I never tried to reproduce it so i couldn't reproduce it.
[/quote]

Oh, way to go Mr. Script Dev. That's sure a smart thing to do!

[quote name='Pig321' timestamp='1338029079' post='9593']
Solution, modify the map.cint cleanup_sub(struct block_list *bl, va_list ap){nullpo_ret(bl);switch(bl->type) {case BL_PC:map_quit((struct map_session_data *) bl);break;case BL_NPC:[color=#008000]//npc_unload((struct npc_data *)bl,false);[/color][color=#ff0000]npc_unload((struct npc_data *)bl,true);[/color]break;case BL_MOB:unit_free(bl,CLR_OUTSIGHT);break;case BL_PET://There is no need for this, the pet is removed together with the player. [Skotlex]break;case BL_ITEM:map_clearflooritem(bl->id);break;case BL_SKILL:skill_delunit((struct skill_unit *) bl);break;}return 1;}
[/quote]

Thanks so much, you're the best.

Hercules Elf Bot - May 26, 2012 21:50

Originally posted by [b]Ind[/b]
Fixed in [rev=16153] Instance NPCs will now be cleared properly. Super-Special Thanks to Pig321

Hercules Elf Bot - May 27, 2012 15:11

Originally posted by [b]emong[/b]
Had error when I recompiled my SVN

View attached photo.

Thanks!

Hercules Elf Bot - May 29, 2012 6:16

Originally posted by [b]xRaisen[/b]
thanks Pig321. a real good find. i cant update it yet. damn this. im heart broken cant think well...........................

Hercules Elf Bot - May 31, 2012 16:14

Originally posted by [b]Ind[/b]
[quote name='emong' timestamp='1338131481' post='9666']
Had error when I recompiled my SVNView attached photo.Thanks!
[/quote]
try to recompile from scratch with a 'make clean && make sql'