SalesReport_Handler::processform() - Code Metrics - sunnysideup/silverstripe-ecommerce_reports - Measure and Improve Code Quality continuously with Scrutinizer

SalesReport_Handler::processform() A
last analyzed 2018-03-20 01:10 UTC

↳ Parent: SalesReport_Handler

Complexity

Conditions	1
Paths	1

Size

Total Lines	6
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	6
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	4
nc	1
nop	0

<?php
/**
 * @author Nicolaas [at] sunnysideup.co.nz
 * @package ecommercextras
 */
class SalesReport extends SS_Report
namespace YourVendor;

class YourClass { }
{
    protected static $full_export_select_statement =
        array(
            "`Order`.`ID`" => "Order number",
            "`Order`.`Created`" => "Order date and time",
            "GROUP_CONCAT(`Payment`.`Message` SEPARATOR ', ')" => "payment gateway reference",
            "GROUP_CONCAT(`Payment`.`ID` SEPARATOR ', ')" =>  "Transaction Id",
            "SUM(IF(Payment.Status = 'Success',`Payment`.`Amount`, 0)) RealPayments" => "Total Order Amount",
            "IF(ProductVariationsForVariations.Title IS NOT NULL, CONCAT(ProductSiteTreeForVariations.Title,' : ', ProductVariationsForVariations.Title), IF(SiteTreeForProducts.Title IS NOT NULL, SiteTreeForProducts.Title, OrderAttribute.ClassName)) ProductOrModifierName" => "Product Name",
            "IF(OrderItem.Quantity IS NOT NULL, OrderItem.Quantity, 1)  ProductOrModifierQuantity" => "Quantity",
            "IF(ProductSiteTreeForVariations.ID IS NOT NULL, ProductVariationsForVariations.Price, 0) +  IF(ProductForProducts.Price IS NOT NULL, ProductForProducts.Price, 0) + IF(OrderModifier.Amount IS NOT NULL, OrderModifier.Amount, 0) ProductOrModifierPrice" => "Amount",
            "IF(ProductForProducts.Price IS NOT NULL, ProductForProducts.Price, 0) ProductOrModifierPriceProduct" => "Product Amount",
            "IF(ProductSiteTreeForVariations.ID IS NOT NULL, ProductVariationsForVariations.Price, 0) ProductOrModifierPriceVariation" => "Variation Amount",
            "IF(OrderModifier.Amount IS NOT NULL, OrderModifier.Amount, 0) ProductOrModifierPriceModifier" => "Modifier Amount",
            "CONCAT(Member.Address, ' ', Member.AddressLine2,' ', Member.City, ' ', Member.Country,' ', Member.HomePhone,' ', Member.MobilePhone,' ', Member.Notes,' ', Member.Notes ) MemberContactDetails" => "Customer contact details",
            "IF(`Order`.`UseShippingAddress`, CONCAT(`Order`.`ShippingName`, ' ',`Order`.`ShippingAddress`, ' ',`Order`.`ShippingAddress2`, ' ',`Order`.`ShippingCity`, ' ',`Order`.`ShippingCountry`), 'no alternative delivery address') MemberShippingDetailsAddress" => "Costumer delivery",
            "`Order`.`Status`" => "Order status",
            "GROUP_CONCAT(`OrderStatusLogWithDetails`.`DispatchTicket` SEPARATOR ', ')" => "Dispatch ticket code",
            "GROUP_CONCAT(`OrderStatusLogWithDetails`.`DispatchedOn` SEPARATOR ', ')" => "Dispatch date",
            "GROUP_CONCAT(`OrderStatusLogWithDetails`.`DispatchedBy` SEPARATOR ', ')" => "Dispatched by",
            "GROUP_CONCAT(`OrderStatusLog`.`Note` SEPARATOR ', ')" => "Dispatch notes"
        );
    public static function set_full_export_select_statement($v)
    {
        self::$full_export_select_statement = $v;
    }
    public static function get_full_export_select_statement()

    {
        $bt = defined('DB::USE_ANSI_SQL') ? "\"" : "`";
        if ($bt == '"') {
            return str_replace('`', '"', self::$full_export_select_statement);
        } else {
            return self::$full_export_select_statement;
        }
    }

