-
Notifications
You must be signed in to change notification settings - Fork 102
Rocket refactoring + starting some features / fixes #331
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
Open
LeMayday
wants to merge
41
commits into
TeamGalacticraft:main
Choose a base branch
from
LeMayday:fix-rocket-bugs
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
db479b9
fix: remove rocket entity when damaged in survival
LeMayday 355c091
fix: rocket does not drop when broken
LeMayday 2634eaf
feature: player dismounts rocket close to the ground
LeMayday 09237df
refactor: improve onPlace readability
LeMayday fb17484
fix: rocket does not break when launchpad is broken
LeMayday f163dbd
fix: dropped rocket will copy NBT data rather than spawn default item
LeMayday ab0229d
Merge remote-tracking branch 'upstream/main' into fix-rocket-bugs
LeMayday ed914cb
Merge branch 'TeamGalacticraft:main' into fix-rocket-bugs
LeMayday 08f8f18
feat: certain projectiles bounce off rocket
LeMayday 2df9783
fix: rocket no longer hangs when exited
LeMayday d0697ab
refactor: moved GC vehicle-related files to new folder
LeMayday 4b2f0f4
refactor: vehicles inherit from GCVehicleEntity
LeMayday afe4ccb
fix: lander gets instantiated on the destination celestial body
LeMayday e5bb153
refactor: added RequiresFuel interface for vehicles
LeMayday 29cf630
refactor: added RideableVehicleEntity class for further subdivision o…
LeMayday 92e309f
refactor: moved fuel related methods out of rocket interface
LeMayday a312475
refactor: reorganized methods to have consistent order
LeMayday 1b57bd7
refactor: reorganized rocketentity code
LeMayday 5a046b1
refactor: added inventory-related methods to GCVehicleEntity
LeMayday 124031e
implemented IDE suggestions
LeMayday 4c1140d
refactor: AbstractLanderEntity extends GCPlayerRideableVehicleEntity
LeMayday d5648a8
refactor: added inventory interfaces to rocket interface
LeMayday bd558f0
refactor: changed rideable vehicle class to PlayerRideable interface
LeMayday 81c2373
refactor: removed assumption that all GCVehicleEntities will have vie…
LeMayday 5f39eb9
refactor: RequiresFuel now extends ScalableFuelLevel
LeMayday 4853665
refactor: created GCFueledVehicleEntity class
LeMayday c631d53
refactor: lander reorganized based on new super classes
LeMayday f0cdcdd
refactor: buggy reorganized based on new superclasses
LeMayday cafbb8b
refactor: rocket reorganized based on new superclasses
LeMayday beee0b9
fix: fixed compiler error where fuel data needed to be a long
LeMayday e3438ac
refactor: removed PlayerRideable interface
LeMayday c5eddd7
refactor: moved lerp back to Rocket
LeMayday 92bee8a
refactor: initialized inventory
LeMayday ea83fad
refactor: reorganized AbstractLanderEntity again
LeMayday 6d2cbb7
refactor: readded isPickable to rocket
LeMayday c3967cb
trying to make the rocket inventory screen work
LeMayday 7106237
fix: changed rocket inventory default key to G
LeMayday 4263491
style: made some changes based on IDEA suggestions
LeMayday 066d1ee
style: style changes
LeMayday b63f88a
refactor: simplified abstract lander entity interaction code
LeMayday 8dbef26
feat: messing around with rocket inventory
LeMayday File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
22 changes: 22 additions & 0 deletions
22
src/main/java/dev/galacticraft/api/rocket/entity/RequiresFuel.java
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,22 @@ | ||
| package dev.galacticraft.api.rocket.entity; | ||
|
|
||
| import dev.galacticraft.mod.content.entity.ScalableFuelLevel; | ||
| import net.fabricmc.fabric.api.transfer.v1.fluid.FluidVariant; | ||
| import net.fabricmc.fabric.api.transfer.v1.storage.Storage; | ||
| import net.minecraft.world.level.material.Fluid; | ||
| import org.jetbrains.annotations.Nullable; | ||
|
|
||
| public interface RequiresFuel extends ScalableFuelLevel { | ||
|
|
||
| @Nullable | ||
| Fluid getFuelTankFluid(); | ||
|
|
||
| long getFuelTankAmount(); | ||
|
|
||
| long getFuelTankCapacity(); | ||
|
|
||
| boolean isTankEmpty(); | ||
|
|
||
| Storage<FluidVariant> getFuelTank(); | ||
|
|
||
| } | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to add the license header at the top of each file.