Release 2.1.0

pom.xml

@@ -5,7 +5,7 @@
     <modelVersion>4.0.0</modelVersion>
     <groupId>com.yahoofinance-api</groupId>
     <artifactId>YahooFinanceAPI</artifactId>
-    <version>2.0.1</version>  
+    <version>2.1.0</version>
     <packaging>jar</packaging>
     <name>YahooFinanceAPI</name>
     <description>This library provides some methods that should make it easy

src/yahoofinance/Utils.java

@@ -69,7 +69,8 @@ public static BigDecimal getBigDecimal(String data) {
             }
             result = new BigDecimal(data).multiply(multiplier);
         } catch (NumberFormatException e) {
-            YahooFinance.logger.log(Level.INFO, "Failed to parse: " + data, e);
+            YahooFinance.logger.log(Level.WARNING, "Failed to parse: " + data);
+            YahooFinance.logger.log(Level.FINEST, "Failed to parse: " + data, e);
         }
         return result;
     }
@@ -108,7 +109,8 @@ public static double getDouble(String data) {
             }
             result = Double.parseDouble(data) * multiplier;
         } catch (NumberFormatException e) {
-            YahooFinance.logger.log(Level.INFO, "Failed to parse: " + data, e);
+            YahooFinance.logger.log(Level.WARNING, "Failed to parse: " + data);
+            YahooFinance.logger.log(Level.FINEST, "Failed to parse: " + data, e);
         }
         return result;
     }
@@ -122,7 +124,8 @@ public static int getInt(String data) {
             data = Utils.cleanNumberString(data);
             result = Integer.parseInt(data);
         } catch (NumberFormatException e) {
-            YahooFinance.logger.log(Level.INFO, "Failed to parse: " + data, e);
+            YahooFinance.logger.log(Level.WARNING, "Failed to parse: " + data);
+            YahooFinance.logger.log(Level.FINEST, "Failed to parse: " + data, e);
         }
         return result;
     }
@@ -136,7 +139,8 @@ public static long getLong(String data) {
             data = Utils.cleanNumberString(data);
             result = Long.parseLong(data);
         } catch (NumberFormatException e) {
-            YahooFinance.logger.log(Level.INFO, "Failed to parse: " + data, e);
+            YahooFinance.logger.log(Level.WARNING, "Failed to parse: " + data);
+            YahooFinance.logger.log(Level.FINEST, "Failed to parse: " + data, e);
         }
         return result;
     }
@@ -201,7 +205,8 @@ public static Calendar parseDividendDate(String date) {
 
             return parsedDate;
         } catch (ParseException ex) {
-            YahooFinance.logger.log(Level.SEVERE, ex.getMessage(), ex);
+            YahooFinance.logger.log(Level.WARNING, "Failed to parse dividend date: " + date);
+            YahooFinance.logger.log(Level.FINEST, "Failed to parse dividend date: " + date, ex);
             return null;
         }
     }
@@ -227,7 +232,8 @@ public static Calendar parseDateTime(String date, String time, TimeZone timeZone
                 return c;
             }
         } catch (ParseException ex) {
-            YahooFinance.logger.log(Level.SEVERE, ex.getMessage(), ex);
+            YahooFinance.logger.log(Level.WARNING, "Failed to parse datetime: " + datetime);
+            YahooFinance.logger.log(Level.FINEST, "Failed to parse datetime: " + datetime, ex);
         }
         return null;
     }
@@ -242,7 +248,8 @@ public static Calendar parseHistDate(String date) {
                 return c;
             }
         } catch (ParseException ex) {
-            YahooFinance.logger.log(Level.SEVERE, ex.getMessage(), ex);
+            YahooFinance.logger.log(Level.WARNING, "Failed to parse hist date: " + date);
+            YahooFinance.logger.log(Level.FINEST, "Failed to parse hist date: " + date, ex);
         }
         return null;
     }

src/yahoofinance/YahooFinance.java

@@ -27,6 +27,9 @@
  * all of the required information in the least amount of 
  * requests possible towards Yahoo Finance.
  * <p>
