Differential D182 Diff 475 modules/in-commerce/units/orders/orders_event_handler.php

Changeset View

Standalone View

modules/in-commerce/units/orders/orders_event_handler.php

	Show First 20 Lines • Show All 58 Lines • ▼ Show 20 Line(s)
	if ( isset($field_values[$status_field]) && $order_dummy->GetDBField($status_field) != $field_values[$status_field] ) {			if ( isset($field_values[$status_field]) && $order_dummy->GetDBField($status_field) != $field_values[$status_field] ) {
	// user can't change status by himself			// user can't change status by himself
	return false;			return false;
	}			}

	if ( $order_dummy->GetDBField($status_field) != ORDER_STATUS_INCOMPLETE ) {			if ( $order_dummy->GetDBField($status_field) != ORDER_STATUS_INCOMPLETE ) {
	// user can't edit orders being processed			// user can't edit orders being processed
	return false;			return false;
	}			}

	if ( $event->Name == 'OnUpdate' ) {
	// all checks were ok -> it's user's order -> allow to modify
	return true;
	}
	}			}
				alexUnsubmitted Not Done Reply Please remove this. Use same pattern for order update events (the `CheckPermission` checks request regardless of the event used + mapPermissions allows any event). alex: Please remove this. --- Use same pattern for order update events (the `CheckPermission`…
	}			}
	}			}

	if ( $event->Name == 'OnQuietPreSave' ) {			if ( $event->Name == 'OnQuietPreSave' ) {
	$section = $event->getSection();			$section = $event->getSection();

	if ( $this->isNewItemCreate($event) ) {			if ( $this->isNewItemCreate($event) ) {
	return $this->Application->CheckPermission($section . '.add', 1);			return $this->Application->CheckPermission($section . '.add', 1);
	▲ Show 20 Lines • Show All 41 Lines • ▼ Show 20 Line(s)
	'OnUpdateCartJSON' => Array('self' => true),			'OnUpdateCartJSON' => Array('self' => true),
	'OnUpdateItemOptions' => Array('self' => true),			'OnUpdateItemOptions' => Array('self' => true),
	'OnCleanupCart' => Array('self' => true),			'OnCleanupCart' => Array('self' => true),
	'OnContinueShopping' => Array('self' => true),			'OnContinueShopping' => Array('self' => true),
	'OnCheckout' => Array('self' => true),			'OnCheckout' => Array('self' => true),
	'OnSelectAddress' => Array('self' => true),			'OnSelectAddress' => Array('self' => true),
	'OnProceedToBilling' => Array('self' => true),			'OnProceedToBilling' => Array('self' => true),
	'OnProceedToPreview' => Array('self' => true),			'OnProceedToPreview' => Array('self' => true),
				'OnProceedToPreviewAjax' => array('self' => true),
	'OnCompleteOrder' => Array('self' => true),			'OnCompleteOrder' => Array('self' => true),
				'OnUpdate' => array('self' => true),
	'OnUpdateAjax' => Array('self' => true),			'OnUpdateAjax' => Array('self' => true),
				alexUnsubmitted Not Done Reply Please: add same line, but for `OnUpdate` event confirm, that user still can update his/her order Use same pattern for order update events (the `CheckPermission` checks request regardless of the event used + mapPermissions allows any event). alex: Please: # add same line, but for `OnUpdate` event # confirm, that user still can update…

	'OnRemoveCoupon' => Array('self' => true),			'OnRemoveCoupon' => Array('self' => true),
	'OnRemoveGiftCertificate' => Array('self' => true),			'OnRemoveGiftCertificate' => Array('self' => true),

	'OnCancelRecurring' => Array('self' => true),			'OnCancelRecurring' => Array('self' => true),
	'OnAddVirtualProductToCart' => Array('self' => true),			'OnAddVirtualProductToCart' => Array('self' => true),
	'OnItemBuild' => Array('self' => true),			'OnItemBuild' => Array('self' => true),
	'OnDownloadLabel' => Array('self' => true, 'subitem' => true),			'OnDownloadLabel' => Array('self' => true, 'subitem' => true),
	▲ Show 20 Lines • Show All 3897 Lines • ▼ Show 20 Line(s)
	$object->SetError('BillingEmail', 'unique');			$object->SetError('BillingEmail', 'unique');
	}			}
	}			}
	}			}

	/**			/**
	* Performs order update and returns results in format, needed by FormManager			* Performs order update and returns results in format, needed by FormManager
	*			*
	* @param kEvent $event			* @param kEvent $event Event.
				*
				* @return void
	*/			*/
	function OnUpdateAjax($event)			protected function OnUpdateAjax(kEvent $event)
	{			{
				/** @var AjaxFormHelper $ajax_form_helper */
	$ajax_form_helper = $this->Application->recallObject('AjaxFormHelper');			$ajax_form_helper = $this->Application->recallObject('AjaxFormHelper');
				alexUnsubmitted Not Done Reply Please remove `&` symbols. The `&` is no longer used in most places since In-Commerce 5.2.0 release. alex: Please remove `&` symbols. --- The `&` is no longer used in most places since In-Commerce 5.2.
	/* @var $ajax_form_helper AjaxFormHelper */

	$ajax_form_helper->transitEvent($event, 'OnUpdate');			$ajax_form_helper->transitEvent($event, 'OnUpdate');
				alexUnsubmitted Not Done Reply Please move this above variable declaration. alex: Please move this above variable declaration.
	}			}

				/**
				* Performs order update after billing step submission and returns results in format, needed by FormManager
				*
				* @param kEvent $event Event.
				*
				* @return void
				*/
				protected function OnProceedToPreviewAjax(kEvent $event)
				{
				/** @var AjaxFormHelper $ajax_form_helper */
				alexUnsubmitted Not Done Reply Please remove `&` symbols. The `&` is no longer used in most places since In-Commerce 5.2.0 release. alex: Please remove `&` symbols. --- The `&` is no longer used in most places since In-Commerce 5.2.
				$ajax_form_helper = $this->Application->recallObject('AjaxFormHelper');
				alexUnsubmitted Not Done Reply Please move this above variable declaration. alex: Please move this above variable declaration.
				$ajax_form_helper->transitEvent($event, 'OnProceedToPreview');
				}

	}			}