View Issue Details

IDProjectCategoryView StatusLast Update
0002456Spring engineLuapublic2011-06-21 21:21
Reporterabma Assigned Toabma  
PrioritynormalSeveritymajorReproducibilityalways
Status resolvedResolutionduplicate 
Product Version0.82.7+git 
Target Version83.0 
Summary0002456: (master) morphing doesn't work
Description[f=0018822] LuaRules::RunCallIn: error = 2, GameFrame, [string "LuaRules/Gadgets/unit_morph.lua"]:521: 'for' limit must be a num



tested in Tech Annihilation + Spring 1944 + Zero-K

Zero-K:
LuaRules::RunCallIn: error = 2, GameFrame, [string "LuaRules/Gadgets/unit_morph.lua"]:602: 'for' limit must be a number
TagsNo tags attached.
Checked infolog.txt for Errors

Relationships

duplicate of 0002467 resolvedabma Remove remaining .n's 

Activities

Kloot

2011-06-01 18:49

developer   ~0006699

caused by https://github.com/spring/spring/commit/10219eed4ee93096dd84733026be61e8f311d83d#diff-2 (note the removal of "hs_n.PushNumber(L, i)")

Since this was done for other Lua API functions as well, I assume the changes are intentional (so https://github.com/spring/spring/commit/bbe00286444af757a215e445f2cc888d64417a0f is probably unwanted) and *gets should be rewritten to use #table instead of table.n, but ask jK for confirmation.

abma

2011-06-12 04:51

administrator   ~0006730

see 0002467 for updates of this

abma

2011-06-21 21:21

administrator   ~0006798

Kloot, can you please review hoijuis commit?

i'm unsure about it because you added more stuff when reverting it:

https://github.com/spring/spring/commit/1874b58f97d7cfab84decdb4ff5f36a810fa9336#L4L1625

Issue History

Date Modified Username Field Change
2011-06-01 18:12 abma New Issue
2011-06-01 18:49 Kloot Note Added: 0006699
2011-06-01 18:50 Kloot Status new => feedback
2011-06-10 16:09 hoijui Target Version => 0.83.0
2011-06-12 04:51 abma Note Added: 0006730
2011-06-12 04:51 abma Relationship added duplicate of 0002467
2011-06-12 04:51 abma Duplicate ID 0 => 2467
2011-06-12 04:51 abma Status feedback => resolved
2011-06-12 04:51 abma Resolution open => duplicate
2011-06-12 04:51 abma Assigned To => abma
2011-06-21 21:21 abma Note Added: 0006798