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

Allow percent_sp feature for all 2k3 versions #1690

Merged
merged 1 commit into from Mar 12, 2019

Conversation

fmatthew5876
Copy link
Contributor

  • Fix for all versions of 2k3
  • Allow in 2k (if the chunk is present) as an EasyRPG extension

Fix: #1684

Vote: Do we feel unsafe enabling this chunk for 2k games in EasyRPG?

@carstene1ns
Copy link
Member

I guess if there is a game converted to 2k3, but shipped with RPG_RT from 2k, this breaks, since is ignored by RPG_RT. We have a workaround that prevents such things elsewhere, but not applicable here.
Chances are too minimal to care... (and likely other things are broken in battles too in this case)

@carstene1ns carstene1ns added this to the 0.6.1 milestone Mar 12, 2019
@fmatthew5876
Copy link
Contributor Author

fmatthew5876 commented Mar 12, 2019

In general, I'd like to be able to use these chunks in 2k games in the future.. But yes the situation you describe is potentially a break.

Still they'd have to convert the game, change sp cost to percentage style, and still keep the old RPG_RT to make this break.

Thats a pretty determined series of steps!

The alternative is Player::IsRPG2k3(), and I'm happy to go with that solution for now if we'd like to punt on the question of easyrpg extensions to the future.

@carstene1ns
Copy link
Member

I'd like to be able to use these chunks in 2k games in the future.

This means either using a different editor (e.g. setting easyrpg mode anyway) or hacking stuff in. So not really detectable.

I feel more safe with this guarded, but as we both said, chances are low.

* Fix for all versions of 2k3
@fmatthew5876 fmatthew5876 changed the title Allow percent_sp feature for all versions Allow percent_sp feature for all 2k3 versions Mar 12, 2019
@fmatthew5876
Copy link
Contributor Author

So I want these custom features for HH3, but I'm very far away from that. I've fixed this to use Player::IsRPG2k3() so we can just fix the bug for now.

Will revisit the customizations question at a later date with a more comprehensive plan. Maybe an ini setting for example?

@carstene1ns carstene1ns merged commit 9bc9715 into EasyRPG:master Mar 12, 2019
@fmatthew5876 fmatthew5876 deleted the percent_sp branch January 16, 2020 03:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

3 participants