My filter adds unwanted classes (bug in Pandoc?)

[pagiraud]

Hi,

Finishing and trying to add an option to a Lua filter, I stumbled upon a problem that I reduced to some sort of MWE. Maybe it’s me doing wrong things (I’m not an expert at all) but this seems quite strange to me.

Consider this markdown file (note the foo class):

---
title: "Untitled"
---

## Some title

![Some caption](teaching.jpg){.foo}

## Without caption

![](teaching.jpg)

One more paragraph.

And then this filter:

local classes = {"aclass", "anotherclass"}
local function bblocks (blocks)
  local newClasses = {}
  local originalClasses = {}
  -- Go from end to start to avoid problems with shifting indices.
  for i = #blocks-1, 1, -1 do
    originalClasses = {}
    newClasses = classes
    if blocks[i].t == "Figure" then
      originalClasses = blocks[i].content[1].content[1].classes
      for k,v in pairs(originalClasses) do table.insert(newClasses, v) end
      blocks[i].content[1].content[1].classes = newClasses
      print("For my Figure:")
      for k,v in pairs(blocks[i].content[1].content[1].classes) do print(v) end
    elseif blocks[i].t == "Para" then
      originalClasses = blocks[i].content[1].classes
      for k,v in pairs(originalClasses) do table.insert(newClasses, v) end
      blocks[i].content[1].classes = newClasses
      print("For my Para:")
      for k,v in pairs(blocks[i].content[1].classes) do print(v) end
    end
  end
  return blocks
end

return {{Blocks = bblocks}}

Then, when I invoke:

pandoc --lua-filter ./testbug.lua -f markdown -t native test.md -o test.json

I get the expected output in the console:

For my Para:
aclass
anotherclass
For my Figure:
aclass
anotherclass
foo

But the generated AST contains also a foo class in the Para block:

[ Header
    2
    ( "some-title" , [] , [] )
    [ Str "Some" , Space , Str "title" ]
, Figure
    ( "" , [] , [] )
    (Caption
       Nothing [ Plain [ Str "Some" , Space , Str "caption" ] ])
    [ Plain
        [ Image
            ( "" , [ "aclass" , "anotherclass" , "foo" ] , [] )
            [ Str "Some" , Space , Str "caption" ]
            ( "teaching.jpg" , "" )
        ]
    ]
, Header
    2
    ( "without-caption" , [] , [] )
    [ Str "Without" , Space , Str "caption" ]
, Para
    [ Image
        ( "" , [ "aclass" , "anotherclass" , "foo" ] , [] )
        []
        ( "teaching.jpg" , "" )
    ]
, Para
    [ Str "One"
    , Space
    , Str "more"
    , Space
    , Str "paragraph."
    ]
]

Since the console output is correct and Blocks are evaluated last to first (so the one with the wanted foo class is the last evaluated), I can’t figure what I’m doing wrong…

[jgm]

Just from a quick glance:

      for k,v in pairs(originalClasses) do table.insert(newClasses, v) end

newClasses scopes over the whole for loop, so you just end up accumulating things, right?

[pagiraud]

Don’t these two lines at the beginning of the loop avoid things to accumulate?

    originalClasses = {}
    newClasses = classes

Initializing originalClasses and newClasses at the beginning of the loop and not of the function doesn’t change anything.