From 0d31da4757a37d322e1e181f2286e8a4c89fbc0c Mon Sep 17 00:00:00 2001
From: Robin Appelman <robin@icewind.nl>
Date: Thu, 24 May 2018 11:48:51 +0200
Subject: [PATCH] move some arrays to constants

---
 src/Wrapped/Parser.php | 19 ++++++++++---------
 src/Wrapped/Share.php  | 15 ++++++++-------
 2 files changed, 18 insertions(+), 16 deletions(-)

diff --git a/src/Wrapped/Parser.php b/src/Wrapped/Parser.php
index 77cdfac..c446ae8 100644
--- a/src/Wrapped/Parser.php
+++ b/src/Wrapped/Parser.php
@@ -45,6 +45,15 @@ class Parser {
 		ErrorCodes::InvalidParameter  => InvalidParameterException::class
 	];
 
+	const MODE_STRINGS = [
+		'R' => FileInfo::MODE_READONLY,
+		'H' => FileInfo::MODE_HIDDEN,
+		'S' => FileInfo::MODE_SYSTEM,
+		'D' => FileInfo::MODE_DIRECTORY,
+		'A' => FileInfo::MODE_ARCHIVE,
+		'N' => FileInfo::MODE_NORMAL
+	];
+
 	/**
 	 * @param TimeZoneProvider $timeZoneProvider
 	 */
@@ -109,15 +118,7 @@ public function checkConnectionError($line) {
 
 	public function parseMode($mode) {
 		$result = 0;
-		$modeStrings = array(
-			'R' => FileInfo::MODE_READONLY,
-			'H' => FileInfo::MODE_HIDDEN,
-			'S' => FileInfo::MODE_SYSTEM,
-			'D' => FileInfo::MODE_DIRECTORY,
-			'A' => FileInfo::MODE_ARCHIVE,
-			'N' => FileInfo::MODE_NORMAL
-		);
-		foreach ($modeStrings as $char => $val) {
+		foreach (self::MODE_STRINGS as $char => $val) {
 			if (strpos($mode, $char) !== false) {
 				$result |= $val;
 			}
diff --git a/src/Wrapped/Share.php b/src/Wrapped/Share.php
index 1d84752..ca88af2 100644
--- a/src/Wrapped/Share.php
+++ b/src/Wrapped/Share.php
@@ -45,6 +45,13 @@ class Share extends AbstractShare {
 	 */
 	private $system;
 
+	const MODE_MAP = [
+		FileInfo::MODE_READONLY => 'r',
+		FileInfo::MODE_HIDDEN   => 'h',
+		FileInfo::MODE_ARCHIVE  => 'a',
+		FileInfo::MODE_SYSTEM   => 's'
+	];
+
 	/**
 	 * @param IServer $server
 	 * @param string $name
@@ -327,13 +334,7 @@ public function write($target) {
 	 */
 	public function setMode($path, $mode) {
 		$modeString = '';
-		$modeMap = array(
-			FileInfo::MODE_READONLY => 'r',
-			FileInfo::MODE_HIDDEN   => 'h',
-			FileInfo::MODE_ARCHIVE  => 'a',
-			FileInfo::MODE_SYSTEM   => 's'
-		);
-		foreach ($modeMap as $modeByte => $string) {
+		foreach (self::MODE_MAP as $modeByte => $string) {
 			if ($mode & $modeByte) {
 				$modeString .= $string;
 			}