com web/php: Adding clarity to the variable names for the JSON/serialized output: releases/index.php

[email protected] (Hannes Magnusson)
Newsgroups php.webmaster
Message-ID <[email protected]>
Commit:    a2106dccfb40ec7443650617632c37b3c853102a
Author:    Ben Ramsey <[email protected]>         Mon, 6 Apr 2015 10:08:42 -0500
Parents:   bbb58d9d6cab1f9a70cc820ad54535cf50caa553
Branches:  master

Link:       http://git.php.net/?p=web/php.git;a=commitdiff;h=a2106dccfb40ec7443650617632c37b3c853102a

Log:
Adding clarity to the variable names for the JSON/serialized output

Changed paths:
  M  releases/index.php


Diff:
diff --git a/releases/index.php b/releases/index.php
index 635d69c..3a43919 100644
--- a/releases/index.php
+++ b/releases/index.php
@@ -8,7 +8,7 @@ if (isset($_GET["serialize"]) || isset($_GET["json"])) {
 	$RELEASES[5][$PHP_5_4_VERSION]["date"] = $PHP_5_4_DATE;
 	$RELEASES                              = $RELEASES + $OLDRELEASES;
 
-	$return = array();
+	$machineReadable = array();
 
 	if (isset($_GET["version"])) {
 		$ver = (int)$_GET["version"];
@@ -20,14 +20,14 @@ if (isset($_GET["serialize"]) || isset($_GET["json"])) {
 				$max = (int)$_GET["max"];
 				if ($max == -1) { $max = PHP_INT_MAX; }
 
-				$return = array($version => $r);
+				$machineReadable = array($version => $r);
 
 				$count = 1;
 
 				/* check if other $RELEASES[$ver] are there */
 				/* e.g., 5_3, 5_4, and 5_5 all exist and have a release */
 				while(($z = each($RELEASES[$ver])) && $count++ < $max) {
-					$return[$z[0]] = $z[1];
+					$machineReadable[$z[0]] = $z[1];
 				}
 
 				foreach($OLDRELEASES[$ver] as $version => $release) {
@@ -35,32 +35,31 @@ if (isset($_GET["serialize"]) || isset($_GET["json"])) {
 						break;
 					}
 
-					$return[$version] = $release;
+					$machineReadable[$version] = $release;
 				}
 			} else {
 				$r["version"] = $version;
 
-				$return = $r;
+				$machineReadable = $r;
 			}
 		} else {
-			$return = array("error" => "Unknown version");
+			$machineReadable = array("error" => "Unknown version");
 		}
 	} else {
-		$array = array();
+		$machineReadable = array();
 		foreach($RELEASES as $major => $release) {
 			list($version, $r) = each($release);
 			$r["version"] = $version;
-			$array[$major] = $r;
+			$machineReadable[$major] = $r;
 		}
-		$return = $array;
 	}
 
 	if (isset($_GET["serialize"])) {
 		header('Content-type: text/plain');
-		echo serialize($return);
+		echo serialize($machineReadable);
 	} elseif (isset($_GET["json"])) {
 		header('Content-Type: application/json');
-		echo json_encode($return);
+		echo json_encode($machineReadable);
 	}
 	return;
 }
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.