Add null checks and move Asset and Tag

src/main/java/seedu/address/logic/commands/AddCommand.java

@@ -81,6 +81,7 @@ public String execute(Model model) throws CommandException {
      * @throws IllegalArgumentException if the user input does not conform the expected format
      */
     public static AddCommand of(String args) throws IllegalArgumentException {
+        requireNonNull(args);
         ArgumentMultimap argMultimap =
                 ArgumentTokenizer.tokenize(args, PREFIX_NAME, PREFIX_PHONE, PREFIX_EMAIL,
                         PREFIX_ADDRESS, PREFIX_TAG, PREFIX_ASSET);

src/main/java/seedu/address/logic/commands/AssetCommand.java

@@ -11,7 +11,7 @@
 import seedu.address.logic.util.ArgumentMultimap;
 import seedu.address.logic.util.ArgumentTokenizer;
 import seedu.address.model.Model;
-import seedu.address.model.asset.Asset;
+import seedu.address.model.person.fields.Asset;
 import seedu.address.model.person.fields.Prefix;
 
 /**

src/main/java/seedu/address/logic/commands/EditCommand.java

@@ -1,6 +1,7 @@
 package seedu.address.logic.commands;
 
 import static java.util.Objects.requireNonNull;
+import static seedu.address.commons.util.CollectionUtil.requireAllNonNull;
 import static seedu.address.logic.Messages.MESSAGE_INVALID_COMMAND_FORMAT;
 import static seedu.address.model.Model.PREDICATE_SHOW_ALL_PERSONS;
 import static seedu.address.model.person.fields.Address.PREFIX_ADDRESS;
@@ -99,7 +100,7 @@ public String execute(Model model) throws CommandException {
      * edited with {@code editPersonDescriptor}.
      */
     private static Person createEditedPerson(Person personToEdit, EditPersonDescriptor editPersonDescriptor) {
-        assert personToEdit != null;
+        requireAllNonNull(personToEdit, editPersonDescriptor);
 
         Name updatedName = editPersonDescriptor.getName().orElse(personToEdit.getName());
         Phone updatedPhone = editPersonDescriptor.getPhone().orElse(personToEdit.getPhone());

src/main/java/seedu/address/logic/commands/RedoCommand.java

@@ -1,5 +1,7 @@
 package seedu.address.logic.commands;
 
+import static java.util.Objects.requireNonNull;
+
 import seedu.address.logic.commands.exceptions.CommandException;
 import seedu.address.model.Model;
 
@@ -19,6 +21,7 @@ public class RedoCommand extends Command {
 
     @Override
     public String execute(Model model) throws CommandException {
+        requireNonNull(model);
         if (!model.canRedo()) {
             throw new CommandException(MESSAGE_REDO_EXCEPTION);
         }

src/main/java/seedu/address/logic/commands/UndoCommand.java

@@ -1,5 +1,7 @@
 package seedu.address.logic.commands;
 
+import static java.util.Objects.requireNonNull;
+
 import seedu.address.logic.commands.exceptions.CommandException;
 import seedu.address.model.Model;
 
@@ -18,6 +20,7 @@ public class UndoCommand extends Command {
 
     @Override
     public String execute(Model model) throws CommandException {
+        requireNonNull(model);
         if (!model.canUndo()) {
             throw new CommandException(MESSAGE_UNDO_EXCEPTION);
         }

src/main/java/seedu/address/logic/util/AddressBookParser.java

@@ -49,15 +49,15 @@ public static Command parseCommand(String userInput) throws ParseException {
         try {
             commandType = CommandType.valueOf(commandWord.toUpperCase());
         } catch (IllegalArgumentException ie) {
-            logger.finer("This user input caused a ParseException: " + userInput);
+            logger.finer("This user command caused a ParseException: " + userInput);
             throw new ParseException(MESSAGE_UNKNOWN_COMMAND);
         }
 
         Command command;
         try {
             command = commandType.createCommand(arguments);
         } catch (IllegalArgumentException ie) {
-            logger.finer("This user input caused a ParseException: " + userInput);
+            logger.finer("These user command arguments caused a ParseException: " + userInput);
             throw new ParseException(ie.getMessage());
         }
 

src/main/java/seedu/address/model/AddressBook.java

@@ -9,10 +9,10 @@
 
 import javafx.collections.ObservableList;
 import seedu.address.commons.util.ToStringBuilder;
-import seedu.address.model.asset.Asset;
 import seedu.address.model.exceptions.AddressBookException;
 import seedu.address.model.person.Person;
 import seedu.address.model.person.UniquePersonList;
+import seedu.address.model.person.fields.Asset;
 
 /**
  * Wraps all data at the address-book level.

src/main/java/seedu/address/model/Model.java

@@ -5,9 +5,9 @@
 
 import javafx.collections.ObservableList;
 import seedu.address.commons.core.GuiSettings;
-import seedu.address.model.asset.Asset;
 import seedu.address.model.exceptions.AddressBookException;
 import seedu.address.model.person.Person;
+import seedu.address.model.person.fields.Asset;
 
 /**
  * The API of the Model component.

src/main/java/seedu/address/model/ModelManager.java

@@ -11,8 +11,8 @@
 import javafx.collections.transformation.FilteredList;
 import seedu.address.commons.core.GuiSettings;
 import seedu.address.commons.core.LogsCenter;
-import seedu.address.model.asset.Asset;
 import seedu.address.model.person.Person;
+import seedu.address.model.person.fields.Asset;
 
 /**
  * Represents the in-memory model of the address book data.

src/main/java/seedu/address/model/person/Person.java

@@ -8,8 +8,8 @@
 import com.fasterxml.jackson.annotation.JsonProperty;
 
 import seedu.address.commons.util.ToStringBuilder;
-import seedu.address.model.asset.Asset;
 import seedu.address.model.person.fields.Address;
+import seedu.address.model.person.fields.Asset;
 import seedu.address.model.person.fields.Assets;
 import seedu.address.model.person.fields.Email;
 import seedu.address.model.person.fields.Name;

src/main/java/seedu/address/model/asset/Asset.java → src/main/java/seedu/address/model/person/fields/Asset.java

@@ -1,4 +1,4 @@
-package seedu.address.model.asset;
+package seedu.address.model.person.fields;
 
 import static java.util.Objects.requireNonNull;
 import static seedu.address.commons.util.AppUtil.checkArgument;

src/main/java/seedu/address/model/person/fields/Assets.java

@@ -10,8 +10,6 @@
 
 import com.fasterxml.jackson.annotation.JsonValue;
 
-import seedu.address.model.asset.Asset;
-
 /**
  * Represents a collection of assets associated to a person in the address book.
  */

src/main/java/seedu/address/model/tag/Tag.java → src/main/java/seedu/address/model/person/fields/Tag.java

@@ -1,4 +1,4 @@
-package seedu.address.model.tag;
+package seedu.address.model.person.fields;
 
 import static java.util.Objects.requireNonNull;
 import static seedu.address.commons.util.AppUtil.checkArgument;

src/main/java/seedu/address/model/person/fields/Tags.java

@@ -9,8 +9,6 @@
 
 import com.fasterxml.jackson.annotation.JsonValue;
 
-import seedu.address.model.tag.Tag;
-
 //@@author aureliony
 /**
  * Represents an abstraction for a list of tags.

src/main/java/seedu/address/ui/PersonCard.java

@@ -7,9 +7,9 @@
 import javafx.scene.layout.FlowPane;
 import javafx.scene.layout.HBox;
 import javafx.scene.layout.Region;
-import seedu.address.model.asset.Asset;
 import seedu.address.model.person.Person;
-import seedu.address.model.tag.Tag;
+import seedu.address.model.person.fields.Asset;
+import seedu.address.model.person.fields.Tag;
 
 /**
  * An UI component that displays information of a {@code Person}.

src/test/java/seedu/address/logic/commands/AddCommandTest.java

@@ -59,8 +59,8 @@
 import seedu.address.model.Model;
 import seedu.address.model.ReadOnlyAddressBook;
 import seedu.address.model.ReadOnlyUserPrefs;
-import seedu.address.model.asset.Asset;
 import seedu.address.model.person.Person;
+import seedu.address.model.person.fields.Asset;
 import seedu.address.testutil.PersonBuilder;
 
 public class AddCommandTest {

src/test/java/seedu/address/logic/commands/AssetCommandTest.java

@@ -10,7 +10,7 @@
 import static seedu.address.logic.commands.AssetCommand.MESSAGE_NOT_EDITED;
 import static seedu.address.logic.commands.AssetCommand.MESSAGE_SUCCESS;
 import static seedu.address.logic.commands.CommandTestUtil.assertParseFailure;
-import static seedu.address.model.asset.Asset.MESSAGE_CONSTRAINTS;
+import static seedu.address.model.person.fields.Asset.MESSAGE_CONSTRAINTS;
 import static seedu.address.testutil.Assert.assertThrows;
 import static seedu.address.testutil.TypicalPersons.getTypicalAddressBook;
 
@@ -20,8 +20,8 @@
 import seedu.address.model.Model;
 import seedu.address.model.ModelManager;
 import seedu.address.model.UserPrefs;
-import seedu.address.model.asset.Asset;
 import seedu.address.model.person.Person;
+import seedu.address.model.person.fields.Asset;
 import seedu.address.testutil.AssetBuilder;
 import seedu.address.testutil.PersonBuilder;
 

src/test/java/seedu/address/model/asset/AssetTest.java → src/test/java/seedu/address/model/person/fields/AssetTest.java

@@ -1,4 +1,4 @@
-package seedu.address.model.asset;
+package seedu.address.model.person.fields;
 
 import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
 import static org.junit.jupiter.api.Assertions.assertEquals;

src/test/java/seedu/address/model/person/fields/AssetsTest.java

@@ -10,8 +10,6 @@
 
 import org.junit.jupiter.api.Test;
 
-import seedu.address.model.asset.Asset;
-
 class AssetsTest {
 
     private static final String INVALID_ASSET = " ";

src/test/java/seedu/address/model/tag/TagTest.java → src/test/java/seedu/address/model/person/fields/TagTest.java

@@ -1,4 +1,4 @@
-package seedu.address.model.tag;
+package seedu.address.model.person.fields;
 
 import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
 import static org.junit.jupiter.api.Assertions.assertEquals;

src/test/java/seedu/address/model/person/fields/TagsTest.java

@@ -10,8 +10,6 @@
 
 import org.junit.jupiter.api.Test;
 
-import seedu.address.model.tag.Tag;
-
 class TagsTest {
 
     private static final String INVALID_TAG = "#friend";

src/test/java/seedu/address/testutil/AssetBuilder.java

@@ -1,6 +1,6 @@
 package seedu.address.testutil;
 
-import seedu.address.model.asset.Asset;
+import seedu.address.model.person.fields.Asset;
 
 /**
  * A utility class to help with building Person objects.