Environment Default trait

The Environment struct only used the Default trait so it could have the same call for both Environment<UsersCache> and Environment<MockUsers>. There’s no reason to keep it around anymore.

src/output/details.rs

@@ -140,7 +140,7 @@ impl<'a> Render<'a> {
         let mut rows = Vec::new();
 
         if let Some(columns) = self.opts.columns {
-            let env = Environment::default();
+            let env = Environment::load_all();
             let colz = columns.for_dir(self.dir);
             let mut table = Table::new(&colz, &self.colours, &env);
 

src/output/grid_details.rs

@@ -47,7 +47,7 @@ impl<'a> Render<'a> {
             None => Vec::new(),
         };
 
-        let env = Environment::default();
+        let env = Environment::load_all();
 
         let drender = self.clone().details();
 

src/output/table.rs

@@ -41,10 +41,8 @@ impl Environment {
     pub fn lock_users(&self) -> MutexGuard<UsersCache> {
         self.users.lock().unwrap()
     }
-}
 
-impl Default for Environment {
+    pub fn load_all() -> Self {
-    fn default() -> Self {
         let tz = match determine_time_zone() {
             Ok(t) => Some(t),
             Err(ref e) => {