Skip to content

Commit

Permalink
fix mixup between lpad/rpad help (fixes #13433)
Browse files Browse the repository at this point in the history
  • Loading branch information
jef-n committed Sep 26, 2015
1 parent 56b5d4e commit b46cbbe
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 11 deletions.
11 changes: 5 additions & 6 deletions resources/function_help/json/lpad
@@ -1,12 +1,11 @@
{
"name": "lpad",
"name": "rpad",
"type": "function",
"description": "Returns a string padded to supplied width using a fill character.",
"arguments": [
{"arg":"string","description":"string to pad"},
{"arg":"width","description":"length of new string"},
{"arg":"fill","description":"character to pad the remaining space with"}
"arguments": [ {"arg":"string","description":"string to pad"},
{"arg":"width","description":"length of new string"},
{"arg":"fill","description":"character to pad the remaining space with"}
],
"examples": [ { "expression":"lpad('Hello', 10, 'x')", "returns":"'Helloxxxxx'"}
"examples": [ { "expression":"rpad('Hello', 10, 'x')", "returns":"'xxxxxHello'"}
]
}
11 changes: 6 additions & 5 deletions resources/function_help/json/rpad
@@ -1,11 +1,12 @@
{
"name": "rpad",
"name": "lpad",
"type": "function",
"description": "Returns a string padded to supplied width using a fill character.",
"arguments": [ {"arg":"string","description":"string to pad"},
{"arg":"width","description":"length of new string"},
{"arg":"fill","description":"character to pad the remaining space with"}
"arguments": [
{"arg":"string","description":"string to pad"},
{"arg":"width","description":"length of new string"},
{"arg":"fill","description":"character to pad the remaining space with"}
],
"examples": [ { "expression":"rpad('Hello', 10, 'x')", "returns":"'xxxxxHello'"}
"examples": [ { "expression":"lpad('Hello', 10, 'x')", "returns":"'Helloxxxxx'"}
]
}

1 comment on commit b46cbbe

@nyalldawson
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jef-n this doesn't look quite right - the function names and examples have now been flipped

Please sign in to comment.