Topic on User talk:Tgr (WMF)

Jump to navigation Jump to search

Casing in abstract schemas

5
Ladsgroup (talkcontribs)

Hey, I just saw Special:Diff/3858844, The code I wrote would very likely explode majestically if we use "notnull" instead of "NotNull". Where did you see the DBAL docs? I can work on it and fix it though, not too complicated.

Tgr (WMF) (talkcontribs)
Ladsgroup (talkcontribs)
Ladsgroup (talkcontribs)

hmm, it's because php function calls are case insensitive: https://stackoverflow.com/a/5643544/2596051 so it doesn't matter which way. There's pros and cons:

- "NotNull" is closer to the actual declaration

- "notnull" is easier to read, looks better IMO.


I don't know which way to go TBH.

Ladsgroup (talkcontribs)

Given the docs, I would say latter. Let's go with the latter.

Reply to "Casing in abstract schemas"