Skip to content

Commit

Permalink
Merge pull request #777 from jpwhite4/fix_jobviewer_deeplink
Browse files Browse the repository at this point in the history
Fix Job Viewer deep link regression (introduced in #733).
  • Loading branch information
jpwhite4 authored Feb 14, 2019
2 parents f25c870 + e1b22b4 commit 156d4c1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion classes/Rest/Controllers/WarehouseControllerProvider.php
Original file line number Diff line number Diff line change
Expand Up @@ -2000,7 +2000,7 @@ private function getJobByPrimaryKey(Application $app, \XDUser $user, $realm, $se
throw new \DataWarehouse\Query\Exceptions\AccessDeniedException;
}

if (isset($searchparams['jobref']) && is_int($searchparams['jobref'])) {
if (isset($searchparams['jobref']) && is_numeric($searchparams['jobref'])) {
$params = array(
'primary_key' => $searchparams['jobref']
);
Expand Down

0 comments on commit 156d4c1

Please sign in to comment.