nazar-pc / CleverStyle-Framework

modules/Uploader/events.php

<?php
/**
 * @package  Uploader
 * @category modules
 * @author   Nazar Mokrynskyi <nazar@mokrynskyi.com>
 * @license  0BSD
 */
namespace cs;

Event::instance()
	->on(
		'System/Page/render/before',
		function () {
			if (!Config::instance()->module('Uploader')->enabled()) {
				return;
			}
			$Config        = Config::instance();
			$Page          = Page::instance();
			$module_data   = $Config->module('Uploader');
			$max_file_size = (int)$module_data->max_file_size;

The property max_file_size does not exist on cs\Config\Module_Properties. Since you implemented __get, consider adding a @property annotation.

			switch (substr(strtolower($module_data->max_file_size), -2)) {

It seems like $module_data->max_file_size can also be of type false; however, parameter $str of strtolower() does only seem to accept string, maybe add an additional type check?

				case 'gb':
					$max_file_size *= 1024;

Consider adding a comment if this fall-through is intended.

				case 'mb':
					$max_file_size *= 1024;

Consider adding a comment if this fall-through is intended.

				case 'kb':
					$max_file_size *= 1024;
			}
			$Page->config(
				[
					'max_file_size' => $max_file_size
				],
				'cs.uploader'
			);
		}
	)
	->on(
		'System/upload_files/add_tag',
		function ($data) {
			if (!Config::instance()->module('Uploader')->enabled()) {
				return true;
			}
			if (!isset($data['url'], $data['tag'])) {
				return false;
			}
			$module_data = Config::instance()->module('Uploader');
			$storage     = Storage::instance()->storage($module_data->storage('files'));
			if (mb_strpos($data['url'], $storage->base_url()) !== 0) {

The method base_url() does not exist on cs\False_class. Since you implemented __call, consider adding a @method annotation.

				return false;
			}
			$cdb = DB::instance()->db_prime($module_data->db('files'));
			$id  = $cdb->qfs(

The method qfs() does not exist on cs\False_class. Since you implemented __call, consider adding a @method annotation.

				"SELECT `id`
				FROM `[prefix]uploader_files`
				WHERE `url` = '%s'
				LIMIT 1",
				$data['url']
			);
			if (!$id) {
				return false;
			}
			return $cdb->q(

The method q() does not exist on cs\False_class. Since you implemented __call, consider adding a @method annotation.

				"INSERT IGNORE INTO `[prefix]uploader_files_tags`
					(`id`, `tag`)
				VALUES
					('%s', '%s')",
				$id,
				$data['tag']
			);
		}
	)
	->on(
		'admin/System/modules/uninstall/before',
		function ($data) {
			if ($data['name'] != 'Uploader') {
				return;
			}
			$module_data = Config::instance()->module('Uploader');
			$storage     = Storage::instance()->storage($module_data->storage('files'));
			$cdb         = DB::instance()->db($module_data->db('files'));
			unset($module_data);
			if (!$storage || !$cdb) {
				return;
			}
			$files = $cdb->q(
				'SELECT `source`
				FROM `[prefix]uploader_files`'
			);
			while ($f = $cdb->f($files, true)) {

The method f() does not exist on cs\False_class. Since you implemented __call, consider adding a @method annotation.

				$storage->unlink($f);

The method unlink() does not exist on cs\False_class. Since you implemented __call, consider adding a @method annotation.

			}
			if ($storage->is_dir('Uploader')) {

The method is_dir() does not exist on cs\False_class. Since you implemented __call, consider adding a @method annotation.

				$storage->rmdir('Uploader');

The method rmdir() does not exist on cs\False_class. Since you implemented __call, consider adding a @method annotation.

			}
		}
	)
	->on(
		'System/upload_files/del_tag',
		function ($data) {
			if (!isset($data['url']) && !isset($data['tag'])) {
				return;
			}
			$module_data = Config::instance()->module('Uploader');
			$storage     = Storage::instance()->storage($module_data->storage('files'));
			if (isset($data['url']) && mb_strpos($data['url'], $storage->base_url()) !== 0) {
				return;
			}
			$cdb = DB::instance()->db_prime($module_data->db('files'));
			if (isset($data['url']) && !isset($data['tag'])) {
				$cdb->q(
					"DELETE FROM `[prefix]uploader_files_tags`
					WHERE `id` IN(
						SELECT `id`
						FROM `[prefix]uploader_files`
						WHERE `url` = '%s'
					)",
					$data['url']
				);
			} elseif (isset($data['url'], $data['tag'])) {
				$cdb->q(
					"DELETE FROM `[prefix]uploader_files_tags`
					WHERE
						`id`	IN(
							SELECT `id`
							FROM `[prefix]uploader_files`
							WHERE `url` = '%s'
						) AND
						`tag`	= '%s'",
					$data['url'],
					$data['tag']
				);
			} else {
				$cdb->q(
					"DELETE FROM `[prefix]uploader_files_tags`
					WHERE `tag` LIKE '%s'",
					$data['tag']
				);
			}
		}
	)
	->on(
		'admin/System/modules/install/after',
		function ($data) {
			if ($data['name'] == 'Uploader') {
				$Config                                        = Config::instance();
				$Config->module('Uploader')->max_file_size     = '5mb';

The property max_file_size does not exist on cs\Config\Module_Properties. Since you implemented __set, consider adding a @property annotation.

				$Config->module('Uploader')->confirmation_time = '900';

The property confirmation_time does not exist on cs\Config\Module_Properties. Since you implemented __set, consider adding a @property annotation.

			}
		}
	);