1. Thomas Waldmann
  2. moin-2.0

Pull requests

#156 Declined
Repository
inducer
Branch
default
Repository
thomaswaldmann
Branch
default

Allow colons in arguments. This allows DateTime() macros imported from 1.9 to work.

Author
  1. Andreas Kloeckner
Reviewers
Description
No description
  • Learn about pull requests

Comments (3)

  1. Thomas Waldmann repo owner

    it would be of course also possible to use double or single quotes, but that would need adding these quotes to all places where the macro gets invoked from the markup.

    not sure if there are negative side effects and which characters we'ld like to allow without requiring quotes.

  2. Andreas Kloeckner author

    Right, it's one or the other. You just have to make a decision. :) My gut feeling says that adding colons is sort of arbitrary, so it might be better to add quoting logic to the importer. I'll send a new pull request.