We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
On line https://github.com/lavary/crunz/blob/master/src/Event.php#L200 there is a call to $this->sudo(), but the definition at https://github.com/lavary/crunz/blob/master/src/Event.php#L222-L225 is protected function sudo($user) with the $user parameter being used.
$this->sudo()
protected function sudo($user)
$user
Also, on https://github.com/lavary/crunz/blob/master/src/Event.php#L207 the call to sudo() is not appended to the $command, so nothing will happen with the returned value.
$command
I believe that sudo() should use $this->user as the value and that https://github.com/lavary/crunz/blob/master/src/Event.php#L207 should append it to $command
sudo()
$this->user
The text was updated successfully, but these errors were encountered:
You are right. Fixed in #85, thank you.
Sorry, something went wrong.
No branches or pull requests
On line https://github.com/lavary/crunz/blob/master/src/Event.php#L200 there is a call to
$this->sudo(), but the definition at https://github.com/lavary/crunz/blob/master/src/Event.php#L222-L225 isprotected function sudo($user)with the$userparameter being used.Also, on https://github.com/lavary/crunz/blob/master/src/Event.php#L207 the call to sudo() is not appended to the
$command, so nothing will happen with the returned value.I believe that
sudo()should use$this->useras the value and that https://github.com/lavary/crunz/blob/master/src/Event.php#L207 should append it to$commandThe text was updated successfully, but these errors were encountered: