Thread:Extension talk:IncludeBikemap/Code review

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.