New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

REACH-Cloning reach profile and partner catalog items #7915

Merged
merged 2 commits into from Dec 10, 2018

Conversation

Projects
None yet
2 participants
@gotlieb
Copy link
Contributor

gotlieb commented Dec 10, 2018

@@ -39,7 +39,7 @@ public function doAction(Zend_Controller_Action $action)
{
if ($resultItem instanceof Kaltura_Client_Exception)
{
if ($resultItem->getCode() == 'SERVICE_FORBIDDEN_CONTENT_BLOCKED')
if (in_array($resultItem->getCode() ,array('SERVICE_FORBIDDEN_CONTENT_BLOCKED', 'FEATURE_FORBIDDEN')))
{
$resultMessage = $resultItem->getMessage(). '. ';

This comment has been minimized.

@MosheMaorKaltura

MosheMaorKaltura Dec 10, 2018

Contributor

Before if $resultMessage = $resultItem->getMessage(). '. ';

@@ -11,6 +11,9 @@ public function shouldConsumeJobStatusEvent(BatchJob $dbBatchJob)
if ($dbBatchJob->getJobType() == ReachPlugin::getBatchJobTypeCoreValue(ReachEntryVendorTasksCsvBatchType::ENTRY_VENDOR_TASK_CSV))
return true;
if ($dbBatchJob->getJobType() == KalturaBatchJobType::COPY_PARTNER && $dbBatchJob->getStatus() == KalturaBatchJobStatus::FINISHED)

This comment has been minimized.

@MosheMaorKaltura
if (!ReachPlugin::isAllowedPartner($fromPartnerId) || !ReachPlugin::isAllowedPartner($toPartnerId))
{
KalturaLog::info("Skip copying reach data to partner. Reach plugin is not enabled");

This comment has been minimized.

@MosheMaorKaltura

MosheMaorKaltura Dec 10, 2018

Contributor

Add from and to in the logs

*/
protected function handleCopyReachDataToPartner(BatchJob $dbBatchJob)
{
try

This comment has been minimized.

@MosheMaorKaltura

MosheMaorKaltura Dec 10, 2018

Contributor

remove try

@gotlieb gotlieb merged commit 5cde903 into Naos-14.10.0 Dec 10, 2018

@yossipapi yossipapi deleted the Naos-14.10.0-REACH-CloningTasks branch Dec 10, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment