[Feedback] Request Features for JASS and the World Editor

[Vinz]

Don't really think it's much of necessity, though for passives it could work out. I doubt people want to hide active abilities in most cases.

I want to hide actives when recreating League's summons, where the ability is temporary replaced by a command one.

Most of the time, one would not reach the 522 limits at all. It's there for Melee balance purposes. So far, I recall Bribe or Purge have a MoveSpeedX to bypass this. I also assume this is related to pathfinding algorithm limitation.

Melees won't exceed it. Auras don't stack with themselves. However, there's no reason for it to be this inflexible when it comes to custom games, especially since there's a field for max movement in Gameplay Constants.
I tried movement speed in Dota 2 in the thousands. It worked fine. That is something Blizzard should aspire to with Reforged.

3. Allows a direct stun from trigger editor. Usually, people relied on the flawed stun systems and the existence of this thing will help a lot on the regard of stun related spells and triggers.

That's wonderful, man. We won't have to create all sorts of abilities to clog our loading screens. I should suggest a triggered slow (could be negative). Perhaps a buff function (directeffect FX (?), integer buffID, boolean debuff) can be intertwined with these natives.

 

[Daffa]

Seemed that I miscalculated. Does new native for moving ability icon will suit your needs? I recall there's a native and a GUI equal in the recent patches to move icons and there's that 0,-11 hiding spot too as other option.

 

[Kyrbi0]

Auras do stack with themselves; prior to one of the recent patches they were just about the only ones that did, in fact.

Also, like Daffa mentioned, both Passive & Active abilities can be set to X:0 & Y:-11 in order to be hidden (while still usable, AFAIK).

 

[Vinz]

Auras with the same buff don't stack with each other. I just tested. I was referring to melee games, where Endurance and Unholy are common speed boosts.

 

[Daffa]

Auras with the same buff don't stack with each other. I just tested. I was referring to melee games, where Endurance and Unholy are common speed boosts.

They stack with different base buff.

 

[Vinz]

Yes.
I was replying to you saying that the limit's there for "melee balance purposes."
I was saying that even if a team brought out multiple Tauren Chieftains, they wouldn't be able to go past 522 in the first place, even without the limit.

 

[Daffa]

Endurance + Haste + Boots of Speed + Unholy?

 

[Vinz]

Alright. But I should add that melee games have a lower limit of 400 due to gameplay constants.
I maintain that there is no good reason for the hard limit of 522.

 

[Wrda]

There's also no good reason for not saying a suggestion value when you don't want a certain limit. I suppose 1000 or around that would be a new reasonable limit.

 

[Vinz]

A new limit?

Shouldn't mapmakers have the freedom to decide what they want that to be?

 

[Wareditor]

There are reasons the limit is 522 obviously. The whole pathing system would need to be redone to be able to go higher than that.

 

[deepstrasz]

There are reasons the limit is 522 obviously. The whole pathing system would need to be redone to be able to go higher than that.

Even for custom maps? I mean, would your dummy projectiles not move because of this pathing system not coping with >522 movement speed?

 

[Wareditor]

Moving a unit via script has nothing to do with it since you don't care about path finding when doing so.

 

[deepstrasz]

Moving a unit via script has nothing to do with it since you don't care about path finding when doing so.

Ordering it to move through triggers is what I had in mind.

 

[Vinz]

There are reasons the limit is 522 obviously. The whole pathing system would need to be redone to be able to go higher than that.

The whole pathing system should be redone. For Reforged, at the very least.

 

[Wareditor]

@deepstrasz You can't go faster than 522 by giving an order. It will be the same as ordering it manually. The only thing you could do is recode the whole pathfinding algorithm and manually move the unit but that would probably work terribly in jass since it's already quite heavy in C++.

@Mythic They will probably not redo it since it's important for it to feel the same as now for the competitive melee scene. The best they could do is optimize it so it runs much faster but I am not sure there is room for that type of optimization.

 

[Wrda]

A new limit?

Shouldn't mapmakers have the freedom to decide what they want that to be?

Mapmakers have no entitlement to decide whether the new limit must be 700 800 900 or any number they want because yes, but give a suggestion.
Also, how does having a new limit destroy your freedom? You can still choose from its range...

 

[Vinz]

My suggestion was to remove it. I did give it a few pages back (Gameplay Section).

E:
I'd like to clarify that I wasn't saying mappers should have the final say on the matter. Rather, I was asking whether or not in principle they should be able to risk breaking their maps with extremely high movement.

 

[deepstrasz]

You can't go faster than 522 by giving an order. It will be the same as ordering it manually. The only thing you could do is recode the whole pathfinding algorithm and manually move the unit but that would probably work terribly in jass since it's already quite heavy in C++.

I meant if they'd increase the speed in a new patch would it matter that much if you'd use that speed for special units like projectiles, considering the pathing?

 

[Wareditor]

@deepstrasz This speed limit is only for ground normally pathed unit. As as I said projectiles are not affected by it at all unless you want to use projectiles using order which is a pretty bad idea.

 

[Devalut]

Some love for the GUI users please

Temporary heath triggers:
* Not adding max health or using mana shield, something akin to anti magic shield but more options! *
- Add x Temporary health to (triggering unit) for Physical type damage for x seconds.
- Modify existing Temporary heath for (triggering unit) to x
- Add X seconds to existing Temporary heath for (triggering unit) to x
- show/hide Temporary health bar under heath bar

Modify unit Ability Data values
- Change (triggering unit) inner fire level 1/X - data - Regeneration Rate/Armor/Duration/STRING to X

Add Dummy triggers
- Cast X spell at Location/Target x with (Triggering unit) as Source - (This would remove the need for dummy unit's)
- Add Buff to (Triggering Unit) and Enable/Disable effect X (I.e. Stun/Poison/Curse/ Using X Ability without need of physical cast, just simply the effect) for X seconds


Multiple copy selection of units/abilities/buffs for transferring over to another map! or an option to inject multiple items into a map without overwriting other ones.

@WeirdEditor
I concur!

 

[MyPad]

I suppose that these natives (types) have already been requested, but having these may benefit mapmakers as a whole:

// For items
type itemevent extends eventid

constant itemevent EVENT_ITEM_CREATED
constant itemevent EVENT_ITEM_REMOVED

// For destructables
type destructableevent extends eventid

constant destructableevent EVENT_DEST_CREATED
constant destructableevent EVENT_DEST_REMOVED

// For the onDamage event
constant native IsEventDamageAttack takes nothing returns boolean
constant native IsEventDamageOrb takes nothing returns boolean

// For the events EVENT_PLAYER_MOUSE_UP and EVENT_PLAYER_MOUSE_DOWN
type mousebutton extends agent

native GetTriggerMouseButton takes nothing returns mousebutton

// An extension to the ability tooltip alteration
native SetUnitAbilityTooltip takes unit whichUnit, integer abilId, integer level, string tooltip returns nothing
native SetUnitAbilityExtendedTooltip takes unit whichUnit, integer abilId, integer level, string tooltip returns nothing
native SetUnitAbilityResearchTooltip takes unit whichUnit, integer abilId, integer level, string tooltip returns nothing

native GetUnitAbilityTooltip takes unit whichUnit, integer abilId, integer level returns string
native GetUnitAbilityExtendedTooltip takes unit whichUnit, integer abilId, integer level returns string
native GetUnitAbilityResearchTooltip takes unit whichUnit, integer abilId, integer level returns string

// A way to make alterations to the base damage permanent
native SetUnitBaseDamageEx takes unit whichUnit, integer index, integer base, boolean makePermanent returns nothing

// More on unit's attack
type missiletype extends agent

constant missiletype MISSILE_INSTANT
constant missiletype MISSILE_NORMAL
constant missiletype MISSILE_ARTILLERY
constant missiletype MISSILE_LINE
constant missiletype MISSILE_BOUNCE

native SetUnitAttackRange takes unit whichUnit, integer index, real range returns nothing
native SetUnitAttackMissileType takes unit whichUnit, integer index, missiletype missType returns nothing
native SetUnitAttackMissileSpeed takes unit whichUnit, integer index, real speed returns nothing
native SetUnitAttackBounceCount takes unit whichUnit, integer index, integer count returns nothing
native SetUnitAttackBounceRange takes unit whichUnit, integer index, real range returns nothing
native SetUnitAttackWeaponType takes unit whichUnit, integer index, weapontype whichWeapon returns nothing

// Already suggested a lot of times by me, but included for the sake of completeness
native GetUnitAttackRange takes unit whichUnit, integer index returns real
native GetUnitAttackMissileType takes unit whichUnit, integer index returns missiletype
native GetUnitAttackMissileSpeed takes unit whichUnit, integer index returns real
native GetUnitAttackBounceCount takes unit whichUnit, integer index returns integer
native GetUnitAttackBounceRange takes unit whichUnit, integer index returns real
native GetUnitAttackWeaponType takes unit whichUnit, integer index returns weapontype

native GetUnitDefaultBaseDamage takes unit whichUnit, integer index returns integer

// On attack-event related natives
constant unitevent EVENT_UNIT_ATTACK_LAUNCH
constant unitevent EVENT_UNIT_ATTACK_MISS

native GetUnitAttackDefaultBackswing takes unit whichUnit, integer index returns real
native GetUnitAttackDefaultCastpoint takes unit whichUnit, integer index returns real
native GetUnitAttackBackswing takes unit whichUnit, integer index returns real
native GetUnitAttackCastpoint takes unit whichUnit, integer index returns real

// These also affect the speed of the attack animation.
native SetUnitAttackBackswing takes unit whichUnit, integer index, real backswing returns nothing
native SetUnitAttackCastpoint takes unit whichUnit, integer index, real castPoint returns nothing

// These deal with the generated projectile for every ranged attack launched.
type projectileevent extends eventid
type projectile extends effect

constant projectileevent EVENT_PROJECTILE_LAUNCH

// Upon hitting the target or 60 seconds (based on Nes' tests), EVENT_PROJECTILE_EXPIRE catches the moment projectile objects are to be destroyed.
constant projectileevent EVENT_PROJECTILE_EXPIRE

// Returns the generated projectile object.
constant native GetTriggerProjectile takes nothing returns projectile

// Allows additional manipulation of the projectile
native GetProjectileSpeed takes projectile whichProj returns real
native GetProjectileDamage takes projectile whichProj returns real
native GetProjectileAttackType takes projectile whichProj returns attacktype
native GetProjectileDamageType takes projectile whichProj returns damagetype
native GetProjectileWeaponType takes projectile whichProj returns weapontype
native GetProjectileTarget takes projectile whichProj returns widget
native GetProjectileTargetX takes projectile whichProj returns real
native GetProjectileTargetY takes projectile whichProj returns real
native GetProjectileTargetZ takes projectile whichProj returns real

native IsProjectileHoming takes projectile whichProj returns boolean
native IsProjectileDamageAttack takes projectile whichProj returns boolean
native GetProjectileOwner takes projectile whichProj returns unit
native GetProjectileArc takes projectile whichProj returns real

// If set to 0, the missile will hit 0-seconds after.
native SetProjectileSpeed takes projectile whichProj, real speed returns nothing
native SetProjectileDamage takes projectile whichProj, real damage returns nothing
native SetProjectileAttackType takes projectile whichProj, attacktype atkType returns nothing
native SetProjectileDamageType takes projectile whichProj, damagetype atkType returns nothing
native SetProjectileWeaponType takes projectile whichProj, weapontype atkType returns nothing
native SetProjectileTarget takes projectile whichProj, widget target returns nothing
native SetProjectileTargetX takes projectile whichProj, real x returns nothing
native SetProjectileTargetY takes projectile whichProj, real y returns nothing
native SetProjectileTargetZ takes projectile whichProj, real z returns nothing

native SetProjectileHoming takes projectile whichProj, boolean homing returns nothing
// If toAttack is true, then the damage inflicted through the projectile is considered as an attack.
native SetProjectileDamageAttack takes projectile whichProj, boolean toAttack returns nothing
native SetProjectileOwner takes projectile whichProj, unit whichUnit returns nothing
native SetProjectileArc takes projectile whichProj, real arc returns nothing

// On spell-events
constant unitevent EVENT_UNIT_SPELL_MISS

// Model related natives
// As I am not really verbose with models, these suggestions may be improved upon.

// Gets the unit's model
type geoset extends agent
type rendermodelpart extends agent

type rendermodel extends handle

native GetUnitModel takes unit whichUnit returns string
native GetUnitTextureModel takes unit whichUnit, geoset whichGeoset returns string
native GetUnitRenderModel takes unit whichUnit returns rendermodel

native SetUnitModel takes unit whichUnit, string model returns nothing
native SetUnitModelTexture takes unit whichUnit, geoset whichGeoset, string texturePath returns nothing

native GetDestructableRenderModel takes destructable whichDest returns rendermodel
native GetItemRenderModel takes destructable whichDest returns rendermodel

// Allows one to create a new animation based on these info.
native RenderModelWriteStart takes rendermodel whichModel returns nothing
// Clears all in-game generated animations from the write list, preventing them from being written upon a RenderModelWrite call.
native RenderModelClear takes rendermodel whichModel returns nothing
// Saves the changes made in-game to the model. 
native RenderModelWrite takes rendermodel whichModel returns nothing

native GetRenderModelTime takes rendermodel whichModel returns real
native SetRenderModelTimer takes rendermodel whichModel, real time returns nothing

// Gets a part, object or event based on the model's info
native GetRenderModelPart takes rendermodel whichModel, integer index returns rendermodelpart
// Localized only to the coordinates of the model
native GetRenderModelPartX takes rendermodel whichModel, rendermodelpart whichPart returns real
native GetRenderModelPartY takes rendermodel whichModel, rendermodelpart whichPart returns real
native GetRenderModelPartZ takes rendermodel whichModel, rendermodelpart whichPart returns real

native SetRenderModelPartX takes rendermodel whichModel, rendermodelpart whichPart, real x returns nothing
native SetRenderModelPartY takes rendermodel whichModel, rendermodelpart whichPart, real y returns nothing
native SetRenderModelPartZ takes rendermodel whichModel, rendermodelpart whichPart, real z returns nothing

 

[Hannibal]

Increasing the Maximum Forces in Scenario Properties to 12 (up from 6) would be helpful and in line with the 24 player increase. Previously you could make 6 teams of 2 but to make 12 teams of 2 would require triggers.

Force 12 ftw!

 

[Napoqe]

Improvement in Resistance Skin ability, to be able to control via World Editor the abilities that will become immune and reduce duration of debuffs.
Better manipulation of skills directly from Object Editor (ignoring Gameplay Constants):

Defend
Ensnare
Magic Immunes
The skills can deactivate
Can change slow values of the Frost Attack, and also can add option to slow the attack speed.

 

[yxq1122]

Add missing natives for Stock UI (very urgent):

// Get the id of an item in stock by coordinates.
native GetItemInStock takes unit whichstock, integer x, integer y returns integer
// Get the x coordinate of an item in stock by id.
native GetItemXInStock takes unit whichstock, integer itemid returns integer
// Get the y coordinate of an item in stock by id.
native GetItemYInStock takes unit whichstock, integer itemid returns integer
// Change the item at the specific coordinate of stock.
native SetItemInStock takes unit whichstock, integer itemid, integer x, integer y returns nothing
// Swap the postions of items in stock.
native SwapItemInStock unit whichstock, integer x1, integer y1, integer x2, integer y2 returns nothing
// Get the icon path of an item in stock.
native GetItemIconInStock unit whichstock, integer x, integer y returns string
// Change the icon of an item in stock and returns the old icon.
native SetItemIconInStock unit whichstock, integer x, integer y, string blpname returns string

I also have a post in the Producer update: natives list thread.

 

[wrsuperboy]

native BlzUnitHideAbility                          takes unit whichUnit, integer abilId, boolean flag returns nothing
native BlzUnitDisableAbility                       takes unit whichUnit, integer abilId, boolean flag, boolean hideUI returns nothing

Please make these two not base on count!!

Based on count:

count=1
UnitHideAbility(.,.,true)-> count--
UnitHideAbility(.,.,false)-> count++
if(count==0) unit hides its ability
else unit reveals its ability​

A earth man's thinking:

flag=true
UnitHideAbility(.,.,true)-> flag=false
UnitHideAbility(.,.,false)-> flag=true
if(flag==false) unit hides its ability
else unit reveals its ability​

 

[Wrda]

I can't comprehend what in the world you're talking about, neither it makes sense.
If you're talking about using countdown timer for this, then use it since it exists already? Instead of asking to add a timer inside the native.

 

[MyPad]

@Wrda,

WeirdEditor meant that the natives should work as most would interpret it, a setter function, instead of a counter-based incremental function.

 

[wrsuperboy]

I can't comprehend what in the world you're talking about, neither it makes sense.
If you're talking about using countdown timer for this, then use it since it exists already? Instead of asking to add a timer inside the native.

What MyPad understood is what I wanted to say. They should be a simply setter funcitons, like SetPlayAbilityAvailble.

UnitAddAbility(hero,'A000',1)
BlzUnitHideAbility(hero,'A000',true)
BlzUnitHideAbility(hero,'A000',true)
BlzUnitHideAbility(hero,'A000',false)//this will not reveal the unit's ability, unless you do it again

UnitAddAbility(hero,'A000',1)
BlzUnitHideAbility(hero,'A000',false)
BlzUnitHideAbility(hero,'A000',true)//this will not hide the unit's ability,unless you do it again

Ridiculous?

 

[wrsuperboy]

Why not integrate BlzSetUnitMaxHP and BlzSetUnitMaxMana into SetUnitState?

 

[Devalut]

Projectile handling so that us lower serfs of the GUI world can make fancy projectiles abilities that for instance STOP projectiles like a barrier

Here's some for thought:

Spoiler

- Create Projectile at Point using X model, source equal to (Triggering Unit)
- Modify X projectile Projectile Damage Type
- Modify X projectile Speed
- Modify X projectile Model
- Destroy X projectile
- Remove X projectile
- Modify X projectile Damage? - ( i'm not sure if damage is stored in the projectile, but this would be neat)
- Modify X projectile Animation
- Modify X Projectile Size
- Pause X projectile
- Change X Projectiles target to X unit / Point
- Point - Position of (Last Created Projectile)
- Projectile Group (Projectiles in X range)
- Set Expiration for X Projectile
- (Last Created Projectile)
- (Last Projectile Launched) - What it says on the tin
- (Source of X projectile) - refer's to source of projectile
- (Projectile source is Spell) Boolean? and it's counter part (Physical attack)
- (Next Projectile) of (Triggering Unit) - What it says on the tin
- (Speed of X Projectile) - What it says on the tin

 

[Vinz]

"Nudge Mode" for terrain editor to use arrow keys for moving the selection, with an option for nudge strength to be relative to zoom level.

Force 12 ftw!

Force 24, please. :]

 

[deepstrasz]

Not sure if this thread is only for triggers and stuff but I'm going to write that the Bronze Dragon Roost has the icon of the Red Dragon Roost.
There is not unique Nether Dragon Roost. It's the one for the Black Dragon, icon and model (soundset too?).

---

EDIT: also, is there no trigger to set the stock delay and replenish of mercenaries to a value?

---

EDIT 2: the Orc and Undead Transport ships are the same.

 

[Wrda]

Force 24, please. :]

