fix: Execute loopcast only after setting the appropriate player data

Fixes #768

src/main/java/vazkii/psi/common/item/ItemLoopcastSpellBullet.java

@@ -10,7 +10,6 @@
 
 import net.minecraft.server.level.ServerPlayer;
 import net.minecraft.world.entity.Entity;
-import net.minecraft.world.item.Item.Properties;
 import net.minecraft.world.item.ItemStack;
 
 import vazkii.psi.api.spell.SpellContext;
@@ -29,12 +28,12 @@ public ItemLoopcastSpellBullet(Properties properties) {
 	public ArrayList<Entity> castSpell(ItemStack stack, SpellContext context) {
 		PlayerDataHandler.PlayerData data = PlayerDataHandler.get(context.caster);
 		if(!data.loopcasting || context.castFrom != data.loopcastHand) {
-			context.cspell.safeExecute(context);
 			data.loopcasting = true;
 			data.loopcastHand = context.castFrom;
 			data.lastTickLoopcastStack = null;
 			data.loopcastTime = 1;
 			data.loopcastAmount = 0;
+			context.cspell.safeExecute(context);
 			if(context.caster instanceof ServerPlayer) {
 				LoopcastTrackingHandler.syncForTrackersAndSelf((ServerPlayer) context.caster);
 			}
@@ -57,9 +56,4 @@ public String getBulletType() {
 	public boolean isCADOnlyContainer(ItemStack stack) {
 		return true;
 	}
-
-	@Override
-	public double getCostModifier(ItemStack stack) {
-		return 1.0;
-	}
 }