Order

Complexity

Total Complexity

551

Size/Duplication

Total Lines	3792
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	67

Importance

Changes

0

173 Methods

Rating	Name	Duplication	Size	Complexity
A	i18n_singular_name()	0	4	1
A	i18n_plural_name()	0	4	1
A	set_needs_recalculating()	0	4	1
A	get_needs_recalculating()	0	4	2
B	getModifierForms()	0	22	6
A	get_order_status_options()	0	4	1
A	get_by_id_if_can_view()	0	14	4
A	get_datalist_of_orders_with_submit_record()	0	22	3
C	scaffoldSearchFields()	0	49	10
A	CMSEditLink()	0	4	1
F	getCMSFields()	0	390	33
A	getOrderItemsField()	0	7	1
A	getModifierTableField()	0	7	1
A	getBillingAddressField()	0	16	1
A	getShippingAddressField()	0	16	1
A	getOrderStatusLogsTableField()	0	15	2
A	getOrderStatusLogsTableFieldEditable()	0	10	1
A	getOrderStatusLogsTableField_Archived()	0	17	4
A	getEmailsTableField()	0	8	1
A	getPaymentsField()	0	9	1
A	OrderStepField()	0	4	1
C	init()	0	60	17
C	tryToFinaliseOrder()	0	55	12
A	doNextStatus()	0	15	4
B	Cancel()	0	33	6
A	Archive()	0	23	3
B	MyStep()	0	22	5
A	RelevantLogEntry()	0	4	1
A	CurrentStepVisibleToCustomer()	0	12	3
A	IsFirstStep()	0	12	4
A	IsInCart()	0	4	2
A	IsPastCart()	0	4	2
A	IsUncomplete()	0	4	1
A	IsProcessing()	0	4	1
A	IsCompleted()	0	4	1
A	IsPaid()	0	8	3
B	PaymentIsPending()	0	16	7
A	RelevantPayments()	0	10	2
A	IsPaidNice()	0	4	1
A	getIsPaidNice()	0	4	2
A	IsCancelled()	0	4	1
A	getIsCancelled()	0	4	2
A	IsCustomerCancelled()	0	8	3
B	IsAdminCancelled()	0	15	5
A	ShopClosed()	0	4	1
C	CreateOrReturnExistingMember()	0	23	8
D	CreateOrReturnExistingAddress()	0	44	13
B	SetCountryFields()	0	19	7
B	SetRegionFields()	0	15	5
A	UpdateCurrency()	0	13	3
A	SetCurrency()	0	4	1
A	sendEmail()	0	15	1
A	sendAdminNotification()	0	15	1
A	renderOrderInEmailFormat()	0	14	1
C	prepareAndSendEmail()	0	60	10
B	createReplacementArrayForEmail()	0	29	3
C	getOrderAttributesByType()	0	24	8
A	Items()	0	8	2
A	OrderItems()	0	4	1
A	Buyables()	0	10	2
A	itemsFromDatabase()	0	14	3
A	OrderModifiers()	0	4	1
A	Modifiers()	0	4	1
A	modifiersFromDatabase()	0	14	3
B	calculateOrderAttributes()	0	14	6
A	calculateOrderItems()	0	15	4
A	calculateModifiers()	0	12	4
D	ModifiersSubTotal()	0	27	10
A	RetrieveModifier()	0	11	4
A	getMemberForCanFunctions()	0	12	3
A	canCreate()	0	11	3
C	canView()	0	45	11
B	canOverrideCanView()	0	21	5
A	IsInSession()	0	6	3
A	LessSecureSessionID()	0	8	4
A	canViewAdminStuff()	0	11	3
B	canEdit()	0	19	6
A	canCheckout()	0	14	4
B	canSubmit()	0	17	5
B	canPay()	0	13	5
B	canCancel()	0	17	5
A	canDelete()	0	16	4
A	CanViewOrderStatusLogs()	0	12	3
A	CustomerViewableOrderStatusLogs()	0	14	4
A	OrderEmail()	0	4	1
B	getOrderEmail()	0	18	9
A	HasPrintOrEmailLink()	0	4	2
A	EmailLink()	0	4	1
A	getEmailLink()	0	8	3
A	PrintLink()	0	4	1
A	getPrintLink()	0	8	3
A	PackingSlipLink()	0	4	1
A	getPackingSlipLink()	0	6	2
A	RetrieveLink()	0	4	1
A	getRetrieveLink()	0	14	3
A	ShareLink()	0	4	1
A	getShareLink()	0	18	2
A	FeedbackLink()	0	4	1
A	getFeedbackLink()	0	8	2
A	DeleteLink()	0	4	1
A	getDeleteLink()	0	8	2
A	CopyOrderLink()	0	4	1
A	getCopyOrderLink()	0	8	3
A	Title()	0	4	1
F	getTitle()	0	59	20
A	SubTotal()	0	4	1
A	getSubTotal()	0	14	4
A	SubTotalAsCurrencyObject()	0	4	1
A	SubTotalAsMoney()	0	4	1
A	getSubTotalAsMoney()	0	4	1
A	ModifiersSubTotalAsCurrencyObject()	0	4	1
A	ModifiersSubTotalAsMoneyObject()	0	4	1
A	Total()	0	4	1
A	getTotal()	0	4	1
A	TotalAsCurrencyObject()	0	4	1
A	TotalAsMoney()	0	4	1
A	getTotalAsMoney()	0	4	1
A	TotalOutstanding()	0	4	1
A	getTotalOutstanding()	0	16	3
A	TotalOutstandingAsCurrencyObject()	0	4	1
A	TotalOutstandingAsMoney()	0	4	1
A	getTotalOutstandingAsMoney()	0	4	1
A	TotalPaid()	0	4	1
B	getTotalPaid()	0	17	6
A	TotalPaidAsCurrencyObject()	0	4	1
A	TotalPaidAsMoney()	0	4	1
A	getTotalPaidAsMoney()	0	4	1
A	TotalItems()	0	4	1
A	getTotalItems()	0	10	3
A	MoreThanOneItemInCart()	0	4	2
A	TotalItemsTimesQuantity()	0	4	1
A	getTotalItemsTimesQuantity()	0	16	3
A	Country()	0	4	1
C	getCountry()	0	37	13
A	FullNameCountry()	0	4	1
A	getFullNameCountry()	0	4	1
A	ExpectedCountryName()	0	4	1
A	getExpectedCountryName()	0	4	1
A	FixedCountry()	0	4	1
A	getFixedCountry()	0	9	2
A	Region()	0	4	1
C	getRegion()	0	33	12
A	HasAlternativeCurrency()	0	4	1
A	getHasAlternativeCurrency()	0	12	3
B	EnsureCorrectExchangeRate()	0	18	5
A	IsSubmitted()	0	4	1
A	getIsSubmitted()	0	12	4
A	IsArchived()	0	10	3
A	SubmissionLog()	0	8	1
A	OrderDate()	0	11	2
A	SecondsSinceBeingSubmitted()	0	8	2
B	SubmitErrors()	0	16	7
A	CustomerStatus()	0	4	1
B	getCustomerStatus()	0	18	6
A	CanHaveShippingAddress()	0	4	1
A	getCanHaveShippingAddress()	0	4	1
B	DisplayPage()	0	18	5
A	Link()	0	16	3
A	APILink()	0	4	1
A	CheckoutLink()	0	15	3
A	ConvertToHTML()	0	10	1
A	ConvertToString()	0	4	1
A	ConvertToJSON()	0	4	1
A	addHasOneAndHasManyAsVariables()	0	13	1
A	AJAXDefinitions()	0	4	1
A	EcomConfig()	0	4	1
B	updateForAjax()	0	53	5
A	SubTotalCartValue()	0	4	1
A	populateDefaults()	0	4	1
A	onBeforeWrite()	0	15	4
C	onAfterWrite()	0	24	7
B	onBeforeDelete()	0	45	3
A	debug()	0	6	1

<?php

/**
 * @description:
 * The order class is a databound object for handling Orders within SilverStripe.
 * Note that it works closely with the ShoppingCart class, which accompanies the Order
 * until it has been paid for / confirmed by the user.
 *
 *
 * CONTENTS:
 * ----------------------------------------------
 * 1. CMS STUFF
 * 2. MAIN TRANSITION FUNCTIONS
 * 3. STATUS RELATED FUNCTIONS / SHORTCUTS
 * 4. LINKING ORDER WITH MEMBER AND ADDRESS
 * 5. CUSTOMER COMMUNICATION
 * 6. ITEM MANAGEMENT
 * 7. CRUD METHODS (e.g. canView, canEdit, canDelete, etc...)
 * 8. GET METHODS (e.g. Total, SubTotal, Title, etc...)
 * 9. TEMPLATE RELATED STUFF
 * 10. STANDARD SS METHODS (requireDefaultRecords, onBeforeDelete, etc...)
 * 11. DEBUG
 *
 * @authors: Nicolaas [at] Sunny Side Up .co.nz
 * @package: ecommerce
 * @sub-package: model
 * @inspiration: Silverstripe Ltd, Jeremy
 *
 * NOTE: This is the SQL for selecting orders in sequence of
 **/
class Order extends DataObject implements EditableEcommerceObject

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    /**
     * API Control.
     *
     * @var array
     */
    private static $api_access = array(

The property $api_access is not used and could be removed.

Consider using a different property name as you override a private property of the parent class.

        'view' => array(
            'OrderEmail',
            'EmailLink',
            'PrintLink',
            'RetrieveLink',
            'ShareLink',
            'FeedbackLink',
            'Title',
            'Total',
            'SubTotal',
            'TotalPaid',
            'TotalOutstanding',
            'ExchangeRate',
            'CurrencyUsed',
            'TotalItems',
            'TotalItemsTimesQuantity',
            'IsCancelled',
            'Country',
            'FullNameCountry',
            'IsSubmitted',
            'CustomerStatus',
            'CanHaveShippingAddress',
            'CancelledBy',
            'CurrencyUsed',
            'BillingAddress',
            'UseShippingAddress',
            'ShippingAddress',
            'Status',
            'Attributes',
            'OrderStatusLogs',
            'MemberID',
        ),
    );

    /**
     * standard SS variable.
     *
     * @var array
     */
    private static $db = array(

Consider using a different property name as you override a private property of the parent class.

The property $db is not used and could be removed.

        'SessionID' => 'Varchar(32)', //so that in the future we can link sessions with Orders.... One session can have several orders, but an order can onnly have one session
        'UseShippingAddress' => 'Boolean',
        'CustomerOrderNote' => 'Text',
        'ExchangeRate' => 'Double',
        //'TotalItems_Saved' => 'Double',

67% of this comment could be valid code. Did you maybe forget this after debugging?

        //'TotalItemsTimesQuantity_Saved' => 'Double'
    );

    private static $has_one = array(

Consider using a different property name as you override a private property of the parent class.

The property $has_one is not used and could be removed.

        'Member' => 'Member',
        'BillingAddress' => 'BillingAddress',
        'ShippingAddress' => 'ShippingAddress',
        'Status' => 'OrderStep',
        'CancelledBy' => 'Member',
        'CurrencyUsed' => 'EcommerceCurrency',
    );

    /**
     * standard SS variable.
     *
     * @var array
     */
    private static $has_many = array(

Consider using a different property name as you override a private property of the parent class.

The property $has_many is not used and could be removed.

        'Attributes' => 'OrderAttribute',
        'OrderStatusLogs' => 'OrderStatusLog',
        'Payments' => 'EcommercePayment',
        'Emails' => 'OrderEmailRecord',
        'OrderProcessQueue' => 'OrderProcessQueue' //there is usually only one.
    );

    /**
     * standard SS variable.
     *
     * @var array
     */
    private static $indexes = array(

Consider using a different property name as you override a private property of the parent class.

The property $indexes is not used and could be removed.

        'SessionID' => true,
    );

    /**
     * standard SS variable.
     *
     * @var string
     */
    private static $default_sort = '"LastEdited" DESC';

Consider using a different property name as you override a private property of the parent class.

The property $default_sort is not used and could be removed.

    /**
     * standard SS variable.
     *
     * @var array
     */
    private static $casting = array(

Consider using a different property name as you override a private property of the parent class.

The property $casting is not used and could be removed.

        'OrderEmail' => 'Varchar',
        'EmailLink' => 'Varchar',
        'PrintLink' => 'Varchar',
        'ShareLink' => 'Varchar',
        'FeedbackLink' => 'Varchar',
        'RetrieveLink' => 'Varchar',
        'Title' => 'Varchar',
        'Total' => 'Currency',
        'TotalAsMoney' => 'Money',
        'SubTotal' => 'Currency',
        'SubTotalAsMoney' => 'Money',
        'TotalPaid' => 'Currency',
        'TotalPaidAsMoney' => 'Money',
        'TotalOutstanding' => 'Currency',
        'TotalOutstandingAsMoney' => 'Money',
        'HasAlternativeCurrency' => 'Boolean',
        'TotalItems' => 'Double',
        'TotalItemsTimesQuantity' => 'Double',
        'IsCancelled' => 'Boolean',
        'IsPaidNice' => 'Varchar',
        'Country' => 'Varchar(3)', //This is the applicable country for the order - for tax purposes, etc....
        'FullNameCountry' => 'Varchar',
        'IsSubmitted' => 'Boolean',
        'CustomerStatus' => 'Varchar',
        'CanHaveShippingAddress' => 'Boolean',
    );

    /**
     * standard SS variable.
     *
     * @var string
     */
    private static $singular_name = 'Order';

Consider using a different property name as you override a private property of the parent class.

The property $singular_name is not used and could be removed.

    public function i18n_singular_name()
    {
        return _t('Order.ORDER', 'Order');
    }

    /**
     * standard SS variable.
     *
     * @var string
     */
    private static $plural_name = 'Orders';

Consider using a different property name as you override a private property of the parent class.

The property $plural_name is not used and could be removed.

    public function i18n_plural_name()
    {
        return _t('Order.ORDERS', 'Orders');
    }

    /**
     * Standard SS variable.
     *
     * @var string
     */
    private static $description = "A collection of items that together make up the 'Order'.  An order can be placed.";

The property $description is not used and could be removed.

    /**
     * Tells us if an order needs to be recalculated
     * can save one for each order...
     *
     * @var array
     */
    private static $_needs_recalculating = array();

    /**
     * @param bool (optional) $b
     * @param int (optional)  $orderID
     *
     * @return bool

Should the return type not be boolean|null?

     */
    public static function set_needs_recalculating($b = true, $orderID = 0)
    {
        self::$_needs_recalculating[$orderID] = $b;
    }

    /**
     * @param int (optional) $orderID
     *
     * @return bool
     */
    public static function get_needs_recalculating($orderID = 0)
    {
        return isset(self::$_needs_recalculating[$orderID]) ? self::$_needs_recalculating[$orderID] : false;
    }

    /**
     * Total Items : total items in cart
     * We start with -1 to easily identify if it has been run before.
     *
     * @var int
     */
    protected $totalItems = null;

    /**
     * Total Items : total items in cart
     * We start with -1 to easily identify if it has been run before.
     *
     * @var float
     */
    protected $totalItemsTimesQuantity = null;

    /**
     * Returns a set of modifier forms for use in the checkout order form,
     * Controller is optional, because the orderForm has its own default controller.
     *
     * This method only returns the Forms that should be included outside
     * the editable table... Forms within it can be called
     * from through the modifier itself.
     *
     * @param Controller $optionalController

Should the type for parameter $optionalController not be null|Controller?

     * @param Validator  $optionalValidator

Should the type for parameter $optionalValidator not be null|Validator?

     *
     * @return ArrayList (ModifierForms) | Null

Should the return type not be ArrayList|null?

     **/
    public function getModifierForms(Controller $optionalController = null, Validator $optionalValidator = null)
    {
        $arrayList = new ArrayList();
        $modifiers = $this->Modifiers();
        if ($modifiers->count()) {
            foreach ($modifiers as $modifier) {
                if ($modifier->ShowForm()) {
                    if ($form = $modifier->getModifierForm($optionalController, $optionalValidator)) {
                        $form->ShowFormInEditableOrderTable = $modifier->ShowFormInEditableOrderTable();
                        $form->ShowFormOutsideEditableOrderTable = $modifier->ShowFormOutsideEditableOrderTable();
                        $form->ModifierName = $modifier->ClassName;
                        $arrayList->push($form);
                    }
                }
            }
        }
        if ($arrayList->count()) {
            return $arrayList;
        } else {
            return;
        }
    }

    /**
     * This function returns the OrderSteps.
     *
     * @return ArrayList (OrderSteps)

Should the return type not be DataList?

     **/
    public static function get_order_status_options()
    {
        return OrderStep::get();
    }

    /**
     * Like the standard byID, but it checks whether we are allowed to view the order.
     *
     * @return: Order | Null
     **/
    public static function get_by_id_if_can_view($id)
    {
        $order = Order::get()->byID($id);
        if ($order && $order->canView()) {
            if ($order->IsSubmitted()) {
                // LITTLE HACK TO MAKE SURE WE SHOW THE LATEST INFORMATION!
                $order->tryToFinaliseOrder();
            }

            return $order;
        }

        return;
    }

    /**
     * returns a Datalist with the submitted order log included
     * this allows you to sort the orders by their submit dates.
     * You can retrieve this list and then add more to it (e.g. additional filters, additional joins, etc...).
     *
     * @param bool $onlySubmittedOrders - only include Orders that have already been submitted.
     * @param bool $includeCancelledOrders - only include Orders that have already been submitted.
     *
     * @return DataList (Orders)
     */
    public static function get_datalist_of_orders_with_submit_record($onlySubmittedOrders = true, $includeCancelledOrders = false)
    {
        if ($onlySubmittedOrders) {
            $submittedOrderStatusLogClassName = EcommerceConfig::get('OrderStatusLog', 'order_status_log_class_used_for_submitting_order');
            $list = Order::get()
                ->LeftJoin('OrderStatusLog', '"Order"."ID" = "OrderStatusLog"."OrderID"')
                ->LeftJoin($submittedOrderStatusLogClassName, '"OrderStatusLog"."ID" = "'.$submittedOrderStatusLogClassName.'"."ID"')
                ->Sort('OrderStatusLog.Created', 'ASC');
            $where = ' ("OrderStatusLog"."ClassName" = \''.$submittedOrderStatusLogClassName.'\') ';
        } else {
            $list = Order::get();
            $where = ' ("StatusID" > 0) ';
        }
        if ($includeCancelledOrders) {

This if statement is empty and can be removed.

            //do nothing...
        } else {
            $where .= ' AND ("CancelledByID" = 0 OR "CancelledByID" IS NULL)';
        }
        $list = $list->where($where);

        return $list;
    }

