Skip to content

Commit

Permalink
Rename HumanInventory to PlayerInventory, add utility getters.
Browse files Browse the repository at this point in the history
Signed-off-by: Chris Sanders <[email protected]>
  • Loading branch information
Zidane committed Sep 18, 2016
1 parent fd6816c commit b88f5eb
Showing 1 changed file with 27 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,12 @@
*/
package org.spongepowered.api.item.inventory.entity;

import org.spongepowered.api.entity.living.Humanoid;
import org.spongepowered.api.entity.living.player.Player;
import org.spongepowered.api.item.inventory.Slot;
import org.spongepowered.api.item.inventory.crafting.CraftingInventory;
import org.spongepowered.api.item.inventory.equipment.EquipmentInventory;
import org.spongepowered.api.item.inventory.type.CarriedInventory;
import org.spongepowered.api.item.inventory.type.GridInventory;

/**
* Represents the inventory of a Humanoid or Player. Implementors of this interface

This comment has been minimized.

Copy link
@jamierocks

jamierocks Sep 19, 2016

Contributor

Javadoc needs updating

This comment has been minimized.

Copy link
@gabizou

gabizou Sep 19, 2016

Member

Comment on the OCD Issue.

Expand All @@ -37,13 +40,34 @@
* {@link CraftingInventory}
* </li></ul>
*/
public interface HumanInventory extends CarriedInventory<Humanoid> {
public interface PlayerInventory extends CarriedInventory<Player> {

/**
* Get the hotbar inventory.
* Gets the hotbar inventory.
*
* @return the hotbar

This comment has been minimized.

Copy link
@liach

liach Sep 19, 2016

Contributor

Should be

@return The hotbar

according to Sponge code style.

This comment has been minimized.

Copy link
@gabizou

gabizou Sep 19, 2016

Member

Comment on the OCD Issue.

This comment has been minimized.

Copy link
@liach

liach Sep 19, 2016

Contributor
*/
Hotbar getHotbar();

/**
* Gets the main inventory.
*
* @return the hotbar

This comment has been minimized.

Copy link
@JonathanBrouwer

JonathanBrouwer Sep 19, 2016

incorrect return

This comment has been minimized.

Copy link
@gabizou

gabizou Sep 19, 2016

Member

Comment on the OCD Issue.

*/
GridInventory getMain();

/**
* Gets the equipment inventory.
*
* @return the hotbar

This comment has been minimized.

Copy link
@JonathanBrouwer

JonathanBrouwer Sep 19, 2016

incorrect return

This comment has been minimized.

Copy link
@gabizou

gabizou Sep 19, 2016

Member

Comment on the OCD Issue.

*/
EquipmentInventory getEquipment();

/**
* Gets the offhand inventory.
*
* @return the hotbar

This comment has been minimized.

Copy link
@JonathanBrouwer

JonathanBrouwer Sep 19, 2016

incorrect return

This comment has been minimized.

Copy link
@gabizou

gabizou Sep 19, 2016

Member

Comment on the OCD Issue.

*/
Slot getOffhand();

}

0 comments on commit b88f5eb

Please sign in to comment.