[slashnhax]

Before, if you tried to interact with an item in your inventory and it wasn't open, it would open the backpack, but recently this doesn't seem to happen.

Is this intended or did something change?

 

[cloud]

Something likely broke.

 

[slashnhax]

This still occurs @Cloud

 

[cloud]

Is it just the inventory or does this also happen for equiped items and such?

 

[slashnhax]

Is it just the inventory or does this also happen for equiped items and such?

I've only tested it with the inventory, I'll try equipped items in a sec.

 

[aidden]

Is it just the inventory or does this also happen for equiped items and such?

Did you end up fixing the inventory opening that i told you about a while back? It wouldn't hover the ribbon to get to the inventory component, but if you hover it yourself it opens the inventory fine

 

[cloud]

Did you end up fixing the inventory opening that i told you about a while back? It wouldn't hover the ribbon to get to the inventory component, but if you hover it yourself it opens the inventory fine

No, it's likely the same issue.
 
Bump guys? @SlashnHax

 

[draggin]

@Cloud Please find this and fix this as it is the only bug in Alpha Fighter.
THE ONLY BUG.

Trying to alch something with the equipment tab open for 9 hours is not good.

 

[slashnhax]

No, it's likely the same issue.
 
Bump guys? @SlashnHax

My bad, yeah, it's the same issue for equipped items (tested with Ring of Duelling).

I tried hovering over the ribbon, like @Aidden said, and then it seemed to be able to open the equipment tab by itself.

@Cloud Please find this and fix this as it is the only bug in Alpha Fighter.
THE ONLY BUG.

Trying to alch something with the equipment tab open for 9 hours is not good.

I've added explicit inventory opening for next update due to this bug xD

 

[draggin]

My bad, yeah, it's the same issue for equipped items (tested with Ring of Duelling).

I tried hovering over the ribbon, like @Aidden said, and then it seemed to be able to open the equipment tab by itself.

I've added explicit inventory opening for next update due to this bug xD

You the man

 

[baddest man on earth]

For anyone who want.

/**
* Interacts with a SpriteItem. Only works if item is in inventory or
* equipped.
*
* @param item   The item to interact with
* @param action The action to interact
* @return Returns true if interaction was successful
*/
public static boolean item(SpriteItem item, String action) {
    if (item != null && item.isValid()) {
        Openable tab;
        SpriteItem.Origin origin = item.getOrigin();
        if (origin.equals(SpriteItem.Origin.INVENTORY)) {
            tab = InterfaceWindows.getInventory();
        } else if (origin.equals(SpriteItem.Origin.EQUIPMENT)) {
            tab = InterfaceWindows.getEquipment();
        } else {
            return false;
        }
        if (tab.isOpen()) {
            return item.interact(action);
        } else {
            if (tab.open()) {
                final Openable t = tab;
                Execution.delayUntil(t::isOpen, Random.nextInt(1500, 3000));
            }
        }
    }
    return false;
}

 

[cloud]

Under what conditions is the ribbon visible? Can someone loan me an account that has that?

 

[aidden]

Under what conditions is the ribbon visible? Can someone loan me an account that has that?

The ribbons always visible afaik. But it doesn't know to hover the icon on the ribbon that then gives you access to the inventory button

 

[cloud]

The ribbons always visible afaik. But it doesn't know to hover the icon on the ribbon that then gives you access to the inventory button

I borrowed an account and it wasn't there, hence why I was asking.

 

[aidden]

I borrowed an account and it wasn't there, hence why I was asking.

DM me on slack and i'll show you

 

[slashnhax]

For anyone who want.

/**
* Interacts with a SpriteItem. Only works if item is in inventory or
* equipped.
*
* @param item   The item to interact with
* @param action The action to interact
* @return Returns true if interaction was successful
*/
public static boolean item(SpriteItem item, String action) {
    if (item != null && item.isValid()) {
        Openable tab;
        SpriteItem.Origin origin = item.getOrigin();
        if (origin.equals(SpriteItem.Origin.INVENTORY)) {
            tab = InterfaceWindows.getInventory();
        } else if (origin.equals(SpriteItem.Origin.EQUIPMENT)) {
            tab = InterfaceWindows.getEquipment();
        } else {
            return false;
        }
        if (tab.isOpen()) {
            return item.interact(action);
        } else {
            if (tab.open()) {
                final Openable t = tab;
                Execution.delayUntil(t::isOpen, Random.nextInt(1500, 3000));
            }
        }
    }
    return false;
}

That name though, wouldn't interact have been a better choice? xD

 

[baddest man on earth]

That name though, wouldn't interact have been a better choice? xD

The class name is Interact...

 

[slashnhax]

The class name is Interact...

Strange, normally you'd name the class a noun, not a verb... Like Interaction or something Just nitpicking, but I thought conventions was one of your pet peeves too?

 

[baddest man on earth]

Strange, normally you'd name the class a noun, not a verb... Like Interaction or something Just nitpicking, but I thought conventions was one of your pet peeves too?

Are we really going to talk about this?

 

[aidden]

Strange, normally you'd name the class a noun, not a verb... Like Interaction or something Just nitpicking, but I thought conventions was one of your pet peeves too?

I'm assuming he's done this because then he can have different interact methods for different types of object in one class, rather than creating an Item class, and Npc class etc with separate interact methods (hence the method being called item). It's a bit backwards, but saves making multiple classes just to have one method in them.