fix: resolved #175 to use hk as default timezone

cli/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "zhang"
-version = "0.1.0-alpha.4"
+version = "0.1.0-alpha.5"
 authors = ["Kilerd <[email protected]>"]
 description = "a plain text double-accounting tool which is compatible with beancount but more powerful"
 edition = "2018"

core/src/constants.rs

@@ -10,6 +10,7 @@ pub const DEFAULT_COMMODITY_PRECISION: i32 = 2;
 pub const DEFAULT_OPERATING_CURRENCY: &str = "CNY";
 pub const DEFAULT_ROUNDING: Rounding = Rounding::RoundDown;
 pub const DEFAULT_BALANCE_TOLERANCE_PRECISION: i32 = 2;
+pub const DEFAULT_TIMEZONE: &str = "Asia/Hong_Kong";
 
 pub const DEFAULT_ROUNDING_PLAIN: &str = "RoundDown";
 pub const DEFAULT_COMMODITY_PRECISION_PLAIN: &str = "2";

core/src/options.rs

@@ -1,12 +1,14 @@
 use itertools::Itertools;
-use log::{error, info};
+use log::{error, info, warn};
 use sqlx::SqliteConnection;
 use std::str::FromStr;
 use std::string::ToString;
 use strum::{AsRefStr, EnumIter, EnumString, IntoEnumIterator};
 use zhang_ast::{Directive, Options, Rounding, SpanInfo, Spanned, ZhangString};
 
-use crate::constants::{DEFAULT_BALANCE_TOLERANCE_PRECISION_PLAIN, DEFAULT_COMMODITY_PRECISION_PLAIN, DEFAULT_OPERATING_CURRENCY, DEFAULT_ROUNDING_PLAIN};
+use crate::constants::{
+    DEFAULT_BALANCE_TOLERANCE_PRECISION_PLAIN, DEFAULT_COMMODITY_PRECISION_PLAIN, DEFAULT_OPERATING_CURRENCY, DEFAULT_ROUNDING_PLAIN, DEFAULT_TIMEZONE,
+};
 use crate::ZhangResult;
 use chrono_tz::Tz;
 
@@ -36,9 +38,16 @@ impl BuiltinOption {
             BuiltinOption::DefaultBalanceTolerancePrecision => DEFAULT_BALANCE_TOLERANCE_PRECISION_PLAIN.to_owned(),
             BuiltinOption::DefaultCommodityPrecision => DEFAULT_COMMODITY_PRECISION_PLAIN.to_owned(),
             BuiltinOption::Timezone => {
-                let system_timezone = iana_time_zone::get_timezone().expect("cannot get the system timezone");
-                info!("detect system timezone is {}", system_timezone);
-                system_timezone
+                match iana_time_zone::get_timezone() {
+                    Ok(timezone) => {
+                        info!("detect system timezone is {}", timezone);
+                        timezone
+                    }
+                    Err(e) => {
+                        warn!("cannot get timezone, fall back to use GMT+8 as default timezone: {}", e);
+                        DEFAULT_TIMEZONE.to_owned()
+                    }
+                }
             }
         }
     }

core/src/utils/logging.rs

@@ -0,0 +1,14 @@
+use log::warn;
+
+pub trait LoggingExit {
+    fn warn_if_none(self, msg: impl AsRef<str>) -> Self;
+}
+
+impl<T> LoggingExit for Option<T> {
+    fn warn_if_none(self, msg: impl AsRef<str>) -> Self {
+        if self.is_none() {
+            warn!("{}", msg.as_ref())
+        }
+        self
+    }
+}

core/src/utils/mod.rs

@@ -7,6 +7,7 @@ pub mod hashmap;
 pub mod id;
 pub mod price_grip;
 pub mod string_;
+pub mod logging;
 
 pub fn has_path_visited<'a>(visited: impl IntoIterator<Item = &'a Pattern>, path: &Path) -> bool {
     visited.into_iter().any(|pattern| pattern.matches_path(path))