diff options
author | Bjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com> | 2023-07-02 11:02:47 +0200 |
---|---|---|
committer | Bjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com> | 2023-07-02 13:04:11 +0200 |
commit | 5afc89f2bfb3e582bface8a9980e60782ece9880 (patch) | |
tree | 325ac10ec7f26da64d7e20f11219d91428f64dba /publisher/htmlElementsCollector.go | |
parent | c1eac616d55945c28ea364f44d1e9ae12e672e11 (diff) |
Rework the build.writeStats struct
Mostly to make it easier to toggle on/off this feature from the env.
See #11191
Diffstat (limited to 'publisher/htmlElementsCollector.go')
-rw-r--r-- | publisher/htmlElementsCollector.go | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/publisher/htmlElementsCollector.go b/publisher/htmlElementsCollector.go index 080502352..c942c46e5 100644 --- a/publisher/htmlElementsCollector.go +++ b/publisher/htmlElementsCollector.go @@ -47,7 +47,7 @@ var ( } ) -func newHTMLElementsCollector(conf config.WriteStats) *htmlElementsCollector { +func newHTMLElementsCollector(conf config.BuildStats) *htmlElementsCollector { return &htmlElementsCollector{ conf: conf, elementSet: make(map[string]bool), @@ -95,7 +95,7 @@ type htmlElement struct { } type htmlElementsCollector struct { - conf config.WriteStats + conf config.BuildStats // Contains the raw HTML string. We will get the same element // several times, and want to avoid costly reparsing when this @@ -117,7 +117,7 @@ func (c *htmlElementsCollector) getHTMLElements() HTMLElements { for _, el := range c.elements { classes = append(classes, el.Classes...) ids = append(ids, el.IDs...) - if c.conf.Tags { + if !c.conf.DisableTags { tags = append(tags, el.Tag) } } @@ -372,7 +372,7 @@ func htmlLexToEndOfComment(w *htmlElementsCollectorWriter) htmlCollectorStateFun func (w *htmlElementsCollectorWriter) parseHTMLElement(elStr string) (el htmlElement, err error) { conf := w.collector.conf - if !conf.IDs && !conf.Classes { + if conf.DisableTags && conf.DisableClasses { // Nothing to do. return } @@ -402,11 +402,11 @@ func (w *htmlElementsCollectorWriter) parseHTMLElement(elStr string) (el htmlEle switch { case strings.EqualFold(a.Key, "id"): // There should be only one, but one never knows... - if conf.IDs { + if !conf.DisableIDs { el.IDs = append(el.IDs, a.Val) } default: - if !conf.Classes { + if conf.DisableClasses { continue } if classAttrRe.MatchString(a.Key) { |