Skip to content
This repository was archived by the owner on Aug 31, 2021. It is now read-only.

[[ Engine ]] Add variant of params() which returns a sequence #6608

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

runrevmark
Copy link
Contributor

This patch adds a variant of the params function.

The params function can now take 0, 1 or 2 parameters.

Calling it with zero parameters has the same behavior as before -
it returns a stringified version of the call that was made.

Calling it with two parameters returns a range of parameters
as a numerically sequenced array. The first parameter indicates the
starting index (from 1), and the second parameter indicates the
finishing index (from 1). Both parameters may be negative, in which
case they are interpreted as being relative to 'the paramCount' in
the same way negative chunk indicies work.

If only one parameter is specified, the second is taken to be -1.

@runrevmark runrevmark force-pushed the params-range_as_array branch 2 times, most recently from 485bdf7 to 15f6c01 Compare July 6, 2018 13:27
@runrevmark runrevmark force-pushed the params-range_as_array branch from 15f6c01 to 521fddd Compare July 20, 2018 16:29
@runrevmark runrevmark added this to the 9.1.0-dp-1 milestone Aug 16, 2018
This patch adds a variant of the params function.

The params function can now take 0, 1 or 2 parameters.

Calling it with zero parameters has the same behavior as before -
it returns a stringified version of the call that was made.

Calling it with two parameters returns a range of parameters
as a numerically sequenced array. The first parameter indicates the
starting index (from 1), and the second parameter indicates the
finishing index (from 1). Both parameters may be negative, in which
case they are interpreted as being relative to 'the paramCount' in
the same way negative chunk indicies work.

If only one parameter is specified, the second is taken to be -1.
@runrevmark runrevmark force-pushed the params-range_as_array branch from 521fddd to 35ec66e Compare August 29, 2018 16:56
@runrevmark runrevmark removed this from the 9.1.0-dp-1 milestone May 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant