Replace of int's with ArtifactID and fix for 1392

Usage of constants should make it a bit easier to find some code related to catapult/spellbook specifics.

Now it's possible to sell spell scrolls:
http://bugs.vcmi.eu/view.php?id=1392
Yeah I know that, but in all functions there is break so I decide stick to this style.