cli, web: fix options breakage

hledger-web/Hledger/Web/Options.hs

@@ -66,8 +66,8 @@ defwebopts = WebOpts
 
 -- instance Default WebOpts where def = defwebopts
 
-toWebOpts :: RawOpts -> IO WebOpts
+rawOptsToWebOpts :: RawOpts -> IO WebOpts
-toWebOpts rawopts = do
+rawOptsToWebOpts rawopts = checkWebOpts <$> do
   cliopts <- rawOptsToCliOpts rawopts
   let p = fromMaybe defport $ maybeintopt "port" rawopts
   return defwebopts {
@@ -80,11 +80,9 @@ toWebOpts rawopts = do
   where
     stripTrailingSlash = reverse . dropWhile (=='/') . reverse -- yesod don't like it
 
-checkWebOpts :: WebOpts -> IO WebOpts
+checkWebOpts :: WebOpts -> WebOpts
-checkWebOpts opts = do
+checkWebOpts = id
-  _ <- checkCliOpts $ cliopts_ opts
-  return opts
 
 getHledgerWebOpts :: IO WebOpts
-getHledgerWebOpts = processArgs webmode >>= return . decodeRawOpts >>= toWebOpts >>= checkWebOpts
+getHledgerWebOpts = processArgs webmode >>= return . decodeRawOpts >>= rawOptsToWebOpts
 

hledger/Hledger/Cli/Main.hs

@@ -144,7 +144,7 @@ argsToCliOpts args addons = do
     args'        = moveFlagsAfterCommand args
     cmdargsopts  = processValue (mainmode addons) args'
     cmdargsopts' = decodeRawOpts cmdargsopts
-  rawOptsToCliOpts cmdargsopts' >>= checkCliOpts
+  rawOptsToCliOpts cmdargsopts'
 
 -- | A hacky workaround for cmdargs not accepting flags before the
 -- subcommand name: try to detect and move such flags after the