gfgtdf

Fixed the crash, we can keep this one open as a remainder to improve the ai, not sure

> I mean where are these displayed? Mainline has similar text, so I'm not sure why it would be a problem here. Iirc in the WoCopedia help and in the...

> `[modify_unit][modifications][effect]` This is wrong syntax it's is supposed to be `[modify_unit][object][effect]` ( or `[modify_unit][trait][effect]` of course)

> EI S7a does `[modify_unit][modifications][trait][effect]apply_to=loyal` I'm not 100% sure what this does, it probably does a config-merge on the [modifications] tag ( meging the apply_to loyal with the first tarit...

We can also just remove the blocker label. I think that even if now no longer draws over the window it probably still draws over it's area since there is...

> Yeah, that'll do a config merge on the modifications tag, but that does mean it should actually work, as the loyal trait is now in the modifications and [modify_unit]...

> Is there any release of 1.14.x that this worked in? I dont think so > This has been added to the 1.14.9 milestone, but it sounds like an API...

> As for the implementation of [modify_unit] in terms of store/unstore, I still believe that that should be changed. I actually did change that for the most part (like 90%...

maybe it should mention that the correct way is to directly use [object], [effect], [trait] in [modify_unit]. (assuming this is what one wants)

> Is that accurate? Yes > If so, then I don't think we need to do anything. We need to update the wrong wml code if it's still in mainline.