+ * You can change the default timeout of 10s for requests to Yahoo Finance by
+ * setting the yahoofinance.connection.timeout system property.
+ * <p>
  * Please be aware that the data received from Yahoo Finance is not always 
  * complete for every single stock. Stocks on the American stock exchanges
  * usually have a lot more data available than stocks on other exchanges.
@@ -46,6 +49,9 @@ public class YahooFinance {
     public static final String QUOTES_CSV_DELIMITER = ",";
     public static final String TIMEZONE = "America/New_York";
 
+    public static final int CONNECTION_TIMEOUT = 
+            Integer.parseInt(System.getProperty("yahoofinance.connection.timeout", "10000"));
+
     public static final Logger logger = Logger.getLogger(YahooFinance.class.getName());
 
     /**

src/yahoofinance/histquotes/HistQuotesRequest.java

@@ -41,11 +41,7 @@ public HistQuotesRequest(String symbol) {
     }
 
     public HistQuotesRequest(String symbol, Interval interval) {
-        this.symbol = symbol;
-        this.interval = interval;
-
-        this.from = DEFAULT_FROM;
-        this.to = DEFAULT_TO;
+        this(symbol, DEFAULT_FROM, DEFAULT_TO, interval);
     }
 
     public HistQuotesRequest(String symbol, Calendar from, Calendar to) {
@@ -54,8 +50,8 @@ public HistQuotesRequest(String symbol, Calendar from, Calendar to) {
 
     public HistQuotesRequest(String symbol, Calendar from, Calendar to, Interval interval) {
         this.symbol = symbol;
-        this.from = from;
-        this.to = to;
+        this.from = this.cleanHistCalendar(from);
+        this.to = this.cleanHistCalendar(to);
         this.interval = interval;
     }
 
@@ -66,13 +62,34 @@ public HistQuotesRequest(String symbol, Date from, Date to) {
     public HistQuotesRequest(String symbol, Date from, Date to, Interval interval) {
         this(symbol, interval);
         this.from.setTime(from);
-        this.to.setTime(to);
+        this.to.setTime(from);
+        this.cleanHistCalendar(this.from);
+        this.cleanHistCalendar(this.to);
+    }
+
+    /**
+     * Put everything smaller than days at 0
+     * @param cal calendar to be cleaned
+     */
+    private Calendar cleanHistCalendar(Calendar cal) {
+        cal.set(Calendar.MILLISECOND, 0);
+        cal.set(Calendar.SECOND, 0);
+        cal.set(Calendar.MINUTE, 0);
+        cal.set(Calendar.HOUR, 0);
+        return cal;
     }
 
     public List<HistoricalQuote> getResult() throws IOException {
 
         List<HistoricalQuote> result = new ArrayList<HistoricalQuote>();
-
+
+        if(this.from.equals(this.to) || this.from.after(this.to)) {
+            YahooFinance.logger.log(Level.WARNING, "Unable to retrieve historical quotes. "
+                    + "From-date should be at least 1 day before to-date. From: " 
+                    + this.from.getTime() + ", to: " + this.to.getTime());
+            return result;
+        }
+
         Map<String, String> params = new LinkedHashMap<String, String>();
         params.put("s", this.symbol);
 
@@ -95,6 +112,8 @@ public List<HistoricalQuote> getResult() throws IOException {
 
         URL request = new URL(url);
         URLConnection connection = request.openConnection();
+        connection.setConnectTimeout(YahooFinance.CONNECTION_TIMEOUT);
+        connection.setReadTimeout(YahooFinance.CONNECTION_TIMEOUT);
         InputStreamReader is = new InputStreamReader(connection.getInputStream());
         BufferedReader br = new BufferedReader(is);
         br.readLine(); // skip the first line

src/yahoofinance/quotes/QuotesRequest.java

@@ -80,6 +80,8 @@ public List<T> getResult() throws IOException {
 
         URL request = new URL(url);
         URLConnection connection = request.openConnection();
+        connection.setConnectTimeout(YahooFinance.CONNECTION_TIMEOUT);
+        connection.setReadTimeout(YahooFinance.CONNECTION_TIMEOUT);
         InputStreamReader is = new InputStreamReader(connection.getInputStream());
         BufferedReader br = new BufferedReader(is);