Some random small things i came across when reviewing the code:

- The quotes in your if statements are obsolete:
if ("$type" == "$value") {

- Don't use switch statements if there's only 1 possible value. Use if() instead

Comments

svendecabooter’s picture

svendecabooter
he/him
Primary language Dutch
Location Gent
commented

And you should re-run the module through coder, as there are still about 2 problems there...

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Closed (fixed)

Thanks, the quotes were a remnant of early debugging. Fix will be posted shortly.