SilverCommerce\DownloadableProducts\DownloadableProduct

Complexity

Total Complexity

15

Size/Duplication

Total Lines	175
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getCMSFields()	0	16	1
A	getDownloadLink()	0	17	2
A	getDownloadFolder()	0	4	1
A	canDownload()	0	27	4
A	onBeforeWrite()	0	26	6
A	getDeliverable()	0	3	1

<?php

namespace SilverCommerce\DownloadableProducts;

use Product;

The type Product was not found. Maybe you did not declare it correctly or list all dependencies?

use SilverStripe\Assets\File;
use SilverStripe\Forms\TextField;
use SilverStripe\Security\Security;
use SilverStripe\Security\Member;
use SilverStripe\Core\Config\Config;
use SilverCommerce\OrdersAdmin\Model\Invoice;
use SilverStripe\AssetAdmin\Forms\UploadField;
use SilverCommerce\DownloadableProducts\DownloadFolder;
use SilverCommerce\DownloadableProducts\FileDownloadController;
use SilverStripe\Core\Injector\Injector;

/**
 * Product class that will allow adding of product to the CMS.
 */
class DownloadableProduct extends Product
{

    /**
     * Set the default DB table name
     * 
     * @var string
     */
    private static $table_name = "DownloadableProduct";

The private property $table_name is not used, and could be removed.

    private static $singular_name = 'Downloadable Product';

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

    private static $plural_name = 'Downloadable Products';

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

    private static $description = "A product that can be downloaded";

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

    private static $db = [

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

        'LinkLife' => 'Int'
    ];

    private static $has_one = [

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

        "File" => File::class
    ];

    private static $casting = [

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

        "DownloadLink" => "Varchar",
        "Deliverable" => "Boolean"
    ];

    private static $owns = [

The private property $owns is not used, and could be removed.

        "File"
    ];

    private static $defaults = [

The private property $defaults is not used, and could be removed.

        'LinkLife' => 7
    ];

    /**
     * Downloadable products are not deliverable. This will be
     * detected by the shopping cart to disable delivery options.
     *
     * @return boolean
     */
    public function getDeliverable()
    {
        return false;
    }

    /**
     * Get the link to download the file associated with this product
     *
     * @param int    $invoice_id the ID of an associated invoice
     * @param string $access_key key of assocaiated invoice
     * 
     * @return string
     */
    public function getDownloadLink($invoice_id, $access_key)
    {
        $file = $this->File();

        if ($file->exists()) {
            $download = Injector::inst()
                ->get(FileDownloadController::class);

            return $download->DownloadLink(
                $file->ID,
                $invoice_id,
                $access_key,
                $file->Name
            );
        }

        return "";
    }

    /**
     * Get the folder to add downloads to
     * 
     * @return DownloadFolder
     */
    public function getDownloadFolder()
    {
        return DownloadFolder::find_or_make(
            Config::inst()->get(DownloadFolder::class, "folder_name")
        );
    }


    public function getCMSFields()
    {
        $fields = parent::getCMSFields();

        $fields->removeByName("Weight");

        $fields->addFieldsToTab(
            "Root.Settings",
            [
                TextField::create('LinkLife', 'Life of download link (in days)'),
                UploadField::create("File")
                    ->setFolderName($this->config()->folder_name)
            ]
        );

        return $fields;
    }

    /**
     * Ensure weight is removed on save and that attached files are moved to
     * the correct folder
     *
     * @return void
     */
    public function onBeforeWrite()
    {
        parent::onBeforeWrite();

        // Set weight
        $this->Weight = 0;

The property Weight does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        // Deal with moving file
        $file = $this->File();
        $folder = $this->getDownloadFolder();
        $move = true;

        // If the file is in the download folder (or an ancestor), don't move
        if ($file->exists() && $file->ParentID == $folder->ID) {
            $move = false;
        } elseif ($file->exists()) {
            $id_list = $folder->getDescendantIDList();

The method getDescendantIDList() does not exist on SilverCommerce\DownloadableProducts\DownloadFolder. Since you implemented __call, consider adding a @method annotation.

            if (in_array($file->ParentID, $id_list)) {
                $move = false;
            }
        }

        // If needed, move the attached file to a new folder
        if ($move) {
            $file->ParentID = $folder->ID;
            $file->write();
        }
    }

    /**
     * Special permission to see if this product can be downloaded by the current member
     *
     * @param Member $member The current member object
     *
     * @return boolean
     */
    public function canDownload(Member $member = null)
    {
        if (!$member) {
            $member = Security::getCurrentUser();
        }

        $contact = null;

        if (isset($member)) {
            $contact = $member->Contact();
        }

        if (isset($contact)) {
            $items = $contact
                ->Invoices()
                ->filter(
                    [
                        "Status" => Config::inst()
                            ->get(Invoice::class, "paid_statuses"),
                        "Items.StockID" => $this->StockID
                    ]
                );

            return $items->exists();
        }

        return false;
    }
}