Skip to content

Conversation

vbaranovskiy-plesk
Copy link
Contributor

No description provided.

*
* @param callable|null $function
*/
public function setVerifyResponseCallback(callable $function = null) {
Copy link
Member

Choose a reason for hiding this comment

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

CC violation, curly brace should be on new line

*
* @param callable|null $function
*/
public function setVerifyResponseCallback(callable $function = null) {
Copy link
Member

Choose a reason for hiding this comment

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

maybe it's better to name it as "setResponseVerification" and the method could accept callback function as an optional parameter

@sibprogrammer sibprogrammer merged commit 5725566 into plesk:master Jul 10, 2017
mrsombre pushed a commit that referenced this pull request Aug 2, 2019
…v-component-PPP-30318 to master

* commit 'cbe908474c939a9cfd41ad31726533824d754341':
  FEATURE PPP-30318 Add Subnav component
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.

2 participants