fixed minor bugs

Civi/DataProcessor/DataFlow/MultipleDataFlows/SimpleNonRequiredJoin.php

@@ -20,8 +20,7 @@ class SimpleNonRequiredJoin  extends  SimpleJoin {
    */
   protected $filterClauses = array();
 
-  public function __construct($left_prefix = null, $left_field = null, $right_prefix = null, $right_field = null, $type = "INNER") {
-    $type = 'LEFT';
+  public function __construct($left_prefix = null, $left_field = null, $right_prefix = null, $right_field = null, $type = "LEFT") {
     parent::__construct($left_prefix, $left_field, $right_prefix, $right_field, $type);
   }
 
@@ -31,7 +30,6 @@ class SimpleNonRequiredJoin  extends  SimpleJoin {
    * @return \Civi\DataProcessor\DataFlow\MultipleDataFlows\JoinInterface
    */
   public function setConfiguration($configuration) {
-    $configuration[' type'] = 'LEFT';
     return parent::setConfiguration($configuration);
   }
 

Civi/DataProcessor/Output/Api.php

@@ -292,8 +292,6 @@ class Api implements OutputInterface, API_ProviderInterface, EventSubscriberInte
    * @throws \Exception
    */
   public function invoke($apiRequest) {
-    $isCountAction = FALSE;
-
     switch (strtolower($apiRequest['action'])) {
       case 'getfields':
         // Do get fields
@@ -310,6 +308,7 @@ class Api implements OutputInterface, API_ProviderInterface, EventSubscriberInte
   }
 
   protected function invokeDataProcessor($apiRequest) {
+    $isCountAction = FALSE;
     $dataProcessorIdSql = "
       SELECT *
       FROM civicrm_data_processor_output o 
@@ -412,6 +411,9 @@ class Api implements OutputInterface, API_ProviderInterface, EventSubscriberInte
   protected function checkForErrors($return) {
     $session = \CRM_Core_Session::singleton();
     $statuses = $session->getStatus(true);
+    if (is_array($statuses)) {
+      return $return;
+    }
     foreach($statuses as $status) {
       if ($status['type'] == 'error') {
         $return['is_error'] = 1;