Skip to content

Commit

Permalink
Release v4.5.8
Browse files Browse the repository at this point in the history
  • Loading branch information
actions-user committed Jan 19, 2025
1 parent f253839 commit 8020475
Show file tree
Hide file tree
Showing 154 changed files with 632 additions and 544 deletions.
2 changes: 1 addition & 1 deletion LICENSE
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
The MIT License (MIT)

Copyright (c) 2014-2019 British Columbia Institute of Technology
Copyright (c) 2019-2024 CodeIgniter Foundation
Copyright (c) 2019-present CodeIgniter Foundation

Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
Expand Down
2 changes: 1 addition & 1 deletion preload.php
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ public function load(): void
$phpFiles = new RegexIterator(
$fullTree,
'/.+((?<!Test)+\.php$)/i',
RecursiveRegexIterator::GET_MATCH
RecursiveRegexIterator::GET_MATCH,
);

foreach ($phpFiles as $key => $file) {
Expand Down
2 changes: 1 addition & 1 deletion public/index.php
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
$message = sprintf(
'Your PHP version must be %s or higher to run CodeIgniter. Current version: %s',
$minPhpVersion,
PHP_VERSION
PHP_VERSION,
);

header('HTTP/1.1 503 Service Unavailable.', true, 503);
Expand Down
2 changes: 1 addition & 1 deletion spark
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ if (version_compare(PHP_VERSION, $minPhpVersion, '<')) {
$message = sprintf(
'Your PHP version must be %s or higher to run CodeIgniter. Current version: %s',
$minPhpVersion,
PHP_VERSION
PHP_VERSION,
);

exit($message);
Expand Down
2 changes: 1 addition & 1 deletion system/API/ResponseTrait.php
Original file line number Diff line number Diff line change
Expand Up @@ -319,7 +319,7 @@ protected function format($data = null)
$mime = $this->request->negotiate(
'media',
$format->getConfig()->supportedResponseFormats,
false
false,
);
}

Expand Down
4 changes: 2 additions & 2 deletions system/Autoloader/Autoloader.php
Original file line number Diff line number Diff line change
Expand Up @@ -347,7 +347,7 @@ public function sanitizeFilename(string $filename): string

throw new InvalidArgumentException(
'The file path contains special characters "' . $chars
. '" that are not allowed: "' . $filename . '"'
. '" that are not allowed: "' . $filename . '"',
);
}
if ($result === false) {
Expand Down Expand Up @@ -386,7 +386,7 @@ private function loadComposerNamespaces(ClassLoader $composer, array $composerPa
throw new RuntimeException(
'Your Composer version is too old.'
. ' Please update Composer (run `composer self-update`) to v2.0.14 or later'
. ' and remove your vendor/ directory, and run `composer update`.'
. ' and remove your vendor/ directory, and run `composer update`.',
);
}
// This method requires Composer 2.0.14 or later.
Expand Down
4 changes: 2 additions & 2 deletions system/Autoloader/FileLocator.php
Original file line number Diff line number Diff line change
Expand Up @@ -295,9 +295,9 @@ public function findQualifiedNameFromPath(string $path)
str_replace(
'/',
'\\',
mb_substr($path, mb_strlen($namespace['path']))
mb_substr($path, mb_strlen($namespace['path'])),
),
'\\'
'\\',
);
// Remove the file extension (.php)
$className = mb_substr($className, 0, -4);
Expand Down
4 changes: 2 additions & 2 deletions system/BaseModel.php
Original file line number Diff line number Diff line change
Expand Up @@ -379,7 +379,7 @@ protected function createDataConverter(): void
$this->converter = new DataConverter(
$this->casts,
$this->castHandlers,
$this->db
$this->db,
);
}
}
Expand Down Expand Up @@ -1081,7 +1081,7 @@ public function updateBatch(?array $set = null, ?string $index = null, int $batc
if ($updateIndex === null) {
throw new InvalidArgumentException(
'The index ("' . $index . '") for updateBatch() is missing in the data: '
. json_encode($row)
. json_encode($row),
);
}

Expand Down
2 changes: 1 addition & 1 deletion system/Boot.php
Original file line number Diff line number Diff line change
Expand Up @@ -288,7 +288,7 @@ protected static function checkMissingExtensions(): void

$message = sprintf(
'The framework needs the following extension(s) installed and loaded: %s.',
implode(', ', $missingExtensions)
implode(', ', $missingExtensions),
);

header('HTTP/1.1 503 Service Unavailable.', true, 503);
Expand Down
8 changes: 4 additions & 4 deletions system/CLI/CLI.php
Original file line number Diff line number Diff line change
Expand Up @@ -289,7 +289,7 @@ public static function promptByKey($text, array $options, $validation = null): s

CLI::isZeroOptions($options);

if ($line = array_shift($text)) {
if (($line = array_shift($text)) !== null) {
CLI::write($line);
}

Expand Down Expand Up @@ -348,7 +348,7 @@ public static function promptByMultipleKeys(string $text, array $options): array
// return the prompt again if $input contain(s) non-numeric character, except a comma.
// And if max from $options less than max from input,
// it means user tried to access null value in $options
if ($pattern === 0 || $maxOptions < $maxInput) {
if ($pattern < 1 || $maxOptions < $maxInput) {
static::error('Please select correctly.');
CLI::newLine();

Expand Down Expand Up @@ -610,11 +610,11 @@ public static function color(string $text, string $foreground, ?string $backgrou
$nonColoredText = preg_replace(
$pattern,
'<<__colored_string__>>',
$text
$text,
);
$nonColoredChunks = preg_split(
'/<<__colored_string__>>/u',
$nonColoredText
$nonColoredText,
);

foreach ($nonColoredChunks as $i => $chunk) {
Expand Down
2 changes: 1 addition & 1 deletion system/CLI/Console.php
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ public function showHeader(bool $suppress = false)
'CodeIgniter v%s Command Line Tool - Server Time: %s UTC%s',
CodeIgniter::CI_VERSION,
date('Y-m-d H:i:s'),
date('P')
date('P'),
), 'green');
CLI::newLine();
}
Expand Down
38 changes: 19 additions & 19 deletions system/CLI/GeneratorTrait.php
Original file line number Diff line number Diff line change
Expand Up @@ -171,7 +171,7 @@ private function generateFile(string $target, string $content): void
CLI::prompt(
'Are you sure you want to continue?',
['y', 'n'],
'required'
'required',
) === 'n'
) {
CLI::newLine();
Expand All @@ -193,7 +193,7 @@ private function generateFile(string $target, string $content): void
CLI::error(
lang('CLI.generator.fileExist', [clean_path($target)]),
'light_gray',
'red'
'red',
);
CLI::newLine();

Expand All @@ -216,7 +216,7 @@ private function generateFile(string $target, string $content): void
CLI::error(
lang('CLI.generator.fileError', [clean_path($target)]),
'light_gray',
'red'
'red',
);
CLI::newLine();

Expand All @@ -227,7 +227,7 @@ private function generateFile(string $target, string $content): void
if ($this->getOption('force') && $isFile) {
CLI::write(
lang('CLI.generator.fileOverwrite', [clean_path($target)]),
'yellow'
'yellow',
);
CLI::newLine();

Expand All @@ -236,7 +236,7 @@ private function generateFile(string $target, string $content): void

CLI::write(
lang('CLI.generator.fileCreate', [clean_path($target)]),
'green'
'green',
);
CLI::newLine();
}
Expand Down Expand Up @@ -326,10 +326,10 @@ private function normalizeInputClassName(): string
'\\',
array_map(
pascalize(...),
explode('\\', str_replace('/', '\\', trim($class)))
)
explode('\\', str_replace('/', '\\', trim($class))),
),
),
'\\/'
'\\/',
);
}

