Skip to content

Commit 8953245

Browse files
authored
Merge pull request #930 from cakephp/5.x-merge
4.x => 5.x merge
2 parents 6366f77 + cbb9321 commit 8953245

6 files changed

Lines changed: 8 additions & 11 deletions

File tree

docs.Dockerfile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# Generate the HTML output.
2-
FROM markstory/cakephp-docs-builder as builder
2+
FROM ghcr.io/cakephp/docs-builder as builder
33

44
RUN pip install git+https://github.com/sphinx-contrib/video.git@master
55

@@ -15,7 +15,7 @@ RUN cd /data/docs-builder \
1515
&& cp /data/docs/static/* /data/website/html/_static/
1616

1717
# Build a small nginx container with just the static site in it.
18-
FROM markstory/cakephp-docs-builder:runtime as runtime
18+
FROM ghcr.io/cakephp/docs-builder:runtime as runtime
1919

2020
# Configure search index script
2121
ENV LANGS="en fr ja pt"

src/Cache/Engine/DebugEngine.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -336,6 +336,7 @@ public function __toString(): string
336336
{
337337
/** @psalm-suppress RedundantPropertyInitializationCheck */
338338
if (isset($this->_engine)) {
339+
// phpcs:ignore SlevomatCodingStandard.Variables.UnusedVariable.UnusedVariable
339340
[$ns, $class] = namespaceSplit(get_class($this->_engine));
340341

341342
return str_replace('Engine', '', $class);

src/DebugPanel.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ class DebugPanel implements EventListenerInterface
4747
*/
4848
public function title(): string
4949
{
50+
// phpcs:ignore SlevomatCodingStandard.Variables.UnusedVariable.UnusedVariable
5051
[$ns, $name] = namespaceSplit(static::class);
5152
$name = substr($name, 0, strlen('Panel') * -1);
5253

@@ -60,6 +61,7 @@ public function title(): string
6061
*/
6162
public function elementName(): string
6263
{
64+
// phpcs:ignore SlevomatCodingStandard.Variables.UnusedVariable.UnusedVariable
6365
[$ns, $name] = namespaceSplit(static::class);
6466
if ($this->plugin) {
6567
return $this->plugin . '.' . Inflector::underscore($name);

tests/TestCase/Controller/RequestsControllerTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ public function setUp(): void
5757
public function testView()
5858
{
5959
$request = $this->makeRequest();
60-
$panel = $this->makePanel($request);
60+
$this->makePanel($request);
6161

6262
$this->configRequest(['headers' => ['Accept' => 'application/json']]);
6363
$this->get("/debug-kit/toolbar/{$request->id}");

tests/TestCase/ToolbarServiceTest.php

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -322,10 +322,6 @@ public function testInjectScriptsLastBodyTag()
322322
*/
323323
public function testInjectScriptsFileBodies()
324324
{
325-
$request = new Request([
326-
'url' => '/articles',
327-
'params' => ['plugin' => null],
328-
]);
329325
$response = new Response([
330326
'statusCode' => 200,
331327
'type' => 'text/html',
@@ -348,10 +344,6 @@ public function testInjectScriptsFileBodies()
348344
*/
349345
public function testInjectScriptsStreamBodies()
350346
{
351-
$request = new Request([
352-
'url' => '/articles',
353-
'params' => ['plugin' => null],
354-
]);
355347
$response = new Response([
356348
'statusCode' => 200,
357349
'type' => 'text/html',

tests/bootstrap.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,8 @@
4242
define('LOGS', TMP);
4343

4444
require_once CORE_PATH . 'config/bootstrap.php';
45+
require_once CAKE . 'Core/functions_global.php';
46+
require_once CAKE . 'Collection/functions_global.php';
4547

4648
date_default_timezone_set('UTC');
4749
mb_internal_encoding('UTF-8');

0 commit comments

Comments
 (0)