From 94fdf827e1157507a6c8672844a49a8b6ad2dba3 Mon Sep 17 00:00:00 2001
From: Periklis Tsirakidis <periklis.tsirakidis@holidaycheck.com>
Date: Tue, 5 Jun 2018 10:55:01 +0200
Subject: [PATCH] Add appropriate option defaults

---
 modules/services/mail/offlineimap.nix | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/modules/services/mail/offlineimap.nix b/modules/services/mail/offlineimap.nix
index b3cdc2fd..e6a5753c 100644
--- a/modules/services/mail/offlineimap.nix
+++ b/modules/services/mail/offlineimap.nix
@@ -25,14 +25,14 @@ in {
 
     startInterval = mkOption {
       type = types.nullOr types.int;
-      default = null;
+      default = 300;
       example = literalExample "300";
       description = "Optional key to start offlineimap services each N seconds";
     };
 
     runQuick = mkOption {
-      type = types.nullOr types.bool;
-      default = null;
+      type = types.bool;
+      default = false;
       description = ''
         Run only quick synchronizations.
         Ignore any flag updates on IMAP servers. If a flag on the remote IMAP changes, and we have the message locally, it will be left untouched in a quick run.
@@ -54,7 +54,7 @@ in {
       command                         = "offlineimap";
       serviceConfig.KeepAlive         = false;
       serviceConfig.RunAtLoad         = true;
-      serviceConfig.ProgramArguments  = [ "-c" "/etc/offlineimaprc" ] ++ optional (cfg.runQuick != null) "-q";
+      serviceConfig.ProgramArguments  = [ "-c" "/etc/offlineimaprc" ] ++ optional (cfg.runQuick) "-q";
       serviceConfig.StartInterval     = cfg.startInterval;
       serviceConfig.StandardErrorPath = "/var/log/offlineimap.log";
       serviceConfig.StandardOutPath   = "/var/log/offlineimap.log";