Dastgir 1246 Posted January 21, 2018 Hello, I feel this is one of the important setting, which was restricted due to packet size. So I have made this plugin to allow UNLIMITED Storage Size. Just enable this plugin and set MAX_STORAGE(from common/mmo.h) to whatever you want. Link: https://github.com/dastgir/HPM-Plugins/blob/master/src/plugins/storage_expand.c Note: This doesn't work with Guild Storage. 4 IndieRO, Nihad, Maple and 1 other reacted to this Quote Share this post Link to post Share on other sites
Nihad 24 Posted January 21, 2018 Way to go dastgir, as always you rock. Tested and works fine. Made the limit 1500 just to test, once I had above 1200 items in storage it started to lag my client when opening storage. But yeah works like a charm, up to 1000 no issues or delays. 1 bWolfie reacted to this Quote Share this post Link to post Share on other sites
bWolfie 139 Posted January 21, 2018 Will try it out. It's a decision you have to make whether to use it or not, since once you increase storage limit, you shouldn't ever reduce it, since players won't be able to access some items until they empty space :| could prove problematic. This is not exclusive to this plugin, though, thought I'd put it out there :P. Thanks as always, Dastgir! 1 Dastgir reacted to this Quote Share this post Link to post Share on other sites
bWolfie 139 Posted January 22, 2018 (edited) Maybe it's the new way storage works...when it's full (even on 600/600) it seems to be laggy af while trying to resolve the storage. Edited January 22, 2018 by Myriad Quote Share this post Link to post Share on other sites
Sephus 203 Posted January 25, 2018 On 22/01/2018 at 11:27 PM, Myriad said: Maybe it's the new way storage works...when it's full (even on 600/600) it seems to be laggy af while trying to resolve the storage. Umm, what do you mean by resolve the storage? If anything it should be laggy before (since it could send over 600 queries iirc), and not at all right now (because it sends a maximum of 4 queries iirc). @Dastgir Not to be offensive or anything, but what if we were to make a PR that allowed splitting of packets?... this plugin would be sort of useless. Quote Share this post Link to post Share on other sites
Dastgir 1246 Posted January 25, 2018 24 minutes ago, Smoke said: Umm, what do you mean by resolve the storage? If anything it should be laggy before (since it could send over 600 queries iirc), and not at all right now (because it sends a maximum of 4 queries iirc). @Dastgir Not to be offensive or anything, but what if we were to make a PR that allowed splitting of packets?... this plugin would be sort of useless. well, I have no issues with it, It happened before too (specialshop and other plugins). If in future, we made the PR for allowing splitting of packet, this plugin may be useful for people using old hercules. Also, I will put a note on topic referring that new hercules don't need this plugin. Quote Share this post Link to post Share on other sites
Sephus 203 Posted January 25, 2018 8 minutes ago, Dastgir said: well, I have no issues with it, It happened before too (specialshop and other plugins). If in future, we made the PR for allowing splitting of packet, this plugin may be useful for people using old hercules. Also, I will put a note on topic referring that new hercules don't need this plugin. Hmm I think you could have created a PR for this. Quote Share this post Link to post Share on other sites
bWolfie 139 Posted January 25, 2018 @Smoke maybe I just didn't notice it before. It takes like 2 seconds for the numbers to appear and I can use my storage. Quote Share this post Link to post Share on other sites
SuyinVC 0 Posted January 20 i try to enable this plugin but if a configure more than 700 in mm.h then show the following error when compile: In file included from ../common/hercules.h:24:0, from db.h:62, from db.c:91: ../common/cbasetypes.h:514:32: error: static assertion failed: "The maximum amount of item slots per account storage is limited by the inter-server communication layout. Use a smaller value!" #define STATIC_ASSERT(ex, msg) _Static_assert(ex, msg) ^ ../common/mmo.h:512:1: note: in expansion of macro ‘STATIC_ASSERT’ STATIC_ASSERT(MAX_STORAGE * sizeof(struct item) + 8 <= 0xFFFF, "The maximum amount of item slots per account storage is limited by the inter-server communication layout. Use a smaller value!"); ^ make[1]: *** [obj_all/db.o] Error 1 make[1]: Leaving directory `/ragnarok/Hercules/src/common' make: *** [common_sql] Error 2 Quote Share this post Link to post Share on other sites
SuyinVC 0 Posted January 20 And if I leave the configuration at 700, it accepts me, when I start the game, it does not allow connections [Error]: Cant set char online. Account not logged in: 2000134 [Error]: chrif_authok: Data size mismatch! 30128 != 95664 [Status]: set users : 1 [Status]: Request for connection of superatomix (ip: ). [Notice]: Authentication accepted (account: , id: 2000114, ip: ) [Status]: Connection of the account '' accepted. [Info]: Closed connection from ''. [Info]: request connect - account_id:2000114/login_id1:1282534782/login_id2:536553747 [Info]: Loading Char Data (2000114) [Info]: Char load request (20) [Info]: Loaded char (20 - 😞 status memo inventory cart skills friends hotkeys mercenary accdata [Info]: Selected char: (Account 2000114: 1 - ) [Info]: Subnet check []: WAN [Status]: set users : 0 [Error]: chrif_authok: Data size mismatch! 30128 != 95664 [Info]: User Count: 0 Quote Share this post Link to post Share on other sites
4144 366 Posted January 20 Data size mismatch! error most time mean you changed headers but nor recompiled all servers run make clean and after build server Quote Share this post Link to post Share on other sites
SuyinVC 0 Posted April 16 it's works ..!! thank you to ALL.. Just a question, how can I make the "@storage 2" or "@storage vip" command only be used by vip players.?????? help me Quote Share this post Link to post Share on other sites