Explv

Thanks for the confirmation, I'll take a look at it

I'm on my phone so the formatting is a bit weird. But it looks like at the bottom of your script you have two global variables bankBooth and fountainObject that store the results of calls to objects.closest If that is the case, that is what is breaking your script. The objects variable will not be initialised before onStart is called, so your script will break. You should move those variables to onStart or onLoop

Thanks, it could be an issue with saving / loading and will be resolved once I push my update

I have written a fix, I plan to push it to the SDN tonight Is this after loading a saved config? Or just a general issue with the GE task?

Seriously? You're going to call me a fool? Firstly, this is a FREE script, I do not make any money on it and nor does OSBot. I wrote this purely for the benefit of yourself and other community members. Secondly the script DOES work, however it makes use of the OSBot web walker. It doesn't guarantee that it can walk from everywhere in RuneScape because the web walker doesn't support every location in RuneScape. If you checked the log you would probably see a message like no path could be found. Thirdly, scripts can break over time, and require the script writer to fix something. Suggesting that posts made months / years ago should be deleted is ridiculous. Finally, when a script doesn't work you should tell the script writer in what way it does not work so that they can evaluate the situation and give you advice or fix their script. Seriously, you really should have more respect. I put my own time into this script for everyone elses benefit, and to call me a fool is pathetic, if anyone is a fool here, it's you.

Use configs. A config consists of an ID and a value. You can view them as they are updated by selecting the configs option in the OSBot settings. As you progress through a quest, an associated config's value will be updated indicating the progress. For example, I think for Tutorial Island the id is 281, so you get the progress using: int progress = getConfigs().get(281); This value will change as you progress, for example, when you start Tutorial Island it would be something like: 281 : 0 After you talk to the RuneScape Instructor it might be something like: 281 : 10 Etc. There is a small section on this in my Scripting 101 tutorial, the link for which is in my signature.

Sure I can add that

Added, thanks again

That's what the scripting help section is for...

Because with prefix x is incremented first then the result is assigned to y With postfix the value of x is assigned to y, then x is incremented

#187AB3 #CDCDCD #191919 #181818 #DDDDDD #535353

Sure I'll add another location

Thanks boss

You could try doing something like this if you just want to get a single potion: Optional<Item> staminaPotion = Arrays.stream(getBank().getItems()) .filter(item -> item != null && item.getName().startsWith("Stamina potion")) .min(Comparator.comparing(Item::getName)); This snippet will construct a Stream from the array of all items in the bank, filter it so that only the items which are not null and has a name beginning with "Stamina potion" remain, and then find the min of those using a Comparator to compare the Item names, this should return Stamina potion (1) first etc. Note: it returns an Optional<Item>, the Optional class can be found here You use it like so: if (staminaPotion.isPresent()) { Item pot = staminaPotion.get(); } Or to get the List of all stamina potions in the bank, sorted by dosage: List<Item> staminaPotion = Arrays.stream(getBank().getItems()) .filter(item -> item != null && item.getName().startsWith("Stamina potion")) .sorted(Comparator.comparing(Item::getName)) .collect(Collectors.toList()); This snippet is similar to the above, but instead of returning the min, it instead sorts the Stream using the same Comparator, and then collects the Items into a List

Detective IHB

Explv designer supreme

Change the sleep timeout to something longer, such as 10000 ms. There isn't much point using a random timeout like you are doing, and 1.5 seconds probably isn't long enough for the teleport to complete. You should also not just call web walk straight after the teleport. The "Break" interaction could fail, and then your bot will attempt to web walk from wherever your player is standing. What you should do is something like If player is in location where you are teleporting from: Teleport Else: Web walk to the bank Alternatively, like other users have suggested you could create a WebWalkEvent which should handle the teleporting for you. Something like: WebWalkEvent webWalkEvent = new WebWalkEvent(Banks.VARROCK_EAST); PathPreferenceProfile profile = new PathPreferenceProfile(); profile.checkInventoryForItems(true); profile.setAllowTeleports(true); webWalkEvent.setPathPreferenceProfile(profile); execute(webWalkEvent); Apologies for any syntax errors and the formatting, I'm on my phone

Don't you want to sleep until you are in Varrock square? I assume you are using a Varrock teleport. Currently you are sleeping until your player is not in Varrock square, which would mean it wouldn't sleep at all.

Perhaps you should try sleeping until the teleport has completed, for example sleeping until your player is in the destination area

Rekt lmao

Good job. I think we already have 6/7 SDN scripts that support Yews though (and more)

Probably some time this evening, if not tomorrow

Pushed a change, the script will be fixed when the SDN is next updated. Sorry for any inconvenience.

Hmm, ok, will take a look after work