Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Request]: Gold Recovery Features #175

Open
Rxann opened this issue Jul 24, 2024 · 14 comments
Open

[Request]: Gold Recovery Features #175

Rxann opened this issue Jul 24, 2024 · 14 comments
Labels
request A request for a feature in HorseMenu

Comments

@Rxann
Copy link
Contributor

Rxann commented Jul 24, 2024

Problem

We have the current method of adding Gold, but we could use transactions to make this process a little bit better.

Solution

Use transactions to award ourselves gold.

Reason

It would improve the recovery experience and make it more robust.

Additional context

@tuyilmaz Do you know anything about this?

@Rxann Rxann added the request A request for a feature in HorseMenu label Jul 24, 2024
@ghost

This comment has been minimized.

@tuyilmaz
Copy link
Contributor

Usage of NETSHOPPING natives in game scripts are really complicated(at least for me), but I'm sure it's either _CASHINVENTORY_TRANSACTION_FIRE_AND_FORGET_ITEM or one of the unknown ones that give awards. I'll look into this.

@AAA-ALR
Copy link
Contributor

AAA-ALR commented Jul 24, 2024

uh, I agree with imstash

You're free to contribute to YimMenu as long as the features are useful, not overly toxic and do not contain anything money related that might get the menu targeted by Take2.

Maybe we need to worry about take2 or how others view it

@Rxann
Copy link
Contributor Author

Rxann commented Jul 24, 2024

uh, I agree with imstash

You're free to contribute to YimMenu as long as the features are useful, not overly toxic and do not contain anything money related that might get the menu targeted by Take2.

Maybe we need to worry about take2 or how others view it

Recovery is allowed in HorseMenu. We make the users agree to a disclaimer saying they are aware there are potential side effects. Take2 has stopped caring about Red Dead Redemption 2 and Red Dead Online for a year or more. They stopped development on it.

@tuyilmaz
Copy link
Contributor

tuyilmaz commented Jul 25, 2024

It seems _CASHINVENTORY_TRANSACTION_FIRE_AND_FORGET_ITEM takes a struct with inventory item and some other vars. Maybe creating inventory GUID for currency_gold_bar with the quantity and calling the native like below will work.

int TransactionID = -1;  
NETSHOPPING::_CASHINVENTORY_TRANSACTION_FIRE_AND_FORGET_ITEM("UPDATE"_J, &TransactionID, &item, sizeof(item) / 8)

@Rxann Have you found anything?

@FIying-Scotsman
Copy link

FIying-Scotsman commented Jul 25, 2024

It seems _CASHINVENTORY_TRANSACTION_FIRE_AND_FORGET_ITEM takes a struct with inventory item and some other vars. Maybe creating inventory GUID for currency_gold_bar with the quantity and calling the native like below will work.

int TransactionID = -1;  
NETSHOPPING::_CASHINVENTORY_TRANSACTION_FIRE_AND_FORGET_ITEM("UPDATE"_J, &TransactionID, &item, sizeof(item) / 8)

@Rxann Have you found anything?

That looks very similar to the basket system from GTA Online (not surprising given RDO is built on GTAO's bare bones implementation). I'd be cautious using it given how easy it is for them to detect dodgy transactions in GTA using that equivalent.

uh, I agree with imstash

You're free to contribute to YimMenu as long as the features are useful, not overly toxic and do not contain anything money related that might get the menu targeted by Take2.

Maybe we need to worry about take2 or how others view it

Recovery is allowed in HorseMenu. We make the users agree to a disclaimer saying they are aware there are potential side effects. Take2 has stopped caring about Red Dead Redemption 2 and Red Dead Online for a year or more. They stopped development on it.

Hm I wouldn't hold out on that opinion. You could argue they 'forgot' about the PS2 era games and yet they still sent C&D's to RE3 and a bunch of mods porting the map into GTA V. The latter happened well before the Defective Edition was released.

Keep an open mind is all I'm saying, or port it into Lua and leave it as an 'optional extra' you can at least deny ownership over similar to how Torrent programs know their software is used for piracy, but because they don't advocate for it themselves - it's very difficult to say they're a piracy enabler when they can't tell users how to use their program.

@tuyilmaz
Copy link
Contributor

That looks very similar to the basket system from GTA Online (not surprising given RDO is built on GTAO's bare bones implementation). I'd be cautious using it given how easy it is for them to detect dodgy transactions in GTA using that equivalent.

It does look similiar, but unlike GTA this method does not take any transaction hashes afaik. This just updates the inventory. That's why I asked before trying.

@FIying-Scotsman
Copy link

FIying-Scotsman commented Jul 25, 2024

That looks very similar to the basket system from GTA Online (not surprising given RDO is built on GTAO's bare bones implementation). I'd be cautious using it given how easy it is for them to detect dodgy transactions in GTA using that equivalent.

It does look similiar, but unlike GTA this method does not take any transaction hashes afaik. This just updates the inventory. That's why I asked before trying.

That's a good point - looks like they replaced the category type with an action (i.e UPDATE) but removed the transaction hash and replaced it with an item instead unless those are in the struct you mentioned.

Would be interesting to see if they've still got a basket system in place as that would allow you to log 'legit' transactions and just replay them (similar to what is done in GTA when it comes to selling Hangar cargo or selling cars in a loop for example).

@tuyilmaz
Copy link
Contributor

Interesting. Would be interesting to see if they've still got a basket system in place as that would allow you to log 'legit' transactions and just replay them (similar to what is done in GTA when it comes to selling Hangar cargo or selling cars in a loop for example).

I think they have it, but with a different approach of adding items to basket.

@Rxann
Copy link
Contributor Author

Rxann commented Jul 26, 2024

Would be interesting to see if they've still got a basket system in place as that would allow you to log 'legit' transactions and just replay them (similar to what is done in GTA when it comes to selling Hangar cargo or selling cars in a loop for example).

I have heard that they do in fact have a basket system but as @tuyilmaz said it is a different approach of doing it.

@tuyilmaz
Copy link
Contributor

@Rxann maybe we can try hooking the basket functions and log the args as @FIying-Scotsman said?

@Rxann
Copy link
Contributor Author

Rxann commented Jul 28, 2024

@Rxann maybe we can try hooking the basket functions and log the args as @FIying-Scotsman said?

Yes, but unfortunately we might have to purchase something(maybe) to get a good log for it.

@tuyilmaz
Copy link
Contributor

Yes, but unfortunately we might have to purchase something(maybe) to get a good log for it.

I think completing a mission or selling items to fence will do the job.

@Rxann
Copy link
Contributor Author

Rxann commented Jul 28, 2024

Yes, but unfortunately we might have to purchase something(maybe) to get a good log for it.

I think completing a mission or selling items to fence will do the job.

Ah yes, you are right. I didn't even think of this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
request A request for a feature in HorseMenu
Projects
None yet
Development

No branches or pull requests

4 participants