Merge pull request #211 from junhaotan/master

Remove unused code

Author: junhaotan

src/main/java/seedu/address/logic/parser/ParserUtil.java

@@ -213,38 +213,6 @@ public static ProductQuantity parseProductQuantity(String quantity) throws Parse
         return new ProductQuantity(trimmedQuantity);
     }
 
-    /**
-     * Parses a {@code String customer} into an {@code Customer}.
-     * Leading and trailing whitespaces will be trimmed.
-     *
-     * @throws ParseException if the given {@code customer} is invalid.
-     */
-    public static String parseCustomer(String customer) throws ParseException {
-        requireNonNull(customer);
-        String trimmedCustomer = customer.trim();
-        //        if (!Sales.isValidSales(trimmedSales)) {
-        //            throw new ParseException(Quantity.MESSAGE_CONSTRAINTS);
-        //        }
-        //        return new Sales(trimmedSales);
-        return trimmedCustomer;
-    }
-
-    /**
-     * Parses a {@code String product} into an {@code product}.
-     * Leading and trailing whitespaces will be trimmed.
-     *
-     * @throws ParseException if the given {@code product} is invalid.
-     */
-    public static String parseProduct(String product) throws ParseException {
-        requireNonNull(product);
-        String trimmedProduct = product.trim();
-        //        if (!Sales.isValidSales(trimmedSales)) {
-        //            throw new ParseException(Quantity.MESSAGE_CONSTRAINTS);
-        //        }
-        //        return new Sales(trimmedSales);
-        return trimmedProduct;
-    }
-
     /**
      * Parses a {@code String product} into an {@code product}.
      * Leading and trailing whitespaces will be trimmed.

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

@@ -4,9 +4,6 @@
 import java.util.logging.Logger;
 
 import javafx.fxml.FXML;
-import javafx.scene.control.Button;
-import javafx.scene.input.Clipboard;
-import javafx.scene.input.ClipboardContent;
 import javafx.scene.media.MediaException;
 import javafx.scene.web.WebView;
 import javafx.stage.Stage;
@@ -22,9 +19,6 @@ public class HelpWindow extends UiPart<Stage> {
     private static final Logger logger = LogsCenter.getLogger(HelpWindow.class);
     private static final String FXML = "HelpWindow.fxml";
 
-    @FXML
-    private Button copyButton;
-
     @FXML
     private WebView webView;
 
@@ -83,28 +77,11 @@ public boolean isShowing() {
         return getRoot().isShowing();
     }
 
-    /**
-     * Hides the help window.
-     */
-    public void hide() {
-        getRoot().hide();
-    }
-
     /**
      * Focuses on the help window.
      */
     public void focus() {
         getRoot().requestFocus();
     }
 
-    /**
-     * Copies the URL to the user guide to the clipboard.
-     */
-    @FXML
-    private void copyUrl() {
-        final Clipboard clipboard = Clipboard.getSystemClipboard();
-        final ClipboardContent url = new ClipboardContent();
-        url.putString(USERGUIDE_URL);
-        clipboard.setContent(url);
-    }
 }

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

@@ -3,15 +3,10 @@
 import java.util.logging.Logger;
 
 import javafx.application.Platform;
-import javafx.event.ActionEvent;
 import javafx.fxml.FXML;
 import javafx.geometry.Pos;
 import javafx.scene.chart.XYChart;
 import javafx.scene.control.Label;
-import javafx.scene.control.MenuItem;
-import javafx.scene.control.TextInputControl;
-import javafx.scene.input.KeyCombination;
-import javafx.scene.input.KeyEvent;
 import javafx.scene.layout.HBox;
 import javafx.scene.layout.Priority;
 import javafx.scene.layout.StackPane;
@@ -83,7 +78,6 @@ public MainWindow(Stage primaryStage, Logic logic) {
 
         // Configure the UI
         setWindowDefaultSize(logic.getGuiSettings());
-        setAccelerators();
 
         helpWindow = new HelpWindow();
         notificationWindow = new NotificationWindow();
@@ -94,39 +88,6 @@ public Stage getPrimaryStage() {
         return primaryStage;
     }
 
-    private void setAccelerators() {
-        // setAccelerator(helpMenuItem, KeyCombination.valueOf("F1"));
-    }
-
-    /**
-     * Sets the accelerator of a MenuItem.
-     * @param keyCombination the KeyCombination value of the accelerator
-     */
-    private void setAccelerator(MenuItem menuItem, KeyCombination keyCombination) {
-        menuItem.setAccelerator(keyCombination);
-
-        /*
-         * TODO: the code below can be removed once the bug reported here
-         * https://bugs.openjdk.java.net/browse/JDK-8131666
-         * is fixed in later version of SDK.
-         *
-         * According to the bug report, TextInputControl (TextField, TextArea) will
-         * consume function-key events. Because CommandBox contains a TextField, and
-         * ResultDisplay contains a TextArea, thus some accelerators (e.g F1) will
-         * not work when the focus is in them because the key event is consumed by
-         * the TextInputControl(s).
-         *
-         * For now, we add following event filter to capture such key events and open
-         * help window purposely so to support accelerators even when focus is
-         * in CommandBox or ResultDisplay.
-         */
-        getRoot().addEventFilter(KeyEvent.KEY_PRESSED, event -> {
-            if (event.getTarget() instanceof TextInputControl && keyCombination.match(event)) {
-                menuItem.getOnAction().handle(new ActionEvent());
-                event.consume();
-            }
-        });
-    }
 
     /**
      * Fills up all the placeholders of this window.

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

@@ -70,13 +70,6 @@ public boolean isShowing() {
         return getRoot().isShowing();
     }
 
-    /**
-     * Hides the notification window.
-     */
-    public void hide() {
-        getRoot().hide();
-    }
-
     /**
      * Focuses on the notification window.
      */

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

@@ -77,13 +77,4 @@ public boolean isShowing() {
     public void focus() {
         getRoot().requestFocus();
     }
-
-
-    /**
-     * Hides the plot window.
-     */
-    public void hide() {
-        getRoot().hide();
-    }
-
 }