It's possible already?

 

[Karifean]

I'd love being able to change whether an ability is physical, magical or universal in the object editor, and if possible, having a field for Gold/Wood Cost of using it on ALL abilities not just the Goblin Shop's Reveal.

 

[Napoqe]

Some things you could do is to have Warcraft 3 support various levels of floors. So, I could do my Medieval Resident Evil better. I did little similar to 1, but with medievel things instead of modern things.
Ah with that implementation, you could add the camera navigation with W, S, A and D. For easy access to specific places. In Blender I use this, to see if it's correctly aligned the model.

 

[MyPad]

Adding native support for calling functions below a line would be quite helpful. (Implemented directly by the JASS Virtual Machine)

In turn, a new annotation above a function might help out those gruesome tasks that may not reach completion without external influence.

Demo:

function foo takes nothing returns nothing
    call bar()
endfunction

//# nooplimit
function bar takes npthing returns nothing
    local integer array I
    loop
        exitwhen I[0] >= 1048576
        set I[0] += 1
    endloop
endfunction

 

[wrsuperboy]

Adding native support for calling functions below a line would be quite helpful. (Implemented directly by the JASS Virtual Machine)

In turn, a new annotation above a function might help out those gruesome tasks that may not reach completion without external influence.

Demo:

function foo takes nothing returns nothing
    call bar()