Expand All @@ -351,7 +351,7 @@ protected function renderTemplate(array $data = []): string
return view(
"CodeIgniter\\Commands\\Generators\\Views\\{$this->template}",
$data,
['debug' => false]
['debug' => false],
);
}
}
Expand All @@ -370,15 +370,15 @@ protected function parseTemplate(
string $class,
array $search = [],
array $replace = [],
array $data = []
array $data = [],
): string {
// Retrieves the namespace part from the fully qualified class name.
$namespace = trim(
implode(
'\\',
array_slice(explode('\\', $class), 0, -1)
array_slice(explode('\\', $class), 0, -1),
),
'\\'
'\\',
);
$search[] = '<@php';
$search[] = '{namespace}';
Expand All @@ -404,7 +404,7 @@ protected function buildContent(string $class): string
&& preg_match(
'/(?P<imports>(?:^use [^;]+;$\n?)+)/m',
$template,
$match
$match,
)
) {
$imports = explode("\n", trim($match['imports']));
Expand Down Expand Up @@ -432,7 +432,7 @@ protected function buildPath(string $class): string
CLI::error(
lang('CLI.namespaceNotDefined', [$namespace]),
'light_gray',
'red'
'red',
);
CLI::newLine();

Expand All @@ -446,16 +446,16 @@ protected function buildPath(string $class): string
. str_replace(
'\\',
DIRECTORY_SEPARATOR,
trim(str_replace($namespace . '\\', '', $class), '\\')
trim(str_replace($namespace . '\\', '', $class), '\\'),
) . '.php';

return implode(
DIRECTORY_SEPARATOR,
array_slice(
explode(DIRECTORY_SEPARATOR, $file),
0,
-1
)
-1,
),
) . DIRECTORY_SEPARATOR . $this->basename($file);
}

