Jump to content

Recommended Posts

Posted

I am having a problem with items in Forge 9.10.0.804. When I create my new item using two old items, I recieve the new one fine, however as soon as I click it my inventory resets to it's previous state. The video explains it better:

[flash=640,360]https://youtube.googleapis.com/v/NXLulVE8x8k

Here is my code for swapping the items:

public void castSpellBlockFirst(ItemStack stack,EntityPlayer player,World world,int x,int y,int z,int side,float hitX,float hitY,float hitZ) {
	if(world.getBlockId(x,y,z)==Items.block_spelltable.blockID&&side==1){
		if(player.inventory.getStackInSlot(0)!=null){
			if(player.inventory.getStackInSlot(0).itemID!=Items.item_apprenticewand.itemID){
				/* Here's the items that can turn into spells */
				switch(player.inventory.getStackInSlot(0).itemID){
				case 370:
					player.inventory.consumeInventoryItem(Items.item_apprenticewand.itemID);
					player.inventory.consumeInventoryItem(Item.ghastTear.itemID);
					player.inventory.addItemStackToInventory(new ItemStack(Items.item_apprenticewand_soul, 1, 0));
					break;
				default:
					player.addChatMessage("§cNo available items found in first slot");
					break;
				}
			}else{
				player.addChatMessage("§cWands cannot be transmuted");
			}
		}else{
			player.addChatMessage("§cNo available items found in first slot");
		}
	}
}

 

Any help would be appreciated. If you need any information, just post or PM.

Aspergers is annoying sometimes :(

Posted

you're not doing the change server side

 

if(!world.isRemote){

//activate effect

}

how to debug 101:http://www.minecraftforge.net/wiki/Debug_101

-hydroflame, author of the forge revolution-

Posted
  On 8/9/2013 at 1:44 PM, hydroflame said:

you're not doing the change server side

 

if(!world.isRemote){

//activate effect

}

 

I thought that was the problem, but ignored it because in another spell I used it and thought I used it in my base function. Thanks!

 

EDIT: Now nothing happens when I right click, and I have a similar problem with my Soul Spell.

 

	public void castSpellBlockFirst(ItemStack stack,EntityPlayer player,World world,int x,int y,int z,int side,float hitX,float hitY,float hitZ) {
	if(!world.isRemote&&world.getBlockId(x,y,z)==Items.block_spelltable.blockID&&side==1){
		if(player.inventory.getStackInSlot(0)!=null){
			if(player.inventory.getStackInSlot(0).itemID!=Items.item_apprenticewand.itemID){
				/* Here's the items that can turn into spells */
				switch(player.inventory.getStackInSlot(0).itemID){
				case 370:
					player.inventory.consumeInventoryItem(Items.item_apprenticewand.itemID);
					player.inventory.consumeInventoryItem(Item.ghastTear.itemID);
					player.inventory.addItemStackToInventory(new ItemStack(Items.item_apprenticewand_soul, 1, 0));
					break;
				default:
					player.addChatMessage("§cNo available items found in first slot");
					break;
				}
			}else{
				player.addChatMessage("§cWands cannot be transmuted");
			}
		}else{
			player.addChatMessage("§cNo available items found in first slot");
		}
	}
}

Aspergers is annoying sometimes :(

Posted

Here's four classes: ItemWand, Spell, NullSpell (the one I'm referencing in my first post) and SoulSpell:

ItemWand.java:

 

  Reveal hidden contents

 

 

Spell.java:

 

  Reveal hidden contents

 

 

NullSpell.ava:

 

  Reveal hidden contents

 

 

SoulSpell.java

 

 

  Reveal hidden contents

 

 

Summary: There are events in ItemWand. In spell, there are functions such as castSpellBlockFirstEvent. These do a few checks, then call castSpellBlockFirst. The event functions are called in ItemWand, and the non-event functions are overriden in the new spell.

 

Aspergers is annoying sometimes :(

Posted

1 just letting you know

onItemFirstUse is called every time a player start usign an item

 

2 in your ItemWand class you have a int cooldown

minecraft is not making a new item for each wand available in the world, so by putting this variable there everybody in the world will have the SAME cooldown

 

  Quote
Now nothing happens when I right click

can you make a bunch of println and try to see which condition is failing ?

how to debug 101:http://www.minecraftforge.net/wiki/Debug_101

-hydroflame, author of the forge revolution-

Posted

	public void castSpellBlockFirst(ItemStack stack,EntityPlayer player,World world,int x,int y,int z,int side,float hitX,float hitY,float hitZ) {
	System.out.println("1");
	if(!world.isRemote&&world.getBlockId(x,y,z)==Items.block_spelltable.blockID&&side==1){
		System.out.println("2");
		if(player.inventory.getStackInSlot(0)!=null){
			System.out.println("3");
			if(player.inventory.getStackInSlot(0).itemID!=Items.item_apprenticewand.itemID){
				/* Here's the items that can turn into spells */
				System.out.println("4");
				switch(player.inventory.getStackInSlot(0).itemID){
				case 370:
					System.out.println("5");
					player.inventory.consumeInventoryItem(Items.item_apprenticewand.itemID);
					player.inventory.consumeInventoryItem(Item.ghastTear.itemID);
					player.inventory.addItemStackToInventory(new ItemStack(Items.item_apprenticewand_soul, 1, 0));
					break;
				default:
					System.out.println("6");
					player.addChatMessage("§cNo available items found in first slot");
					break;
				}
				System.out.println("7");
			}else{
				System.out.println("8");
				player.addChatMessage("§cWands cannot be transmuted");
			}
		}else{
			System.out.println("9");
			player.addChatMessage("§cNo available items found in first slot");
		}
		System.out.println("10");
	}
	System.out.println("11");
}

 

2013-08-09 15:10:12 [iNFO] [sTDOUT] 1
2013-08-09 15:10:12 [iNFO] [sTDOUT] 11

Aspergers is annoying sometimes :(

Posted

Spell:

public boolean castSpellBlockFirstEvent(ItemStack stack,EntityPlayer player,World world,int x,int y,int z,int side,float hitX,float hitY,float hitZ){
if(FMLCommonHandler.instance().getEffectiveSide.isServer()){
System.out.println("server side thinks: "+castBlockFirst);
}else{
System.out.println("client side thinks: "+castBlockFirst);
}
	if(!castBlockFirst) return false;
	if(stack.getMaxDamage()-stack.getItemDamage()>=castChargeRequired){
		stack.damageItem(castChargeRequired,player);
		castSpellBlockFirst(stack,player,world,x,y,z,side,hitX,hitY,hitZ);
	}
	return true;
}

 

 

whats the output of thsi ?

how to debug 101:http://www.minecraftforge.net/wiki/Debug_101

-hydroflame, author of the forge revolution-

Posted

Description Resource Path Location Type

getEffectiveSide cannot be resolved or is not a field Spell.java /Minecraft/src/lemmy/eldercraft line 49 Java Problem

 

Edit: Added parenthasis to the end of getEffectiveSide, testing now.

 

2013-08-09 15:19:19 [iNFO] [sTDOUT] client side thinks: true

Aspergers is annoying sometimes :(

Posted

server side thinks what ? nothing ? hmmm

 

 

try the same print in onItemFirstUse

 

if nothing appears there, that mean onItemFirstUse is never called server side :\

how to debug 101:http://www.minecraftforge.net/wiki/Debug_101

-hydroflame, author of the forge revolution-

Posted

After adding castAir=true; to the constructor of NullItem, and adding your code to castAirEvent, then finally right clicking towards the sky, I just got 2013-08-09 15:25:29 [iNFO] [sTDOUT] client side thinks: true. No mention of serverside.

Aspergers is annoying sometimes :(

Posted
  Quote
if nothing appears there, that mean onItemFirstUse is never called server side :\

 

youll have to use something else then :)

how to debug 101:http://www.minecraftforge.net/wiki/Debug_101

-hydroflame, author of the forge revolution-

Posted

*reasearch a bit more in the issue*

*suspicious look*

*wtfbbq*

soooo onItemUseFirst is actually called by a method in ItemInWorldManager which is a class that is mainly used server side ... *suspicious look x2*

 

honestly you should start by changing the cooldown and spell reference to the nbt because its going to cause a butlaod of problem anyway later. and it will probably fix the current one too

how to debug 101:http://www.minecraftforge.net/wiki/Debug_101

-hydroflame, author of the forge revolution-

Posted

It's not too much of a problem, I can simply use onBlockActivated on the Spell Table block. Unfortunately, I cannot use these functions for other spells

 

Aspergers is annoying sometimes :(

Posted

which is why you should start by changing spell and cooldown to the nbt, everything will work wonderfully after that

how to debug 101:http://www.minecraftforge.net/wiki/Debug_101

-hydroflame, author of the forge revolution-

Posted
  On 8/9/2013 at 2:38 PM, hydroflame said:

which is why you should start by changing spell and cooldown to the nbt, everything will work wonderfully after that

 

Spell should be an int, right? I only really need the object instance for the variables, nothing is stored in it really.

Edit: How would I go about storing NBT in an item?

 

Aspergers is annoying sometimes :(

Posted

well spell cannot be an int becasue you cant add method to int

but you COUDL store the spellID and whenever someone cast a wand it would check in the SpellLibrary (that you have to create btw) for which spell he is actually casting

 

pseudo code:

public class SpellLibrary{
    public static Spell[] allSpells;

    public static void populateLibrary(){
        allSpells = new Spell[10];
        allSpells[0] = new SpellWtv();
        allSpells[1] = new SpellEtc();
        allSpells[2] = new SpellMore();

    }
}

onItemUse(){
int spellid = getSpellId();
Spell aboutToCast = SpellLibrary.allSpells[spellid];
aboutToCast.cast();
}

how to debug 101:http://www.minecraftforge.net/wiki/Debug_101

-hydroflame, author of the forge revolution-

Posted

not in the item but in the itemStack

 

ItemStack.java(line ~51):

/**
     * A NBTTagMap containing data about an ItemStack. Can only be used for non stackable items
     */
    public NBTTagCompound stackTagCompound;

 

conviniently, the itemStack used in "onItemUseFirst" and similar method is provided by the caller

 

how to debug 101:http://www.minecraftforge.net/wiki/Debug_101

-hydroflame, author of the forge revolution-

Posted

pretty much, just make sure to create a new NBTTagCompound if its null at this point

how to debug 101:http://www.minecraftforge.net/wiki/Debug_101

-hydroflame, author of the forge revolution-

Posted
  On 8/9/2013 at 2:49 PM, hydroflame said:

pretty much, just make sure to create a new NBTTagCompound if its null at this point

 

So far, it works like a charm! Thanks!

Aspergers is annoying sometimes :(

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.