Skip to content
This repository has been archived by the owner on Oct 25, 2022. It is now read-only.

Commit

Permalink
Merge pull request #95 from virgiliopontes/patch-1
Browse files Browse the repository at this point in the history
JSON driver issue with username.
  • Loading branch information
cossou authored Jun 13, 2018
2 parents 7129131 + f20bd3f commit c4d9f7a
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/JasperPHP/JasperPHP.php
Original file line number Diff line number Diff line change
Expand Up @@ -102,8 +102,10 @@ public function process($input_file, $output_file = false, $format = array("pdf"
if( count($db_connection) > 0 )
{
$command .= " -t " . $db_connection['driver'];
$command .= " -u " . $db_connection['username'];

if( isset($db_connection['username']) && !empty($db_connection['username']) )
$command .= " -u " . $db_connection['username'];

if( isset($db_connection['password']) && !empty($db_connection['password']) )
$command .= " -p " . $db_connection['password'];

Expand Down

0 comments on commit c4d9f7a

Please sign in to comment.