SemanticMediaWiki / SemanticScribunto

Provides service functions to support the Scribunto extension
Other
24 stars 14 forks source link

mw.set drops property names with numbers in key-value mode #48

Closed honestleaf closed 6 years ago

honestleaf commented 6 years ago

Setup and configuration

Issue

When setting properties by using key-value form, if the "key" part contains numbers, then the property will not be accepted (more like simply get ignored).

Steps to reproduce the observation (recommendation is to use the sandbox): Using following table to setup properties. (source)

local settings = {
    ['hello'] = 'world',
    ['hell0'] = 'world',
    'HELLO=WORLD',
    'HELL0=WORLD',
}

Only ['hell0'] = 'world' doesn't show up on browse page.

oetterer commented 6 years ago

@kghbln could you please assign this issue to me. I'm looking into it today or on the morrow.

kghbln commented 6 years ago

@oetterer This is apparently not so easy. I sent you some invitations via GitHub hopeing that this will allow me to select you. Touching wood.

oetterer commented 6 years ago

Worked fine, thank you. 👍 Courious, that you couldn't find me in assignees....

kghbln commented 6 years ago

You probably had to be part of a group with extra powers. :)