Extension talk:IncludeBikemap/Flow

About this board

MaxSem (talkcontribs)

Hi, just noticed this extension and I see some issues with code:

  • First of all, great to see that this extension is XSS-safe, however you don't need to apply escaping to $wgIBMsize - it is not outputted to pages directly.
  • We use prefix wg for global configuration variables (wg stands for "wiki global"), you don't need to use it for internal variables.
  • wfMsg*() functions are used to retrieve messages from message files and optionally transform them (by parsing wikitext, substituting parameters like $1 with provided values and so on). They can't be used with random input like wfMsgForContent('uncorrect id: '). See Localisation for details.
Singlespeedfahrer (talkcontribs)

Hi, many thanks for your code review! I have to learn a lot about writing those extensions. So I'm pleased of your fast response. I fixed the code now as you have suggested. Best wishes.

There are no older topics
Return to "IncludeBikemap/Flow" page.