Missing argument in route when using `url_to` or `route_to` with regex routes
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 48/100
Rechercherichtung
Reproduzieren Sie den Fehler mithilfe der Routen in Routes.php und vergleichen Sie url_to und route_to mit beiden Einstellungen von multipleSegmentsOneParam in Routing.php. Beginnen Sie damit, die Behandlung von Routenparametern ausgehend von den benannten Regex-Routen nachzuverfolgen; fertig ist die Aufgabe, wenn alle gezeigten Links ohne InvalidArgumentException generiert werden und die erwarteten Parameter beibehalten.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
PHP Version
8.3
CodeIgniter4 Version
CodeIgniter 4.5.5
CodeIgniter4 Installation Method
Composer (using codeigniter4/appstarter)
Which operating systems have you tested for this bug?
Debian 12
Which server did you use?
fpm-fcgi
Database
N/A
What happened?
The following route definition works fine as long as you don't use url_to or route_to to refer to it:
$routes->get('/test(/(:any))?', 'Test::direct/$1', ['as' => 'test']);
Linking as below works:
<a href="/test">No parameter</a>
<a href="/test/param1">1 parameter</a>
<a href="/test/param1/param2">2 parameters</a>
But using both url_to or route_to fails:
<a href="<?= url_to('test') ?>">No parameter</a>
<a href="<?= url_to('test', 'param1') ?>">1 parameter</a>
<a href="<?= url_to('test', 'param1', 'param2') ?>">2 parameters</a>
Steps to Reproduce
- Add the new routes in
Routes.php:
$routes->get('/test(/(:any))?', 'Test::direct/$1', ['as' => 'test']);
$routes->get('/test_urlto(/(:any))?', 'Test::urlto/$1', ['as' => 'urlto']);
$routes->get('/test_routeto(/(:any))?', 'Test::routeto/$1', ['as' => 'routeto']);
- Create the
Testcontroller:
<?php
namespace App\Controllers;
class Test extends BaseController
{
public function direct(...$params): string
{
return view('test', ['params' => $params]);
}
public function urlto(...$params): string
{
return view('test_urlto', ['params' => $params]);
}
public function routeto(...$params): string
{
return view('test_routeto', ['params' => $params]);
}
}
- Create the respective views:
A) direct view
<!DOCTYPE html>
<html lang="en">
<head>
<meta charset="UTF-8">
<title>Test</title>
</head>
<body>
<h1>Testing CI4 Routing with Parameters</h1>
<h2>Test using string directly</h2>
<ul>
<li>
<a href="/test">No parameter</a>
</li>
<li>
<a href="/test/param1">1 parameter</a>
</li>
<li>
<a href="/test/param1/param2">2 parameters</a>
</li>
<li>
<a href="/test/param1/param2/param3">3 parameters</a>
</li>
</ul>
<h3>Passed Parameters</h3>
<ul>
<?php foreach ($params as $param) : ?>
<li><?= $param ?></li>
<?php endforeach; ?>
</body>
</html>
B) view using url_to
<!DOCTYPE html>
<html lang="en">
<head>
<meta charset="UTF-8">
<title>Test</title>
</head>
<body>
<h1>Testing CI4 Routing with Parameters</h1>
<h2>Test using <code>url_to</code></h2>
<ul>
<li>
<a href="<?= url_to('urlto') ?>">No parameter</a>
</li>
<li>
<a href="<?= url_to('urlto', 'param1') ?>">1 parameter</a>
</li>
<li>
<a href="<?= url_to('urlto', 'param1', 'param2') ?>">2 parameters</a>
</li>
<li>
<a href="<?= url_to('urlto', 'param1', 'param2', 'param3') ?>">3 parameters</a>
</li>
</ul>
<h3>Passed Parameters</h3>
<ul>
<?php foreach ($params as $param) : ?>
<li><?= $param ?></li>
<?php endforeach; ?>
</body>
</html>
- Access those views at: /test and /test_urlto
Expected Output
No InvalidArgumentException, Missing argument for "(/(:any)" in route "test_urlto(/(:any))?" while using both url_to and route_to .
Anything else?
-
One can try it with
public bool $multipleSegmentsOneParam = true;orpublic bool $multipleSegmentsOneParam = false;inRouting.php. The error is the same in both situations (just the parameters are treated differently, as expected). -
One can also use a regex like
'(/(.+))?'instead of(/(:any))?. The error remains.
- Vorherrschende Sprache
- PHP
- Sterne
- 6k
- Forks
- 2k
- Ø Merge
- 1 T. 22 Std.
- Gemergte PRs (30 T.)
- 59
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus codeigniter4/CodeIgniter4
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
codeigniter4/CodeIgniter4#10616 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100
codeigniter4/CodeIgniter4#10618 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 62/100
codeigniter4/CodeIgniter4#10617 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100
codeigniter4/CodeIgniter4#10615 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
codeigniter4/CodeIgniter4#10589 · 3 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in codeigniter4/CodeIgniter4
Ähnliche Issues
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 84/100
WordPress/hosting-handbook#506 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Installing a plugin with composer_packages installs dev dependencies and breaks the autoloader (Error 500)Evtl. vergeben @Boy132 hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
Maintainer antworten meist innerhalb von 1 Tag
-
nginx: static-file locations drop the security headers, and the dotfile deny is matched after themOffenbug Low Security
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 78/100
backdrop-contrib/commerce#60 ·