Add support for numeric map keys

[Richtermeister]

I have data from an external source which contains map[string]interface{} instances with numeric keys:

map[string]interface{}{
  "0": "foo",
  "1": "bar",
}

I'm trying to access this map via this template {{ mydict.0 }} as I would a regular map, but Pongo2 doesn't seem to allow this:

Can't access an index on type map (variable mydict.0)

This PR adds this capability.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[Richtermeister]

Hi Florian, The test is here: https://github.com/flosch/pongo2/pull/269/files#diff-60c354206e504d66cb5b2fab9b3bab7b8a9cb3067ee0a31d9ca3b3eb333cddabR31 Is this insufficient? – Daniel

…

On Fri, Dec 10, 2021 at 3:41 AM Florian Schlachter ***@***.***> wrote: Please add some tests, thanks! — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#269 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAEYSGIRVOJTNNVTHUJZEZ3UQHRQJANCNFSM5BHRTAIA> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[Richtermeister]

@flosch Sorry to bother, could I please get some feedback on this PR? It seems straight-forward and has test coverage. If there's anything else I can do to unblock this, please let me know. Thank you for your time.

[flosch]

Hi, this should be resolved by the introduced subscript syntax (#281): mydict["0"]

If not, feel free to re-open/answer again. Thanks!

[Richtermeister]

Yes that looks good, I'll give that a shot! Thank you for your efforts on this library, I really appreciate it.