Skip to content

Commit

Permalink
Merge pull request #17098 from mattwire/removeunusedparameterjob
Browse files Browse the repository at this point in the history
Remove unused parameter from function
  • Loading branch information
colemanw authored Apr 17, 2020
2 parents 0aa61b6 + 41b74a9 commit 816fa1d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 7 deletions.
4 changes: 1 addition & 3 deletions CRM/Admin/Page/Job.php
Original file line number Diff line number Diff line change
Expand Up @@ -136,10 +136,8 @@ public function run() {

/**
* Browse all jobs.
*
* @param null $action
*/
public function browse($action = NULL) {
public function browse() {
// check if non-prod mode is enabled.
if (CRM_Core_Config::environment() != 'Production') {
CRM_Core_Session::setStatus(ts('Execution of scheduled jobs has been turned off by default since this is a non-production environment. You can override this for particular jobs by adding runInNonProductionEnvironment=TRUE as a parameter.'), ts("Non-production Environment"), "warning", array('expires' => 0));
Expand Down
5 changes: 1 addition & 4 deletions CRM/Admin/Page/JobLog.php
Original file line number Diff line number Diff line change
Expand Up @@ -71,11 +71,8 @@ public function run() {

/**
* Browse all jobs.
*
* @param null $action
*/
public function browse($action = NULL) {

public function browse() {
$jid = CRM_Utils_Request::retrieve('jid', 'Positive', $this);

$sj = new CRM_Core_JobManager();
Expand Down

0 comments on commit 816fa1d

Please sign in to comment.