Page MenuHomePhabricator

Normalise usage of parameter names in parameter descriptions
Closed, ResolvedPublic

Description

Per irc, we should normalise the style. In this case, it seems appending the module prefix is probably better.

Some are hardcoded with the prefix, some use fields etc.

<Reedy> RoanKattouw, in ApiQueryRevisions, we use the literal rvsections in a description... Every other param references is minus prefix... Presume it should be removed?
<RoanKattouw> This is probably inconsistent across the API as well. Let's pick one and stick with it
<Reedy> it's that, or do something with a prefux
<RoanKattouw> Maybe see which style is the most frequently used one. I have no idea
<Reedy> RoanKattouw, I suspect, if we're going to normalise it to using the prefix, we shouldn't hard code in.. Incase we inherit from modules etc etc?
<RoanKattouw> Reedy: Yes, and for multi-mode modules. So that means {$this->getModulePrefix()} or whatever the name is all over
<RoanKattouw> See ApiQueryBacklinks
<Reedy> Mmmm
<RoanKattouw> Or do $p = $this->getModulePrefix(); and use {$p} , probably nicer


Version: unspecified
Severity: trivial

Details

Reference
bz23461

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 11:10 PM
bzimport set Reference to bz23461.

r66232

Probably not perfect, but somewhere near