MantisBT

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0003001VCMIModspublic2018-08-25 20:052018-08-27 21:26
Reporterasgraf 
Assigned ToAVS 
PrioritynormalSeverityfeatureReproducibilityN/A
StatusassignedResolutionopen 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Summary0003001: Ability to learn new spells should be moddable
DescriptionI would like to request new bonus type ALLOW_LEARN_SPELLS that grands hero ability to learn certain spells without need of having Wisdom skill of appropriate level

This new bonus type should be allowed to be limited by:
- spell level (ability to learn spells of certain level including spells added by other mods)
- spell school (ability to learn spells of certain magic school including spells added by other mods)
- spell flags (ability to learn all spells with certain flag like summoning including spells added by other mods)
- hand-picked selection of spell id's (white-listed spells can be learn without Wisdom)

Example use case:
https://forum.vcmi.eu/t/spell-schools-modding/4679 [^]
TagsNo tags attached.
Attached Files

- Relationships

-  Notes
(0007631)
asgraf (reporter)
2018-08-25 20:07
edited on: 2018-08-25 20:13

Additionally BAN_SPELLS that prevents spells from being casted (and/or learned) with similar limiters would be useful for custom artifacts

(0007634)
AVS (administrator)
2018-08-26 18:06

Please provide examples of desired JSON configuration.
(0007639)
AVS (administrator)
2018-08-27 20:09

As this is bonus type, it should use https://wiki.vcmi.eu/Bonus_Format, [^] or it most be an extension of it.
(0007640)
asgraf (reporter)
2018-08-27 20:58
edited on: 2018-08-27 20:58

[
    {
        "type": "ALLOW_LEARN_SPELLS",
        "subtype": 0,//0 allow spells | 1 ban spells
        "val" : 3,//spell level
        "description" : "Unlock Hero ability to learn level 3 fire spells",
        "limiters": [
            {
                "type":"SPELL_SCHOOL_LIMITER",
                "parameters":["fire"]
            }
        ]
    },
    {
        "type": "ALLOW_LEARN_SPELLS",
        "subtype": 0,//0 allow spells | 1 ban spells
        "val" : 5,//spell level
        "description" : "Unlock Hero ability to learn all summon spells",
        "limiters": [
            {
                "type":"SPELL_EFFECT_LIMITER",
                "parameters":["summon"]
            }
        ]
    },
    {
        "type": "ALLOW_LEARN_SPELLS",
        "description" : "Unlock Hero ability to learn Summon Elemental spells",
        "limiters": [
            {
                "type":"SPELL_ID_LIMITER",
                "parameters":["airElemental","waterElemental","earthElemental","fireElemental"]
            }
        ]
    },
    {
        "type": "ALLOW_LEARN_SPELLS",
        "val" : 3,
        "description" : "Basic Wisdom Equivalent"
    },
    {
        "type": "ALLOW_LEARN_SPELLS",
        "val" : 4,
        "description" : "Advanced Wisdom Equivalent"
    },
    {
        "type": "ALLOW_LEARN_SPELLS",
        "val" : 5,
        "description" : "Expert Wisdom Equivalent"
    }
]

(0007641)
asgraf (reporter)
2018-08-27 21:19
edited on: 2018-08-27 21:20

Since new bonus is extension of wisdom bonus, old wisdom effect probably should be deprecated or treated as alias of new ALLOW_LEARN_SPELLS effect.
To avoid functional duplication Wisdom skill should be changed to use this new bonus

(0007642)
asgraf (reporter)
2018-08-27 21:26

I don't know if i restrictions should be listed as limiters or if they should be part of effect in addInfo key.

- Issue History
Date Modified Username Field Change
2018-08-25 20:05 asgraf New Issue
2018-08-25 20:07 asgraf Note Added: 0007631
2018-08-25 20:11 asgraf Note Edited: 0007631 View Revisions
2018-08-25 20:13 asgraf Note Edited: 0007631 View Revisions
2018-08-26 18:06 AVS Note Added: 0007634
2018-08-26 18:06 AVS Assigned To => AVS
2018-08-26 18:06 AVS Status new => feedback
2018-08-27 20:03 asgraf Note Added: 0007635
2018-08-27 20:03 asgraf Status feedback => assigned
2018-08-27 20:05 asgraf Note Added: 0007636
2018-08-27 20:07 asgraf Note Added: 0007637
2018-08-27 20:07 asgraf Note Edited: 0007637 View Revisions
2018-08-27 20:08 asgraf Note Added: 0007638
2018-08-27 20:09 AVS Note Added: 0007639
2018-08-27 20:57 asgraf Note View State: 0007635: private
2018-08-27 20:57 asgraf Note View State: 0007636: private
2018-08-27 20:57 asgraf Note View State: 0007637: private
2018-08-27 20:57 asgraf Note View State: 0007638: private
2018-08-27 20:58 asgraf Note Added: 0007640
2018-08-27 20:58 asgraf Note Edited: 0007640 View Revisions
2018-08-27 20:59 asgraf Note Deleted: 0007638
2018-08-27 20:59 asgraf Note Deleted: 0007637
2018-08-27 20:59 asgraf Note Deleted: 0007636
2018-08-27 20:59 asgraf Note Deleted: 0007635
2018-08-27 21:19 asgraf Note Added: 0007641
2018-08-27 21:20 asgraf Note Edited: 0007641 View Revisions
2018-08-27 21:26 asgraf Note Added: 0007642

Site | Forums | Wiki | Slack | GitHub


Copyright © 2000 - 2018 MantisBT Team
Hosting provided by DigitalOcean