Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Corrected process() argument syntax to dispel PHP warning #24

Merged
merged 1 commit into from
Apr 20, 2021
Merged

Corrected process() argument syntax to dispel PHP warning #24

merged 1 commit into from
Apr 20, 2021

Conversation

Finetuned
Copy link
Contributor

Editing the TV on a resource generates a PHP warning like:

Declaration of MapsTvInputRender::process(, = Array) should be compatible with modTemplateVarRender::process(, array = Array)

A simple addition of array to the second argument:

public function process($value, $params = array())

to
public function process($value, array $params = array())

Fixes #23

…on of MapsTvInputRender::process(, = Array) should be compatible with modTemplateVarRender::process(, array = Array)
@Finetuned Finetuned mentioned this pull request Jan 17, 2018
@Serifenlos
Copy link

Thanks. This helped me to turn off the error-massage. I will keep an eye to that function.

@joeke joeke merged commit 34425d9 into Sterc:master Apr 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

issues with modTemplateVarRender
3 participants