updated and fix files using php-cs-fixer

This commit is contained in:
iamflorencejay 2021-05-14 20:10:54 +08:00 committed by Jeremy Kauffman
parent f8b413326c
commit de10165d5e
34 changed files with 394 additions and 395 deletions

View file

@ -2,7 +2,7 @@
class Controller
{
const CACHE_CLEAR_PATH = '/clear-cache';
public const CACHE_CLEAR_PATH = '/clear-cache';
protected static $queuedFunctions = [];

View file

@ -2,10 +2,10 @@
class Request
{
const GET = 'GET';
const POST = 'POST';
const HEAD = 'HEAD';
const OPTIONS = 'OPTIONS';
public const GET = 'GET';
public const POST = 'POST';
public const HEAD = 'HEAD';
public const OPTIONS = 'OPTIONS';
protected static $method;

View file

@ -2,10 +2,10 @@
class Session
{
const KEY_LIST_SUB_ERROR = 'list_error',
public const KEY_LIST_SUB_ERROR = 'list_error',
KEY_USER_CULTURE = 'user_culture';
const NAMESPACE_DEFAULT = 'default',
public const NAMESPACE_DEFAULT = 'default',
NAMESPACE_FLASH = 'flash',
NAMESPACE_FLASH_REMOVE = 'flash_remove',
USER_ID = 'user_id',

View file

@ -2,7 +2,7 @@
class ContentActions extends Actions
{
const
public const
SLUG_RSS = 'rss.xml',
SLUG_NEWS = 'news',
SLUG_FAQ = 'faq',

View file

@ -3,8 +3,8 @@
class DownloadActions extends Actions
{
//bad, fix me!
const ANDROID_STORE_URL = 'https://play.google.com/store/apps/details?id=io.lbry.browser';
const IOS_STORE_URL = 'https://apps.apple.com/us/app/odysee/id1539444143';
public const ANDROID_STORE_URL = 'https://play.google.com/store/apps/details?id=io.lbry.browser';
public const IOS_STORE_URL = 'https://apps.apple.com/us/app/odysee/id1539444143';
public static function executeDownloadPrereleaseAsset(string $repo, string $ext)
{
@ -49,18 +49,13 @@ class DownloadActions extends Actions
'os' => $os
];
if ($os === OS::OS_ANDROID)
{
if ($os === OS::OS_ANDROID) {
$params['downloadUrl'] = static::ANDROID_STORE_URL;
$params['osScreenshotSrc'] = 'https://spee.ch/@lbry:3f/android-08-homepage.gif';
}
else if ($os === OS::OS_IOS)
{
} elseif ($os === OS::OS_IOS) {
$params['downloadUrl'] = static::IOS_STORE_URL;
$params['osScreenshotSrc'] = 'https://spee.ch/odyseeiosimage.png';
}
else
{
} else {
$asset = Github::getRepoAsset(GitHub::REPO_LBRY_DESKTOP, $os, $params['preferredExt']);
$params['downloadUrl'] = $asset ? $asset['browser_download_url'] : null;
}
@ -144,7 +139,7 @@ class DownloadActions extends Actions
if ($os === OS::OS_ANDROID) {
$asset = ['browser_download_url' => static::ANDROID_STORE_URL];
} else if ($os === OS::OS_IOS) {
} elseif ($os === OS::OS_IOS) {
$asset = ['browser_download_url' => static::IOS_STORE_URL];
} else {
$asset = Github::getRepoAsset(GitHub::REPO_LBRY_DESKTOP, $os, $vars['preferredExt'] ?? '');
@ -190,7 +185,7 @@ class DownloadActions extends Actions
if ($os === OS::OS_ANDROID) {
$asset = ['browser_download_url' => static::ANDROID_STORE_URL, 'size' => 0];
$release = [];
} else if ($os === OS::OS_IOS) {
} elseif ($os === OS::OS_IOS) {
$asset = ['browser_download_url' => static::IOS_STORE_URL, 'size' => 0];
$release = [];
} else {

View file

@ -83,7 +83,7 @@ class MailActions extends Actions
$tags = LBRY::listTags($vars['token']);
$tagMetadata = [];
foreach($tags as $tag) {
foreach ($tags as $tag) {
if ($tag['is_user_addable']) {
$tagMetadata[$tag['name']] = [
'label' => $tag['display_name'],

View file

@ -2,7 +2,6 @@
class ReportActions extends Actions
{
/**
* Handles all routing requests without a claimId as parameter.
* E.g. /dmca
@ -10,8 +9,8 @@ class ReportActions extends Actions
* @return array
* @throws Exception
*/
public static function executeDmca() {
public static function executeDmca()
{
self::setResponseHeader();
self::redirectIfPostRequest(false, '');
@ -35,8 +34,8 @@ class ReportActions extends Actions
* @return array
* @throws Exception
*/
public static function executeDmcaWithClaimId(string $claimId): array {
public static function executeDmcaWithClaimId(string $claimId): array
{
$claimId = htmlspecialchars($claimId);
self::setResponseHeader();
@ -63,7 +62,6 @@ class ReportActions extends Actions
*/
private static function validateForm(array $values, array $errors)
{
foreach (['name', 'email', 'rightsholder', 'identifier'] as $field) {
$value = Request::getPostParam($field);
@ -82,7 +80,6 @@ class ReportActions extends Actions
Session::setFlash('success', '<h3>Report Submitted</h3><p>We will respond shortly.</p><p>This ID for this report is <strong>' . $values['report_id'] . '</strong>. Please reference this ID when contacting us regarding this report.</p>');
return Controller::redirect(Request::getRelativeUri(), 303);
}
}
/**
@ -101,8 +98,8 @@ class ReportActions extends Actions
*
* @return array
*/
private static function redirectIfPostRequest(bool $hasClaimId = false, string $claimId = '') {
private static function redirectIfPostRequest(bool $hasClaimId = false, string $claimId = '')
{
if ($hasClaimId && !empty($claimId)) {
$returnValue = ['report/dmca', ['claimId' => $claimId]];
} else {

File diff suppressed because it is too large Load diff

View file

@ -69,12 +69,12 @@ class Dispatcher
return false;
}
return (boolean)$routes[$httpMethod];
return (bool)$routes[$httpMethod];
}
try {
$handler = $this->dispatchVariableRoute($httpMethod, $uri);
return (boolean)$handler;
return (bool)$handler;
} catch (HttpRouteNotFoundException | HttpMethodNotAllowedException $e) {
return false;
}

View file

@ -7,7 +7,7 @@ class HandlerResolver implements HandlerResolverInterface
public function resolve($handler)
{
if (is_array($handler) && is_string($handler[0])) {
$handler[0] = new $handler[0];
$handler[0] = new $handler[0]();
}
return $handler;

View file

@ -7,19 +7,19 @@ class Route
/**
* Constants for before and after filters
*/
const BEFORE = 'before';
const AFTER = 'after';
const PREFIX = 'prefix';
public const BEFORE = 'before';
public const AFTER = 'after';
public const PREFIX = 'prefix';
/**
* Constants for common HTTP methods
*/
const ANY = 'ANY';
const GET = 'GET';
const HEAD = 'HEAD';
const POST = 'POST';
const PUT = 'PUT';
const PATCH = 'PATCH';
const DELETE = 'DELETE';
const OPTIONS = 'OPTIONS';
public const ANY = 'ANY';
public const GET = 'GET';
public const HEAD = 'HEAD';
public const POST = 'POST';
public const PUT = 'PUT';
public const PATCH = 'PATCH';
public const DELETE = 'DELETE';
public const OPTIONS = 'OPTIONS';
}

View file

@ -4,9 +4,9 @@ namespace Routing;
class RouteCollector
{
const DEFAULT_CONTROLLER_ROUTE = 'index';
public const DEFAULT_CONTROLLER_ROUTE = 'index';
const APPROX_CHUNK_SIZE = 10;
public const APPROX_CHUNK_SIZE = 10;
/**
* @var RouteParser

View file

@ -9,7 +9,6 @@ namespace Routing;
*/
class RouteParser
{
/**
* Search through the given route looking for dynamic portions.
*
@ -22,7 +21,7 @@ class RouteParser
*
* Finally we look for an optional '?' which is used to signify an optional route.
*/
const VARIABLE_REGEX =
public const VARIABLE_REGEX =
"~\{
\s* ([a-zA-Z0-9_]*) \s*
(?:
@ -33,7 +32,7 @@ class RouteParser
/**
* The default parameter character restriction (One or more characters that is not a '/').
*/
const DEFAULT_DISPATCH_REGEX = '[^/]+';
public const DEFAULT_DISPATCH_REGEX = '[^/]+';
private $parts;

View file

@ -2,7 +2,7 @@
class Github
{
const REPO_LBRY_DESKTOP = 'lbry-desktop';
public const REPO_LBRY_DESKTOP = 'lbry-desktop';
public static function isAssetForOs(array $asset, string $os)
{

View file

@ -2,7 +2,7 @@
class LBRY
{
const DEFAULT_TIMEOUT = 10;
public const DEFAULT_TIMEOUT = 10;
public static function getApiUrl($endpoint)
{

View file

@ -2,12 +2,12 @@
class Mailgun
{
const BASE_URL = 'https://api.mailgun.net/v3';
public const BASE_URL = 'https://api.mailgun.net/v3';
const TOP_DOMAIN = 'lbry.com';
const MAIL_DOMAIN = 'mail.lbry.com';
public const TOP_DOMAIN = 'lbry.com';
public const MAIL_DOMAIN = 'mail.lbry.com';
const LIST_GENERAL = 'lbryians@lbry.com';
public const LIST_GENERAL = 'lbryians@lbry.com';
public static function sendDmcaReport($data)
{

View file

@ -2,7 +2,7 @@
class Salesforce
{
const
public const
API_URL = 'https://api.salesforceiq.com/v2/',
DEFAULT_LIST_ID = '58387a94e4b0a1fea2c76f4a';

View file

@ -5,7 +5,7 @@ class Transifex
{
public static function isConfigured()
{
return (boolean)Config::get(Config::TRANSIFEX_API_KEY);
return (bool)Config::get(Config::TRANSIFEX_API_KEY);
}
public static function getTranslationResourceFile($project, $resource, $language, $cache = true)

View file

@ -2,26 +2,26 @@
class Config
{
const HELP_CONTACT_EMAIL = 'hello@lbry.com';
public const HELP_CONTACT_EMAIL = 'hello@lbry.com';
//Constant to help with managing strings
const IS_PROD = "is_prod";
const CHAINQUERY_DSN = 'chainquery_dsn';
const CHAINQUERY_USERNAME = 'chainquery_username';
const CHAINQUERY_PASSWORD = 'chainquery_password';
const GITHUB_KEY = "github_key";
const GITHUB_APP_CLIENT_ID = "github_app_client_id";
const GITHUB_APP_CLIENT_SECRET = "github_app_client_secret";
const GITHUB_PERSONAL_AUTH_TOKEN = 'github_personal_auth_token';
const LBRY_API_SERVER = "lbry_api_server";
const MAILCHIMP_KEY = "mailchimp_key";
const TRANSIFEX_API_KEY = 'transifex_api_key';
const AWS_LOG_ACCESS_KEY = "aws_log_access_key";
const AWS_LOG_SECRET_KEY = "aws_log_secret_key";
const MAILGUN_API_KEY = "mailgun_api_key";
const SALESFORCE_KEY = "salesforce_key";
const SALESFORCE_SECRET = "salesforce_secret";
const SLACK_ERROR_NOTIFICATION_URL = "slack_error_notification_url";
public const IS_PROD = "is_prod";
public const CHAINQUERY_DSN = 'chainquery_dsn';
public const CHAINQUERY_USERNAME = 'chainquery_username';
public const CHAINQUERY_PASSWORD = 'chainquery_password';
public const GITHUB_KEY = "github_key";
public const GITHUB_APP_CLIENT_ID = "github_app_client_id";
public const GITHUB_APP_CLIENT_SECRET = "github_app_client_secret";
public const GITHUB_PERSONAL_AUTH_TOKEN = 'github_personal_auth_token';
public const LBRY_API_SERVER = "lbry_api_server";
public const MAILCHIMP_KEY = "mailchimp_key";
public const TRANSIFEX_API_KEY = 'transifex_api_key';
public const AWS_LOG_ACCESS_KEY = "aws_log_access_key";
public const AWS_LOG_SECRET_KEY = "aws_log_secret_key";
public const MAILGUN_API_KEY = "mailgun_api_key";
public const SALESFORCE_KEY = "salesforce_key";
public const SALESFORCE_SECRET = "salesforce_secret";
public const SLACK_ERROR_NOTIFICATION_URL = "slack_error_notification_url";
protected static $loaded = false;

View file

@ -2,7 +2,7 @@
class Curl
{
const
public const
GET = 'GET',
POST = 'POST',
PUT = 'PUT',

View file

@ -2,7 +2,7 @@
class CurlWithCache extends Curl
{
const DEFAULT_CACHE = 600000;
public const DEFAULT_CACHE = 600000;
public static function doCurl($method, $url, $params = [], $options = [])
{

View file

@ -5,7 +5,7 @@ class OS
/*
* if changing below constants, you should add permanent redirects for old OS names used in URLs
*/
const OS_ANDROID = 'android',
public const OS_ANDROID = 'android',
OS_IOS = 'ios',
OS_LINUX = 'linux',
OS_OSX = 'osx',

View file

@ -13,11 +13,11 @@
*/
class Smaz
{
const CODEBOOK_DEFAULT = 'default';
const CODEBOOK_EMAIL = 'email';
public const CODEBOOK_DEFAULT = 'default';
public const CODEBOOK_EMAIL = 'email';
const VERBATIM_CHAR = 254;
const VERBATIM_STR = 255;
public const VERBATIM_CHAR = 254;
public const VERBATIM_STR = 255;
protected static $encodeBooks = [];
protected static $decodeBooks = [

View file

@ -1,6 +1,8 @@
<?php
class PostException extends Exception {}
class PostException extends Exception
{
}
class PostNotFoundException extends PostException
{
@ -12,7 +14,7 @@ class PostMalformedException extends PostException
class Post
{
const SORT_DATE_DESC = 'sort_date_desc',
public const SORT_DATE_DESC = 'sort_date_desc',
SORT_ORD_ASC = 'sort_ord_asc';
protected static $slugMap = [];
@ -81,7 +83,7 @@ class Post
public static function find($folder, $sort = null)
{
$posts = array_filter(array_map(function($file) {
$posts = array_filter(array_map(function ($file) {
try {
return static::load($file);
} catch (PostException $e) {

Binary file not shown.

View file

@ -2,18 +2,18 @@
class Response
{
const HEADER_STATUS = 'Status';
const HEADER_LOCATION = 'Location';
public const HEADER_STATUS = 'Status';
public const HEADER_LOCATION = 'Location';
const HEADER_CACHE_CONTROL = 'Cache-Control';
const HEADER_ETAG = 'Etag';
public const HEADER_CACHE_CONTROL = 'Cache-Control';
public const HEADER_ETAG = 'Etag';
const HEADER_CONTENT_TYPE = 'Content-Type';
const HEADER_CONTENT_LENGTH = 'Content-Length';
const HEADER_CONTENT_DISPOSITION = 'Content-Disposition';
const HEADER_CONTENT_TYPE_OPTIONS = 'X-Content-Type-Options';
const HEADER_CONTENT_ENCODING = 'Content-Encoding';
const HEADER_CROSS_ORIGIN = 'Access-Control-Allow-Origin';
public const HEADER_CONTENT_TYPE = 'Content-Type';
public const HEADER_CONTENT_LENGTH = 'Content-Length';
public const HEADER_CONTENT_DISPOSITION = 'Content-Disposition';
public const HEADER_CONTENT_TYPE_OPTIONS = 'X-Content-Type-Options';
public const HEADER_CONTENT_ENCODING = 'Content-Encoding';
public const HEADER_CROSS_ORIGIN = 'Access-Control-Allow-Origin';
protected static $metaDescription = '';
protected static $metaTitle = '';

View file

@ -12,14 +12,14 @@ function js_end()
class View
{
const LAYOUT_PARAMS = '_layout_params';
public const LAYOUT_PARAMS = '_layout_params';
const WEB_DIR = ROOT_DIR . '/web';
const COMPONENTS_DIR = self::WEB_DIR . '/components/sass';
const COLORS_DIR = self::WEB_DIR . '/scss/color';
const SCSS_DIR = self::WEB_DIR . '/scss';
const CSS_DIR = self::WEB_DIR . '/css';
const JS_DIR = self::WEB_DIR . '/js';
public const WEB_DIR = ROOT_DIR . '/web';
public const COMPONENTS_DIR = self::WEB_DIR . '/components/sass';
public const COLORS_DIR = self::WEB_DIR . '/scss/color';
public const SCSS_DIR = self::WEB_DIR . '/scss';
public const CSS_DIR = self::WEB_DIR . '/css';
public const JS_DIR = self::WEB_DIR . '/js';
public static function render($template, array $vars = []): string
{
@ -193,8 +193,7 @@ class View
}
return $parser->saveHTML($dom);
}
catch (Error $e) {
} catch (Error $e) {
Slack::slackGrin();
return $html;
}

View file

@ -1 +1,3 @@
<?php echo json_encode($json ?? [], JSON_PRETTY_PRINT);
<?php
echo json_encode($json ?? [], JSON_PRETTY_PRINT);

View file

@ -1 +1,3 @@
<?php echo file_get_contents($zipPath);
<?php
echo file_get_contents($zipPath);

View file

@ -12,7 +12,7 @@
<?php else: ?>
<form id="<?php echo $formId ?>" novalidate style="display: none">
<section class="email-section">
<div style="min-height: 33px"> <?php //hack fix to prevent jumping ?>
<div style="min-height: 33px"> <?php //hack fix to prevent jumping?>
<div class="notice notice-error hide spacer-half"></div>
<div class="notice notice-success hide spacer-half">Your email preferences have been updated.</div>
</div>
@ -34,7 +34,7 @@
</section>
<section class="tag-section spacer1">
<div style="min-height: 33px"> <?php //hack fix to prevent jumping ?>
<div style="min-height: 33px"> <?php //hack fix to prevent jumping?>
<div class="notice notice-error hide spacer-half"></div>
<div class="notice notice-success hide spacer-half">Your email preferences have been updated.</div>
</div>
@ -42,7 +42,9 @@
<?php $tagIndex = 0 ?>
<?php foreach ($tags as $tag => $enabled): ?>
<?php if (!isset($tagMetadata[$tag])) continue; ?>
<?php if (!isset($tagMetadata[$tag])) {
continue;
} ?>
<?php $tagId = 'tag_' . (++$tagIndex) ?>
<div>
<checkbox-element>

View file

@ -1,7 +1,7 @@
<?php $error = $error ?? null ?>
<?php $tag = $tag ?? null ?>
<?php $largeInput = $largeInput ?? false ?>
<?php $forceUrl = $forceUrl ?? '' //for cross-domain landing pages ?>
<?php $forceUrl = $forceUrl ?? '' //for cross-domain landing pages?>
<form id="mail_form" action="<?php echo $forceUrl ?>/list/subscribe" method="POST" novalidate>
<?php if ($error): ?>