[web-php] master: Add void return types
| From: | Mathias Reker ⚡️ via GitHub | Date: | Fri, 17 Jun 2022 17:22:28 +0000 |
| Subject: | [web-php] master: Add void return types | ||
| Groups: | php.webmaster | ||
| Request: | Send a blank email to php-webmaster+get-30100@lists.php.net to get a copy of this message | ||
Author: Mathias Reker ⚡️ (MathiasReker)
Committer: GitHub (web-flow)
Pusher: cmb69
Date: 2022-06-17T19:16:52+02:00
Commit: https://github.com/php/web-php/commit/530b28e95b4d66323b3078cd8b6bf65c390c2e50
Raw diff: https://github.com/php/web-php/commit/530b28e95b4d66323b3078cd8b6bf65c390c2e50.diff
Add void return types
Add void return type to functions with missing or empty return statements.
Closes GH-535.
Changed paths:
M cal.php
M download-logos.php
M images/logo.php
M index.php
M mailing-lists.php
M quickref.php
M releases/index.php
M submit-event.php
M urlhowto.php
Diff:
diff --git a/cal.php b/cal.php
index ed02e43d0..3b5399959 100644
--- a/cal.php
+++ b/cal.php
@@ -230,7 +230,7 @@ function date_for_recur($recur, $day, $bom, $eom)
}
// Display a <div> for each of the events that are on a given day
-function display_events_for_day($day, $events)
+function display_events_for_day($day, $events): void
{
// For preservation of state in the links
global $cm, $cy, $COUNTRY;
diff --git a/download-logos.php b/download-logos.php
index 02158900a..85f59dd1c 100644
--- a/download-logos.php
+++ b/download-logos.php
@@ -13,13 +13,13 @@
site_header("Download Logos", array("current" => "downloads"));
// Print recommended star cell
-function print_star()
+function print_star(): void
{
echo "<td class=\"star\"><span class=\"star\"
title=\"recommended\">*</span></td>\n";
}
// Provide a random bgcolor setting for a cell
-function random_bgcolor($min, $max)
+function random_bgcolor($min, $max): void
{
echo "style=\"background-color: #" .
sprintf('%02x%02x%02x', rand($min, $max)*51, rand($min, $max)*51, rand($min,
$max)*51) .
diff --git a/images/logo.php b/images/logo.php
index 525efb7da..0c64158c5 100644
--- a/images/logo.php
+++ b/images/logo.php
@@ -53,7 +53,7 @@ function get_accepted_encodings() {
}
-function serve_compressed_if_available($logo) {
+function serve_compressed_if_available($logo): void {
$encodings = get_accepted_encodings();
if (!empty($encodings)) {
foreach ($encodings as $encoding) {
diff --git a/index.php b/index.php
index 34f7b2a30..352d5750e 100644
--- a/index.php
+++ b/index.php
@@ -1,5 +1,5 @@
<?php // vim: et
-(function($uri) {
+(function($uri): void {
// Special redirect cases not able to be captured in error.php
$shortcuts = [
'/?:' =>
'/language.operators.comparison#language.operators.comparison.ternary',
diff --git a/mailing-lists.php b/mailing-lists.php
index a2a5232b8..e134096af 100644
--- a/mailing-lists.php
+++ b/mailing-lists.php
@@ -304,7 +304,7 @@
);
// Print out a table for a given list array
-function output_lists_table($mailing_lists)
+function output_lists_table($mailing_lists): void
{
echo '<table cellpadding="5" border="0" class="standard
mailing-lists">', "\n";
foreach ($mailing_lists as $listinfo) {
diff --git a/quickref.php b/quickref.php
index da786b9ca..940165521 100644
--- a/quickref.php
+++ b/quickref.php
@@ -24,7 +24,7 @@
// Print out the table of found (or all) functions. The HTML comments are
// needed to support MyCroft search (Mozilla browser family and Sherlock for MacOSX)
-function quickref_table($functions, $sort = true)
+function quickref_table($functions, $sort = true): void
{
global $LANG;
diff --git a/releases/index.php b/releases/index.php
index 4f141f8f2..b2bf361e2 100644
--- a/releases/index.php
+++ b/releases/index.php
@@ -186,7 +186,7 @@ function mk_rel(int $major,
/* bool | array */ $announcement,
array $source,
array $windows,
- bool $museum) {
+ bool $museum): void {
printf("<a id=\"%s\"></a>\n<h2>%1\$s</h2>\n<ul>\n
<li>Released: %s</li>\n <li>Announcement: ",
($pos = strpos($ver, " ")) ? substr($ver, 0, $pos) : $ver,
$date);
diff --git a/submit-event.php b/submit-event.php
index 055c694d6..adba380c1 100644
--- a/submit-event.php
+++ b/submit-event.php
@@ -265,7 +265,7 @@
site_footer();
// Display an option list with one selected
-function display_options($options, $current)
+function display_options($options, $current): void
{
foreach ($options as $k => $v) {
echo '<option value="', $k, '"',
diff --git a/urlhowto.php b/urlhowto.php
index 196be2bec..0ea3c0329 100644
--- a/urlhowto.php
+++ b/urlhowto.php
@@ -25,7 +25,7 @@
';
site_header("URL Howto", array("current" => "help"));
-function a($href) {
+function a($href): void {
global $MYSITE;
echo '<a href="' . $MYSITE . $href . '">' . $MYSITE . $href .
'</a>';
}