    protected static $full_export_join_statement = '
			INNER JOIN `Order` ON `Order`.`ID` = `OrderAttribute`.`OrderID`
			INNER JOIN `Member` On `Order`.`MemberID` = `Member`.`ID`
			LEFT JOIN `Payment` ON `Payment`.`OrderID` = `Order`.`ID`
			LEFT JOIN `Product_versions` ProductForProducts ON `Product_OrderItem`.`ProductID` = ProductForProducts.`RecordID` AND `Product_OrderItem`.`ProductVersion` = ProductForProducts.`Version`
			LEFT JOIN `SiteTree_versions` SiteTreeForProducts ON SiteTreeForProducts.`RecordID` = `Product_OrderItem`.`ProductID` AND `Product_OrderItem`.`ProductVersion` = SiteTreeForProducts.`Version`
			LEFT JOIN `ProductVariation_versions` ProductVariationsForVariations ON `ProductVariation_OrderItem`.`ProductVariationID` = ProductVariationsForVariations.`RecordID` AND `ProductVariation_OrderItem`.`ProductVariationVersion` = ProductVariationsForVariations.`Version`
			LEFT JOIN `SiteTree_versions` ProductSiteTreeForVariations ON `ProductSiteTreeForVariations`.`RecordID` = `Product_OrderItem`.`ProductID` AND `Product_OrderItem`.`ProductVersion` = `ProductSiteTreeForVariations`.`Version`
			LEFT JOIN `OrderStatusLog` ON `OrderStatusLog`.`OrderID` = `Order`.`ID`
			LEFT JOIN `OrderStatusLogWithDetails` ON `OrderStatusLogWithDetails`.`ID` = `OrderStatusLog`.`ID`';
    public static function set_full_export_join_statement($v)
    {
        self::$full_export_join_statement = $v;
    }
    public static function get_full_export_join_statement()

    {
        $bt = defined('DB::USE_ANSI_SQL') ? "\"" : "`";
        if ($bt == '"') {
            return str_replace('`', '"', self::$full_export_join_statement);
        } else {
            return self::$full_export_join_statement;
        }
    }

    protected static $full_export_file_name = "SalesExport";
    public static function set_full_export_file_name($v)
    {
        self::$full_export_file_name = $v;
    }
    public static function get_full_export_file_name()

    {
        $bt = defined('DB::USE_ANSI_SQL') ? "\"" : "`";
        if ($bt == '"') {
            return str_replace('`', '"', self::$full_export_file_name);
        } else {
            return self::$full_export_file_name;
        }
    }

    protected $title = 'All Orders';

    protected static $sales_array = array();

    protected $description = 'Search Orders';

    /**
     * Return a {@link ComplexTableField} that shows
     * all Order instances that are not printed. That is,
     * Order instances with the property "Printed" value
     * set to "0".
     *
     * @return ComplexTableField
     */
    public function getReportField()
    {
        // Get the fields used for the table columns
        Order::$summary_fields["Created"] = "Received";
        $fields = Order::$summary_fields;

        // Add some fields specific to this report
        $fields['Invoice'] = '';
        $fields['PackingSlip'] = '';
        $fields['ChangeStatus'] = '';

        $table = new TableListField(
            'Orders',
            'Order',
            $fields
        );

        // Customise the SQL query for Order, because we don't want it querying
        // all the fields. Invoice and Printed are dummy fields that just have some
        // text in them, which would be automatically queried if we didn't specify
        // a custom query.

        $table->setCustomQuery($this->getCustomQuery());

        // Set the links to the Invoice and Print fields allowing a user to view
        // another template for viewing an Order instance
        $table->setFieldFormatting(array(
            'Invoice' => '<a href=\"OrderReportWithLog_Popup/invoice/$ID\" class=\"makeIntoPopUp\">Invoice and Update</a>',
            'PackingSlip' => '<a href=\"OrderReport_Popup/packingslip/$ID\" class=\"makeIntoPopUp\">Packing Slip</a>',
            'ChangeStatus' => '<a href=\"#\" class=\"statusDropdownChange\" rel=\"$ID\">$Status</a><span class=\"outcome\"></span>'
        ));

        $table->setFieldCasting(array(
            'Total' => 'Currency->Nice'
        ));

        $table->setPermissions(array(
            'edit',
            'show',
            'export'
        ));
        $table->setPageSize(250);


        $table->setFieldListCsv($this->getExportFields());

        $table->setCustomCsvQuery($this->getExportQuery());

        //$tableField->removeCsvHeader();


        return $table;
    }

    public function getCustomQuery()

    {
        if ("SalesReport" == $this->class) {
            user_error('Please implement getCustomQuery() on ' . $this->class, E_USER_ERROR);
        } else {
            //buildSQL($filter = "", $sort = "", $limit = "", $join = "", $restrictClasses = true, $having = "")

            $query = singleton('Order')->buildSQL('', 'Order.Created DESC');
            $query->groupby[] = 'Order.ID';
            return $query;
        }
    }

    public function getExportFields()
    {
        if ("SalesReport" == $this->class) {
            user_error('Please implement getExportFields() on ' . $this->class, E_USER_ERROR);
        } else {
if (rand(1, 6) > 3) {
print "Check failed";
} else {
    //print "Check succeeded";
}
        }
    }

    public function getExportQuery()

    {
        if ("SalesReport" == $this->class) {
            user_error('Please implement getExportFields() on ' . $this->class, E_USER_ERROR);
        } else {
            $query = singleton('Order')->buildSQL('', 'Order.Created DESC');
            $query->groupby[] = 'Order.Created';
            return $query;
        }
    }

    protected function statistic($type)

    {
        if (!count(self::$sales_array)) {
            $data = $this->getCustomQuery()->execute();
            if ($data) {
                $array = array();
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
                foreach ($data as $row) {
                    if ($row["RealPayments"]) {
                        self::$sales_array[] = $row["RealPayments"];
                    }
                }
            }
        }
        if (count(self::$sales_array)) {
            switch ($type) {
                case "count":
                    return count(self::$sales_array);
                    break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
                case "sum":
                    return array_sum(self::$sales_array);
                    break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
                case "avg":
                    return array_sum(self::$sales_array) / count(self::$sales_array);
                    break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
                case "min":
                    asort(self::$sales_array);
                    foreach (self::$sales_array as $item) {
                        return $item;
                    }
                    break;
                case "max":
                    arsort(self::$sales_array);
                    foreach (self::$sales_array as $item) {
                        return $item;
                    }
                    break;
                default:
                    user_error("Wrong statistic type speficied in SalesReport::statistic", E_USER_ERROR);
            }
        }
        return -1;
    }

    public function processform()
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    {
        if ("SalesReport" == $this->class) {
            user_error('Please implement processform() on ' . $this->class, E_USER_ERROR);
        } else {
            die($_REQUEST);

        }
    }
}

class SalesReport_Handler extends Controller
namespace YourVendor;

class YourClass { }
{
    public function processform()

    {
        $ClassName = Director::URLParam("ID");

        $object = new $ClassName;
        return $object->processform();
    }


    public function setstatus()

    {
        $id = $this->urlParams['ID'];
        if (!is_numeric($id)) {
            return "could not update order status";
        }
        $order = DataObject::get_by_id('Order', $id);
        if ($order) {
            $oldStatus = $order->Status;
            $newStatus = $this->urlParams['OtherID'];
            if ($oldStatus != $newStatus) {
                $order->Status = $newStatus;
                $order->write();
                $orderlog = new OrderStatusLog();
                $orderlog->OrderID = $order->ID;
                $orderlog->Status = "Status changed from ".$oldStatus." to ".$newStatus.".";
                $orderlog->Note = "Status changed from ".$oldStatus." to ".$newStatus.".";
                $orderlog->write();
            } else {
                return "no change";
            }
        } else {
            return "order not found";
        }
        return "updated to ".$newStatus;
    }

    public function fullsalesexport()
    {
        $exportReport = new SearchableProductSalesReport();
        $query = $exportReport->getCustomQuery();
        $query->select = null;
        $array = SalesReport::get_full_export_select_statement();
        if (is_array($array) && count($array)) {
            $fileData = '"row number"';
            foreach ($array as $sql => $name) {
                $query->select[] = $sql;
                $fileData .= ',"'.$name.'"';
            }
            $data = $query->execute();
            if ($data) {
                $i = 0;
                foreach ($data as $row) {
                    $i++;
                    $fileData .= "\r\n".'"'.$i.'"';
                    foreach ($row as $fieldName => $value) {
                        $fileData .= ',"'.$value.'"';
                    }
                }
            } else {
                $fileData = "no data available";
            }
        } else {
            $fileData = "please select fields first";
        }
        $fileName = SalesReport::get_full_export_file_name()."-".date("Y-m-d", strtotime("today")).".csv";
        header("Content-Type: text/csv; name=\"" . addslashes($fileName) . "\"");
        header("Content-Disposition: attachment; filename=\"" . addslashes($fileName) . "\"");
        header("Content-length: " . strlen($fileData));
        echo $fileData;
        exit();

    }
}


1		<?php
2		/**
3		* @author Nicolaas [at] sunnysideup.co.nz
4		* @package ecommercextras
5		*/
6		class SalesReport extends SS_Report
		0 ignored issues – show Coding Style Compatibility introduced 2017-01-03 01:11 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
7		{
8		protected static $full_export_select_statement =
9		array(
10		"`Order`.`ID`" => "Order number",
11		"`Order`.`Created`" => "Order date and time",
12		"GROUP_CONCAT(`Payment`.`Message` SEPARATOR ', ')" => "payment gateway reference",
13		"GROUP_CONCAT(`Payment`.`ID` SEPARATOR ', ')" => "Transaction Id",
14		"SUM(IF(Payment.Status = 'Success',`Payment`.`Amount`, 0)) RealPayments" => "Total Order Amount",
15		"IF(ProductVariationsForVariations.Title IS NOT NULL, CONCAT(ProductSiteTreeForVariations.Title,' : ', ProductVariationsForVariations.Title), IF(SiteTreeForProducts.Title IS NOT NULL, SiteTreeForProducts.Title, OrderAttribute.ClassName)) ProductOrModifierName" => "Product Name",
16		"IF(OrderItem.Quantity IS NOT NULL, OrderItem.Quantity, 1) ProductOrModifierQuantity" => "Quantity",
17		"IF(ProductSiteTreeForVariations.ID IS NOT NULL, ProductVariationsForVariations.Price, 0) + IF(ProductForProducts.Price IS NOT NULL, ProductForProducts.Price, 0) + IF(OrderModifier.Amount IS NOT NULL, OrderModifier.Amount, 0) ProductOrModifierPrice" => "Amount",
18		"IF(ProductForProducts.Price IS NOT NULL, ProductForProducts.Price, 0) ProductOrModifierPriceProduct" => "Product Amount",
19		"IF(ProductSiteTreeForVariations.ID IS NOT NULL, ProductVariationsForVariations.Price, 0) ProductOrModifierPriceVariation" => "Variation Amount",
20		"IF(OrderModifier.Amount IS NOT NULL, OrderModifier.Amount, 0) ProductOrModifierPriceModifier" => "Modifier Amount",
21		"CONCAT(Member.Address, ' ', Member.AddressLine2,' ', Member.City, ' ', Member.Country,' ', Member.HomePhone,' ', Member.MobilePhone,' ', Member.Notes,' ', Member.Notes ) MemberContactDetails" => "Customer contact details",
22		"IF(`Order`.`UseShippingAddress`, CONCAT(`Order`.`ShippingName`, ' ',`Order`.`ShippingAddress`, ' ',`Order`.`ShippingAddress2`, ' ',`Order`.`ShippingCity`, ' ',`Order`.`ShippingCountry`), 'no alternative delivery address') MemberShippingDetailsAddress" => "Costumer delivery",
23		"`Order`.`Status`" => "Order status",
24		"GROUP_CONCAT(`OrderStatusLogWithDetails`.`DispatchTicket` SEPARATOR ', ')" => "Dispatch ticket code",
25		"GROUP_CONCAT(`OrderStatusLogWithDetails`.`DispatchedOn` SEPARATOR ', ')" => "Dispatch date",
26		"GROUP_CONCAT(`OrderStatusLogWithDetails`.`DispatchedBy` SEPARATOR ', ')" => "Dispatched by",
27		"GROUP_CONCAT(`OrderStatusLog`.`Note` SEPARATOR ', ')" => "Dispatch notes"
28		);
29		public static function set_full_export_select_statement($v)
30		{
31		self::$full_export_select_statement = $v;
32		}
33	View Code Duplication	public static function get_full_export_select_statement()
		0 ignored issues – show Duplication introduced 2017-01-03 01:11 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
34		{
35		$bt = defined('DB::USE_ANSI_SQL') ? "\"" : "`";
36		if ($bt == '"') {
37		return str_replace('`', '"', self::$full_export_select_statement);
38		} else {

sunnysideup / silverstripe-ecommerce_reports

SalesReport_Handler::processform() A last analyzed 2018-03-20 01:10 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

SalesReport_Handler::processform() A
last analyzed 2018-03-20 01:10 UTC