Change the parameter name into the award check() method #155
This commit is contained in:
@@ -32,15 +32,15 @@ class PilotFlightAwards extends AwardInterface
|
||||
*
|
||||
* If no parameter is passed in, just default it to 100. You should check if there
|
||||
* is a parameter or not
|
||||
* @param null $params The parameters passed in from the UI
|
||||
* @param null $parameter The parameters passed in from the UI
|
||||
* @return bool
|
||||
*/
|
||||
public function check($params = null): bool
|
||||
public function check($parameter = null): bool
|
||||
{
|
||||
if(!$params) {
|
||||
$params = 100;
|
||||
if(!$parameter) {
|
||||
$parameter = 100;
|
||||
}
|
||||
|
||||
return $this->user->flights >= $params;
|
||||
return $this->user->flights >= $parameter;
|
||||
}
|
||||
}
|
||||
|
||||
@@ -22,10 +22,10 @@ abstract class AwardInterface
|
||||
/**
|
||||
* Each award class just needs to return true or false if it should actually
|
||||
* be awarded to a user. This is the only method that needs to be implemented
|
||||
* @param null $params Optional parameters that are passed in from the UI
|
||||
* @param null $parameter Optional parameters that are passed in from the UI
|
||||
* @return bool
|
||||
*/
|
||||
abstract public function check($params = null): bool;
|
||||
abstract public function check($parameter = null): bool;
|
||||
|
||||
/**
|
||||
* AwardInterface constructor.
|
||||
|
||||
Reference in New Issue
Block a user