Shop System Issue

#1 have you declared the itemlist array?

#2 Yes I'm sure I have. I'll post my full code just to make sure.

moveMap.bindCmd(keyboard, "insert", Shop @ ".toggleMenu();", "");

$STIMPACKS = 0;
$WEAPONS = 1;
$ACCESSORIES = 2;

$Stim[$STIMPACKS,0] = "Smallstimpack";
$Stim[$STIMPACKS,1] = "Mediumstimpack";
$Stim[$STIMPACKS,2] = "Largestimpack";

function Shop::toggleMenu(%this)
{
	if(!%this.isAwake())
	{
		Canvas.pushDialog(%this);
	}
	else if(!ShopItems.isAwake())
	{
		Canvas.popDialog(%this);
	}
}

function Shop::buy(%this)
{
	if(!ShopItems.isAwake())
	{
		ShopItems.initialize();
		ShopItems.PopulateList();
		Canvas.pushDialog(ShopItems);
	}
	
	textBuy.visible = true;
	textSell.visible = false;
}

function Shop::sell(%this)
{
	if(!ShopItems.isAwake())
	{
		ShopItems.initialize();
		
		Canvas.pushDialog(ShopItems);
	}
	
	textBuy.visible = false;
	textSell.visible = true;
}

function ShopItems::initialize(%this)
{
	%dialogBox = ShopItems.findObjectByInternalName("ShopItemWin");
	
	%Items = %this.findObjectByInternalName("Items");
	%totaltext = %this.findObjectByInternalName("totaltext");
	%totalitemtext = %this.findObjectByInternalName("totalitemtext");
	%amountbuy = %this.findObjectByInternalName("amountbuy");
	%amountsell = %this.findObjectByInternalName("amountsell");
	
}

function ShopItems::PopulateList(%this)
{
	ItemList.clearItems();
	
	for(%i = 0; %i < 3; %i++)
	{
		ItemList.addItem($Stim[$STIMPACKS,%i]);
		
		if(ItemList.getSelectedItems([$STIMPACKS,%i]))
		{
			buyAmount.setValue("10");
		}
	}
}

function ShopItems::SelectItems(%this)
{
}

function ShopItems::confirm(%this)
{
}

function Shop::exit(%this)
{
	Canvas.popDialog(%this);
}

function ShopItems::cancel(%this)
{
	Canvas.popDialog(%this);
}

#3 cut it down in to chunks outside of your functions, and see if its doing what it's meant to..

ItemList.clearItems();  
      
    for(%i = 0; %i < 3; %i++)  
    {  
        ItemList.addItem($Stim[$STIMPACKS,%i]);  
          
        if(ItemList.getSelectedItems([$STIMPACKS,%i]))  
        {  
            buyAmount.setValue("10");  
        }  
    }

then echo out ItemList.getSelectedItems([0,0]);

and see if you are populating it.

#4 Well the item list is being populated with items. What I mean is that when you select an item from the list, the value it supposed to display in the buy text section dosen't show.

#5 after:

buyAmount.setValue("10");

add:

buyAmount.echo();

What do you get in the console?

#6 Also with

setValue() - (int value, int index) Set the value of the array element at the submitted index

Don't you need a value, and an index number?

#7

function ShopItems::PopulateList(%this)
{
	ItemList.clearItems();
	
	for(%i = 0; %i < 3; %i++)
	{
		ItemList.addItem($Stim[$STIMPACKS,%i]);
		
		buyAmount.setValue("10",%i);
		
		buyAmount.echo();
	}
}

I get these errors in the console. Wrong number of parameters for setValue(), I also get the error, no return value.

#8 I'll have my dev environment setup again next week, so I'll take a look then. Unless someone else comes up with a solution for you.

#9 Thanks for your help.

#10 not sure but something I would try

function ShopItems::PopulateList(%this)
{
ItemList.clearItems();

for(%i = 0; %i < 3; %i++)
{
ItemList.addItem($Stim[$STIMPACKS,%i]);

buyAmount.setValue("10",%i);

buyAmount.echo();
}
return buyAmount;
}

and the
buyAmount.setValue("10",%i);
if you say it has the wrong amount of parameters then that means it should only have 1 value so your %i shouldn't be there.

Something like this
buyAmount.setValue("10");

#11 Thanks, but it dosen't work the way I want it too. I need it so that when the user clicks on an item in the list, it will display the buy price of the selected item.

#12 Maybe something like,

%Price = ItemList.getSelectedItems([$STIMPACKS,%i]

then in the gui have the text = %Price,

Sorry haven't got the torque engine yet still in demo.

#13 Well, if you are using a Text List Control, I think there is a method like so:

function someTextListCtrl::onSelect(%this,%a,%b,%c)
{
     //Echo what each of the arguments are because I don't know off hand.
}