Much better to use tag ids than text strings - then you don't have to worry that in someone's home game, cure moderate has been renamed "Touch of Sarenrae":
Code:
foreach pick in hero from BaseSpell where "thingid.spCureMod2 | thingid.spCureMod3 | all the other cure moderate spells out there"
@valid = 1
nexteach
Hint on #1 - do you see that code used only in scripts on one editor tab? Or on class specials used only by the things on that one tab?
Also, you say you're trying a script, but you don't say what phase and priority you're trying it at - that's always critical when you're trying to alter the way an existing thing works.