Paper icon indicating copy to clipboard operation
Paper copied to clipboard

Better Stats API

Open Machine-Maker opened this issue 4 years ago • 10 comments

Still needs some javadocs

And if this is good, replacing uses of the bukkit statistic enum.

Machine-Maker avatar May 20 '21 08:05 Machine-Maker

Rebased for 1.17.1

Machine-Maker avatar Aug 19 '21 05:08 Machine-Maker

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Dec 04 '21 10:12 stale[bot]

Rebased for 1.18.2. Also changed the PlayerIncrementStatisticEvent to use the new api, forgot about that one.

Machine-Maker avatar Mar 28 '22 20:03 Machine-Maker

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar May 30 '22 21:05 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jul 30 '22 14:07 stale[bot]

Ok, think I finally finished integrating with upstream's new Criteria API. Unsure if we should deprecate Bukkit#getScoreboardCriteria. I really do not know why this object needed an Craft implementation, rather than just a conversion method. Maybe that's what my criteria API pr could do, just get rid of CraftCriteria entirely.

Machine-Maker avatar Aug 10 '22 19:08 Machine-Maker

Rebased for 1.19.4

Machine-Maker avatar Apr 01 '23 22:04 Machine-Maker

can we use your code generation stuff to generate the keys?

MiniDigger avatar Aug 21 '23 07:08 MiniDigger

can we use your code generation stuff to generate the keys?

Yes, absolutely. If we want to start using the code generation stuff, then we need to move away from putting all the constants in the same file as the actual type. So like in this case, CustomStatistics would be the holder class for all of them, while CustomStatistic would be in source. That way, the generation is just the keys, and doesn't have to include any methods.

Machine-Maker avatar Aug 21 '23 13:08 Machine-Maker

Rebased and updated for 1.20.4.

I refactored the API types into interfaces backed by server impls. Much more future proof doing that.

Machine-Maker avatar Dec 21 '23 07:12 Machine-Maker