add addmonsterdrop and delmonsterdrop scriptcommands

[evilpuncker]

they are already implemented in rAthena, maybe someone could make a pull request here? They are very useful for events

hope everyone agrees

/* Work In Progress [Lupus] BUILDIN(addmonsterdrop) { int class_,item_id,chance; class_=script_getnum(st,2); item_id=script_getnum(st,3); chance=script_getnum(st,4); if(class_>1000 && item_id>500 && chance>0) { script_pushint(st,1); } else { script_pushint(st,0); } } BUILDIN(delmonsterdrop) { int class_,item_id; class_=script_getnum(st,2); item_id=script_getnum(st,3); if(class_>1000 && item_id>500) { script_pushint(st,1); } else { script_pushint(st,0); } } */ 

 

[jaBote]

I'm from my cellphone but... it seems that code doesn't do anything except just returning 1 or 0 depending on the parameters value.

 

[evilpuncker]

the code I posted is the current one we got (commented out and marked as WIP) the rAthena one is here

 

[Dastgir]

the code I posted is the current one we got (commented out and marked as WIP) the rAthena one is here

According to the script you posted, the drop item is added to empty drop slot, if all 10 drop slots are full, it wont add the dropitem, so accord7ng to me, i dont think its of much use, if it was adding extra(more than 10), then it might be useful.

 

[evilpuncker]

anyone agree on implementing this? If yes I'll add a pull request

 

[Mhalicot]

Agreed, This is useful for making/adding custom mobs. 

Please pull this..

 

[jaBote]

If I remember well, Haruna was making some upstream merges from rAthena. There's an open issue there on GitHub, but I don't know if the addmonsterdrop and delmonsterdrop rAthena commits were referenced as commits to merge. 

 

[Mhalicot]

wish it merge soon.

 

[evilpuncker]

If I remember well, Haruna was making some upstream merges from rAthena. There's an open issue there on GitHub, but I don't know if the addmonsterdrop and delmonsterdrop rAthena commits were referenced as commits to merge. 

I don't think so

 

[Dastgir]

https://github.com/HerculesWS/Hercules/issues/144 The Merges that will be taken from rAthena is tracked there.

 

[evilpuncker]

added pull request

https://github.com/HerculesWS/Hercules/pull/191

 

[jaBote]

Thanks to @evilpuncker's pull request and some minot edits by this is now done in 7ddcff9a.

Much appreciated!