Expand All @@ -470,9 +470,9 @@ protected function getNamespace(): string
str_replace(
'/',
'\\',
$this->getOption('namespace') ?? APP_NAMESPACE
$this->getOption('namespace') ?? APP_NAMESPACE,
),
'\\'
'\\',
);
}

Expand Down
6 changes: 3 additions & 3 deletions system/Cache/Handlers/MemcachedHandler.php
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ public function initialize()
$this->memcached->addServer(
$this->config['host'],
$this->config['port'],
$this->config['weight']
$this->config['weight'],
);

// attempt to get status of servers
Expand All @@ -103,7 +103,7 @@ public function initialize()
// Check if we can connect to the server
$canConnect = $this->memcached->connect(
$this->config['host'],
$this->config['port']
$this->config['port'],
);

// If we can't connect, throw a CriticalError exception
Expand All @@ -116,7 +116,7 @@ public function initialize()
$this->config['host'],
$this->config['port'],
true,
$this->config['weight']
$this->config['weight'],
);
} else {
throw new CriticalError('Cache: Not support Memcache(d) extension.');
Expand Down
2 changes: 1 addition & 1 deletion system/Cache/Handlers/PredisHandler.php
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ public function get(string $key)

$data = array_combine(
['__ci_type', '__ci_value'],
$this->redis->hmget($key, ['__ci_type', '__ci_value'])
$this->redis->hmget($key, ['__ci_type', '__ci_value']),
);

if (! isset($data['__ci_type'], $data['__ci_value']) || $data['__ci_value'] === false) {
Expand Down
2 changes: 1 addition & 1 deletion system/Cache/ResponseCache.php
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,7 @@ public function make($request, ResponseInterface $response): bool
return $this->cache->save(
$this->generateCacheKey($request),
serialize(['headers' => $headers, 'output' => $response->getBody()]),
$this->ttl
$this->ttl,
);
}

Expand Down
10 changes: 5 additions & 5 deletions system/CodeIgniter.php
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ class CodeIgniter
/**
* The current version of CodeIgniter Framework
*/
public const CI_VERSION = '4.5.7';
public const CI_VERSION = '4.5.8';

/**
* App startup time.
Expand Down Expand Up @@ -325,7 +325,7 @@ public function run(?RouteCollectionInterface $routes = null, bool $returnRespon
if ($this->context === null) {
throw new LogicException(
'Context must be set before run() is called. If you are upgrading from 4.1.x, '
. 'you need to merge `public/index.php` and `spark` file from `vendor/codeigniter4/framework`.'
. 'you need to merge `public/index.php` and `spark` file from `vendor/codeigniter4/framework`.',
);
}

Expand Down Expand Up @@ -799,7 +799,7 @@ public function displayPerformanceMetrics(string $output): string
return str_replace(
['{elapsed_time}', '{memory_usage}'],
[(string) $this->totalTime, number_format(memory_get_peak_usage() / 1024 / 1024, 3)],
$output
$output,
);
}

Expand Down Expand Up @@ -979,7 +979,7 @@ protected function display404errors(PageNotFoundException $e)

// Throws new PageNotFoundException and remove exception message on production.
throw PageNotFoundException::forPageNotFound(
(ENVIRONMENT !== 'production' || ! $this->isWeb()) ? $e->getMessage() : null
(ENVIRONMENT !== 'production' || ! $this->isWeb()) ? $e->getMessage() : null,
);
}

Expand Down Expand Up @@ -1064,7 +1064,7 @@ public function storePreviousURL($uri)
$uri->getAuthority(),
$uri->getPath(),
$uri->getQuery(),
$uri->getFragment()
$uri->getFragment(),
));
}
}
Expand Down
Loading

0 comments on commit 8020475

Please sign in to comment.