/*******************************************************
   * 1. CMS STUFF
*******************************************************/

    /**
     * fields that we remove from the parent::getCMSFields object set.
     *
     * @var array
     */
    protected $fieldsAndTabsToBeRemoved = array(
        'MemberID',
        'Attributes',
        'SessionID',
        'Emails',
        'BillingAddressID',
        'ShippingAddressID',
        'UseShippingAddress',
        'OrderStatusLogs',
        'Payments',
        'OrderDate',
        'ExchangeRate',
        'CurrencyUsedID',
        'StatusID',
        'Currency',
    );

    /**
     * STANDARD SILVERSTRIPE STUFF.
     **/
    private static $summary_fields = array(

Consider using a different property name as you override a private property of the parent class.

The property $summary_fields is not used and could be removed.

        'Title' => 'Title',
        'Status.Title' => 'Next Step',
        'Member.Surname' => 'Name',
        'Member.Email' => 'Email',
        'TotalAsMoney.Nice' => 'Total',
        'TotalItemsTimesQuantity' => 'Units',
        'IsPaidNice' => 'Paid'
    );

    /**
     * STANDARD SILVERSTRIPE STUFF.
     *
     * @todo: how to translate this?
     **/
    private static $searchable_fields = array(

Consider using a different property name as you override a private property of the parent class.

The property $searchable_fields is not used and could be removed.

        'ID' => array(
            'field' => 'NumericField',
            'title' => 'Order Number',
        ),
        'MemberID' => array(
            'field' => 'TextField',
            'filter' => 'OrderFilters_MemberAndAddress',
            'title' => 'Customer Details',
        ),
        'Created' => array(
            'field' => 'TextField',
            'filter' => 'OrderFilters_AroundDateFilter',
            'title' => 'Date (e.g. Today, 1 jan 2007, or last week)',
        ),
        //make sure to keep the items below, otherwise they do not show in form
        'StatusID' => array(
            'filter' => 'OrderFilters_MultiOptionsetStatusIDFilter',
        ),
        'CancelledByID' => array(
            'filter' => 'OrderFilters_HasBeenCancelled',
            'title' => 'Cancelled by ...',
        ),
    );

    /**
     * Determine which properties on the DataObject are
     * searchable, and map them to their default {@link FormField}
     * representations. Used for scaffolding a searchform for {@link ModelAdmin}.
     *
     * Some additional logic is included for switching field labels, based on
     * how generic or specific the field type is.
     *
     * Used by {@link SearchContext}.
     *
     * @param array $_params

Should the type for parameter $_params not be array|null?

     *                       'fieldClasses': Associative array of field names as keys and FormField classes as values
     *                       'restrictFields': Numeric array of a field name whitelist
     *
     * @return FieldList
     */
    public function scaffoldSearchFields($_params = null)

scaffoldSearchFields uses the super-global variable $_GET which is generally not recommended.

    {
        $fieldList = parent::scaffoldSearchFields($_params);

        //for sales to action only show relevant ones ...
        if(Controller::curr() && Controller::curr()->class === 'SalesAdmin') {
            $statusOptions = OrderStep::admin_manageable_steps();
        } else {
            $statusOptions = OrderStep::get();
        }
        if ($statusOptions && $statusOptions->count()) {
            $createdOrderStatusID = 0;

$createdOrderStatusID is not used, you could remove the assignment.

            $preSelected = array();
            $createdOrderStatus = $statusOptions->First();
            if ($createdOrderStatus) {
                $createdOrderStatusID = $createdOrderStatus->ID;

$createdOrderStatusID is not used, you could remove the assignment.

            }
            $arrayOfStatusOptions = clone $statusOptions->map('ID', 'Title');
            $arrayOfStatusOptionsFinal = array();
            if (count($arrayOfStatusOptions)) {
                foreach ($arrayOfStatusOptions as $key => $value) {
                    if (isset($_GET['q']['StatusID'][$key])) {
                        $preSelected[$key] = $key;
                    }
                    $count = Order::get()
                        ->Filter(array('StatusID' => intval($key)))
                        ->count();
                    if ($count < 1) {

This if statement is empty and can be removed.

                        //do nothing
                    } else {
                        $arrayOfStatusOptionsFinal[$key] = $value." ($count)";
                    }
                }
            }
            $statusField = new CheckboxSetField(
                'StatusID',
                Injector::inst()->get('OrderStep')->i18n_singular_name(),
                $arrayOfStatusOptionsFinal,
                $preSelected

$preSelected is of type array, but the function expects a string.

            );
            $fieldList->push($statusField);
        }
        $fieldList->push(new DropdownField('CancelledByID', 'Cancelled', array(-1 => '(Any)', 1 => 'yes', 0 => 'no')));

        //allow changes
        $this->extend('scaffoldSearchFields', $fieldList, $_params);

        return $fieldList;
    }

    /**
     * link to edit the record.
     *
     * @param string | Null $action - e.g. edit
     *
     * @return string
     */
    public function CMSEditLink($action = null)
    {
        return CMSEditLinkAPI::find_edit_link_for_object($this, $action, 'sales-advanced');

The call to CMSEditLinkAPI::find_edit_link_for_object() has too many arguments starting with 'sales-advanced'.

    }

    /**
     * STANDARD SILVERSTRIPE STUFF
     * broken up into submitted and not (yet) submitted.
     **/
    public function getCMSFields()

The return type could not be reliably inferred; please add a @return annotation.

getCMSFields uses the super-global variable $_SERVER which is generally not recommended.

    {
        $fields = $this->scaffoldFormFields(array(
            // Don't allow has_many/many_many relationship editing before the record is first saved
            'includeRelations' => false,
            'tabbed' => true,
            'ajaxSafe' => true
        ));
        $fields->insertBefore(
            Tab::create(
                'Next',
                _t('Order.NEXT_TAB', 'Action')
            ),
            'Main'

'Main' is of type string, but the function expects a object<FormField>.

        );
        $fields->addFieldsToTab(
            'Root',
            array(
                Tab::create(
                    "Items",
                    _t('Order.ITEMS_TAB', 'Items')
                ),
                Tab::create(
                    "Extras",
                    _t('Order.MODIFIERS_TAB', 'Adjustments')
                ),
                Tab::create(
                    'Emails',
                    _t('Order.EMAILS_TAB', 'Emails')
                ),
                Tab::create(
                    'Payments',
                    _t('Order.PAYMENTS_TAB', 'Payment')
                ),
                Tab::create(
                    'Account',
                    _t('Order.ACCOUNT_TAB', 'Account')
                ),
                Tab::create(
                    'Currency',
                    _t('Order.CURRENCY_TAB', 'Currency')
                ),
                Tab::create(
                    'Addresses',
                    _t('Order.ADDRESSES_TAB', 'Addresses')
                ),
                Tab::create(
                    'Log',
                    _t('Order.LOG_TAB', 'Notes')
                ),
                Tab::create(
                    'Cancellations',
                    _t('Order.CANCELLATION_TAB', 'Cancel')
                ),
            )
        );
        //as we are no longer using the parent:;getCMSFields
        // we had to add the updateCMSFields hook.
        $this->extend('updateCMSFields', $fields);
        $currentMember = Member::currentUser();
        if (!$this->exists() || !$this->StatusID) {

The property StatusID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

            $firstStep = DataObject::get_one('OrderStep');
            $this->StatusID = $firstStep->ID;

The property StatusID does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

            $this->write();
        }
        $submitted = $this->IsSubmitted() ? true : false;
        if ($submitted) {
            //TODO
            //Having trouble here, as when you submit the form (for example, a payment confirmation)
            //as the step moves forward, meaning the fields generated are incorrect, causing an error
            //"I can't handle sub-URLs of a Form object." generated by the RequestHandler.
            //Therefore we need to try reload the page so that it will be requesting the correct URL to generate the correct fields for the current step
            //Or something similar.
            //why not check if the URL == $this->CMSEditLink()
            //and only tryToFinaliseOrder if this is true....
            if ($_SERVER['REQUEST_URI'] == $this->CMSEditLink() || $_SERVER['REQUEST_URI'] == $this->CMSEditLink('edit')) {
                $this->tryToFinaliseOrder();
            }
        } else {
            $this->init(true);
            $this->calculateOrderAttributes(true);
            Session::set('EcommerceOrderGETCMSHack', $this->ID);
        }
        if ($submitted) {
            $this->fieldsAndTabsToBeRemoved[] = 'CustomerOrderNote';
        } else {
            $this->fieldsAndTabsToBeRemoved[] = 'Emails';
        }
        foreach ($this->fieldsAndTabsToBeRemoved as $field) {
            $fields->removeByName($field);
        }
        $orderSummaryConfig = GridFieldConfig_Base::create();
        $orderSummaryConfig->removeComponentsByType('GridFieldToolbarHeader');
        // $orderSummaryConfig->removeComponentsByType('GridFieldSortableHeader');

75% of this comment could be valid code. Did you maybe forget this after debugging?

        $orderSummaryConfig->removeComponentsByType('GridFieldFilterHeader');
        $orderSummaryConfig->removeComponentsByType('GridFieldPageCount');
        $orderSummaryConfig->removeComponentsByType('GridFieldPaginator');
        $nextFieldArray = array(
            LiteralField::create('CssFix', '<style>#Root_Next h2.form-control {padding: 0!important; margin: 0!important; padding-top: 4em!important;}</style>'),
            HeaderField::create('MyOrderStepHeader', _t('Order.CURRENT_STATUS', '1. Current Status')),
            $this->OrderStepField(),
            GridField::create(
                'OrderSummary',
                _t('Order.CURRENT_STATUS', 'Summary'),
                ArrayList::create(array($this)),
                $orderSummaryConfig
            )
        );
        $keyNotes = OrderStatusLog::get()->filter(
            array(
                'OrderID' => $this->ID,
                'ClassName' => 'OrderStatusLog'
            )
        );
        if ($keyNotes->count()) {
            $notesSummaryConfig = GridFieldConfig_RecordViewer::create();
            $notesSummaryConfig->removeComponentsByType('GridFieldToolbarHeader');
            $notesSummaryConfig->removeComponentsByType('GridFieldFilterHeader');
            // $orderSummaryConfig->removeComponentsByType('GridFieldSortableHeader');

75% of this comment could be valid code. Did you maybe forget this after debugging?

            $notesSummaryConfig->removeComponentsByType('GridFieldPageCount');
            $notesSummaryConfig->removeComponentsByType('GridFieldPaginator');
            $nextFieldArray = array_merge(
                $nextFieldArray,
                array(
                    HeaderField::create('KeyNotesHeader', _t('Order.KEY_NOTES_HEADER', 'Key Notes')),
                    GridField::create(
                        'OrderStatusLogSummary',
                        _t('Order.CURRENT_KEY_NOTES', 'Key Notes'),
                        $keyNotes,
                        $notesSummaryConfig
                    )
                )
            );
        }
        $nextFieldArray = array_merge(
            $nextFieldArray,
            array(
                EcommerceCMSButtonField::create(
                    'AddNoteButton',
                    '/admin/sales/Order/EditForm/field/Order/item/' . $this->ID . '/ItemEditForm/field/OrderStatusLog/item/new',
                    _t('Order.ADD_NOTE', 'Add Note')
                )
            )
        );
        $nextFieldArray = array_merge(
            $nextFieldArray,
            array(

            )
        );

         //is the member is a shop admin they can always view it

        if (EcommerceRole::current_member_can_process_orders(Member::currentUser())) {
            $lastStep = OrderStep::last_order_step();
            if($this->StatusID != $lastStep->ID) {

The property StatusID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                $queueObjectSingleton = Injector::inst()->get('OrderProcessQueue');
                if ($myQueueObject = $queueObjectSingleton->getQueueObject($this)) {

$myQueueObject is not used, you could remove the assignment.

                    $myQueueObjectField = GridField::create(
                        'MyQueueObjectField',
                        _t('Order.QUEUE_DETAILS', 'Queue Details'),
                        $this->OrderProcessQueue(),

The method OrderProcessQueue does not exist on object<Order>? Since you implemented __call, maybe consider adding a @method annotation.

                        GridFieldConfig_RecordEditor::create()
                    );
                } else {
                    $myQueueObjectField = LiteralField::create('MyQueueObjectField', '<p>'._t('Order.NOT_QUEUED','This order is not queued for future processing.').'</p>');
                }
                $nextFieldArray = array_merge(
                    $nextFieldArray,
                    array(
                        HeaderField::create('OrderStepNextStepHeader', _t('Order.ACTION_NEXT_STEP', '2. Action Next Step')),
                        $myQueueObjectField,
                        HeaderField::create('ActionNextStepManually', _t('Order.MANUAL_STATUS_CHANGE', '3. Move Order Along')),
                        LiteralField::create('OrderStepNextStepHeaderExtra', '<p>'._t('Order.NEEDTOREFRESH', 'Once you have made any changes to the order then you will have to refresh below or save it to move it along.').'</p>'),
                        EcommerceCMSButtonField::create(
                            'StatusIDExplanation',
                            $this->CMSEditLink(),
                            _t('Order.REFRESH', 'refresh now')
                        )
                    )
                );
            }
        }
        $fields->addFieldsToTab(
            'Root.Next',
            $nextFieldArray
        );

        $this->MyStep()->addOrderStepFields($fields, $this);

        if ($submitted) {
            $permaLinkLabel = _t('Order.PERMANENT_LINK', 'Customer Link');
            $html = '<p>'.$permaLinkLabel.': <a href="'.$this->getRetrieveLink().'">'.$this->getRetrieveLink().'</a></p>';
            $shareLinkLabel = _t('Order.SHARE_LINK', 'Share Link');
            $html .= '<p>'.$shareLinkLabel.': <a href="'.$this->getShareLink().'">'.$this->getShareLink().'</a></p>';
            $feedbackLinkLabel = _t('Order.FEEDBACK_LINK', 'Feedback Link');
            $html .= '<p>'.$feedbackLinkLabel.': <a href="'.$this->getFeedbackLink().'">'.$this->getFeedbackLink().'</a></p>';
            $js = "window.open(this.href, 'payment', 'toolbar=0,scrollbars=1,location=1,statusbar=1,menubar=0,resizable=1,width=800,height=600'); return false;";
            $link = $this->getPrintLink();
            $label = _t('Order.PRINT_INVOICE', 'invoice');
            $linkHTML = '<a href="'.$link.'" onclick="'.$js.'">'.$label.'</a>';
            $linkHTML .= ' | ';
            $link = $this->getPackingSlipLink();
            $label = _t('Order.PRINT_PACKING_SLIP', 'packing slip');
            $labelPrint = _t('Order.PRINT', 'Print');
            $linkHTML .= '<a href="'.$link.'" onclick="'.$js.'">'.$label.'</a>';
            $html .= '<h3>';
            $html .= $labelPrint.': '.$linkHTML;
            $html .= '</h3>';

            $fields->addFieldToTab(
                'Root.Main',
                LiteralField::create('getPrintLinkANDgetPackingSlipLink', $html)
            );

            //add order here as well.
            $fields->addFieldToTab(
                'Root.Main',
                new LiteralField(
                    'MainDetails',
                    '<iframe src="'.$this->getPrintLink().'" width="100%" height="2500" style="border: 5px solid #2e7ead; border-radius: 2px;"></iframe>')
            );
            $fields->addFieldsToTab(
                'Root.Items',
                array(
                    GridField::create(
                        'Items_Sold',
                        'Items Sold',
                        $this->Items(),
                        new GridFieldConfig_RecordViewer
                    )
                )
            );
            $fields->addFieldsToTab(
                'Root.Extras',
                array(
                    GridField::create(
                        'Modifications',
                        'Price (and other) adjustments',
                        $this->Modifiers(),
                        new GridFieldConfig_RecordViewer
                    )
                )
            );
            $fields->addFieldsToTab(
                'Root.Emails',
                array(
                    $this->getEmailsTableField()
                )
            );
            $fields->addFieldsToTab(
                'Root.Payments',
                array(
                    $this->getPaymentsField(),
                    new ReadOnlyField('TotalPaidNice', _t('Order.TOTALPAID', 'Total Paid'), $this->TotalPaidAsCurrencyObject()->Nice()),
                    new ReadOnlyField('TotalOutstandingNice', _t('Order.TOTALOUTSTANDING', 'Total Outstanding'), $this->getTotalOutstandingAsMoney()->Nice())
                )
            );
            if ($this->canPay()) {
                $link = EcommercePaymentController::make_payment_link($this->ID);
                $js = "window.open(this.href, 'payment', 'toolbar=0,scrollbars=1,location=1,statusbar=1,menubar=0,resizable=1,width=800,height=600'); return false;";
                $header = _t('Order.MAKEPAYMENT', 'make payment');
                $label = _t('Order.MAKEADDITIONALPAYMENTNOW', 'make additional payment now');
                $linkHTML = '<a href="'.$link.'" onclick="'.$js.'">'.$label.'</a>';
                $fields->addFieldToTab('Root.Payments', new HeaderField('MakeAdditionalPaymentHeader', $header, 3));
                $fields->addFieldToTab('Root.Payments', new LiteralField('MakeAdditionalPayment', $linkHTML));
            }
            //member
            $member = $this->Member();

The method Member() does not exist on Order. Did you maybe mean CreateOrReturnExistingMember()?

            if ($member && $member->exists()) {
                $fields->addFieldToTab('Root.Account', new LiteralField('MemberDetails', $member->getEcommerceFieldsForCMS()));
            } else {
                $fields->addFieldToTab('Root.Account', new LiteralField('MemberDetails',
                    '<p>'._t('Order.NO_ACCOUNT', 'There is no --- account --- associated with this order').'</p>'
                ));
            }
            if($this->getFeedbackLink()) {

The expression $this->getFeedbackLink() of type string|null is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

                $fields->addFieldToTab(
                    'Root.Account',
                    GridField::create(
                        'OrderFeedback',
                        Injector::inst()->get('OrderFeedback')->singular_name(),
                        OrderFeedback::get()->filter(array('OrderID' => $this->ID)),
                        GridFieldConfig_RecordViewer::create()
                    )
                );
            }
            $cancelledField = $fields->dataFieldByName('CancelledByID');
            $fields->removeByName('CancelledByID');
            $shopAdminAndCurrentCustomerArray = EcommerceRole::list_of_admins(true);
            if ($member && $member->exists()) {
                $shopAdminAndCurrentCustomerArray[$member->ID] = $member->getName();
            }
            if ($this->CancelledByID) {

The property CancelledByID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                if ($cancellingMember = $this->CancelledBy()) {

The method CancelledBy does not exist on object<Order>? Since you implemented __call, maybe consider adding a @method annotation.

                    $shopAdminAndCurrentCustomerArray[$this->CancelledByID] = $cancellingMember->getName();

The property CancelledByID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                }
            }
            if ($this->canCancel()) {
                $fields->addFieldsToTab(
                    'Root.Cancellations',
                    array(
                        DropdownField::create(
                            'CancelledByID',
                            $cancelledField->Title(),
                            $shopAdminAndCurrentCustomerArray
                        )
                    )
                );
            } else {
                $cancelledBy = isset($shopAdminAndCurrentCustomerArray[$this->CancelledByID]) && $this->CancelledByID ? $shopAdminAndCurrentCustomerArray[$this->CancelledByID] : _t('Order.NOT_CANCELLED', 'not cancelled');

The property CancelledByID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                $fields->addFieldsToTab(
                    'Root.Cancellations',
                    ReadonlyField::create(
                        'CancelledByDisplay',
                        $cancelledField->Title(),
                        $cancelledBy

                    )
                );
            }
            $fields->addFieldToTab('Root.Log', $this->getOrderStatusLogsTableField_Archived());
            $submissionLog = $this->SubmissionLog();
            if ($submissionLog) {
                $fields->addFieldToTab('Root.Log',
                    ReadonlyField::create(
                        'SequentialOrderNumber',
                        _t('Order.SEQUENTIALORDERNUMBER', 'Consecutive order number'),
                        $submissionLog->SequentialOrderNumber
                    )->setRightTitle('e.g. 1,2,3,4,5...')
                );
            }
        } else {
            $linkText = _t(
                'Order.LOAD_THIS_ORDER',
                'load this order'
            );
            $message = _t(
                'Order.NOSUBMITTEDYET',
                'No details are shown here as this order has not been submitted yet. You can {link} to submit it... NOTE: For this, you will be logged in as the customer and logged out as (shop)admin .',
                array('link' => '<a href="'.$this->getRetrieveLink().'" data-popup="true">'.$linkText.'</a>')

array('link' => '<a href="' . $this->getRetrieveLink() . '" data-popup="true">' . $linkText . '</a>') is of type array<string,string,{"link":"string"}>, but the function expects a string.

            );
            $fields->addFieldToTab('Root.Next', new LiteralField('MainDetails', '<p>'.$message.'</p>'));
            $fields->addFieldToTab('Root.Items', $this->getOrderItemsField());
            $fields->addFieldToTab('Root.Extras', $this->getModifierTableField());

            //MEMBER STUFF
            $specialOptionsArray = array();
            if ($this->MemberID) {

The property MemberID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                $specialOptionsArray[0] = _t('Order.SELECTCUSTOMER', '--- Remover Customer ---');
                $specialOptionsArray[$this->MemberID] = _t('Order.LEAVEWITHCURRENTCUSTOMER', '- Leave with current customer: ').$this->Member()->getTitle();

The property MemberID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

The method Member() does not exist on Order. Did you maybe mean CreateOrReturnExistingMember()?

            } elseif ($currentMember) {
                $specialOptionsArray[0] = _t('Order.SELECTCUSTOMER', '--- Select Customers ---');
                $currentMemberID = $currentMember->ID;
                $specialOptionsArray[$currentMemberID] = _t('Order.ASSIGNTHISORDERTOME', '- Assign this order to me: ').$currentMember->getTitle();
            }
            //MEMBER FIELD!!!!!!!

70% of this comment could be valid code. Did you maybe forget this after debugging?

            $memberArray = $specialOptionsArray + EcommerceRole::list_of_customers(true);
            $fields->addFieldToTab('Root.Next', new DropdownField('MemberID', _t('Order.SELECTCUSTOMER', 'Select Customer'), $memberArray), 'CustomerOrderNote');
            $memberArray = null;

$memberArray is not used, you could remove the assignment.

        }
        $fields->addFieldToTab('Root.Addresses', new HeaderField('BillingAddressHeader', _t('Order.BILLINGADDRESS', 'Billing Address')));

        $fields->addFieldToTab('Root.Addresses', $this->getBillingAddressField());

        if (EcommerceConfig::get('OrderAddress', 'use_separate_shipping_address')) {
            $fields->addFieldToTab('Root.Addresses', new HeaderField('ShippingAddressHeader', _t('Order.SHIPPINGADDRESS', 'Shipping Address')));
            $fields->addFieldToTab('Root.Addresses', new CheckboxField('UseShippingAddress', _t('Order.USESEPERATEADDRESS', 'Use separate shipping address?')));
            if ($this->UseShippingAddress) {

The property UseShippingAddress does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                $fields->addFieldToTab('Root.Addresses', $this->getShippingAddressField());
            }
        }
        $currencies = EcommerceCurrency::get_list();
        if ($currencies && $currencies->count()) {
            $currencies = $currencies->map()->toArray();
            $fields->addFieldToTab('Root.Currency', new ReadOnlyField('ExchangeRate ', _t('Order.EXCHANGERATE', 'Exchange Rate'), $this->ExchangeRate));

The property ExchangeRate does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

            $fields->addFieldToTab('Root.Currency', $currencyField = new DropdownField('CurrencyUsedID', _t('Order.CurrencyUsed', 'Currency Used'), $currencies));
            if ($this->IsSubmitted()) {
                $fields->replaceField('CurrencyUsedID', $fields->dataFieldByName('CurrencyUsedID')->performReadonlyTransformation());
            }
        } else {
            $fields->addFieldToTab('Root.Currency', new LiteralField('CurrencyInfo', '<p>You can not change currencies, because no currencies have been created.</p>'));
            $fields->replaceField('CurrencyUsedID', $fields->dataFieldByName('CurrencyUsedID')->performReadonlyTransformation());
        }
        $fields->addFieldToTab('Root.Log', new ReadonlyField('Created', _t('Root.CREATED', 'Created')));
        $fields->addFieldToTab('Root.Log', new ReadonlyField('LastEdited', _t('Root.LASTEDITED', 'Last saved')));
        $this->extend('updateCMSFields', $fields);

        return $fields;
    }

    /**
     * Field to add and edit Order Items.
     *
     * @return GridField
     */
    protected function getOrderItemsField()
    {
        $gridFieldConfig = GridFieldConfigForOrderItems::create();
        $source = $this->OrderItems();

        return new GridField('OrderItems', _t('OrderItems.PLURALNAME', 'Order Items'), $source, $gridFieldConfig);
    }

    /**
     * Field to add and edit Modifiers.
     *
     * @return GridField
     */
    public function getModifierTableField()
    {
        $gridFieldConfig = GridFieldConfigForOrderItems::create();
        $source = $this->Modifiers();

        return new GridField('OrderModifiers', _t('OrderItems.PLURALNAME', 'Order Items'), $source, $gridFieldConfig);
    }

    /**
     *@return GridField
     **/
    protected function getBillingAddressField()
    {
        $this->CreateOrReturnExistingAddress('BillingAddress');
        $gridFieldConfig = GridFieldConfig::create()->addComponents(
            new GridFieldToolbarHeader(),
            new GridFieldSortableHeader(),
            new GridFieldDataColumns(),
            new GridFieldPaginator(10),
            new GridFieldEditButton(),
            new GridFieldDetailForm()
        );
        //$source = $this->BillingAddress();

60% of this comment could be valid code. Did you maybe forget this after debugging?

        $source = BillingAddress::get()->filter(array('OrderID' => $this->ID));

        return new GridField('BillingAddress', _t('BillingAddress.SINGULARNAME', 'Billing Address'), $source, $gridFieldConfig);
    }

    /**
     *@return GridField
     **/
    protected function getShippingAddressField()
    {
        $this->CreateOrReturnExistingAddress('ShippingAddress');
        $gridFieldConfig = GridFieldConfig::create()->addComponents(
            new GridFieldToolbarHeader(),
            new GridFieldSortableHeader(),
            new GridFieldDataColumns(),
            new GridFieldPaginator(10),
            new GridFieldEditButton(),
            new GridFieldDetailForm()
        );
        //$source = $this->ShippingAddress();

60% of this comment could be valid code. Did you maybe forget this after debugging?

        $source = ShippingAddress::get()->filter(array('OrderID' => $this->ID));

        return new GridField('ShippingAddress', _t('BillingAddress.SINGULARNAME', 'Shipping Address'), $source, $gridFieldConfig);
    }

    /**
     * Needs to be public because the OrderStep::getCMSFIelds accesses it.
     *
     * @param string    $sourceClass
     * @param string    $title
     *
     * @return GridField
     **/
    public function getOrderStatusLogsTableField(
        $sourceClass = 'OrderStatusLog',
        $title = ''
    ) {
        $gridFieldConfig = GridFieldConfig_RecordViewer::create()->addComponents(
            new GridFieldAddNewButton('toolbar-header-right'),
            new GridFieldDetailForm()
        );
        $title ? $title : $title = _t('OrderStatusLog.PLURALNAME', 'Order Status Logs');
        $source = $this->OrderStatusLogs()->Filter(array('ClassName' => $sourceClass));

The method OrderStatusLogs() does not exist on Order. Did you maybe mean getOrderStatusLogsTableField()?

        $gf = new GridField($sourceClass, $title, $source, $gridFieldConfig);
        $gf->setModelClass($sourceClass);

        return $gf;
    }

    /**
     * Needs to be public because the OrderStep::getCMSFIelds accesses it.
     *
     * @param string    $sourceClass
     * @param string    $title
     *
     * @return GridField
     **/
    public function getOrderStatusLogsTableFieldEditable(
        $sourceClass = 'OrderStatusLog',
        $title = ''
    ) {
        $gf = $this->getOrderStatusLogsTableField($sourceClass, $title);
        $gf->getConfig()->addComponents(
            new GridFieldEditButton()
        );
        return $gf;
    }

    /**
     * @param string    $sourceClass
     * @param string    $title
     * @param FieldList $fieldList          (Optional)

Should the type for parameter $fieldList not be null|FieldList?

     * @param FieldList $detailedFormFields (Optional)

Should the type for parameter $detailedFormFields not be null|FieldList?

     *
     * @return GridField
     **/
    protected function getOrderStatusLogsTableField_Archived(
        $sourceClass = 'OrderStatusLog',
        $title = '',
        FieldList $fieldList = null,

The parameter $fieldList is not used and could be removed.

        FieldList $detailedFormFields = null

The parameter $detailedFormFields is not used and could be removed.

    ) {
        $title ? $title : $title = _t('OrderLog.PLURALNAME', 'Order Log');
        $source = $this->OrderStatusLogs();

The method OrderStatusLogs() does not exist on Order. Did you maybe mean getOrderStatusLogsTableField()?

        if ($sourceClass != 'OrderStatusLog' && class_exists($sourceClass)) {
            $source = $source->filter(array('ClassName' => ClassInfo::subclassesFor($sourceClass)));
        }
        $gridField = GridField::create($sourceClass, $title, $source, $config = GridFieldConfig_RelationEditor::create());
        $config->removeComponentsByType('GridFieldAddExistingAutocompleter');
        $config->removeComponentsByType('GridFieldDeleteAction');

        return $gridField;
    }

    /**
     * @return GridField
     **/
    public function getEmailsTableField()
    {
        $gridFieldConfig = GridFieldConfig_RecordViewer::create()->addComponents(
            new GridFieldDetailForm()
        );

        return new GridField('Emails', _t('Order.CUSTOMER_EMAILS', 'Customer Emails'), $this->Emails(), $gridFieldConfig);

The method Emails() does not exist on Order. Did you maybe mean getEmailsTableField()?

    }

    /**
     * @return GridField
     */
    protected function getPaymentsField()
    {
        $gridFieldConfig = GridFieldConfig_RecordViewer::create()->addComponents(
            new GridFieldDetailForm(),
            new GridFieldEditButton()
        );

        return new GridField('Payments', _t('Order.PAYMENTS', 'Payments'), $this->Payments(), $gridFieldConfig);

The method Payments() does not exist on Order. Did you maybe mean getPaymentsField()?

    }

    /**
     * @return OrderStepField
     */
    public function OrderStepField()
    {
        return OrderStepField::create($name = 'MyOrderStep', $this, Member::currentUser());
    }

