Hello, I've been looking around this community for a few days and I decided to make my first script to learn more about programming and runescape scripting more specifically.

The script should be really simple: tanning cowhides into hard leather

I've been following @Explv guide which can be found here: 

I'm stuck at one part, though.

Whenever I trade the NPC (Ellis), the following screen will pop up: 

 

If I right-click Hard Leather:

I'm not so sure how to tan all the Hard Leathers. Part of my code, which was based in an example found in the tutorial I mentioned earlier, is the following:

private boolean canMakeLeather(){
        return getInventory().contains("Cowhide");
    }

    private boolean tradeNPC(){
        return getNpcs().closest("Ellis").interact("Trade");

    }

    private boolean isTanningScreenVisible(){
        return getWidgets().getWidgetContainingText("What hides would you like tanning") != null;
    }

    private boolean tanningHides(){
        return getWidgets().getWidgetContainingText("Hard Leather").interact("Tan all");
    }

 

The example script code (smelting iron bars):

private boolean canSmeltIronBars() {
        return getInventory().contains("Iron ore");
    }

    private boolean useFurnace() {
        return getObjects().closest("Furnace").interact("Smelt");
    }

    private boolean isSmeltScreenVisible() {
        return getWidgets().getWidgetContainingText("What would you like to smelt?") != null;
    }

    private boolean smeltXIron() {
        return getWidgets().getWidgetContainingText("Iron").interact("Smelt X Iron");
    }

 

I'm just wondering if there is any mistake in the code or anything I could improve.

Obviously not the full script code, I just didn't really understand how to work with widgets.

Edited April 16, 20187 yr by RagBoys III

I have a guide relating to configs & widgets if you would like to take a look at it

 

@RagBoys III Looks like the widget text and interaction text are incorrect, it should be "Hard leather" not "Hard Leather", and "Tan All" not "Tan all", the case matters.

Edited April 16, 20187 yr by Explv