Index: branches/5.2.x/units/orders/orders_event_handler.php =================================================================== diff -u -N -r16469 -r16516 --- branches/5.2.x/units/orders/orders_event_handler.php (.../orders_event_handler.php) (revision 16469) +++ branches/5.2.x/units/orders/orders_event_handler.php (.../orders_event_handler.php) (revision 16516) @@ -1,6 +1,6 @@ Application->RecallVar($event->getPrefixSpecial(true) . '_id'); + /** @var OrdersItem $order_dummy */ $order_dummy = $this->Application->recallObject($event->Prefix . '.-item', null, Array ('skip_autoload' => true)); - /* @var $order_dummy OrdersItem */ foreach ($items_info as $id => $field_values) { if ( $order_session_id != $id ) { @@ -176,8 +176,8 @@ function OnQuietPreSave($event) { + /** @var kDBItem $object */ $object = $event->getObject(); - /* @var $object kDBItem */ $object->IgnoreValidation = true; $event->CallSubEvent('OnPreSave'); @@ -195,21 +195,21 @@ return ; } + /** @var OrdersItem $object */ $object = $event->getObject(); - /* @var $object OrdersItem */ $shipping_address_id = $this->Application->GetVar('shipping_address_id'); $billing_address_id = $this->Application->GetVar('billing_address_id'); if ($shipping_address_id || $billing_address_id) { + /** @var kCountryStatesHelper $cs_helper */ $cs_helper = $this->Application->recallObject('CountryStatesHelper'); - /* @var $cs_helper kCountryStatesHelper */ + /** @var AddressesItem $address */ $address = $this->Application->recallObject('addr.-item','addr', Array('skip_autoload' => true)); - /* @var $address AddressesItem */ + /** @var AddressesList $addr_list */ $addr_list = $this->Application->recallObject('addr', 'addr_List', Array('per_page'=>-1, 'skip_counting'=>true) ); - /* @var $addr_list AddressesList */ $addr_list->Query(); } @@ -301,8 +301,8 @@ */ function updateUserID($order_id, $event) { + /** @var UsersItem $user */ $user = $this->Application->recallObject('u.current'); - /* @var $user UsersItem */ $affiliate_id = $this->isAffiliate( $user->GetID() ); @@ -327,8 +327,8 @@ function isAffiliate($user_id) { + /** @var kDBItem $affiliate_user */ $affiliate_user = $this->Application->recallObject('affil.-item', null, Array('skip_autoload' => true) ); - /* @var $affiliate_user kDBItem */ $affiliate_user->Load($user_id, 'PortalUserId'); @@ -383,8 +383,8 @@ function PrepareCoupons($event, &$order) { + /** @var kDBList $order_items */ $order_items = $this->Application->recallObject('orditems.-inv','orditems_List',Array('skip_counting'=>true,'per_page'=>-1) ); - /* @var $order_items kDBList */ $order_items->linkToParent($order->Special); $order_items->Query(); @@ -399,8 +399,9 @@ if (isset($item_data['AssignedCoupon']) && $item_data['AssignedCoupon']) { $coupon_id = $item_data['AssignedCoupon']; // clone coupon, get new coupon ID + /** @var kDBItem $coupon */ $coupon = $this->Application->recallObject('coup',null,array('skip_autload' => true)); - /* @var $coupon kDBItem */ + $coupon->Load($coupon_id); if (!$coupon->isLoaded()) continue; @@ -415,7 +416,7 @@ } } - /* @var $order OrdersItem */ + /** @var OrdersItem $order */ if ($assigned_coupons) { $comments = $order->GetDBField('AdminComment'); if ($comments) $comments .= "\r\n"; @@ -445,8 +446,8 @@ $this->ReserveItems($event); + /** @var OrdersItem $order */ $order = $event->getObject(); - /* @var $order OrdersItem */ $charge_result = $this->ChargeOrder($order); @@ -464,8 +465,8 @@ } // call CompleteOrder events for items in order BEFORE SplitOrder (because ApproveEvents are called there) + /** @var kDBList $order_items */ $order_items = $this->Application->recallObject('orditems.-inv','orditems_List',Array('skip_counting'=>true,'per_page'=>-1) ); - /* @var $order_items kDBList */ $order_items->linkToParent($order->Special); $order_items->Query(true); @@ -539,8 +540,8 @@ */ function setBillingAddress($event) { + /** @var OrdersItem $object */ $object = $event->getObject(); - /* @var $object OrdersItem */ if ( $object->HasTangibleItems() ) { if ( $this->Application->GetVar('same_address') ) { @@ -585,8 +586,8 @@ function OnContinueShopping($event) { + /** @var OrderHelper $order_helper */ $order_helper = $this->Application->recallObject('OrderHelper'); - /* @var $order_helper OrderHelper */ $template = $this->Application->GetVar('continue_shopping_template'); @@ -602,8 +603,8 @@ { $this->OnUpdateCart($event); if ( !$event->getEventParam('RecalculateChangedCart') ) { + /** @var OrdersItem $object */ $object = $event->getObject(); - /* @var $object OrdersItem */ if ( !$object->HasTangibleItems() ) { $object->SetDBField('ShippingTo', ''); @@ -680,8 +681,8 @@ if ( $items_info ) { list($id, $field_values) = each($items_info); + /** @var kDBItem $object */ $object = $event->getObject(); - /* @var $object kDBItem */ $payment_type_id = $object->GetDBField('PaymentType'); @@ -708,8 +709,8 @@ */ protected function OnCancelRecurring($event) { + /** @var OrdersItem $order */ $order = $event->getObject(); - /* @var $order OrdersItem */ $order->SetDBField('IsRecurringBilling', 0); $order->Update(); @@ -730,8 +731,8 @@ { parent::OnAfterItemUpdate($event); + /** @var OrdersItem $object */ $object = $event->getObject(); - /* @var $object OrdersItem */ $cvv2 = $object->GetDBField('PaymentCVV2'); @@ -766,8 +767,8 @@ } else { // strange: recalculate total amount on error + /** @var OrdersItem $object */ $object = $event->getObject(); - /* @var $object OrdersItem */ $object->SetDBField('TotalAmount', $object->getTotalAmount()); } @@ -784,16 +785,16 @@ return ; } + /** @var kDBItem $object */ $object = $event->getObject(); - /* @var $object kDBItem */ + /** @var kDBList $addr_list */ $addr_list = $this->Application->recallObject('addr', 'addr_List', Array ('per_page' => -1, 'skip_counting' => true)); - /* @var $addr_list kDBList */ $addr_list->Query(); + /** @var AddressesItem $address_dummy */ $address_dummy = $this->Application->recallObject('addr.-item', null, Array ('skip_autoload' => true)); - /* @var $address_dummy AddressesItem */ $address_prefixes = Array ('Billing', 'Shipping'); $address_fields = Array ( @@ -851,8 +852,8 @@ return; } + /** @var kDBItem $object */ $object = $event->getObject(); - /* @var $object kDBItem */ // 1. delete given order item by id $delete_id = $this->Application->GetVar('delete_id'); @@ -882,8 +883,8 @@ // 4. remove "orditems" object of kDBItem class, since getOrderInfo uses kDBList object under same prefix $this->Application->removeObject('orditems'); + /** @var OrderHelper $order_helper */ $order_helper = $this->Application->recallObject('OrderHelper'); - /* @var $order_helper OrderHelper */ $event->status = kEvent::erSTOP; $currency = $this->Application->GetVar('currency', 'selected'); @@ -938,8 +939,8 @@ $item_data = $default_item_data; + /** @var ProductsItem $product */ $product = $this->Application->recallObject('p', null, Array('skip_autoload' => true)); - /* @var $product ProductsItem */ $product->Load($item_id); @@ -969,8 +970,8 @@ { $package_content_ids = $product->GetPackageContentIds(); + /** @var ProductsItem $product_package_item */ $product_package_item = $this->Application->recallObject('p.-packageitem'); - /* @var $product_package_item ProductsItem */ $package_item_data = array(); @@ -1123,8 +1124,8 @@ if (!$options) return; + /** @var kDBItem $ord_item */ $ord_item = $this->Application->recallObject('orditems.-opt', null, Array ('skip_autoload' => true)); - /* @var $ord_item kDBItem */ $ord_item->Load($this->Application->GetVar('orditems_id')); @@ -1186,11 +1187,11 @@ return ; } + /** @var OrdersItem $object */ $object = $event->getObject(); - /* @var $object OrdersItem */ + /** @var kDBItem $coupon */ $coupon = $this->Application->recallObject('coup', null, Array ('skip_autoload' => true)); - /* @var $coupon kDBItem */ $coupon->Load($code, 'Code'); @@ -1231,8 +1232,9 @@ $coupon->Update(); $this->Application->setUnitOption('ord', 'AutoLoad', true); + + /** @var OrdersItem $order */ $order = $this->Application->recallObject('ord'); - /* @var $order OrdersItem */ $order->SetDBField('CouponId', $coupon->GetDBField('CouponId')); $order->SetDBField('CouponName', $coupon->GetDBField('Name')); // calculated field @@ -1251,8 +1253,8 @@ */ function OnRemoveCoupon($event) { + /** @var OrdersItem $object */ $object = $event->getObject(); - /* @var $object OrdersItem */ $this->RemoveCoupon($object); $object->setCheckoutError(OrderCheckoutErrorType::COUPON, OrderCheckoutError::COUPON_REMOVED); @@ -1267,8 +1269,8 @@ */ function RemoveCoupon(&$object) { + /** @var kDBItem $coupon */ $coupon = $this->Application->recallObject('coup', null, Array('skip_autoload' => true)); - /* @var $coupon kDBItem */ $coupon->Load( $object->GetDBField('CouponId') ); @@ -1411,8 +1413,8 @@ // if we already know, that there is no such order, // then don't run database query, that will confirm that + /** @var kDBItem $object */ $object = $event->getObject(); - /* @var $object kDBItem */ $object->Clear($id); return; @@ -1428,17 +1430,17 @@ */ function _createNewCart($event) { + /** @var kDBItem $object */ $object = $event->getObject( Array('skip_autoload' => true) ); - /* @var $object kDBItem */ $this->setNextOrderNumber($event); $object->SetDBField('Status', ORDER_STATUS_INCOMPLETE); $object->SetDBField('VisitId', $this->Application->RecallVar('visit_id') ); // get user if ( $this->Application->LoggedIn() ) { + /** @var UsersItem $user */ $user = $this->Application->recallObject('u.current'); - /* @var $user UsersItem */ $user_id = $user->GetID(); $object->SetDBField('BillingEmail', $user->GetDBField('Email')); @@ -1522,11 +1524,11 @@ */ function SetStepRequiredFields($event) { + /** @var OrdersItem $order */ $order = $event->getObject(); - /* @var $order OrdersItem */ + /** @var kCountryStatesHelper $cs_helper */ $cs_helper = $this->Application->recallObject('CountryStatesHelper'); - /* @var $cs_helper kCountryStatesHelper */ $items_info = $this->Application->GetVar($event->getPrefixSpecial(true)); if ($items_info) { @@ -1586,8 +1588,8 @@ return; } + /** @var OrdersItem $order */ $order = $event->getObject(); - /* @var $order OrdersItem */ $ses_user = $this->Application->RecallVar('user_id'); @@ -1616,8 +1618,8 @@ { parent::OnPreCreate($event); + /** @var kDBItem $object */ $object = $event->getObject(); - /* @var $object kDBItem */ $this->setNextOrderNumber($event); @@ -1665,8 +1667,8 @@ function OnReserveItems($event) { + /** @var kDBList $order_items */ $order_items = $this->Application->recallObject('orditems.-inv','orditems_List',Array('skip_counting'=>true,'per_page'=>-1) ); - /* @var $order_items kDBList */ $order_items->linkToParent('-inv'); // force re-query, since we are updateing through orditem ITEM, not the list, and @@ -1678,16 +1680,16 @@ // query all combinations used in this order + /** @var kCatDBItem $product_object */ $product_object = $this->Application->recallObject('p', null, Array('skip_autoload' => true)); - /* @var $product_object kCatDBItem */ $product_object->SwitchToLive(); + /** @var kDBItem $order_item */ $order_item = $this->Application->recallObject('orditems.-item', null, Array('skip_autoload' => true)); - /* @var $order_item kDBItem */ + /** @var kDBItem $combination_item */ $combination_item = $this->Application->recallObject('poc.-item', null, Array('skip_autoload' => true)); - /* @var $combination_item kDBItem */ $combinations = $this->queryCombinations($order_items); @@ -1758,8 +1760,8 @@ $from_tab = substr($event->Name, strlen('OnResetTo')); // load values from db + /** @var kDBItem $object */ $object = $event->getObject(); - /* @var $object kDBItem */ // update values from submit $field_values = $this->getSubmittedFields($event); @@ -1947,16 +1949,16 @@ */ function restoreOrder(&$order_items) { + /** @var kCatDBItem $product_object */ $product_object = $this->Application->recallObject('p', null, Array('skip_autoload' => true)); - /* @var $product_object kCatDBItem */ $product_object->SwitchToLive(); + /** @var kDBItem $order_item */ $order_item = $this->Application->recallObject('orditems.-item', null, Array('skip_autoload' => true)); - /* @var $order_item kDBItem */ + /** @var kDBItem $combination_item */ $combination_item = $this->Application->recallObject('poc.-item', null, Array('skip_autoload' => true)); - /* @var $combination_item kDBItem */ $combinations = $this->queryCombinations($order_items); @@ -1985,8 +1987,8 @@ $inv_object->SetDBField('QtyInStock', $inv_object->GetDBField('QtyInStock') + $rec['QuantityReserved']); $inv_object->Update(); + /** @var ProductsEventHandler $product_h */ $product_h = $this->Application->recallObject('p_EventHandler'); - /* @var $product_h ProductsEventHandler */ if ($product_object->GetDBField('InventoryStatus') == 2) { // inventory by options, then restore changed combination values back to common $combinations array !!! @@ -2024,8 +2026,8 @@ } // process order products + /** @var kDBItem $object */ $object = $this->Application->recallObject($event->Prefix . '.-inv', null, Array ('skip_autoload' => true)); - /* @var $object kDBItem */ $ids = $this->StoreSelectedIDs($event); @@ -2057,15 +2059,15 @@ 'OnOrderProcess' => ORDER_STATUS_TOSHIP, ); + /** @var kDBList $order_items */ $order_items = $this->Application->recallObject('orditems.-inv','orditems_List',Array('skip_counting'=>true,'per_page'=>-1) ); - /* @var $order_items kDBList */ $order_items->linkToParent('-inv'); $order_items->Query(); $order_items->GoFirst(); + /** @var OrdersItem $object */ $object = $this->Application->recallObject($event->Prefix.'.-inv'); - /* @var $object OrdersItem */ if ($object->GetDBField('OnHold')) { // any actions have no effect while on hold @@ -2113,8 +2115,8 @@ $object->SetDBField('GWResult2', $gateway_object->getGWResponce() ); if ($charge_result) { + /** @var ProductsItem $product_object */ $product_object = $this->Application->recallObject('p', null, Array('skip_autoload' => true)); - /* @var $product_object ProductsItem */ foreach ($order_items->Records as $product_item) { if (!$product_item['ProductId']) { @@ -2192,17 +2194,17 @@ $shipping_info = $object->GetDBField('ShippingInfo'); if ($shipping_info) { + /** @var ShippingQuoteCollector $quote_engine_collector */ $quote_engine_collector = $this->Application->recallObject('ShippingQuoteCollector'); - /* @var $quote_engine_collector ShippingQuoteCollector */ $shipping_info = unserialize($shipping_info); $sqe_class_name = $quote_engine_collector->GetClassByType($shipping_info, 1); } // try to create usps order if (($object->GetDBField('ShippingType') == 0) && ($sqe_class_name !== false)) { + /** @var ShippingQuoteEngine $shipping_quote_engine */ $shipping_quote_engine = $this->Application->recallObject($sqe_class_name); - /* @var $shipping_quote_engine ShippingQuoteEngine */ $ret = $shipping_quote_engine->MakeOrder($object); } @@ -2411,8 +2413,8 @@ { parent::OnAfterItemLoad($event); + /** @var kDBItem $object */ $object = $event->getObject(); - /* @var $object kDBItem */ // get user fields $user_id = $object->GetDBField('PortalUserId'); @@ -2436,8 +2438,8 @@ $object->SetDBField('PaymentCVV2', $this->Application->RecallVar('CVV2Code')); + /** @var kCountryStatesHelper $cs_helper */ $cs_helper = $this->Application->recallObject('CountryStatesHelper'); - /* @var $cs_helper kCountryStatesHelper */ $cs_helper->PopulateStates($event, 'ShippingState', 'ShippingCountry'); $cs_helper->PopulateStates($event, 'BillingState', 'BillingCountry'); @@ -2459,8 +2461,8 @@ { parent::OnBeforeItemCreate($event); + /** @var kCountryStatesHelper $cs_helper */ $cs_helper = $this->Application->recallObject('CountryStatesHelper'); - /* @var $cs_helper kCountryStatesHelper */ $cs_helper->PopulateStates($event, 'ShippingState', 'ShippingCountry'); $cs_helper->PopulateStates($event, 'BillingState', 'BillingCountry'); @@ -2477,8 +2479,8 @@ { parent::OnBeforeItemUpdate($event); + /** @var OrdersItem $object */ $object = $event->getObject(); - /* @var $object OrdersItem */ $old_payment_type = $object->GetOriginalField('PaymentType'); $new_payment_type = $object->GetDBField('PaymentType'); @@ -2495,8 +2497,8 @@ } } + /** @var kCountryStatesHelper $cs_helper */ $cs_helper = $this->Application->recallObject('CountryStatesHelper'); - /* @var $cs_helper kCountryStatesHelper */ $cs_helper->PopulateStates($event, 'ShippingState', 'ShippingCountry'); $cs_helper->PopulateStates($event, 'BillingState', 'BillingCountry'); @@ -2532,8 +2534,8 @@ // guess fields from "One Step Checkout" form if ( $object->GetDBField('PaymentAccount') ) { + /** @var OrderHelper $order_helper */ $order_helper = $this->Application->recallObject('OrderHelper'); - /* @var $order_helper OrderHelper */ $object->SetDBField( 'PaymentCardType', @@ -2562,11 +2564,11 @@ */ protected function createAccountFromOrder($event) { + /** @var OrdersItem $order */ $order = $event->getObject(); - /* @var $order OrdersItem */ + /** @var OrderHelper $order_helper */ $order_helper = $this->Application->recallObject('OrderHelper'); - /* @var $order_helper OrderHelper */ $user_fields = $order_helper->getUserFields($order); $user_fields['Password'] = $order->GetDBField('UserPassword_plain'); @@ -2578,8 +2580,8 @@ $this->Application->HandleEvent(new kEvent('u.register:OnCreate')); } else { + /** @var UsersItem $user */ $user = $this->Application->recallObject('u.current'); - /* @var $user UsersItem */ $user->SetFieldsFromHash($user_fields); if ( !$user->Update() ) { @@ -2600,8 +2602,8 @@ { parent::SetCustomQuery($event); + /** @var kDBList $object */ $object = $event->getObject(); - /* @var $object kDBList */ $types = $event->getEventParam('types'); if ( $types == 'myorders' || $types == 'myrecentorders' ) { @@ -2908,8 +2910,8 @@ $skip_types = Array (PRODUCT_TYPE_TANGIBLE, PRODUCT_TYPE_DOWNLOADABLE); foreach ($sub_orders as $sub_order_data) { + /** @var OrdersItem $sub_order */ $sub_order = $this->Application->recallObject('ord.-sub'.$next_sub_number, 'ord'); - /* @var $sub_order OrdersItem */ if ( $this->UseTempTables($event) && $next_sub_number == 0 ) { $sub_order =& $order; @@ -3114,16 +3116,16 @@ } // Loading product to add + /** @var kDBItem $product */ $product = $this->Application->recallObject('p.toadd', null, Array('skip_autoload' => true)); - /* @var $product kDBItem */ $product->Load($item_id); + /** @var kDBItem $object */ $object = $this->Application->recallObject('orditems.-item', null, Array('skip_autoload' => true)); - /* @var $object kDBItem */ + /** @var kDBItem $order */ $order = $this->Application->recallObject('ord'); - /* @var $order kDBItem */ if (!$order->isLoaded() && !$this->Application->isAdmin) { // no order was created before -> create one now @@ -3145,8 +3147,8 @@ return; } + /** @var OrderManager $manager */ $manager = $this->Application->recallObject('OrderManager'); - /* @var $manager OrderManager */ $manager->setOrder($order); $manager->addProduct($product, $event->getEventParam('ItemData'), $qty, $package_num); @@ -3166,8 +3168,8 @@ return; } + /** @var OrdersItem $object */ $object = $event->getObject(); - /* @var $object OrdersItem */ $shipping_total = $insurance_fee = 0; $shipping_info = $object->GetDBField('ShippingInfo') ? unserialize($object->GetDBField('ShippingInfo')) : false; @@ -3202,8 +3204,8 @@ return ; } + /** @var OrdersItem $order */ $order = $event->getObject(); - /* @var $order OrdersItem */ if ( !$order->isLoaded() ) { $this->LoadItem($event); // try to load @@ -3218,8 +3220,8 @@ return; } + /** @var OrderManager $manager */ $manager = $this->Application->recallObject('OrderManager'); - /* @var $manager OrderManager */ $manager->setOrder($order); $result = $manager->calculate(); @@ -3328,8 +3330,8 @@ */ function UpdateShippingTypes($event) { + /** @var OrdersItem $object */ $object = $event->getObject(); - /* @var $object OrdersItem */ $ord_id = $object->GetID(); @@ -3347,17 +3349,17 @@ $template = $this->Application->GetVar('t'); $shipping_templates = Array ('in-commerce/checkout/shipping', 'in-commerce/orders/orders_edit_shipping'); + /** @var ShippingQuoteCollector $quote_engine_collector */ $quote_engine_collector = $this->Application->recallObject('ShippingQuoteCollector'); - /* @var $quote_engine_collector ShippingQuoteCollector */ foreach ($shipping_ids as $package => $id) { // try to validate $shipping_types[$package] = $last_shippings[$package][$id]; $sqe_class_name = $quote_engine_collector->GetClassByType($shipping_types, $package); if (($object->GetDBField('ShippingType') == 0) && ($sqe_class_name !== false) && in_array($template, $shipping_templates)) { + /** @var ShippingQuoteEngine $shipping_quote_engine */ $shipping_quote_engine = $this->Application->recallObject($sqe_class_name); - /* @var $shipping_quote_engine ShippingQuoteEngine */ // USPS related part // TODO: remove USPS condition from here @@ -3410,8 +3412,8 @@ function RecalculateTax($event) { + /** @var OrdersItem $object */ $object = $event->getObject(); - /* @var $object OrdersItem */ if ($object->GetDBField('Status') > ORDER_STATUS_PENDING) { return; @@ -3436,8 +3438,8 @@ function UpdateTotals($event) { + /** @var OrdersItem $object */ $object = $event->getObject(); - /* @var $object OrdersItem */ $object->UpdateTotals(); } @@ -3504,8 +3506,8 @@ $prefix_special = $event->Prefix.'.'.$this->getSpecialByType($order_status); + /** @var kDBList $orders_list */ $orders_list = $this->Application->recallObject($prefix_special, $event->Prefix.'_List', Array('per_page'=>-1) ); - /* @var $orders_list kDBList */ $orders_list->Query(); @@ -3529,8 +3531,8 @@ */ function OnResetToPending($event) { + /** @var kDBItem $object */ $object = $event->getObject( Array('skip_autoload' => true) ); - /* @var $object kDBItem */ $items_info = $this->Application->GetVar($event->getPrefixSpecial(true)); @@ -3665,8 +3667,8 @@ $this->Application->InitParser(); $o = $this->Application->ParseBlock(array('name'=>'in-commerce/orders/orders_pdf')); + /** @var FileHelper $file_helper */ $file_helper = $this->Application->recallObject('FileHelper'); - /* @var $file_helper FileHelper */ $file_helper->CheckFolder(EXPORT_PATH); @@ -3763,8 +3765,8 @@ $fields['Number']['format'] = str_replace('%06d', '%0'.$order_number.'d', $fields['Number']['format']); $fields['SubNumber']['format'] = str_replace('%03d', '%0'.$order_sub_number.'d', $fields['SubNumber']['format']); + /** @var SiteHelper $site_helper */ $site_helper = $this->Application->recallObject('SiteHelper'); - /* @var $site_helper SiteHelper */ $fields['BillingCountry']['default'] = $site_helper->getDefaultCountry('Billing'); $fields['ShippingCountry']['default'] = $site_helper->getDefaultCountry('Shipping'); @@ -3921,8 +3923,8 @@ return true; } + /** @var kDBItem $object */ $object = $event->getObject(); - /* @var $object kDBItem */ if ( !$object->isLoaded() ) { return true; @@ -3945,11 +3947,11 @@ return; } + /** @var OrdersItem $object */ $object = $event->getObject(); - /* @var $object OrdersItem */ + /** @var kDBItem $gift_certificate */ $gift_certificate = $this->Application->recallObject('gc', null, Array ('skip_autoload' => true)); - /* @var $gift_certificate kDBItem */ $gift_certificate->Load($code, 'Code'); @@ -3986,8 +3988,8 @@ */ function OnRemoveGiftCertificate($event) { + /** @var OrdersItem $object */ $object = $event->getObject(); - /* @var $object OrdersItem */ $this->RemoveGiftCertificate($object); $object->setCheckoutError(OrderCheckoutErrorType::GIFT_CERTIFICATE, OrderCheckoutError::GC_REMOVED); @@ -4002,8 +4004,8 @@ function RecalculateGift($event) { + /** @var OrdersItem $object */ $object = $event->getObject(); - /* @var $object OrdersItem */ if ($object->GetDBField('Status') > ORDER_STATUS_PENDING) { return ; @@ -4034,8 +4036,8 @@ ini_set('memory_limit', '300M'); ini_set('max_execution_time', '0'); + /** @var kDBItem $object */ $object = $event->getObject(); - /* @var $object kDBItem */ $file = $object->GetDBField('ShippingTracking') . '.pdf'; $full_path = USPS_LABEL_FOLDER . $file; @@ -4062,8 +4064,8 @@ $create_account = $this->Application->GetVar('create_account'); + /** @var kDBItem $object */ $object = $event->getObject(); - /* @var $object kDBItem */ $required_fields = Array ('UserPassword', 'UserPassword_plain', 'VerifyUserPassword', 'VerifyUserPassword_plain'); $object->setRequired($required_fields, $create_account);