Nickbur / Sunrise-CMS

application/controller/account/download.php

<?php

/* 	Divine CMS - Open source CMS for widespread use.
    Copyright (c) 2019 Mykola Burakov (burakov.work@gmail.com)

    See SOURCE.txt for other and additional information.

    This file is part of Divine CMS.

    This program is free software: you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
    the Free Software Foundation, either version 3 of the License, or
    (at your option) any later version.

    This program is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
    GNU General Public License for more details.

    You should have received a copy of the GNU General Public License
    along with this program. If not, see <http://www.gnu.org/licenses/>. */

class ControllerAccountDownload extends \Divine\Engine\Core\Controller

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

{
    public function index()

Expected 2 blank lines before function; 0 found

    {
        if (!$this->customer->isLogged()) {
            $this->session->data['redirect'] = $this->url->link('account/download', '', true);

            $this->response->redirect($this->url->link('account/login', '', true));
        }

        $this->load->language('account/download');

        $this->document->setTitle($this->language->get('heading_title'));

        $data['breadcrumbs'] = array();

$data was never initialized. Although not strictly required by PHP, it is generally a good practice to add $data = array(); before regardless.

        $data['breadcrumbs'][] = array(
            'text' => $this->language->get('text_home'),
            'href' => $this->url->link('common/home')
        );

        $data['breadcrumbs'][] = array(
            'text' => $this->language->get('text_account'),
            'href' => $this->url->link('account/account', '', true)
        );

        $data['breadcrumbs'][] = array(
            'text' => $this->language->get('text_downloads'),
            'href' => $this->url->link('account/download', '', true)
        );

        $this->load->model('account/download');

        $data['heading_title'] = $this->language->get('heading_title');

        $data['text_empty'] = $this->language->get('text_empty');

        $data['column_order_id'] = $this->language->get('column_order_id');
        $data['column_name'] = $this->language->get('column_name');
        $data['column_size'] = $this->language->get('column_size');
        $data['column_date_added'] = $this->language->get('column_date_added');

        $data['button_download'] = $this->language->get('button_download');
        $data['button_continue'] = $this->language->get('button_continue');

        if (isset($this->request->get['page'])) {
            $page = $this->request->get['page'];
        } else {
            $page = 1;
        }

        $data['downloads'] = array();

        $download_total = $this->model_account_download->getTotalDownloads();

        $results = $this->model_account_download->getDownloads(($page - 1) * $this->config->get('config_limit_store'), $this->config->get('config_limit_store'));

        foreach ($results as $result) {
            if (file_exists($_SERVER['DOCUMENT_ROOT'] . '/storage/download/' . $result['filename'])) {
                $size = filesize($_SERVER['DOCUMENT_ROOT'] . '/storage/download/' . $result['filename']);

                $i = 0;

                $suffix = array(
                    'B',
                    'KB',
                    'MB',
                    'GB',
                    'TB',
                    'PB',
                    'EB',
                    'ZB',
                    'YB'
                );

                while (($size / 1024) > 1) {
                    $size = $size / 1024;
                    $i++;
                }

                $data['downloads'][] = array(
                    'order_id'   => $result['order_id'],
                    'date_added' => date($this->language->get('date_format_short'), strtotime($result['date_added'])),
                    'name'       => $result['name'],
                    'size'       => round(substr($size, 0, strpos($size, '.') + 4), 2) . $suffix[$i],

substr($size, 0, strpos($size, '.') + 4) of type string is incompatible with the type double expected by parameter $val of round().

                    'href'       => $this->url->link('account/download/download', 'download_id=' . $result['download_id'], true)
                );
            }
        }

        $pagination = new \Divine\Engine\Library\Pagination();
        $pagination->total = $download_total;
        $pagination->page = $page;
        $pagination->limit = $this->config->get($this->config->get('config_theme') . '_product_limit');
        $pagination->url = $this->url->link('account/download', 'page={page}', true);

        $data['pagination'] = $pagination->render();

        $data['results'] = sprintf($this->language->get('text_pagination'), ($download_total) ? (($page - 1) * $this->config->get($this->config->get('config_theme') . '_product_limit')) + 1 : 0, ((($page - 1) * $this->config->get($this->config->get('config_theme') . '_product_limit')) > ($download_total - $this->config->get($this->config->get('config_theme') . '_product_limit'))) ? $download_total : ((($page - 1) * $this->config->get($this->config->get('config_theme') . '_product_limit')) + $this->config->get($this->config->get($this->config->get('config_theme') . '_theme') . '_product_limit')), $download_total, ceil($download_total / $this->config->get($this->config->get('config_theme') . '_product_limit')));

        $data['continue'] = $this->url->link('account/account', '', true);

        $data['column'] = $this->load->controller('common/column');

        $data['content_top'] = $this->load->controller('common/content_top');
        $data['content_bottom'] = $this->load->controller('common/content_bottom');
        $data['footer'] = $this->load->controller('common/footer');
        $data['header'] = $this->load->controller('common/header');

        $this->response->setOutput($this->load->view('account/download', $data));
    }

    public function download()
    {
        if (!$this->customer->isLogged()) {
            $this->session->data['redirect'] = $this->url->link('account/download', '', true);

            $this->response->redirect($this->url->link('account/login', '', true));
        }

        $this->load->model('account/download');

        if (isset($this->request->get['download_id'])) {
            $download_id = $this->request->get['download_id'];
        } else {
            $download_id = 0;
        }

        $download_info = $this->model_account_download->getDownload($download_id);

        if ($download_info) {
            $file = $_SERVER['DOCUMENT_ROOT'] . '/storage/download/' . $download_info['filename'];
            $mask = basename($download_info['mask']);

            if (!headers_sent()) {
                if (file_exists($file)) {
                    header('Content-Type: application/octet-stream');
                    header('Content-Disposition: attachment; filename="' . ($mask ? $mask : basename($file)) . '"');
                    header('Expires: 0');
                    header('Cache-Control: must-revalidate, post-check=0, pre-check=0');
                    header('Pragma: public');
                    header('Content-Length: ' . filesize($file));

                    if (ob_get_level()) {
                        ob_end_clean();
                    }

                    readfile($file, 'rb');

'rb' of type string is incompatible with the type boolean expected by parameter $use_include_path of readfile().

                    exit();

Using exit here is not recommended.

                } else {
                    exit('Error: Could not find file ' . $file . '!');

Using exit here is not recommended.

                }
            } else {
                exit('Error: Headers already sent out!');

Using exit here is not recommended.

            }
        } else {
            $this->response->redirect($this->url->link('account/download', '', true));
        }
    }
}