Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lua Modifier Properties not correctly linked #26

Open
1 of 4 tasks
SinZ163 opened this issue Nov 24, 2016 · 7 comments
Open
1 of 4 tasks

Lua Modifier Properties not correctly linked #26

SinZ163 opened this issue Nov 24, 2016 · 7 comments

Comments

@SinZ163
Copy link
Member

SinZ163 commented Nov 24, 2016

  • MODIFIER_PROPERTY_SPELL_AMPLIFY_PERCENTAGE
    Currently the only work around is via messing with everything via Damage Filters, but it would be preferable if this modifier property worked as intended.
    This would make Allow modifying what heroes get per attribute point #16 fully redundant as everything associated to stats will be changeable (with the possible exception of base armour)

    • Doesn't exist in DD (Citation Needed)
  • MODIFIER_PROPERTY_CAST_RANGE_BONUS
    Current workarounds include messing with Aether lens (which is simply a hard-coded version of this very property.)

    • Doesn't exist in DD (Citation Needed)
  • MODIFIER_PROPERTY_COOLDOWN_PERCENTAGE

    • Works as intended in DD
  • MODIFIER_PROPERTY_MANACOST_PERCENTAGE

    • Doesn't exist in DD (Citation Needed)
@DoctorGester
Copy link

Probably need a collection of issues/one single big issue, it's not just spell amplify but many other things.

@fcalife
Copy link

fcalife commented Nov 24, 2016

Adding MODIFIER_PROPERTY_CAST_RANGE_BONUS to the list of broken modifiers. This one doesn't work in DD or lua. Tested @ Nov 24th 2016

--Edit: Added to post - SinZ

@SinZ163 SinZ163 changed the title MODIFIER_PROPERTY_SPELL_AMPLIFY_PERCENTAGE Not correctly linked to Lua Modifiers Lua Modifier Properties not correctly linked Nov 24, 2016
@Yahnich
Copy link

Yahnich commented Nov 24, 2016

MODIFIER_PROPERTY_COOLDOWN_PERCENTAGE works in datadriven ~~~but not lua~~~ EDIT: Works fine in Lua - DoctorGester
MODIFIER_PROPERTY_MANACOST_PERCENTAGE is broken in both
--Edit: Added to post - SinZ

@DoctorGester
Copy link

Not true, cooldown percentage indeed works in lua for me.

@Wouterz90
Copy link

MODIFIER_PROPERTY_SPELL_AMPLIFY_PERCENTAGE can be done by MODIFIER_PROPERTY_TOTALDAMAGEOUTGOING_PERCENTAGE and checking if there was an inflictor. Still would be nice to have this though.

@Yahnich
Copy link

Yahnich commented Jan 27, 2017

MODIFIER_PROPERTY_SPELL_AMPLIFY_PERCENTAGE
MODIFIER_PROPERTY_CAST_RANGE_BONUS
MODIFIER_PROPERTY_CAST_RANGE_BONUS_STACKING
MODIFIER_PROPERTY_HEAL_AMPLIFY_PERCENTAGE
MODIFIER_PROPERTY_MANACOST_PERCENTAGE

are all confirmed fixed for LUA modifiers

@Yunten
Copy link

Yunten commented Jul 5, 2017

- [x] MODIFIER_PROPERTY_ALWAYS_ALLOW_ATTACK
- [x] MODIFIER_PROPERTY_PREATTACK_TARGET_CRITICALSTRIKE
- [x] MODIFIER_PROPERTY_TEMPEST_DOUBLE
- [ ] MODIFIER_PROPERTY_SUPER_ILLUSION
- [ ] MODIFIER_PROPERTY_SUPER_ILLUSION_WITH_ULTIMATE
- [x] MODIFIER_PROPERTY_MAGICAL_CONSTANT_BLOCK

Not working in LUA

@SinZ163 SinZ163 pinned this issue Sep 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants