Merge pull request #8400 from kaltura/Naos-14.20.0-REACH2-577

REACH2-577: When vendor marks task as error set its price to 0 while saving old price value

plugins/reach/lib/kReachManager.php

@@ -352,7 +352,13 @@ private function updateReachProfileCreditUsage(EntryVendorTask $entryVendorTask)
 
 	private function handleErrorTask(EntryVendorTask $entryVendorTask)
 	{
+		//Refund credit for tasks which could not be handled by the service provider
 		ReachProfilePeer::updateUsedCredit($entryVendorTask->getReachProfileId(), -$entryVendorTask->getPrice());
+
+		//Rest task price so that reports will be alligned with the total used credit
+		$entryVendorTask->setOldPrice($entryVendorTask->getPrice());
+		$entryVendorTask->setPrice(0);
+		$entryVendorTask->save();
 	}
 
 	private function invalidateAccessKey(EntryVendorTask $entryVendorTask)

plugins/reach/lib/model/EntryVendorTask.php

@@ -27,6 +27,7 @@ class EntryVendorTask extends BaseEntryVendorTask implements IRelatedObject, IIn
 	const CUSTOM_DATA_CREATION_MODE =       'creation_mode';
 	const CUSTOM_DATA_IS_REQUEST_MODERATED ='request_moderated';
 	const CUSTOM_DATA_TASK_DATA =       	'task_data';
+	const CUSTOM_DATA_OLD_PRICE =       	'old_price';
 
 	//setters
 
@@ -89,6 +90,11 @@ public function setTaskJobData($v)
 	{
 		$this->putInCustomData(self::CUSTOM_DATA_TASK_DATA, $v);
 	}
+
+	public function setOldPrice($v)
+	{
+		$this->putInCustomData(self::CUSTOM_DATA_OLD_PRICE, $v);
+	}
 
 	//getters
 
@@ -166,6 +172,11 @@ public function getTaskJobData()
 	{
 		return $this->getFromCustomData(self::CUSTOM_DATA_TASK_DATA);
 	}
+
+	public function getOldPrice()
+	{
+		return $this->getFromCustomData(self::CUSTOM_DATA_OLD_PRICE);
+	}
 
 	/* (non-PHPdoc)
  	 * @see BaseEntryVendorTask::preSave()