Issue information

Issue ID
#7400
Status
Fixed
Severity
None
Started
Samuel
Jun 19, 2013 3:16
Last Post
Joseph
Jul 3, 2013 21:37
Confirmation
N/A

Samuel - Jun 19, 2013 3:16

I have a question, is this a typo or just an accidentally copy paste problem?

npc/re/merchants/refiner.txt

[b]line 298:[/b][quote]
if (.@refinecnt < 1 || .@refinecheck > 10) {[/quote]

should it be?[quote]
if (.@refinecnt < 1 || .@refinecheck > [b]20[/b]) {[/quote]

because if I use the:
[quote]
callfunc "refinenew","Bestry",[b]1[/b],0;[/quote]

I can't get past with that check and always saying:
[quote]
mes "[" + getarg(0) + "]";
mes "I can't refine this item that many times.";
close;[/quote]

Euphy - Jun 19, 2013 3:47

The equipment has to be at least +10 to get to that point. You can't refine something over +20.

Samuel - Jun 19, 2013 3:51

The equipment I'm refining is just +10, and even if I input 1, it still says "I can't refine this item that many times."

Also tried using a +11 and +12 item, still the same.. :)

This post has been edited by Samuel on Jun 19, 2013 4:05

malufett - Jun 19, 2013 4:56

use the other refiner that refines more than 10..:D

:meow:

Euphy - Jun 19, 2013 6:00

Oh, my mistake, I didn't read the line above it.[code=auto:0]set .@refinecheck,.@refinecnt + getequiprefinerycnt(.@part);[/code]It's totally wrong, then.

Samuel - Jun 19, 2013 6:12

Yup, that's why there is a need to increase the refinecheck value in order to pass

Euphy - Jun 19, 2013 7:02

I fixed this on rAthena, Herc can merge it.

Joseph - Jul 3, 2013 21:37

Fixed in [url="https://github.com/HerculesWS/Hercules/commit/28a9fec0916db3d92629ae6e6cd62b5777b09e73"]https://github.com/HerculesWS/Hercules/commit/28a9fec0916db3d92629ae6e6cd62b5777b09e73[/url]