Skip to content

Event:sudo is not correct #55

Closed
Closed
@cjunge-work

Description

@cjunge-work

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.

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->user as the value and that https://github.com/lavary/crunz/blob/master/src/Event.php#L207 should append it to $command

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions