Skip to content

Commit 71684f5

Browse files
committed
ref:import: simplify renderTemplate and friends
renderTemplate and its ancillary functions did not need the HledgerFieldName argument, so remove it. Signed-off-by: Jonathan Dowland <[email protected]>
1 parent b7027c8 commit 71684f5

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

hledger-lib/Hledger/Read/RulesReader.hs

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -795,12 +795,12 @@ isBlockActive rules record CB{..} = any (all matcherMatches) $ groupedMatchers c
795795

796796
-- | Render a field assignment's template, possibly interpolating referenced
797797
-- CSV field values or match groups. Outer whitespace is removed from interpolated values.
798-
renderTemplate :: CsvRules -> CsvRecord -> HledgerFieldName -> FieldTemplate -> Text
799-
renderTemplate rules record f t =
798+
renderTemplate :: CsvRules -> CsvRecord -> FieldTemplate -> Text
799+
renderTemplate rules record t =
800800
maybe t mconcat $ parseMaybe
801801
(many
802802
( literaltextp
803-
<|> (matchrefp <&> replaceRegexGroupReference rules record f)
803+
<|> (matchrefp <&> replaceRegexGroupReference rules record)
804804
<|> (fieldrefp <&> replaceCsvFieldReference rules record)
805805
)
806806
)
@@ -818,13 +818,13 @@ renderTemplate rules record f t =
818818

819819
-- | Replace something that looks like a Regex match group reference with the
820820
-- resulting match group value after applying the Regex.
821-
replaceRegexGroupReference :: CsvRules -> CsvRecord -> HledgerFieldName -> MatchGroupReference -> Text
822-
replaceRegexGroupReference rules record f s = case T.uncons s of
823-
Just ('\\', group) -> fromMaybe "" $ regexMatchValue rules record f group
821+
replaceRegexGroupReference :: CsvRules -> CsvRecord -> MatchGroupReference -> Text
822+
replaceRegexGroupReference rules record s = case T.uncons s of
823+
Just ('\\', group) -> fromMaybe "" $ regexMatchValue rules record group
824824
_ -> s
825825

826-
regexMatchValue :: CsvRules -> CsvRecord -> HledgerFieldName -> Text -> Maybe Text
827-
regexMatchValue rules record f sgroup = let
826+
regexMatchValue :: CsvRules -> CsvRecord -> Text -> Maybe Text
827+
regexMatchValue rules record sgroup = let
828828
matchgroups = concatMap (getMatchGroups rules record)
829829
$ concatMap cbMatchers
830830
$ filter (isBlockActive rules record)
@@ -1228,7 +1228,7 @@ getAmount rules record currency p1IsVirtual n =
12281228

12291229
-- assignments to any of these field names with non-empty values
12301230
assignments = [(f,a') | f <- fieldnames
1231-
, Just v <- [T.strip . renderTemplate rules record f <$> hledgerField rules record f]
1231+
, Just v <- [T.strip . renderTemplate rules record <$> hledgerField rules record f]
12321232
, not $ T.null v
12331233
-- XXX maybe ignore rule-generated values like "", "-", "$", "-$", "$-" ? cf CSV FORMAT -> "amount", "Setting amounts",
12341234
, let a = parseAmount rules record currency v

0 commit comments

Comments
 (0)