fix: allow period to be specified with any case

fixes #63
howeyc · Dec 31, 2024 · bfa5aa0 · bfa5aa0
1 parent 599ad1b
commit bfa5aa0
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 4 deletions.
diff --git a/ledger/cmd/printBalance.go b/ledger/cmd/printBalance.go
@@ -23,7 +23,7 @@ var balanceCmd = &cobra.Command{
 		if period == "" {
 			PrintBalances(ledger.GetBalances(generalLedger, args), showEmptyAccounts, transactionDepth, columnWidth)
 		} else {
-			lperiod := ledger.Period(period)
+			lperiod := ledger.Period(strings.Title(period))
 			rtrans := ledger.TransactionsByPeriod(generalLedger, lperiod)
 			for rIdx, rt := range rtrans {
 				balances := ledger.GetBalances(rt.Transactions, args)

diff --git a/ledger/cmd/printRegister.go b/ledger/cmd/printRegister.go
@@ -23,7 +23,7 @@ var registerCmd = &cobra.Command{
 		if period == "" {
 			PrintRegister(generalLedger, args, columnWidth)
 		} else {
-			lperiod := ledger.Period(period)
+			lperiod := ledger.Period(strings.Title(period))
 			rtrans := ledger.TransactionsByPeriod(generalLedger, lperiod)
 			for rIdx, rt := range rtrans {
 				if len(rt.Transactions) < 1 {

diff --git a/ledger/cmd/webHandlerReport.go b/ledger/cmd/webHandlerReport.go
@@ -15,7 +15,7 @@ import (
 )
 
 func getRangeAndPeriod(dateRange, dateFreq string) (start, end time.Time, period ledger.Period, err error) {
-	period = ledger.Period(dateFreq)
+	period = ledger.Period(strings.Title(dateFreq))
 
 	start, end, err = pdr.ParseRange(dateRange, time.Now())
 

diff --git a/ledger/man/ledger.1 b/ledger/man/ledger.1
@@ -80,7 +80,11 @@ End date of transactions to include in processing.
 Filter transactions used in processing to payees that contain this string.
 .It Fl \-period Ar STR
 Split output into multiple results based on specified period. Valid options are:
+.Sy Daily ,
+.Sy Weekly ,
+.Sy BiWeekly ,
 .Sy Monthly ,
+.Sy BiMonthly ,
 .Sy Quarterly ,
 .Sy SemiYearly ,
 .Sy Yearly
@@ -126,7 +130,11 @@ End date of transactions to include in processing.
 Filter transactions used in processing to payees that contain this string.
 .It Fl \-period Ar STR
 Split output into multiple results based on specified period. Valid options are:
+.Sy Daily ,
+.Sy Weekly ,
+.Sy BiWeekly ,
 .Sy Monthly ,
+.Sy BiMonthly ,
 .Sy Quarterly ,
 .Sy SemiYearly ,
 .Sy Yearly
@@ -165,7 +173,9 @@ match the
 .Ar account-filter .
 The purpose of this is to close the books for a period. The equity transaction
 can be used as an inital transaction in a new file to start a new period. The
-amount necessary to balance the transaction is assigned to the Equity account.
+amount necessary to balance the transaction is assigned to the
+.Sy Equity
+account.
 Options available for this command are:
 .Bl -tag -compact -width "--begin-date (b) YYYY-mm-dd "
 .It Fl \-begin-date ( Fl b ) Ar YYYY-mm-dd