endfunction

//# nooplimit
function bar takes npthing returns nothing
    local integer array I
    loop
        exitwhen I[0] >= 1048576
        set I[0] += 1
    endloop
endfunction

ExecuteFunc("bar") is not that slow, I think.

 

[MyPad]

ExecuteFunc("bar") is not that slow, I think.

Perhaps, if looking at most practical applications thereof (GUI initialization, for example)

Reason behind suggestion is to circumvent the comparatively more costly way of emulating such a behavior, such as a forced trigger evaluation/execution or a force enumeration.

The no-oplimit notation will effectively prevent repeated virtual thread generation for a repetitive task that is anticipated to cost that much, by bypassing the op-limit with a drawback, it only applies to the function below itself.

 

[deepstrasz]

Not a bug or anything. However, spells like Thunder Clap have a wider aggro area of effect than the zone the spell acts upon. Neutral hostile/enemy units sense Thunder Clap from a distance even if they were not hit by the spell. It's annoying in many maps. I'm not sure if this can be changed in the gameplay constants or not or if it is hard-coded.

 

[MyPad]

A custom autocast attack modifier would be of interest, with the ability to define stacking behavior with other abilities via bitsets, just like the dummy channel ability.

 

[Devalut]

Custom Fonts!
Turning (error squares) ‍ into for neat floating text options or ability descriptions:

Spoiler

- (Health Remaining)
‍ 5% Execute chance
-10 Frost Damage
️ - 14 Lightning Damage
50% crit vs ranged units
10 defense when facing - jazz musicians
-36 Arcane Damage
️ - Rooted
S͔̱̙e̹̘̺̞͚r̟̜͈̯͚̲̭̀io̗̫͝ͅư͈̯̟s̭͈l̴̫y̳̼̗̬̯̥ ͔̮̥͙̳̳͜ͅC҉̙͙͚r̝e҉̫e̡͈͇̜p̗̭̰y̞̥͇͈͕͠ ͈̟̖̜͔s̯̘̲̝t͡u̯ͅͅf̧͍̫͎̬f͕̫̣̤͎̭͞

 

[Wrda]

Everytime I see a map with custom fonts it makes me throw up, because the texts are ineligible...

 

[Devalut]

You're supposed to put filth like that in the toilet friend, not on a suggestion thread for new features~

Spoiler

thousand hours in mspaint

ETA:

Custom Text has a special place

Spoiler

 

[Wrda]

Epic picture. You just forgot one thing.
To put custom fonts in the trash

 

[Devalut]

Making use of extra post, see my previous post for the silly stuff:

Suggestion - Framework for reoccurring systems of popular maps
so that simpletons of the GUI need not consult filthy knife eared elves for lessons in moon runes:

