Aleos Posted December 26, 2012 Group: Development Manager Topic Count: 56 Topics Per Day: 0.01 Content Count: 732 Reputation: 525 Joined: 12/13/11 Last Seen: June 13, 2024 Share Posted December 26, 2012 Whenever a server has RENEWAL_EXP or RENEWAL_DROPS active, the @mi command becomes somewhat skewed informatively to the player. Pre-renewal just had to read directly from the mob_db since there were no variations of player level versus monster level. Now that there is a variation, should @mi be modified to adjust for this or should a separate command be created? Link to comment Share on other sites More sharing options...
Emistry Posted December 26, 2012 Group: Forum Moderator Topic Count: 93 Topics Per Day: 0.02 Content Count: 10018 Reputation: 2369 Joined: 10/28/11 Last Seen: Sunday at 05:32 PM Share Posted December 26, 2012 isnt that we have a setting that can manipulate this ? trunk/conf/battle/gm.conf // requires RENEWAL_DROP to be enabled (src/map/config/renewal.h) // modifies @mobinfo to display the users' real drop rate as per renewal_drop formula // modifies @iteminfo to not display the minimum item drop rate (since it can't tell the mob level) atcommand_mobinfo_type: 0 i think enable users to select which settings they want would be better. 1 Link to comment Share on other sites More sharing options...
Aleos Posted December 26, 2012 Group: Development Manager Topic Count: 56 Topics Per Day: 0.01 Content Count: 732 Reputation: 525 Joined: 12/13/11 Last Seen: June 13, 2024 Author Share Posted December 26, 2012 Ah! So I knew I wasn't going crazy. I could have sworn something was implemented for it, so should one be added for exp too then since drop is there? Edit: It was added in r16337. But looking at the revision diff and the current atcommand.c, it seems it was completely removed from mobinfo at some point! Link to comment Share on other sites More sharing options...
MarkZD Posted December 27, 2012 Group: Members Topic Count: 6 Topics Per Day: 0.00 Content Count: 134 Reputation: 35 Joined: 02/27/12 Last Seen: April 5, 2022 Share Posted December 27, 2012 IMO, it should be done with a special parameter or on another function and let the @mobinfo keeps regarding only to the mobs. So remove the atcommand_mobinfo_type, and let the new command enabled by default. #ifdef RENEWAL_DROP &&|| RENEWAL_EXP, shows the referenced infos, otherwise show user a message informing it just work with those renewal settings or just shows the default drop, exp. Link to comment Share on other sites More sharing options...
Aleos Posted March 6, 2013 Group: Development Manager Topic Count: 56 Topics Per Day: 0.01 Content Count: 732 Reputation: 525 Joined: 12/13/11 Last Seen: June 13, 2024 Author Share Posted March 6, 2013 Applied in r17173. Link to comment Share on other sites More sharing options...
Recommended Posts