ashcsi1322 Posted January 19, 2015 Share Posted January 19, 2015 Hi All,I have downloaded the latest v2 jrs-rest-php-client-master.zip file and started using it. But while going through the Job class I found that its constructor is not setting the $mailNotification object in the constructor. The code is as mentioned below: /** * Mail notification settings * @var MailNotification */ public $mailNotification; public function __construct($label = null, $trigger = null, $source = null, $baseOutputFilename = null, $outputFormats = null, $repositoryDestination = null) { $this->label = $label; $this->trigger = $trigger; $this->source = $source; $this->baseOutputFilename = $baseOutputFilename; $this->outputFormats = $outputFormats; $this->repositoryDestination = $repositoryDestination; }[/code]This might cause problem for users who want to set up mailnotification while scheduling reports/jobs. Could anyone let me know if it is indeed missing to update the mailNotification object or is intended to work as it is ? ThanksAshy Link to comment Share on other sites More sharing options...
ashcsi1322 Posted January 23, 2015 Author Share Posted January 23, 2015 Hi All, I fixed the following issue. There is no need to add $mailNotification object in the constructor. But it can be set after creating the job object for e.g.: $job->mailNotification; thanks ashy Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now