This of course is just referring to framework, the bare minimum:

• Inventory system (Exceeding past the legendary 6 of course.)
• Skill Tree system
• Max unit's selected
• Maximum number of floating texts
• Knockback

If there's a chance to add something from the current times that appears in something as new as Diablo 3 we can make this game last even longer!

Imagine Warcraft 3 Reforged: REANIMATED in 20 years time.

 

[deepstrasz]

Items like Ancient Janggo of Endurance and others should mention the values they give.

 

[Warseeker]

• Fix the Spellbook Shared Cooldown : As it is now the Shared Cooldown works only for Level 1 of the ability.
• AI Building Upgrades : The AI doesn't recognize custom upgraded buildings. For example if I upgrade my custom townhall to custom keep, the AI will build an extra townhall. I know there is a workaround for this nevertheless.
• I don't know if this was already reported/fixed or not but the Pan Camera as necessary thing is known to cause desync in multiplayer.

As for the natives. I would like to request this :

native GetUnitColor takes unit whichunit returns playercolor

 

[Tommi Gustafsson]

A Trigger Action to set the order of hero portrait icons, so that we could control which hero is first (F1), which one second (F2), and so on.

 

[deepstrasz]

Hey, so, I don't know if it was fixed in newer patches but you cannot make a building only be constructed on shallow water no matter what combination(s) you choose in the Placement Prevented by and Placement Requires Object Editor fields for said unit. All I can do is either make it build only on Blight, any type of water (deep and shallow), buildable (non-water) or on all (non-blight).

By the way, a manner to make Blight be placed on shallow water?

EDIT:
Give us an option to edit the targets for the Build abilities (they don't even appear in the Object Editor). For instance I'm trying to place a Haunted Gold Mine on a flying non-moving Gold Mine. Why? Because I placed the mine on water and without it flying the height field isn't considered and the mine remains underwater.
But imagine maybe I'd want to make some sort of air workers and air mines.
It works with Float though.

EDIT2:
More fields on some abilities like Return resources. For instance the Targets don't do anything. I tried to exclude organic units from bringing gold.

EDIT3:
Would be useful for a haunted mine build ability that would result in a building of the mapper's choice, not necessarily for it to be placed on a mine and for it to be a mine.