Cydh Posted April 23, 2014 Posted April 23, 2014 I and @Lighta was talking about to move hardoceds SC's config to db files, lighta started it and made prototype the status_db file (Oct 16th, 2013). He knows I like db, then he asked me to continue it. Then, in any reason, I stopped it. Now, I'm starting it back, and still testing also checking the configs for SC since many SC changes (@Aleos did it!)The config files are: sc_config_db.txt, contains the SC, its SI, SCS, SCB, Opt1, Opt2, Opt3, Option, some flags, and buff type. the doc of sc_config_db.txt sc_endsc_db.txt, "if the SC is active, the other SC will be ended" sc_failingsc_db, "if the SC attempts to start, but there are SC that causing failed, it will be failed" sc_script_db.txt, additional script that will be executed when status is active skill_sc_assoc_db, SC-Skill relation Try it! Here in sc_config branch on rAthena repo (added in May 27th, 2014) I hope at least, I get 1 tester. Or, you can test on my test server, use this clientinfo.xml, (2013-08-07a Ragexe client) Quote
QQfoolsorellina Posted April 23, 2014 Posted April 23, 2014 That ideas sounds good!,how about add additional bonus script field which will be run while specific sc sate was started SC,SI,SCS,SCB,OPT1,OPT2,OPT3,Option,Flag,Type {,{ bonus xxx.... } } Quote
Cydh Posted April 23, 2014 Author Posted April 23, 2014 That's already crossed on my mind, then I dropped it because I just got other idea to make some new labels, like OnSCStarted and OnSCEnded, then maybe can make new script command to check sc_isactive(<SC>), that maybe can be used on OnPCCalcEvent. Then I think maybe later, at least finish the main issue first. Quote
Aleos Posted April 23, 2014 Posted April 23, 2014 Keep up the good work. I know I broke the original diff with all the 3rd skill changes. Quote
Cydh Posted May 3, 2014 Author Posted May 3, 2014 there, need someone to use it. my fork, sc_config branch https://github.com/cydh/rathena/tree/sc_config pull req, Pull: 71, Pull: 71.patch Quote
Lighta Posted May 3, 2014 Posted May 3, 2014 Look good look good, didn't review it yet but if you people want to try it and give us feedback that will help. The main idea when I started was this: -Offert a way to easily add/remove Status_Icone without the need of recompiling server. -Allow Status_Bonus, to be enable or disable by simply removing SCB from it. (not just prevent the recalculation like actual but really doesn't trigger the bonus) -Allow Status listing mode alteration on the fly (without the need of reboot). like if you wanted put a status that was removed on death permanent just alter his flag. (whereas adding it to a hardcoded list actually). In short remove as possible the hardcoded list into the emulator and offert the possibility of admin to edit this easily without the need of src edit. @Cydh In sc_config_db.txt I didn't get the purpose of type (buff/debuff) and it seem you have forget to put it in your exemple =). What does this implies? SCO_REM_ONLOGOUT = Remove when player logs out Shouldn't this be the default ? so we should have SCM_NO_REM_ONLOGOUT instead ?? Quote
Shade Posted May 3, 2014 Posted May 3, 2014 This is nice!Does it mean we can use the visual effect like (Stone Curse/Poison) on a custom status? Quote
Lighta Posted May 4, 2014 Posted May 4, 2014 yes you should be able, are you talking about the option part? Quote
Cydh Posted May 4, 2014 Author Posted May 4, 2014 @Cydh In sc_config_db.txt I didn't get the purpose of type (buff/debuff) and it seem you have forget to put it in your exemple =). What does this implies? SCO_REM_ONLOGOUT = Remove when player logs out Shouldn't this be the default ? so we should have SCM_NO_REM_ONLOGOUT instead ?? yo lighta, that my snippets aren't updated yet. I changed the naming for SCO to SCF (status change flag). and, everything that in my fork (sc_config branch) are updated now. I'll update my code snippets. For SC type, which before rA using 'debuff_on_logout' to decide which buff type that will be removed when player logs out. Now, if 'debuff_on_logout' = 0, remove only SC that has SC_NO_SAVE, when 'debuff_on_logout' is 1, it will remove SC with SC_NO_SAVE and as negative negative buff (SC with type 1), so by default, SC is type 0, as positive buff which will be removed when player logs out when 'debuff_on_logout' is 2. Also, before, there are some SC that typed as "Debuff" or "Buff" that removed by status_change_clear_buffs(). Maybe if rA wants, can make branch sc_config so I can do PR for that branch instead of the master branch. Quote
Lighta Posted May 4, 2014 Posted May 4, 2014 Ah ok for the status_change_clear_buffs part, for the debuff_on_logout part coupled with SC_NO_SAVE I have a feeling that a bit too overly complex. I mean for logout we should just say, if he have NO_SAVE drop it that it. Quote
Shade Posted May 4, 2014 Posted May 4, 2014 yes you should be able, are you talking about the option part? Just to be clear, if you use Stone Curse VISUAL effect, you won't be immobile right? Since you're only taking the visual effects. Quote
Cydh Posted May 5, 2014 Author Posted May 5, 2014 yes you should be able, are you talking about the option part? Just to be clear, if you use Stone Curse VISUAL effect, you won't be immobile right? Since you're only taking the visual effects. Actually, I want just like that, "just give the visual effect", but like or not, by adding the OPT1, player marked as 'pc_cant_act'. Seems need to make flag that really can decide which status that make immobilze, cannot cast, cannot attacking, and something. Then we can use OPTs only as visual effect, by asking clif_changeoption(). (you can try by using @send 0x229 L<account_id> 1) add sc->cant.* in same line beside pc_cant_act() and pc_cant_act2(). Quote
Cydh Posted May 27, 2014 Author Posted May 27, 2014 I've added sc_config branch in rAthena main repository. It's the 12,000th commit (even is not in master branch ) Quote
Helly Posted March 28, 2016 Posted March 28, 2016 (edited) great work! Edited March 28, 2016 by Namine210 Quote
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.