-
-
Notifications
You must be signed in to change notification settings - Fork 0
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
GH-58 Add basic Parcel locker business logic #58
Conversation
…til, fix ParcelLockerPlaceController
Co-authored-by: DMK <81445555+imDMK@users.noreply.github.com>
…ageGUI.java Co-authored-by: DMK <81445555+imDMK@users.noreply.github.com>
…java Co-authored-by: DMK <81445555+imDMK@users.noreply.github.com>
….java Co-authored-by: DMK <81445555+imDMK@users.noreply.github.com>
@imDMK I changed the structure, as you requested, and applied your fixes. Let me know if there are any codestyle issues, as the Gradle Checkstyle plugin hasn't detected anything now. |
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.
- command.ParcelCommand -> parcel.command.ParcelCommand
- command.argument.ParcelArgument -> parcel.command.ParcelArgument
src/main/java/com/eternalcode/parcellockers/parcel/gui/ParcelListGUI.java
Outdated
Show resolved
Hide resolved
src/main/java/com/eternalcode/parcellockers/parcel/gui/ParcelListGUI.java
Outdated
Show resolved
Hide resolved
src/main/java/com/eternalcode/parcellockers/parcel/gui/ParcelListGUI.java
Outdated
Show resolved
Hide resolved
private boolean setupEconomy() { | ||
if (this.getServer().getPluginManager().getPlugin("Vault") == null) { | ||
return false; | ||
} | ||
|
||
RegisteredServiceProvider<Economy> rsp = this.getServer().getServicesManager().getRegistration(Economy.class); | ||
if (rsp == null) { | ||
return false; | ||
} | ||
|
||
this.economy = rsp.getProvider(); | ||
return true; | ||
} | ||
|
||
public Economy getEconomy() { | ||
return this.economy; | ||
} |
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.
I don't like this method in main class.
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.
I'm sorry to hear that, where else can I move this method to?
…istGUI.java Co-authored-by: DMK <81445555+imDMK@users.noreply.github.com>
src/main/java/com/eternalcode/parcellockers/parcel/gui/ParcelListGUI.java
Outdated
Show resolved
Hide resolved
…istGUI.java Co-authored-by: DMK <81445555+imDMK@users.noreply.github.com>
# Conflicts: # src/main/java/com/eternalcode/parcellockers/parcel/gui/ParcelListGUI.java
src/main/java/com/eternalcode/parcellockers/locker/database/LockerDatabaseService.java
Outdated
Show resolved
Hide resolved
…ckerDatabaseService.java Co-authored-by: DMK <81445555+imDMK@users.noreply.github.com>
-[ ] Parcel receiving- [ ] Parcel statuses IN PROGRESS- [ ] Parcel redeem time update- [ ] Parcel locker deletion by administrator (further logic, what will happen to parcels)Closes #72