/*******************************************************
   * 2. MAIN TRANSITION FUNCTIONS
*******************************************************/

    /**
     * init runs on start of a new Order (@see onAfterWrite)
     * it adds all the modifiers to the orders and the starting OrderStep.
     *
     * @param bool $recalculate
     *
     * @return DataObject (Order)
     **/
    public function init($recalculate = false)
    {
        if ($this->IsSubmitted()) {
            user_error('Can not init an order that has been submitted', E_USER_NOTICE);
        } else {
            //to do: check if shop is open....
            if ($this->StatusID || $recalculate) {

The property StatusID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                if (!$this->StatusID) {

The property StatusID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                    $createdOrderStatus = DataObject::get_one('OrderStep');
                    if (!$createdOrderStatus) {
                        user_error('No ordersteps have been created', E_USER_WARNING);
                    }
                    $this->StatusID = $createdOrderStatus->ID;

The property StatusID does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

                }
                $createdModifiersClassNames = array();
                $modifiersAsArrayList = new ArrayList();
                $modifiers = $this->modifiersFromDatabase($includingRemoved = true);

$includingRemoved = true is of type boolean, but the function expects a string.

                if ($modifiers->count()) {
                    foreach ($modifiers as $modifier) {
                        $modifiersAsArrayList->push($modifier);
                    }
                }
                if ($modifiersAsArrayList->count()) {
                    foreach ($modifiersAsArrayList as $modifier) {
                        $createdModifiersClassNames[$modifier->ID] = $modifier->ClassName;
                    }
                } else {

This else statement is empty and can be removed.

                }
                $modifiersToAdd = EcommerceConfig::get('Order', 'modifiers');
                if (is_array($modifiersToAdd) && count($modifiersToAdd) > 0) {
                    foreach ($modifiersToAdd as $numericKey => $className) {
                        if (!in_array($className, $createdModifiersClassNames)) {
                            if (class_exists($className)) {
                                $modifier = new $className();
                                //only add the ones that should be added automatically
                                if (!$modifier->DoNotAddAutomatically()) {
                                    if (is_a($modifier, 'OrderModifier')) {
                                        $modifier->OrderID = $this->ID;
                                        $modifier->Sort = $numericKey;
                                        //init method includes a WRITE
                                        $modifier->init();
                                        //IMPORTANT - add as has_many relationship  (Attributes can be a modifier OR an OrderItem)
                                        $this->Attributes()->add($modifier);

The method Attributes() does not exist on Order. Did you maybe mean getOrderAttributesByType()?

                                        $modifiersAsArrayList->push($modifier);
                                    }
                                }
                            } else {
                                user_error('reference to a non-existing class: '.$className.' in modifiers', E_USER_NOTICE);
                            }
                        }
                    }
                }
                $this->extend('onInit', $this);
                //careful - this will call "onAfterWrite" again
                $this->write();
            }
        }

        return $this;
    }

    /**
     * @var array
     */
    private static $_try_to_finalise_order_is_running = array();

    /**
     * Goes through the order steps and tries to "apply" the next status to the order.
     *
     * @param bool $runAgain
     * @param bool $fromOrderQueue - is it being called from the OrderProcessQueue (or similar)
     *
     * @return null
     **/
    public function tryToFinaliseOrder($runAgain = false, $fromOrderQueue = false)
    {
        if (empty(self::$_try_to_finalise_order_is_running[$this->ID]) || $runAgain) {
            self::$_try_to_finalise_order_is_running[$this->ID] = true;

            //if the order has been cancelled then we do not process it ...
            if ($this->CancelledByID) {

The property CancelledByID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                $this->Archive(true);

                return;
            }
            // if it is in the queue it has to run from the queue tasks
            // if it ruins from the queue tasks then it has to be one currently processing.
            $queueObjectSingleton = Injector::inst()->get('OrderProcessQueue');
            if ($myQueueObject = $queueObjectSingleton->getQueueObject($this)) {
                if($fromOrderQueue) {
                    if ( ! $myQueueObject->InProcess) {

                        return;
                    }
                } else {

                    return;
                }
            }
            //a little hack to make sure we do not rely on a stored value
            //of "isSubmitted"
            $this->_isSubmittedTempVar = -1;

The property $_isSubmittedTempVar was declared of type boolean, but -1 is of type integer. Maybe add a type cast?

            //status of order is being progressed
            $nextStatusID = $this->doNextStatus();
            if ($nextStatusID) {
                $nextStatusObject = OrderStep::get()->byID($nextStatusID);
                if ($nextStatusObject) {
                    $delay = $nextStatusObject->CalculatedDeferTimeInSeconds($this);
                    if ($delay > 0) {
                        //adjust delay time from seconds since being submitted
                        if ($nextStatusObject->DeferFromSubmitTime) {
                            $delay = $delay - $this->SecondsSinceBeingSubmitted();
                            if ($delay < 0) {
                                $delay = 0;
                            }
                        }
                        $queueObjectSingleton->AddOrderToQueue(
                            $this,
                            $delay
                        );
                    } else {
                        //status has been completed, so it can be released
                        self::$_try_to_finalise_order_is_running[$this->ID] = false;
                        $this->tryToFinaliseOrder($runAgain, $fromOrderQueue);
                    }
                }
            }
        }
    }

    /**
     * Goes through the order steps and tries to "apply" the next step
     * Step is updated after the other one is completed...
     *
     * @return int (StatusID or false if the next status can not be "applied")
     **/
    public function doNextStatus()
    {
        if ($this->MyStep()->initStep($this)) {
            if ($this->MyStep()->doStep($this)) {
                if ($nextOrderStepObject = $this->MyStep()->nextStep($this)) {
                    $this->StatusID = $nextOrderStepObject->ID;

The property StatusID does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

                    $this->write();

                    return $this->StatusID;

The property StatusID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                }
            }
        }

        return 0;
    }

    /**
     * cancel an order.
     *
     * @param Member $member - (optional) the user cancelling the order

Should the type for parameter $member not be Member|null?

     * @param string $reason - (optional) the reason the order is cancelled
     *
     * @return OrderStatusLog_Cancel
     */
    public function Cancel($member = null, $reason = '')
    {
        if($member && $member instanceof Member) {

This if statement is empty and can be removed.

            //we have a valid member
        } else {
            $member = EcommerceRole::get_default_shop_admin_user();
        }
        if($member) {
            //archive and write
            $this->Archive($avoidWrites = true);
            if($avoidWrites) {
                DB::query('Update "Order" SET CancelledByID = '.$member->ID.' WHERE ID = '.$this->ID.' LIMIT 1;');
            } else {
                $this->CancelledByID = $member->ID;

The property CancelledByID does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

                $this->write();
            }
            //create log ...
            $log = OrderStatusLog_Cancel::create();
            $log->AuthorID = $member->ID;

The property AuthorID does not exist on object<OrderStatusLog_Cancel>. Since you implemented __set, maybe consider adding a @property annotation.

            $log->OrderID = $this->ID;

The property OrderID does not exist on object<OrderStatusLog_Cancel>. Since you implemented __set, maybe consider adding a @property annotation.

            $log->Note = $reason;

The property Note does not exist on object<OrderStatusLog_Cancel>. Since you implemented __set, maybe consider adding a @property annotation.

            if ($member->IsShopAdmin()) {
                $log->InternalUseOnly = true;

The property InternalUseOnly does not exist on object<OrderStatusLog_Cancel>. Since you implemented __set, maybe consider adding a @property annotation.

            }
            $log->write();
            //remove from queue ...
            $queueObjectSingleton = Injector::inst()->get('OrderProcessQueue');
            $ordersinQueue = $queueObjectSingleton->removeOrderFromQueue($this);

$ordersinQueue is not used, you could remove the assignment.

            $this->extend('doCancel', $member, $log);

            return $log;
        }
    }

    /**
     * returns true if successful.
     *
     * @param bool $avoidWrites
     *
     * @return bool
     */
    public function Archive($avoidWrites = true)
    {
        $lastOrderStep = OrderStep::last_order_step();
        if ($lastOrderStep) {
            if ($avoidWrites) {
                DB::query('
                    UPDATE "Order"
                    SET "Order"."StatusID" = '.$lastOrderStep->ID.'
                    WHERE "Order"."ID" = '.$this->ID.'
                    LIMIT 1
                ');

                return true;
            } else {
                $this->StatusID = $lastOrderStep->ID;

The property StatusID does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

                $this->write();

                return true;
            }
        }

        return false;
    }

/*******************************************************
   * 3. STATUS RELATED FUNCTIONS / SHORTCUTS
*******************************************************/

    /**
     * Avoids caching of $this->Status().
     *
     * @return DataObject (current OrderStep)
     */
    public function MyStep()
    {
        $step = null;
        if ($this->StatusID) {

The property StatusID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

            $step = OrderStep::get()->byID($this->StatusID);

The property StatusID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

        }
        if (! $step) {
            $step = DataObject::get_one(
                'OrderStep',
                null,
                $cacheDataObjectGetOne = false
            );
        }
        if (! $step) {
            $step = OrderStep_Created::create();
        }
        if (! $step) {
            user_error('You need an order step in your Database.');
        }

        return $step;
    }

    /**
     * Return the OrderStatusLog that is relevant to the Order status.
     *
     * @return OrderStatusLog
     */
    public function RelevantLogEntry()
    {
        return $this->MyStep()->RelevantLogEntry($this);
    }

    /**
     * @return OrderStep (current OrderStep that can be seen by customer)
     */
    public function CurrentStepVisibleToCustomer()
    {
        $obj = $this->MyStep();
        if ($obj->HideStepFromCustomer) {
            $obj = OrderStep::get()->where('"OrderStep"."Sort" < '.$obj->Sort.' AND "HideStepFromCustomer" = 0')->Last();
            if (!$obj) {
                $obj = DataObject::get_one('OrderStep');
            }
        }

        return $obj;
    }

    /**
     * works out if the order is still at the first OrderStep.
     *
     * @return bool
     */
    public function IsFirstStep()
    {
        $firstStep = DataObject::get_one('OrderStep');
        $currentStep = $this->MyStep();
        if ($firstStep && $currentStep) {
            if ($firstStep->ID == $currentStep->ID) {
                return true;
            }
        }

        return false;
    }

    /**
     * Is the order still being "edited" by the customer?
     *
     * @return bool
     */
    public function IsInCart()
    {
        return (bool) $this->IsSubmitted() ? false : true;
    }

    /**
     * The order has "passed" the IsInCart phase.
     *
     * @return bool
     */
    public function IsPastCart()
    {
        return (bool) $this->IsInCart() ? false : true;
    }

    /**
     * Are there still steps the order needs to go through?
     *
     * @return bool
     */
    public function IsUncomplete()
    {
        return (bool) $this->MyStep()->ShowAsUncompletedOrder;
    }

    /**
     * Is the order in the :"processing" phaase.?
     *
     * @return bool
     */
    public function IsProcessing()
    {
        return (bool) $this->MyStep()->ShowAsInProcessOrder;
    }

    /**
     * Is the order completed?
     *
     * @return bool
     */
    public function IsCompleted()
    {
        return (bool) $this->MyStep()->ShowAsCompletedOrder;
    }

    /**
     * Has the order been paid?
     * TODO: why do we check if there is a total at all?
     *
     * @return bool
     */
    public function IsPaid()
    {
        if ($this->IsSubmitted()) {
            return (bool) (($this->Total() >= 0) && ($this->TotalOutstanding() <= 0));
        }

        return false;
    }

    /**
     * @alias for getIsPaidNice
     * @return string
     */
    public function IsPaidNice()
    {
        return $this->getIsPaidNice();
    }


    public function getIsPaidNice()
    {
        return $this->IsPaid() ? 'yes' : 'no';
    }


    /**
     * Has the order been paid?
     * TODO: why do we check if there is a total at all?
     *
     * @return bool
     */
    public function PaymentIsPending()
    {
        if ($this->IsSubmitted()) {
            if ($this->IsPaid()) {

This if statement is empty and can be removed.

                //do nothing;
            } elseif (($payments = $this->Payments()) && $payments->count()) {

The method Payments() does not exist on Order. Did you maybe mean getPaymentsField()?

                foreach ($payments as $payment) {
                    if ('Pending' == $payment->Status) {
                        return true;
                    }
                }
            }
        }

        return false;
    }

    /**
     * shows payments that are meaningfull
     * if the order has been paid then only show successful payments.
     *
     * @return DataList
     */
    public function RelevantPayments()
    {
        if ($this->IsPaid()) {
            return $this->Payments("\"Status\" = 'Success'");

The method Payments() does not exist on Order. Did you maybe mean getPaymentsField()?

            //EcommercePayment::get()->

67% of this comment could be valid code. Did you maybe forget this after debugging?

            //	filter(array("OrderID" => $this->ID, "Status" => "Success"));

64% of this comment could be valid code. Did you maybe forget this after debugging?

        } else {
            return $this->Payments();

The method Payments() does not exist on Order. Did you maybe mean getPaymentsField()?

        }
    }


    /**
     * Has the order been cancelled?
     *
     * @return bool
     */
    public function IsCancelled()
    {
        return $this->getIsCancelled();
    }
    public function getIsCancelled()
    {
        return $this->CancelledByID ? true : false;

The property CancelledByID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    /**
     * Has the order been cancelled by the customer?
     *
     * @return bool
     */
    public function IsCustomerCancelled()
    {
        if ($this->MemberID > 0 && $this->MemberID == $this->IsCancelledID) {

The property MemberID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

The property IsCancelledID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

This if statement, and the following return statement can be replaced with return $this->MemberID > 0 && $this->MemberID == $this->IsCancelledID;.

            return true;
        }

        return false;
    }

    /**
     * Has the order been cancelled by the  administrator?
     *
     * @return bool
     */
    public function IsAdminCancelled()
    {
        if ($this->IsCancelled()) {
            if (!$this->IsCustomerCancelled()) {
                $admin = Member::get()->byID($this->CancelledByID);

The property CancelledByID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                if ($admin) {
                    if ($admin->IsShopAdmin()) {
                        return true;
                    }
                }
            }
        }

        return false;
    }

    /**
     * Is the Shop Closed for business?
     *
     * @return bool
     */
    public function ShopClosed()
    {
        return EcomConfig()->ShopClosed;
    }

/*******************************************************
   * 4. LINKING ORDER WITH MEMBER AND ADDRESS
*******************************************************/

    /**
     * Returns a member linked to the order.
     * If a member is already linked, it will return the existing member.
     * Otherwise it will return a new Member.
     *
     * Any new member is NOT written, because we dont want to create a new member unless we have to!
     * We will not add a member to the order unless a new one is created in the checkout
     * OR the member is logged in / logs in.
     *
     * Also note that if a new member is created, it is not automatically written
     *
     * @param bool $forceCreation - if set to true then the member will always be saved in the database.
     *
     * @return Member
     **/
    public function CreateOrReturnExistingMember($forceCreation = false)
    {
        if ($this->IsSubmitted()) {
            return $this->Member();

The method Member() does not exist on Order. Did you maybe mean CreateOrReturnExistingMember()?

        }
        if ($this->MemberID) {

The property MemberID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

            $member = $this->Member();

The method Member() does not exist on Order. Did you maybe mean CreateOrReturnExistingMember()?

$member is not used, you could remove the assignment.

        } elseif ($member = Member::currentUser()) {
            if (!$member->IsShopAdmin()) {
                $this->MemberID = $member->ID;

The property MemberID does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

                $this->write();
            }
        }
        $member = $this->Member();

The method Member() does not exist on Order. Did you maybe mean CreateOrReturnExistingMember()?

        if (!$member) {
            $member = new Member();
        }
        if ($member && $forceCreation) {
            $member->write();
        }

        return $member;
    }

    /**
     * Returns either the existing one or a new Order Address...
     * All Orders will have a Shipping and Billing address attached to it.
     * Method used to retrieve object e.g. for $order->BillingAddress(); "BillingAddress" is the method name you can use.
     * If the method name is the same as the class name then dont worry about providing one.
     *
     * @param string $className             - ClassName of the Address (e.g. BillingAddress or ShippingAddress)
     * @param string $alternativeMethodName - method to retrieve Address
     **/
    public function CreateOrReturnExistingAddress($className = 'BillingAddress', $alternativeMethodName = '')

The return type could not be reliably inferred; please add a @return annotation.

    {
        if ($this->exists()) {
            $methodName = $className;
            if ($alternativeMethodName) {
                $methodName = $alternativeMethodName;
            }
            if ($this->IsSubmitted()) {
                return $this->$methodName();
            }
            $variableName = $className.'ID';
            $address = null;
            if ($this->$variableName) {
                $address = $this->$methodName();
            }
            if (!$address) {
                $address = new $className();
                if ($member = $this->CreateOrReturnExistingMember()) {
                    if ($member->exists()) {
                        $address->FillWithLastAddressFromMember($member, $write = false);
                    }
                }
            }
            if ($address) {
                if (!$address->exists()) {
                    $address->write();
                }
                if ($address->OrderID != $this->ID) {
                    $address->OrderID = $this->ID;
                    $address->write();
                }
                if ($this->$variableName != $address->ID) {
                    if (!$this->IsSubmitted()) {
                        $this->$variableName = $address->ID;
                        $this->write();
                    }
                }

                return $address;
            }
        }

        return;
    }

    /**
     * Sets the country in the billing and shipping address.
     *
     * @param string $countryCode            - code for the country e.g. NZ
     * @param bool   $includeBillingAddress
     * @param bool   $includeShippingAddress
     **/
    public function SetCountryFields($countryCode, $includeBillingAddress = true, $includeShippingAddress = true)
    {
        if ($this->IsSubmitted()) {
            user_error('Can not change country in submitted order', E_USER_NOTICE);
        } else {
            if ($includeBillingAddress) {
                if ($billingAddress = $this->CreateOrReturnExistingAddress('BillingAddress')) {
                    $billingAddress->SetCountryFields($countryCode);
                }
            }
            if (EcommerceConfig::get('OrderAddress', 'use_separate_shipping_address')) {
                if ($includeShippingAddress) {
                    if ($shippingAddress = $this->CreateOrReturnExistingAddress('ShippingAddress')) {
                        $shippingAddress->SetCountryFields($countryCode);
                    }
                }
            }
        }
    }

    /**
     * Sets the region in the billing and shipping address.
     *
     * @param int $regionID - ID for the region to be set
     **/
    public function SetRegionFields($regionID)
    {
        if ($this->IsSubmitted()) {
            user_error('Can not change country in submitted order', E_USER_NOTICE);
        } else {
            if ($billingAddress = $this->CreateOrReturnExistingAddress('BillingAddress')) {
                $billingAddress->SetRegionFields($regionID);
            }
            if ($this->CanHaveShippingAddress()) {
                if ($shippingAddress = $this->CreateOrReturnExistingAddress('ShippingAddress')) {
                    $shippingAddress->SetRegionFields($regionID);
                }
            }
        }
    }

    /**
     * Stores the preferred currency of the order.
     * IMPORTANTLY we store the exchange rate for future reference...
     *
     * @param EcommerceCurrency $currency

There is no parameter named $currency. Did you maybe mean $newCurrency?

     */
    public function UpdateCurrency($newCurrency)
    {
        if ($this->IsSubmitted()) {
            user_error('Can not set the currency after the order has been submitted', E_USER_NOTICE);
        } else {
            if (! is_a($newCurrency, Object::getCustomClass('EcommerceCurrency'))) {
                $newCurrency = EcommerceCurrency::default_currency();
            }
            $this->CurrencyUsedID = $newCurrency->ID;

The property CurrencyUsedID does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

            $this->ExchangeRate = $newCurrency->getExchangeRate();

The property ExchangeRate does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

            $this->write();
        }
    }

    /**
     * alias for UpdateCurrency.
     *
     * @param EcommerceCurrency $currency
     */
    public function SetCurrency($currency)
    {
        $this->UpdateCurrency($currency);
    }

/*******************************************************
   * 5. CUSTOMER COMMUNICATION
*******************************************************/

    /**
     * Send the invoice of the order by email.
     *
     * @param string $emailClassName     (optional) class used to send email
     * @param string $subject            (optional) subject for the email
     * @param string $message            (optional) the main message in the email
     * @param bool   $resend             (optional) send the email even if it has been sent before
     * @param bool   $adminOnlyOrToEmail (optional) sends the email to the ADMIN ONLY, if you provide an email, it will go to the email...
     *
     * @return bool TRUE on success, FALSE on failure
     */
    public function sendEmail(
        $emailClassName = 'Order_InvoiceEmail',
        $subject = '',
        $message = '',
        $resend = false,
        $adminOnlyOrToEmail = false
    ) {
        return $this->prepareAndSendEmail(
            $emailClassName,
            $subject,
            $message,
            $resend,
            $adminOnlyOrToEmail
        );
    }

    /**
     * Sends a message to the shop admin ONLY and not to the customer
     * This can be used by ordersteps and orderlogs to notify the admin of any potential problems.
     *
     * @param string         $emailClassName       - (optional) template to be used ...
     * @param string         $subject              - (optional) subject for the email
     * @param string         $message              - (optional) message to be added with the email
     * @param bool           $resend               - (optional) can it be sent twice?
     * @param bool | string  $adminOnlyOrToEmail   - (optional) sends the email to the ADMIN ONLY, if you provide an email, it will go to the email...
     *
     * @return bool TRUE for success, FALSE for failure (not tested)
     */
    public function sendAdminNotification(
        $emailClassName = 'Order_ErrorEmail',
        $subject = '',
        $message = '',
        $resend = false,
        $adminOnlyOrToEmail = true
    ) {
        return $this->prepareAndSendEmail(
            $emailClassName,
            $subject,
            $message,
            $resend,
            $adminOnlyOrToEmail
        );
    }

    /**
     * returns the order formatted as an email.
     *
     * @param string $emailClassName - template to use.
     * @param string $subject        - (optional) the subject (which can be used as title in email)
     * @param string $message        - (optional) the additional message
     *
     * @return string (html)
     */
    public function renderOrderInEmailFormat(
        $emailClassName,
        $subject = '',
        $message = ''
    )
    {
        $arrayData = $this->createReplacementArrayForEmail($subject, $message);
        Config::nest();
        Config::inst()->update('SSViewer', 'theme_enabled', true);
        $html = $arrayData->renderWith($emailClassName);
        Config::unnest();

        return Order_Email::emogrify_html($html);
    }

    /**
     * Send a mail of the order to the client (and another to the admin).
     *
     * @param string         $emailClassName       - (optional) template to be used ...
     * @param string         $subject              - (optional) subject for the email
     * @param string         $message              - (optional) message to be added with the email
     * @param bool           $resend               - (optional) can it be sent twice?
     * @param bool | string  $adminOnlyOrToEmail   - (optional) sends the email to the ADMIN ONLY, if you provide an email, it will go to the email...
     *
     * @return bool TRUE for success, FALSE for failure (not tested)
     */
    protected function prepareAndSendEmail(
        $emailClassName = 'Order_InvoiceEmail',
        $subject,

Parameters which have default values should be placed at the end.

        $message,
        $resend = false,
        $adminOnlyOrToEmail = false
    ) {
        $arrayData = $this->createReplacementArrayForEmail($subject, $message);
        $from = Order_Email::get_from_email();
        //why are we using this email and NOT the member.EMAIL?
        //for historical reasons????

56% of this comment could be valid code. Did you maybe forget this after debugging?

        if ($adminOnlyOrToEmail) {
            if (filter_var($adminOnlyOrToEmail, FILTER_VALIDATE_EMAIL)) {
                $to = $adminOnlyOrToEmail;
                // invalid e-mail address
            } else {
                $to = Order_Email::get_from_email();
            }
        } else {
            $to = $this->getOrderEmail();
        }
        if ($from && $to) {
            if(! class_exists($emailClassName)) {
                user_error('Invalid Email ClassName provided: '. $emailClassName, E_USER_ERROR);
            }
            $email = new $emailClassName();
            if (!(is_a($email, Object::getCustomClass('Email')))) {
                user_error('No correct email class provided.', E_USER_ERROR);
            }
            $email->setFrom($from);
            $email->setTo($to);
            //we take the subject from the Array Data, just in case it has been adjusted.
            $email->setSubject($arrayData->getField('Subject'));
            //we also see if a CC and a BCC have been added
            ;
            if ($cc = $arrayData->getField('CC')) {
                $email->setCc($cc);
            }
            if ($bcc = $arrayData->getField('BCC')) {
                $email->setBcc($bcc);
            }
            $email->populateTemplate($arrayData);
            // This might be called from within the CMS,
            // so we need to restore the theme, just in case
            // templates within the theme exist
            Config::nest();
            Config::inst()->update('SSViewer', 'theme_enabled', true);
            $email->setOrder($this);
            $email->setResend($resend);
            $result = $email->send(null);
            Config::unnest();
            if (Director::isDev()) {
                return true;
            } else {
                return $result;
            }
        }

        return false;
    }

    /**
     * returns the Data that can be used in the body of an order Email
     * we add the subject here so that the subject, for example, can be added to the <title>
     * of the email template.
     * we add the subject here so that the subject, for example, can be added to the <title>
     * of the email template.
     *
     * @param string $subject  - (optional) subject for email
     * @param string $message  - (optional) the additional message
     *
     * @return ArrayData
     *                   - Subject - EmailSubject
     *                   - Message - specific message for this order
     *                   - Message - custom message
     *                   - OrderStepMessage - generic message for step
     *                   - Order
     *                   - EmailLogo
     *                   - ShopPhysicalAddress
     *                   - CurrentDateAndTime
     *                   - BaseURL
     *                   - CC
     *                   - BCC
     */
    protected function createReplacementArrayForEmail($subject = '', $message = '')
    {
        $step = $this->MyStep();
        $config = $this->EcomConfig();
        $replacementArray = array();
        //set subject
        if ( ! $subject) {
            $subject = $step->EmailSubject;
        }
        if( ! $message) {
            $message = $step->CustomerMessage;
        }
        $subject = str_replace('[OrderNumber]', $this->ID, $subject);
        //set other variables
        $replacementArray['Subject'] = $subject;
        $replacementArray['To'] = '';
        $replacementArray['CC'] = '';
        $replacementArray['BCC'] = '';
        $replacementArray['OrderStepMessage'] = $message;
        $replacementArray['Order'] = $this;
        $replacementArray['EmailLogo'] = $config->EmailLogo();

The method EmailLogo does not exist on object<EcommerceDBConfig>? Since you implemented __call, maybe consider adding a @method annotation.

        $replacementArray['ShopPhysicalAddress'] = $config->ShopPhysicalAddress;

The property ShopPhysicalAddress does not exist on object<EcommerceDBConfig>. Since you implemented __set, maybe consider adding a @property annotation.

        $replacementArray['CurrentDateAndTime'] = DBField::create_field('SS_Datetime', 'Now');
        $replacementArray['BaseURL'] = Director::baseURL();
        $arrayData = ArrayData::create($replacementArray);
        $this->extend('updateReplacementArrayForEmail', $arrayData);

        return $arrayData;
    }

/*******************************************************
   * 6. ITEM MANAGEMENT
*******************************************************/

    /**
     * returns a list of Order Attributes by type.
     *
     * @param array | String $types
     *
     * @return ArrayList
     */
    public function getOrderAttributesByType($types)
    {
        if (!is_array($types) && is_string($types)) {
            $types = array($types);
        }
        if (!is_array($al)) {

The variable $al seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

            user_error('wrong parameter (types) provided in Order::getOrderAttributesByTypes');
        }
        $al = new ArrayList();
        $items = $this->Items();
        foreach ($items as $item) {
            if (in_array($item->OrderAttributeType(), $types)) {
                $al->push($item);
            }
        }
        $modifiers = $this->Modifiers();
        foreach ($modifiers as $modifier) {
            if (in_array($modifier->OrderAttributeType(), $types)) {
                $al->push($modifier);
            }
        }

        return $al;
    }

    /**
     * Returns the items of the order.
     * Items are the order items (products) and NOT the modifiers (discount, tax, etc...).
     *
     * N. B. this method returns Order Items
     * also see Buaybles

     *
     * @param string filter - where statement to exclude certain items OR ClassName (e.g. 'TaxModifier')
     *
     * @return DataList (OrderItems)
     */
    public function Items($filterOrClassName = '')
    {
        if (!$this->exists()) {
            $this->write();
        }

        return $this->itemsFromDatabase($filterOrClassName);
    }

    /**
     * @alias function of Items
     *
     * N. B. this method returns Order Items
     * also see Buaybles
     *
     * @param string filter - where statement to exclude certain items.
     * @alias for Items
     * @return DataList (OrderItems)
     */
    public function OrderItems($filterOrClassName = '')
    {
        return $this->Items($filterOrClassName);
    }

    /**
     * returns the buyables asscoiated with the order items.
     *
     * NB. this method retursn buyables
     *
     * @param string filter - where statement to exclude certain items.
     *
     * @return ArrayList (Buyables)
     */
    public function Buyables($filterOrClassName = '')
    {
        $items = $this->Items($filterOrClassName);
        $arrayList = new ArrayList();
        foreach ($items as $item) {
            $arrayList->push($item->Buyable());
        }

        return $arrayList;
    }

    /**
     * Return all the {@link OrderItem} instances that are
     * available as records in the database.
     *
     * @param string filter - where statement to exclude certain items,
     *   you can also pass a classname (e.g. MyOrderItem), in which case only this class will be returned (and any class extending your given class)
     *
     * @return DataList (OrderItems)
     */
    protected function itemsFromDatabase($filterOrClassName = '')
    {
        $className = 'OrderItem';
        $extrafilter = '';
        if ($filterOrClassName) {
            if (class_exists($filterOrClassName)) {
                $className = $filterOrClassName;
            } else {
                $extrafilter = " AND $filterOrClassName";
            }
        }

        return $className::get()->filter(array('OrderID' => $this->ID))->where($extrafilter);
    }

    /**
     * @alias for Modifiers
     *
     * @return DataList (OrderModifiers)
     */
    public function OrderModifiers()
    {
        return $this->Modifiers();
    }

    /**
     * Returns the modifiers of the order, if it hasn't been saved yet
     * it returns the modifiers from session, if it has, it returns them
     * from the DB entry. ONLY USE OUTSIDE ORDER.
     *
     * @param string filter - where statement to exclude certain items OR ClassName (e.g. 'TaxModifier')
     *
     * @return DataList (OrderModifiers)
     */
    public function Modifiers($filterOrClassName = '')
    {
        return $this->modifiersFromDatabase($filterOrClassName);
    }

    /**
     * Get all {@link OrderModifier} instances that are
     * available as records in the database.
     * NOTE: includes REMOVED Modifiers, so that they do not get added again...
     *
     * @param string filter - where statement to exclude certain items OR ClassName (e.g. 'TaxModifier')
     *
     * @return DataList (OrderModifiers)
     */
    protected function modifiersFromDatabase($filterOrClassName = '')
    {
        $className = 'OrderModifier';
        $extrafilter = '';
        if ($filterOrClassName) {
            if (class_exists($filterOrClassName)) {
                $className = $filterOrClassName;
            } else {
                $extrafilter = " AND $filterOrClassName";
            }
        }

        return $className::get()->where('"OrderAttribute"."OrderID" = '.$this->ID." $extrafilter");
    }

    /**
     * Calculates and updates all the order attributes.
     *
     * @param bool $recalculate - run it, even if it has run already
     */
    public function calculateOrderAttributes($recalculate = false)
    {
        if ($this->IsSubmitted()) {

This if statement is empty and can be removed.

            //submitted orders are NEVER recalculated.
            //they are set in stone.
        } elseif (self::get_needs_recalculating($this->ID) || $recalculate) {
            if ($this->StatusID || $this->TotalItems()) {

The property StatusID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                $this->ensureCorrectExchangeRate();
                $this->calculateOrderItems($recalculate);
                $this->calculateModifiers($recalculate);
                $this->extend('onCalculateOrder');
            }
        }
    }

    /**
     * Calculates and updates all the product items.
     *
     * @param bool $recalculate - run it, even if it has run already
     */
    protected function calculateOrderItems($recalculate = false)
    {
        //check if order has modifiers already
        //check /re-add all non-removable ones
        //$start = microtime();

50% of this comment could be valid code. Did you maybe forget this after debugging?

        $orderItems = $this->itemsFromDatabase();
        if ($orderItems->count()) {
            foreach ($orderItems as $orderItem) {
                if ($orderItem) {
                    $orderItem->runUpdate($recalculate);
                }
            }
        }
        $this->extend('onCalculateOrderItems', $orderItems);
    }

    /**
     * Calculates and updates all the modifiers.
     *
     * @param bool $recalculate - run it, even if it has run already
     */
    protected function calculateModifiers($recalculate = false)
    {
        $createdModifiers = $this->modifiersFromDatabase();
        if ($createdModifiers->count()) {
            foreach ($createdModifiers as $modifier) {
                if ($modifier) {
                    $modifier->runUpdate($recalculate);
                }
            }
        }
        $this->extend('onCalculateModifiers', $createdModifiers);
    }

    /**
     * Returns the subtotal of the modifiers for this order.
     * If a modifier appears in the excludedModifiers array, it is not counted.
     *
     * @param string|array $excluded               - Class(es) of modifier(s) to ignore in the calculation.

Should the type for parameter $excluded not be string|array|null?

     * @param bool         $stopAtExcludedModifier - when this flag is TRUE, we stop adding the modifiers when we reach an excluded modifier.
     *
     * @return float

Should the return type not be integer?

     */
    public function ModifiersSubTotal($excluded = null, $stopAtExcludedModifier = false)
    {
        $total = 0;
        $modifiers = $this->Modifiers();
        if ($modifiers->count()) {
            foreach ($modifiers as $modifier) {
                if (!$modifier->IsRemoved()) { //we just double-check this...
                    if (is_array($excluded) && in_array($modifier->ClassName, $excluded)) {
                        if ($stopAtExcludedModifier) {
                            break;
                        }
                        //do the next modifier
                        continue;
                    } elseif (is_string($excluded) && ($modifier->ClassName == $excluded)) {
                        if ($stopAtExcludedModifier) {
                            break;
                        }
                        //do the next modifier
                        continue;
                    }
                    $total += $modifier->CalculationTotal();
                }
            }
        }

        return $total;
    }

    /**
     * returns a modifier that is an instanceof the classname
     * it extends.
     *
     * @param string $className: class name for the modifier

There is no parameter named $className:. Did you maybe mean $className?

     *
     * @return DataObject (OrderModifier)
     **/
    public function RetrieveModifier($className)
    {
        $modifiers = $this->Modifiers();
        if ($modifiers->count()) {
            foreach ($modifiers as $modifier) {
                if (is_a($modifier, Object::getCustomClass($className))) {
                    return $modifier;
                }
            }
        }
    }

/*******************************************************
   * 7. CRUD METHODS (e.g. canView, canEdit, canDelete, etc...)
*******************************************************/

    /**
     * @param Member $member
     *
     * @return DataObject (Member)
     **/
     //TODO: please comment why we make use of this function
    protected function getMemberForCanFunctions(Member $member = null)
    {
        if (!$member) {
            $member = Member::currentUser();
        }
        if (!$member) {
            $member = new Member();
            $member->ID = 0;
        }

        return $member;
    }

    /**
     * @param Member $member

Should the type for parameter $member not be Member|null?

     *
     * @return bool
     **/
    public function canCreate($member = null)
    {
        $member = $this->getMemberForCanFunctions($member);
        $extended = $this->extendedCan(__FUNCTION__, $member);

$member is of type object<DataObject>, but the function expects a object<Member>|integer.

        if ($extended !== null) {
            return $extended;
        }
        if ($member->exists()) {
            return $member->IsShopAdmin();
        }
    }

    /**
     * Standard SS method - can the current member view this order?
     *
     * @param Member $member

Should the type for parameter $member not be Member|null?

     *
     * @return bool
     **/
    public function canView($member = null)
    {
        if (!$this->exists()) {
            return true;
        }
        $member = $this->getMemberForCanFunctions($member);
        //check if this has been "altered" in any DataExtension
        $extended = $this->extendedCan(__FUNCTION__, $member);

$member is of type object<DataObject>, but the function expects a object<Member>|integer.

        if ($extended !== null) {
            return $extended;
        }
        //is the member is a shop admin they can always view it
        if (EcommerceRole::current_member_is_shop_admin($member)) {
            return true;
        }

        //is the member is a shop assistant they can always view it
        if (EcommerceRole::current_member_is_shop_assistant($member)) {
            return true;
        }
        //if the current member OWNS the order, (s)he can always view it.
        if ($member->exists() && $this->MemberID == $member->ID) {

The property MemberID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

            return true;
        }
        //it is the current order
        if ($this->IsInSession()) {
            //we do some additional CHECKS for session hackings!
            if ($member->exists() && $this->MemberID) {

The property MemberID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                //can't view the order of another member!
                //shop admin exemption is already captured.
                //this is always true
                if ($this->MemberID != $member->ID) {

The property MemberID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                    return false;
                }
            } else {
                //order belongs to someone, but current user is NOT logged in...
                //this is allowed!
                //the reason it is allowed is because we want to be able to
                //add order to non-existing member
                return true;
            }
        }

        return false;
    }

    /**
     * @param Member $member optional

Should the type for parameter $member not be Member|null?

     * @return bool
     */
    public function canOverrideCanView($member = null)
    {
        if ($this->canView($member)) {
            //can view overrides any concerns
            return true;
        } else {
            $tsOrder = strtotime($this->LastEdited);
            $tsNow = time();
            $minutes = EcommerceConfig::get('Order', 'minutes_an_order_can_be_viewed_without_logging_in');
            if ($minutes && ((($tsNow - $tsOrder) / 60) < $minutes)) {

                //has the order been edited recently?
                return true;
            } elseif ($orderStep = $this->MyStep()) {

                // order is being processed ...
                return $orderStep->canOverrideCanViewForOrder($this, $member);

The method canOverrideCanViewForOrder() does not exist on DataObject. Did you maybe mean canView()?

            }
        }
        return false;
    }

    /**
     * @return bool
     */
    public function IsInSession()
    {
        $orderInSession = ShoppingCart::session_order();

        return $orderInSession && $this->ID && $this->ID == $orderInSession->ID;
    }

    /**
     * returns a pseudo random part of the session id.
     *
     * @param int $size
     *
     * @return string
     */
    public function LessSecureSessionID($size = 7, $start = null)
    {
        if (!$start || $start < 0 || $start > (32 - $size)) {
            $start = 0;
        }

        return substr($this->SessionID, $start, $size);

The property SessionID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

    }
    /**
     *
     * @param Member (optional) $member

Should the type for parameter $member not be optional|null?

     *
     * @return bool

Should the return type not be boolean|null?

     **/
    public function canViewAdminStuff($member = null)
    {
        $member = $this->getMemberForCanFunctions($member);
        $extended = $this->extendedCan(__FUNCTION__, $member);

$member is of type object<DataObject>, but the function expects a object<Member>|integer.

        if ($extended !== null) {
            return $extended;
        }
        if (Permission::checkMember($member, Config::inst()->get('EcommerceRole', 'admin_permission_code'))) {
            return true;
        }
    }

    /**
     * if we set canEdit to false then we
     * can not see the child records
     * Basically, you can edit when you can view and canEdit (even as a customer)
     * Or if you are a Shop Admin you can always edit.
     * Otherwise it is false...
     *
     * @param Member $member

Should the type for parameter $member not be Member|null?

     *
     * @return bool
     **/
    public function canEdit($member = null)
    {
        $member = $this->getMemberForCanFunctions($member);
        $extended = $this->extendedCan(__FUNCTION__, $member);

$member is of type object<DataObject>, but the function expects a object<Member>|integer.

        if ($extended !== null) {
            return $extended;
        }
        if ($this->canView($member) && $this->MyStep()->CustomerCanEdit) {

$member is of type object<DataObject>, but the function expects a object<Member>|null.

            return true;
        }
        if (Permission::checkMember($member, Config::inst()->get('EcommerceRole', 'admin_permission_code'))) {
            return true;
        }
        //is the member is a shop assistant they can always view it
        if (EcommerceRole::current_member_is_shop_assistant($member)) {

This if statement, and the following return statement can be replaced with return \EcommerceRole::current_member_is_shop_assistant($member);.

            return true;
        }
        return false;
    }

    /**
     * is the order ready to go through to the
     * checkout process.
     *
     * This method checks all the order items and order modifiers
     * If any of them need immediate attention then this is done
     * first after which it will go through to the checkout page.
     *
     * @param Member (optional) $member

Should the type for parameter $member not be null|Member?

     *
     * @return bool
     **/
    public function canCheckout(Member $member = null)
    {
        $member = $this->getMemberForCanFunctions($member);
        $extended = $this->extendedCan(__FUNCTION__, $member);

$member is of type object<DataObject>, but the function expects a object<Member>|integer.

        if ($extended !== null) {
            return $extended;
        }
        $submitErrors = $this->SubmitErrors();
        if ($submitErrors && $submitErrors->count()) {

This if statement, and the following return statement can be replaced with return !($submitErrors && $submitErrors->count());.

            return false;
        }

        return true;
    }

    /**
     * Can the order be submitted?
     * this method can be used to stop an order from being submitted
     * due to something not being completed or done.
     *
     * @see Order::SubmitErrors
     *
     * @param Member $member

Should the type for parameter $member not be null|Member?

     *
     * @return bool
     **/
    public function canSubmit(Member $member = null)
    {
        $member = $this->getMemberForCanFunctions($member);
        $extended = $this->extendedCan(__FUNCTION__, $member);

$member is of type object<DataObject>, but the function expects a object<Member>|integer.

        if ($extended !== null) {
            return $extended;
        }
        if ($this->IsSubmitted()) {
            return false;
        }
        $submitErrors = $this->SubmitErrors();
        if ($submitErrors && $submitErrors->count()) {

This if statement, and the following return statement can be replaced with return !($submitErrors && $submitErrors->count());.

            return false;
        }

        return true;
    }

    /**
     * Can a payment be made for this Order?
     *
     * @param Member $member

Should the type for parameter $member not be null|Member?

     *
     * @return bool
     **/
    public function canPay(Member $member = null)
    {
        $member = $this->getMemberForCanFunctions($member);
        $extended = $this->extendedCan(__FUNCTION__, $member);

$member is of type object<DataObject>, but the function expects a object<Member>|integer.

        if ($extended !== null) {
            return $extended;
        }
        if ($this->IsPaid() || $this->IsCancelled() || $this->PaymentIsPending()) {
            return false;
        }

        return $this->MyStep()->CustomerCanPay;
    }

    /**
     * Can the given member cancel this order?
     *
     * @param Member $member

Should the type for parameter $member not be null|Member?

     *
     * @return bool
     **/
    public function canCancel(Member $member = null)
    {
        //if it is already cancelled it can not be cancelled again
        if ($this->CancelledByID) {

The property CancelledByID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

            return false;
        }
        $member = $this->getMemberForCanFunctions($member);
        $extended = $this->extendedCan(__FUNCTION__, $member);

$member is of type object<DataObject>, but the function expects a object<Member>|integer.

        if ($extended !== null) {
            return $extended;
        }
        if (EcommerceRole::current_member_can_process_orders($member)) {
            return true;
        }

        return $this->MyStep()->CustomerCanCancel && $this->canView($member);

$member is of type object<DataObject>, but the function expects a object<Member>|null.

    }

    /**
     * @param Member $member

Should the type for parameter $member not be Member|null?

     *
     * @return bool
     **/
    public function canDelete($member = null)
    {
        $member = $this->getMemberForCanFunctions($member);
        $extended = $this->extendedCan(__FUNCTION__, $member);

$member is of type object<DataObject>, but the function expects a object<Member>|integer.

        if ($extended !== null) {
            return $extended;
        }
        if ($this->IsSubmitted()) {
            return false;
        }
        if (Permission::checkMember($member, Config::inst()->get('EcommerceRole', 'admin_permission_code'))) {

This if statement, and the following return statement can be replaced with return (bool) \Permission::checkMember($member, \Config::inst()->get('EcommerceRole', 'admin_permission_code'));.

            return true;
        }

        return false;
    }

    /**
     * Returns all the order logs that the current member can view
     * i.e. some order logs can only be viewed by the admin (e.g. suspected fraud orderlog).
     *
     * @return ArrayList (OrderStatusLogs)
     **/
    public function CanViewOrderStatusLogs()
    {
        $canViewOrderStatusLogs = new ArrayList();
        $logs = $this->OrderStatusLogs();

The method OrderStatusLogs() does not exist on Order. Did you maybe mean getOrderStatusLogsTableField()?

        foreach ($logs as $log) {
            if ($log->canView()) {
                $canViewOrderStatusLogs->push($log);
            }
        }

        return $canViewOrderStatusLogs;
    }

    /**
     * returns all the logs that can be viewed by the customer.
     *
     * @return ArrayList (OrderStausLogs)
     */
    public function CustomerViewableOrderStatusLogs()
    {
        $customerViewableOrderStatusLogs = new ArrayList();
        $logs = $this->OrderStatusLogs();

The method OrderStatusLogs() does not exist on Order. Did you maybe mean getOrderStatusLogsTableField()?

        if ($logs) {
            foreach ($logs as $log) {
                if (!$log->InternalUseOnly) {
                    $customerViewableOrderStatusLogs->push($log);
                }
            }
        }

        return $customerViewableOrderStatusLogs;
    }

/*******************************************************
   * 8. GET METHODS (e.g. Total, SubTotal, Title, etc...)
*******************************************************/

    /**
     * returns the email to be used for customer communication.
     *
     * @return string
     */
    public function OrderEmail()
    {
        return $this->getOrderEmail();
    }
    public function getOrderEmail()

The return type could not be reliably inferred; please add a @return annotation.

    {
        $email = '';
        if ($this->BillingAddressID && $this->BillingAddress()) {

The property BillingAddressID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

The method BillingAddress() does not exist on Order. Did you maybe mean getBillingAddressField()?

            $email = $this->BillingAddress()->Email;

The method BillingAddress() does not exist on Order. Did you maybe mean getBillingAddressField()?

        }
        if (! $email) {
            if ($this->MemberID && $this->Member()) {

The property MemberID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

The method Member() does not exist on Order. Did you maybe mean CreateOrReturnExistingMember()?

                $email = $this->Member()->Email;

The method Member() does not exist on Order. Did you maybe mean CreateOrReturnExistingMember()?

            }
        }
        $extendedEmail = $this->extend('updateOrderEmail', $email);
        if ($extendedEmail !== null && is_array($extendedEmail) && count($extendedEmail)) {
            $email = implode(';', $extendedEmail);
        }

        return $email;
    }

    /**
     * Returns true if there is a prink or email link.
     *
     * @return bool
     */
    public function HasPrintOrEmailLink()
    {
        return $this->EmailLink() || $this->PrintLink();

The expression $this->PrintLink() of type string|null is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

    }

    /**
     * returns the absolute link to the order that can be used in the customer communication (email).
     *
     * @return string

Should the return type not be false|string|null?

     */
    public function EmailLink($type = 'Order_StatusEmail')

The parameter $type is not used and could be removed.

    {
        return $this->getEmailLink();
    }
    public function getEmailLink($type = 'Order_StatusEmail')

The parameter $type is not used and could be removed.

getEmailLink uses the super-global variable $_REQUEST which is generally not recommended.

    {
        if (!isset($_REQUEST['print'])) {
            if ($this->IsSubmitted()) {
                return Director::AbsoluteURL(OrderConfirmationPage::get_email_link($this->ID, $this->MyStep()->getEmailClassName(), $actuallySendEmail = true));
            }
        }
    }

    /**
     * returns the absolute link to the order for printing.
     *
     * @return string

Should the return type not be string|null?

     */
    public function PrintLink()
    {
        return $this->getPrintLink();
    }
    public function getPrintLink()

getPrintLink uses the super-global variable $_REQUEST which is generally not recommended.

    {
        if (!isset($_REQUEST['print'])) {
            if ($this->IsSubmitted()) {
                return Director::AbsoluteURL(OrderConfirmationPage::get_order_link($this->ID)).'?print=1';
            }
        }
    }

    /**
     * returns the absolute link to the order for printing.
     *
     * @return string

Should the return type not be string|null?

     */
    public function PackingSlipLink()
    {
        return $this->getPackingSlipLink();
    }
    public function getPackingSlipLink()
    {
        if ($this->IsSubmitted()) {
            return Director::AbsoluteURL(OrderConfirmationPage::get_order_link($this->ID)).'?packingslip=1';
        }
    }

    /**
     * returns the absolute link that the customer can use to retrieve the email WITHOUT logging in.
     *
     * @return string

Should the return type not be string|false?

     */
    public function RetrieveLink()
    {
        return $this->getRetrieveLink();
    }

    public function getRetrieveLink()
    {
        //important to recalculate!
        if ($this->IsSubmitted($recalculate = true)) {
            //add session ID if not added yet...
            if (!$this->SessionID) {

The property SessionID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                $this->write();
            }

            return Director::AbsoluteURL(OrderConfirmationPage::find_link()).'retrieveorder/'.$this->SessionID.'/'.$this->ID.'/';

The property SessionID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

        } else {
            return Director::AbsoluteURL('/shoppingcart/loadorder/'.$this->ID.'/');
        }
    }

    public function ShareLink()
    {
        return $this->getShareLink();
    }

    public function getShareLink()
    {
        $orderItems = $this->itemsFromDatabase();
        $action = 'share';
        $array = array();
        foreach ($orderItems as $orderItem) {
            $array[] = implode(
                ',',
                array(
                    $orderItem->BuyableClassName,
                    $orderItem->BuyableID,
                    $orderItem->Quantity
                )
            );
        }

        return Director::AbsoluteURL(CartPage::find_link($action.'/'.implode('-', $array)));
    }

    /**
     * @alias for getFeedbackLink
     * @return string

Should the return type not be string|null?

     */
    public function FeedbackLink()
    {
        return $this->getFeedbackLink();
    }

    /**
     * @return string | null

Should the return type not be string|null?

     */
    public function getFeedbackLink()
    {
        $orderConfirmationPage = DataObject::get_one('OrderConfirmationPage');
        if($orderConfirmationPage->IsFeedbackEnabled) {

            return Director::AbsoluteURL($this->getRetrieveLink()).'#OrderForm_Feedback_FeedbackForm';

It seems like $this->getRetrieveLink() targeting Order::getRetrieveLink() can also be of type false; however, Director::absoluteURL() does only seem to accept string, did you maybe forget to handle an error condition?

        }
    }

    /**
     * link to delete order.
     *
     * @return string
     */
    public function DeleteLink()
    {
        return $this->getDeleteLink();
    }
    public function getDeleteLink()
    {
        if ($this->canDelete()) {
            return ShoppingCart_Controller::delete_order_link($this->ID);
        } else {
            return '';
        }
    }

    /**
     * link to copy order.
     *
     * @return string

Should the return type not be string|null?

     */
    public function CopyOrderLink()
    {
        return $this->getCopyOrderLink();
    }
    public function getCopyOrderLink()
    {
        if ($this->canView() && $this->IsSubmitted()) {
            return ShoppingCart_Controller::copy_order_link($this->ID);
        } else {
            return '';
        }
    }

    /**
     * A "Title" for the order, which summarises the main details (date, and customer) in a string.
     *
     * @param string $dateFormat  - e.g. "D j M Y, G:i T"

Should the type for parameter $dateFormat not be string|null?

     * @param bool   $includeName - e.g. by Mr Johnson
     *
     * @return string
     **/
    public function Title($dateFormat = null, $includeName = false)
    {
        return $this->getTitle($dateFormat, $includeName);
    }
    public function getTitle($dateFormat = null, $includeName = false)

The return type could not be reliably inferred; please add a @return annotation.

    {
        if ($this->exists()) {
            if ($dateFormat === null) {
                $dateFormat = EcommerceConfig::get('Order', 'date_format_for_title');
            }
            if ($includeName === null) {
                $includeName = EcommerceConfig::get('Order', 'include_customer_name_in_title');
            }
            $title = $this->i18n_singular_name()." #".number_format($this->ID);
            if ($dateFormat) {
                if ($submissionLog = $this->SubmissionLog()) {
                    $dateObject = $submissionLog->dbObject('Created');
                    $placed = _t('Order.PLACED', 'placed');
                } else {
                    $dateObject = $this->dbObject('Created');
                    $placed = _t('Order.STARTED', 'started');
                }
                $title .= ' - '.$placed.' '.$dateObject->Format($dateFormat);
            }
            $name = '';
            if ($this->CancelledByID) {

The property CancelledByID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                $name = ' - '._t('Order.CANCELLED', 'CANCELLED');
            }
            if ($includeName) {
                $by = _t('Order.BY', 'by');
                if (!$name) {
                    if ($this->BillingAddressID) {

The property BillingAddressID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                        if ($billingAddress = $this->BillingAddress()) {

The method BillingAddress() does not exist on Order. Did you maybe mean getBillingAddressField()?

                            $name = ' - '.$by.' '.$billingAddress->Prefix.' '.$billingAddress->FirstName.' '.$billingAddress->Surname;
                        }
                    }
                }
                if (!$name) {
                    if ($this->MemberID) {

The property MemberID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                        if ($member = $this->Member()) {

The method Member() does not exist on Order. Did you maybe mean CreateOrReturnExistingMember()?

                            if ($member->exists()) {
                                if ($memberName = $member->getName()) {
                                    if (!trim($memberName)) {
                                        $memberName = _t('Order.ANONYMOUS', 'anonymous');
                                    }
                                    $name = ' - '.$by.' '.$memberName;
                                }
                            }
                        }
                    }
                }
            }
            $title .= $name;
        } else {
            $title = _t('Order.NEW', 'New').' '.$this->i18n_singular_name();
        }
        $extendedTitle = $this->extend('updateTitle', $title);
        if ($extendedTitle !== null && is_array($extendedTitle) && count($extendedTitle)) {
            $title = implode('; ', $extendedTitle);
        }

        return $title;
    }

    /**
     * Returns the subtotal of the items for this order.
     *
     * @return float

Should the return type not be integer?

     */
    public function SubTotal()
    {
        return $this->getSubTotal();
    }
    public function getSubTotal()
    {
        $result = 0;
        $items = $this->Items();
        if ($items->count()) {
            foreach ($items as $item) {
                if (is_a($item, Object::getCustomClass('OrderAttribute'))) {
                    $result += $item->Total();
                }
            }
        }

        return $result;
    }

    /**
     * @return Currency (DB Object)
     **/
    public function SubTotalAsCurrencyObject()
    {
        return DBField::create_field('Currency', $this->SubTotal());
    }

    /**
     * @return Money
     **/
    public function SubTotalAsMoney()
    {
        return $this->getSubTotalAsMoney();
    }
    public function getSubTotalAsMoney()
    {
        return EcommerceCurrency::get_money_object_from_order_currency($this->SubTotal(), $this);
    }

    /**
     * @param string|array $excluded               - Class(es) of modifier(s) to ignore in the calculation.

Should the type for parameter $excluded not be string|array|null?

     * @param bool         $stopAtExcludedModifier - when this flag is TRUE, we stop adding the modifiers when we reach an excluded modifier.
     *
     * @return Currency (DB Object)
     **/
    public function ModifiersSubTotalAsCurrencyObject($excluded = null, $stopAtExcludedModifier = false)
    {
        return DBField::create_field('Currency', $this->ModifiersSubTotal($excluded, $stopAtExcludedModifier));
    }

    /**
     * @param string|array $excluded               - Class(es) of modifier(s) to ignore in the calculation.

Should the type for parameter $excluded not be string|array|null?

     * @param bool         $stopAtExcludedModifier - when this flag is TRUE, we stop adding the modifiers when we reach an excluded modifier.
     *
     * @return Money (DB Object)
     **/
    public function ModifiersSubTotalAsMoneyObject($excluded = null, $stopAtExcludedModifier = false)
    {
        return EcommerceCurrency::get_money_object_from_order_currency($this->ModifiersSubTotal($excluded, $stopAtExcludedModifier), $this);
    }

    /**
     * Returns the total cost of an order including the additional charges or deductions of its modifiers.
     *
     * @return float

Should the return type not be integer?

     */
    public function Total()
    {
        return $this->getTotal();
    }
    public function getTotal()
    {
        return $this->SubTotal() + $this->ModifiersSubTotal();
    }

    /**
     * @return Currency (DB Object)
     **/
    public function TotalAsCurrencyObject()
    {
        return DBField::create_field('Currency', $this->Total());
    }

    /**
     * @return Money
     **/
    public function TotalAsMoney()
    {
        return $this->getTotalAsMoney();
    }
    public function getTotalAsMoney()
    {
        return EcommerceCurrency::get_money_object_from_order_currency($this->Total(), $this);
    }

    /**
     * Checks to see if any payments have been made on this order
     * and if so, subracts the payment amount from the order.
     *
     * @return float

Should the return type not be double|integer?

     **/
    public function TotalOutstanding()
    {
        return $this->getTotalOutstanding();
    }
    public function getTotalOutstanding()
    {
        if ($this->IsSubmitted()) {
            $total = $this->Total();
            $paid = $this->TotalPaid();
            $outstanding = $total - $paid;
            $maxDifference = EcommerceConfig::get('Order', 'maximum_ignorable_sales_payments_difference');
            if (abs($outstanding) < $maxDifference) {
                $outstanding = 0;
            }

            return floatval($outstanding);
        } else {
            return 0;
        }
    }

    /**
     * @return Currency (DB Object)
     **/
    public function TotalOutstandingAsCurrencyObject()
    {
        return DBField::create_field('Currency', $this->TotalOutstanding());
    }

    /**
     * @return Money
     **/
    public function TotalOutstandingAsMoney()
    {
        return $this->getTotalOutstandingAsMoney();
    }
    public function getTotalOutstandingAsMoney()
    {
        return EcommerceCurrency::get_money_object_from_order_currency($this->TotalOutstanding(), $this);
    }

    /**
     * @return float

Should the return type not be integer|double?

     */
    public function TotalPaid()
    {
        return $this->getTotalPaid();
    }
    public function getTotalPaid()
    {
        $paid = 0;
        if ($payments = $this->Payments()) {

The method Payments() does not exist on Order. Did you maybe mean getPaymentsField()?

            foreach ($payments as $payment) {
                if ($payment->Status == 'Success') {
                    $paid += $payment->Amount->getAmount();
                }
            }
        }
        $reverseExchange = 1;
        if ($this->ExchangeRate && $this->ExchangeRate != 1) {

The property ExchangeRate does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

            $reverseExchange = 1 / $this->ExchangeRate;

The property ExchangeRate does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

        }

        return $paid * $reverseExchange;
    }

    /**
     * @return Currency (DB Object)
     **/
    public function TotalPaidAsCurrencyObject()
    {
        return DBField::create_field('Currency', $this->TotalPaid());
    }

    /**
     * @return Money
     **/
    public function TotalPaidAsMoney()
    {
        return $this->getTotalPaidAsMoney();
    }
    public function getTotalPaidAsMoney()
    {
        return EcommerceCurrency::get_money_object_from_order_currency($this->TotalPaid(), $this);
    }

    /**
     * returns the total number of OrderItems (not modifiers).
     * This is meant to run as fast as possible to quickly check
     * if there is anything in the cart.
     *
     * @param bool $recalculate - do we need to recalculate (value is retained during lifetime of Object)
     *
     * @return int
     **/
    public function TotalItems($recalculate = false)
    {
        return $this->getTotalItems($recalculate);
    }
    public function getTotalItems($recalculate = false)
    {
        if ($this->totalItems === null || $recalculate) {
            $this->totalItems = OrderItem::get()
                ->where('"OrderAttribute"."OrderID" = '.$this->ID.' AND "OrderItem"."Quantity" > 0')
                ->count();
        }

        return $this->totalItems;
    }

    /**
     * Little shorthand.
     *
     * @param bool $recalculate
     *
     * @return bool
     **/
    public function MoreThanOneItemInCart($recalculate = false)
    {
        return $this->TotalItems($recalculate) > 1 ? true : false;
    }

    /**
     * returns the total number of OrderItems (not modifiers) times their respectective quantities.
     *
     * @param bool $recalculate - force recalculation
     *
     * @return float

Should the return type not be integer|double?

     **/
    public function TotalItemsTimesQuantity($recalculate = false)
    {
        return $this->getTotalItemsTimesQuantity($recalculate);
    }
    public function getTotalItemsTimesQuantity($recalculate = false)
    {
        if ($this->totalItemsTimesQuantity === null || $recalculate) {
            //to do, why do we check if you can edit ????
            $this->totalItemsTimesQuantity = DB::query('

The property $totalItemsTimesQuantity was declared of type double, but \DB::query(' SELECT SUM("OrderItem"."Quantity") FROM "OrderItem" INNER JOIN "OrderAttribute" ON "OrderAttribute"."ID" = "OrderItem"."ID" WHERE "OrderAttribute"."OrderID" = ' . $this->ID . ' AND "OrderItem"."Quantity" > 0')->value() is of type string. Maybe add a type cast?

                SELECT SUM("OrderItem"."Quantity")
                FROM "OrderItem"
                    INNER JOIN "OrderAttribute" ON "OrderAttribute"."ID" = "OrderItem"."ID"
                WHERE
                    "OrderAttribute"."OrderID" = '.$this->ID.'
                    AND "OrderItem"."Quantity" > 0'
            )->value();
        }

        return $this->totalItemsTimesQuantity - 0;
    }

    /**
     *
     * @return string (country code)
     **/
    public function Country()
    {
        return $this->getCountry();
    }

    /**
    * Returns the country code for the country that applies to the order.
    * @alias  for getCountry
    *
    * @return string - country code e.g. NZ
     */
    public function getCountry()
    {
        $countryCodes = array(
            'Billing' => '',
            'Shipping' => '',
        );
        $code = null;

$code is not used, you could remove the assignment.

        if ($this->BillingAddressID) {

The property BillingAddressID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

            $billingAddress = BillingAddress::get()->byID($this->BillingAddressID);

The property BillingAddressID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

            if ($billingAddress) {
                if ($billingAddress->Country) {
                    $countryCodes['Billing'] = $billingAddress->Country;
                }
            }
        }
        if ($this->ShippingAddressID && $this->UseShippingAddress) {

The property ShippingAddressID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

The property UseShippingAddress does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

            $shippingAddress = ShippingAddress::get()->byID($this->ShippingAddressID);

The property ShippingAddressID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

            if ($shippingAddress) {
                if ($shippingAddress->ShippingCountry) {
                    $countryCodes['Shipping'] = $shippingAddress->ShippingCountry;
                }
            }
        }
        if (
            (EcommerceConfig::get('OrderAddress', 'use_shipping_address_for_main_region_and_country') && $countryCodes['Shipping'])
            ||
            (!$countryCodes['Billing'] && $countryCodes['Shipping'])
        ) {
            $code = $countryCodes['Shipping'];
        } elseif ($countryCodes['Billing']) {
            $code = $countryCodes['Billing'];
        } else {
            $code = EcommerceCountry::get_country_from_ip();
        }

        return $code;
    }

    /**
     * @alias for getFullNameCountry
     *
     * @return string - country name
     **/
    public function FullNameCountry()
    {
        return $this->getFullNameCountry();
    }

    /**
     * returns name of coutry.
     *
     * @return string - country name
     **/
    public function getFullNameCountry()
    {
        return EcommerceCountry::find_title($this->Country());
    }

    /**
     * @alis for getExpectedCountryName
     * @return string - country name
     **/
    public function ExpectedCountryName()
    {
        return $this->getExpectedCountryName();
    }

    /**
     * returns name of coutry that we expect the customer to have
     * This takes into consideration more than just what has been entered
     * for example, it looks at GEO IP.
     *
     * @todo: why do we dont return a string IF there is only one item.
     *
     * @return string - country name
     **/
    public function getExpectedCountryName()
    {
        return EcommerceCountry::find_title(EcommerceCountry::get_country(false, $this->ID));
    }

    /**
     * return the title of the fixed country (if any).
     *
     * @return string | empty string
     **/
    public function FixedCountry()
    {
        return $this->getFixedCountry();
    }
    public function getFixedCountry()
    {
        $code = EcommerceCountry::get_fixed_country_code();
        if ($code) {
            return EcommerceCountry::find_title($code);
        }

        return '';
    }

    /**
     * Returns the region that applies to the order.
     * we check both billing and shipping, in case one of them is empty.
     *
     * @return DataObject | Null (EcommerceRegion)

Should the return type not be DataObject|null?

     **/
    public function Region()
    {
        return $this->getRegion();
    }
    public function getRegion()
    {
        $regionIDs = array(
            'Billing' => 0,
            'Shipping' => 0,
        );
        if ($this->BillingAddressID) {

The property BillingAddressID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

            if ($billingAddress = $this->BillingAddress()) {

The method BillingAddress() does not exist on Order. Did you maybe mean getBillingAddressField()?

                if ($billingAddress->RegionID) {
                    $regionIDs['Billing'] = $billingAddress->RegionID;
                }
            }
        }
        if ($this->CanHaveShippingAddress()) {
            if ($this->ShippingAddressID) {

The property ShippingAddressID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                if ($shippingAddress = $this->ShippingAddress()) {

The method ShippingAddress() does not exist on Order. Did you maybe mean getShippingAddressField()?

                    if ($shippingAddress->ShippingRegionID) {
                        $regionIDs['Shipping'] = $shippingAddress->ShippingRegionID;
                    }
                }
            }
        }
        if (count($regionIDs)) {
            //note the double-check with $this->CanHaveShippingAddress() and get_use_....
            if ($this->CanHaveShippingAddress() && EcommerceConfig::get('OrderAddress', 'use_shipping_address_for_main_region_and_country') && $regionIDs['Shipping']) {
                return EcommerceRegion::get()->byID($regionIDs['Shipping']);
            } else {
                return EcommerceRegion::get()->byID($regionIDs['Billing']);
            }
        } else {
            return EcommerceRegion::get()->byID(EcommerceRegion::get_region_from_ip());
        }
    }

    /**
     * Casted variable
     * Currency is not the same as the standard one?
     *
     * @return bool
     **/
    public function HasAlternativeCurrency()
    {
        return $this->getHasAlternativeCurrency();
    }
    public function getHasAlternativeCurrency()
    {
        if ($currency = $this->CurrencyUsed()) {

The method CurrencyUsed does not exist on object<Order>? Since you implemented __call, maybe consider adding a @method annotation.

            if ($currency->IsDefault()) {

The if-else statement can be simplified to return !$currency->IsDefault();.

                return false;
            } else {
                return true;
            }
        } else {
            return false;
        }
    }

    /**
     * Makes sure exchange rate is updated and maintained before order is submitted
     * This method is public because it could be called from a shopping Cart Object.
     **/
    public function EnsureCorrectExchangeRate()
    {
        if (!$this->IsSubmitted()) {
            $oldExchangeRate = $this->ExchangeRate;

The property ExchangeRate does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

            if ($currency = $this->CurrencyUsed()) {

The method CurrencyUsed does not exist on object<Order>? Since you implemented __call, maybe consider adding a @method annotation.

                if ($currency->IsDefault()) {
                    $this->ExchangeRate = 0;

The property ExchangeRate does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

                } else {
                    $this->ExchangeRate = $currency->getExchangeRate();

The property ExchangeRate does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

                }
            } else {
                $this->ExchangeRate = 0;

The property ExchangeRate does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

            }
            if ($this->ExchangeRate != $oldExchangeRate) {

The property ExchangeRate does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                $this->write();
            }
        }
    }

    /**
     * speeds up processing by storing the IsSubmitted value
     * we start with -1 to know if it has been requested before.
     *
     * @var bool
     */
    protected $_isSubmittedTempVar = -1;

    /**
     * Casted variable - has the order been submitted?
     * alias
     * @param bool $recalculate
     *
     * @return bool
     **/
    public function IsSubmitted($recalculate = true)
    {
        return $this->getIsSubmitted($recalculate);
    }

    /**
     * Casted variable - has the order been submitted?
     *
     * @param bool $recalculate
     *
     * @return bool
     **/
    public function getIsSubmitted($recalculate = false)
    {
        if ($this->_isSubmittedTempVar === -1 || $recalculate) {
            if ($this->SubmissionLog()) {
                $this->_isSubmittedTempVar = true;
            } else {
                $this->_isSubmittedTempVar = false;
            }
        }

        return $this->_isSubmittedTempVar;
    }

    /**
     *
     *
     * @return bool
     */
    public function IsArchived()
    {
        $lastStep = OrderStep::last_order_step();
        if ($lastStep) {
            if ($lastStep->ID == $this->StatusID) {

The property StatusID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                return true;
            }
        }
        return false;
    }

    /**
     * Submission Log for this Order (if any).
     *
     * @return Submission Log (OrderStatusLog_Submitted) | Null
     **/
    public function SubmissionLog()
    {
        $className = EcommerceConfig::get('OrderStatusLog', 'order_status_log_class_used_for_submitting_order');

        return $className::get()
            ->Filter(array('OrderID' => $this->ID))
            ->Last();
    }

    /**
     * Submission Log for this Order (if any).
     *
     * @return DateTime

Should the return type not be DBField?

     **/
    public function OrderDate()
    {
        $object = $this->SubmissionLog();
        if($object) {
            $created = $object->Created;
        } else {
            $created = $this->LastEdited;
        }

        return DBField::create_field('SS_Datetime', $created);
    }

    /**
     * @return int
     */
    public function SecondsSinceBeingSubmitted()
    {
        if ($submissionLog = $this->SubmissionLog()) {
            return time() - strtotime($submissionLog->Created);
        } else {
            return 0;
        }
    }

    /**
     * if the order can not be submitted,
     * then the reasons why it can not be submitted
     * will be returned by this method.
     *
     * @see Order::canSubmit
     *
     * @return ArrayList | null

Should the return type not be ArrayList|null?

     */
    public function SubmitErrors()
    {
        $al = null;
        $extendedSubmitErrors = $this->extend('updateSubmitErrors');
        if ($extendedSubmitErrors !== null && is_array($extendedSubmitErrors) && count($extendedSubmitErrors)) {
            $al = ArrayList::create();
            foreach ($extendedSubmitErrors as $returnResultArray) {
                foreach ($returnResultArray as $issue) {
                    if ($issue) {
                        $al->push(ArrayData::create(array("Title" => $issue)));
                    }
                }
            }
        }
        return $al;
    }

    /**
     * Casted variable - has the order been submitted?
     *
     * @param bool $withDetail
     *
     * @return string
     **/
    public function CustomerStatus($withDetail = true)
    {
        return $this->getCustomerStatus($withDetail);
    }
    public function getCustomerStatus($withDetail = true)
    {
        $str = '';
        if ($this->MyStep()->ShowAsUncompletedOrder) {
            $str = _t('Order.UNCOMPLETED', 'Uncompleted');
        } elseif ($this->MyStep()->ShowAsInProcessOrder) {
            $str = _t('Order.IN_PROCESS', 'In Process');
        } elseif ($this->MyStep()->ShowAsCompletedOrder) {
            $str = _t('Order.COMPLETED', 'Completed');
        }
        if ($withDetail) {
            if (!$this->HideStepFromCustomer) {

The property HideStepFromCustomer does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                $str .= ' ('.$this->MyStep()->Name.')';
            }
        }

        return $str;
    }

    /**
     * Casted variable - does the order have a potential shipping address?
     *
     * @return bool

Should the return type not be array|integer|double|string|boolean?

     **/
    public function CanHaveShippingAddress()
    {
        return $this->getCanHaveShippingAddress();
    }
    public function getCanHaveShippingAddress()
    {
        return EcommerceConfig::get('OrderAddress', 'use_separate_shipping_address');
    }

    /**
     * returns the link to view the Order
     * WHY NOT CHECKOUT PAGE: first we check for cart page.
     *
     * @return CartPage | Null
     */
    public function DisplayPage()
    {
        if ($this->MyStep() && $this->MyStep()->AlternativeDisplayPage()) {
            $page = $this->MyStep()->AlternativeDisplayPage();
        } elseif ($this->IsSubmitted()) {
            $page = DataObject::get_one('OrderConfirmationPage');
        } else {
            $page = DataObject::get_one(
                'CartPage',
                array('ClassName' => 'CartPage')
            );
            if (!$page) {
                $page = DataObject::get_one('CheckoutPage');
            }
        }

        return $page;
    }

    /**
     * returns the link to view the Order
     * WHY NOT CHECKOUT PAGE: first we check for cart page.
     * If a cart page has been created then we refer through to Cart Page.
     * Otherwise it will default to the checkout page.
     *
     * @param string $action - any action that should be added to the link.

Should the type for parameter $action not be string|null?

     *
     * @return String(URLSegment)

The doc-type String(URLSegment) could not be parsed: Expected "|" or "end of type", but got "(" at position 6. (view supported doc-types)

     */
    public function Link($action = null)

The return type could not be reliably inferred; please add a @return annotation.

    {
        $page = $this->DisplayPage();
        if ($page) {
            return $page->getOrderLink($this->ID, $action);
        } else {
            user_error('A Cart / Checkout Page + an Order Confirmation Page needs to be setup for the e-commerce module to work.', E_USER_NOTICE);
            $page = DataObject::get_one(
                'ErrorPage',
                array('ErrorCode' => '404')
            );
            if ($page) {
                return $page->Link();
            }
        }
    }

    /**
     * Returns to link to access the Order's API.
     *
     * @param string $version
     * @param string $extension
     *
     * @return String(URL)

The doc-type String(URL) could not be parsed: Expected "|" or "end of type", but got "(" at position 6. (view supported doc-types)

     */
    public function APILink($version = 'v1', $extension = 'xml')
    {
        return Director::AbsoluteURL("/api/ecommerce/$version/Order/".$this->ID."/.$extension");
    }

    /**
     * returns the link to finalise the Order.
     *
     * @return String(URLSegment)

The doc-type String(URLSegment) could not be parsed: Expected "|" or "end of type", but got "(" at position 6. (view supported doc-types)

     */
    public function CheckoutLink()

The return type could not be reliably inferred; please add a @return annotation.

    {
        $page = DataObject::get_one('CheckoutPage');
        if ($page) {
            return $page->Link();
        } else {
            $page = DataObject::get_one(
                'ErrorPage',
                array('ErrorCode' => '404')
            );
            if ($page) {
                return $page->Link();
            }
        }
    }

    /**
     * Converts the Order into HTML, based on the Order Template.
     *
     * @return HTML Object

Should the return type not be DBField?

     **/
    public function ConvertToHTML()
    {
        Config::nest();
        Config::inst()->update('SSViewer', 'theme_enabled', true);
        $html = $this->renderWith('Order');
        Config::unnest();
        $html = preg_replace('/(\s)+/', ' ', $html);

        return DBField::create_field('HTMLText', $html);
    }

    /**
     * Converts the Order into a serialized string
     * TO DO: check if this works and check if we need to use special sapphire serialization code.
     *
     * @return string - serialized object
     **/
    public function ConvertToString()
    {
        return serialize($this->addHasOneAndHasManyAsVariables());
    }

    /**
     * Converts the Order into a JSON object
     * TO DO: check if this works and check if we need to use special sapphire JSON code.
     *
     * @return string -  JSON
     **/
    public function ConvertToJSON()
    {
        return json_encode($this->addHasOneAndHasManyAsVariables());
    }

    /**
     * returns itself wtih more data added as variables.
     * We add has_one and has_many as variables like this: $this->MyHasOne_serialized = serialize($this->MyHasOne()).
     *
     * @return Order - with most important has one and has many items included as variables.
     **/
    protected function addHasOneAndHasManyAsVariables()
    {
        $object = clone $this;
        $object->Member_serialized = serialize($this->Member());

The property Member_serialized does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

The method Member() does not exist on Order. Did you maybe mean CreateOrReturnExistingMember()?

        $object->BillingAddress_serialized = serialize($this->BillingAddress());

The property BillingAddress_serialized does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

The method BillingAddress() does not exist on Order. Did you maybe mean getBillingAddressField()?

        $object->ShippingAddress_serialized = serialize($this->ShippingAddress());

The property ShippingAddress_serialized does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

The method ShippingAddress() does not exist on Order. Did you maybe mean getShippingAddressField()?

        $object->Attributes_serialized = serialize($this->Attributes());

The property Attributes_serialized does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

The method Attributes() does not exist on Order. Did you maybe mean getOrderAttributesByType()?

        $object->OrderStatusLogs_serialized = serialize($this->OrderStatusLogs());

The property OrderStatusLogs_serialized does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

The method OrderStatusLogs() does not exist on Order. Did you maybe mean getOrderStatusLogsTableField()?

        $object->Payments_serialized = serialize($this->Payments());

The property Payments_serialized does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

The method Payments() does not exist on Order. Did you maybe mean getPaymentsField()?

        $object->Emails_serialized = serialize($this->Emails());

The property Emails_serialized does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

The method Emails() does not exist on Order. Did you maybe mean getEmailsTableField()?

        return $object;
    }

/*******************************************************
   * 9. TEMPLATE RELATED STUFF
*******************************************************/

    /**
     * returns the instance of EcommerceConfigAjax for use in templates.
     * In templates, it is used like this:
     * $EcommerceConfigAjax.TableID.
     *
     * @return EcommerceConfigAjax

Should the return type not be EcommerceConfigAjaxDefinitions?

     **/
    public function AJAXDefinitions()
    {
        return EcommerceConfigAjax::get_one($this);
    }

    /**
     * returns the instance of EcommerceDBConfig.
     *
     * @return EcommerceDBConfig
     **/
    public function EcomConfig()
    {
        return EcommerceDBConfig::current_ecommerce_db_config();
    }

    /**
     * Collects the JSON data for an ajax return of the cart.
     *
     * @param array $js
     *
     * @return array (for use in AJAX for JSON)
     **/
    public function updateForAjax(array $js)
    {
        $function = EcommerceConfig::get('Order', 'ajax_subtotal_format');
        if (is_array($function)) {
            list($function, $format) = $function;
        }
        $subTotal = $this->$function();
        if (isset($format)) {
            $subTotal = $subTotal->$format();
            unset($format);
        }
        $function = EcommerceConfig::get('Order', 'ajax_total_format');
        if (is_array($function)) {
            list($function, $format) = $function;
        }
        $total = $this->$function();
        if (isset($format)) {
            $total = $total->$format();
        }
        $ajaxObject = $this->AJAXDefinitions();
        $js[] = array(
            't' => 'id',
            's' => $ajaxObject->TableSubTotalID(),
            'p' => 'innerHTML',
            'v' => $subTotal,
        );
        $js[] = array(
            't' => 'id',
            's' => $ajaxObject->TableTotalID(),
            'p' => 'innerHTML',
            'v' => $total,
        );
        $js[] = array(
            't' => 'class',
            's' => $ajaxObject->TotalItemsClassName(),
            'p' => 'innerHTML',
            'v' => $this->TotalItems($recalculate = true),
        );
        $js[] = array(
            't' => 'class',
            's' => $ajaxObject->TotalItemsTimesQuantityClassName(),
            'p' => 'innerHTML',
            'v' => $this->TotalItemsTimesQuantity(),
        );
        $js[] = array(
            't' => 'class',
            's' => $ajaxObject->ExpectedCountryClassName(),
            'p' => 'innerHTML',
            'v' => $this->ExpectedCountryName(),
        );

        return $js;
    }

    /**
     * @ToDO: move to more appropriate class
     *
     * @return float
     **/
    public function SubTotalCartValue()
    {
        return $this->SubTotal;

The property SubTotal does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

    }

/*******************************************************
   * 10. STANDARD SS METHODS (requireDefaultRecords, onBeforeDelete, etc...)
*******************************************************/

    /**
     *standard SS method.
     **/
    public function populateDefaults()
    {
        parent::populateDefaults();
    }

    public function onBeforeWrite()
    {
        parent::onBeforeWrite();
        if (! $this->getCanHaveShippingAddress()) {
            $this->UseShippingAddress = false;

The property UseShippingAddress does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

        }
        if (!$this->CurrencyUsedID) {

The property CurrencyUsedID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

            $this->CurrencyUsedID = EcommerceCurrency::default_currency_id();

The property CurrencyUsedID does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

        }
        if (!$this->SessionID) {

The property SessionID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

            $generator = Injector::inst()->create('RandomGenerator');
            $token = $generator->randomToken('sha1');
            $this->SessionID = substr($token, 0, 32);

The property SessionID does not exist on object<Order>. Since you implemented __set, maybe consider adding a @property annotation.

        }
    }

    /**
     * standard SS method
     * adds the ability to update order after writing it.
     **/
    public function onAfterWrite()

onAfterWrite uses the super-global variable $_REQUEST which is generally not recommended.

    {
        parent::onAfterWrite();
        //crucial!
        self::set_needs_recalculating(true, $this->ID);
        // quick double-check
        if ($this->IsCancelled() && ! $this->IsArchived()) {
            $this->Archive($avoidWrites = true);
        }
        if ($this->IsSubmitted($recalculate = true)) {

This if statement is empty and can be removed.

            //do nothing
        } else {
            if ($this->StatusID) {

The property StatusID does not exist on object<Order>. Since you implemented __get, maybe consider adding a @property annotation.

                $this->calculateOrderAttributes($recalculate = false);
                if (EcommerceRole::current_member_is_shop_admin()) {
                    if (isset($_REQUEST['SubmitOrderViaCMS'])) {
                        $this->tryToFinaliseOrder();
                        //just in case it writes again...
                        unset($_REQUEST['SubmitOrderViaCMS']);
                    }
                }
            }
        }
    }

    /**
     *standard SS method.
     *
     * delete attributes, statuslogs, and payments
     * THIS SHOULD NOT BE USED AS ORDERS SHOULD BE CANCELLED NOT DELETED
     */
    public function onBeforeDelete()
    {
        parent::onBeforeDelete();
        if ($attributes = $this->Attributes()) {

The method Attributes() does not exist on Order. Did you maybe mean getOrderAttributesByType()?

            foreach ($attributes as $attribute) {
                $attribute->delete();
                $attribute->destroy();
            }
        }

        //THE REST WAS GIVING ERRORS - POSSIBLY DUE TO THE FUNNY RELATIONSHIP (one-one, two times...)
        /*

66% of this comment could be valid code. Did you maybe forget this after debugging?

        if($billingAddress = $this->BillingAddress()) {
            if($billingAddress->exists()) {
                $billingAddress->delete();
                $billingAddress->destroy();
            }
        }
        if($shippingAddress = $this->ShippingAddress()) {
            if($shippingAddress->exists()) {
                $shippingAddress->delete();
                $shippingAddress->destroy();
            }
        }

        if($statuslogs = $this->OrderStatusLogs()){
            foreach($statuslogs as $log){
                $log->delete();
                $log->destroy();
            }
        }
        if($payments = $this->Payments()){
            foreach($payments as $payment){
                $payment->delete();
                $payment->destroy();
            }
        }
        if($emails = $this->Emails()) {
            foreach($emails as $email){
                $email->delete();
                $email->destroy();
            }
        }
        */
    }

/*******************************************************
   * 11. DEBUG
*******************************************************/

    /**
     * Debug helper method.
     * Can be called from /shoppingcart/debug/.
     *
     * @return string
     */
    public function debug()
    {
        $this->calculateOrderAttributes(true);

        return EcommerceTaskDebugCart::debug_object($this);
    }
}