summaryrefslogtreecommitdiffstats
path: root/apps/files_external/lib
diff options
context:
space:
mode:
authorJoas Schilling <coding@schilljs.com>2017-05-10 14:12:58 +0200
committerMorris Jobke <hey@morrisjobke.de>2017-08-01 13:56:12 +0200
commit7a62fbd205874355abb3b551b182bc61b9c3d3e1 (patch)
tree8fba151653401d4af74f2091fb7d2e6add45f00e /apps/files_external/lib
parente30287cf814bca99b01561df0e60fb00c59ade42 (diff)
downloadnextcloud-server-7a62fbd205874355abb3b551b182bc61b9c3d3e1.tar.gz
nextcloud-server-7a62fbd205874355abb3b551b182bc61b9c3d3e1.zip
Fix comparisons in the files external app
Signed-off-by: Joas Schilling <coding@schilljs.com>
Diffstat (limited to 'apps/files_external/lib')
-rw-r--r--apps/files_external/lib/Command/Import.php4
-rw-r--r--apps/files_external/lib/Lib/Storage/Dropbox.php8
-rw-r--r--apps/files_external/lib/Lib/Storage/FTP.php2
-rw-r--r--apps/files_external/lib/Lib/Storage/OwnCloud.php4
-rw-r--r--apps/files_external/lib/Lib/Storage/SFTP.php14
-rw-r--r--apps/files_external/lib/Lib/Storage/SMB.php4
-rw-r--r--apps/files_external/lib/Lib/Storage/Swift.php4
7 files changed, 20 insertions, 20 deletions
diff --git a/apps/files_external/lib/Command/Import.php b/apps/files_external/lib/Command/Import.php
index 96afc86ba2c..712a8851c8a 100644
--- a/apps/files_external/lib/Command/Import.php
+++ b/apps/files_external/lib/Command/Import.php
@@ -161,8 +161,8 @@ class Import extends Base {
if (
$existingMount->getMountPoint() === $mount->getMountPoint() &&
$existingMount->getApplicableGroups() === $mount->getApplicableGroups() &&
- $existingMount->getApplicableUsers() == $mount->getApplicableUsers() &&
- $existingMount->getBackendOptions() == $mount->getBackendOptions()
+ $existingMount->getApplicableUsers() === $mount->getApplicableUsers() &&
+ $existingMount->getBackendOptions() === $mount->getBackendOptions()
) {
$output->writeln("<error>Duplicate mount (" . $mount->getMountPoint() . ")</error>");
return 1;
diff --git a/apps/files_external/lib/Lib/Storage/Dropbox.php b/apps/files_external/lib/Lib/Storage/Dropbox.php
index d2ba1cca751..fad85650e9b 100644
--- a/apps/files_external/lib/Lib/Storage/Dropbox.php
+++ b/apps/files_external/lib/Lib/Storage/Dropbox.php
@@ -47,7 +47,7 @@ class Dropbox extends \OC\Files\Storage\Common {
private $oauth;
public function __construct($params) {
- if (isset($params['configured']) && $params['configured'] == 'true'
+ if (isset($params['configured']) && $params['configured'] === 'true'
&& isset($params['app_key'])
&& isset($params['app_secret'])
&& isset($params['token'])
@@ -187,12 +187,12 @@ class Dropbox extends \OC\Files\Storage\Common {
}
public function filetype($path) {
- if ($path == '' || $path == '/') {
+ if ($path === '' || $path === '/') {
return 'dir';
} else {
$metaData = $this->getDropBoxMetaData($path);
if ($metaData) {
- if ($metaData['is_dir'] == 'true') {
+ if ($metaData['is_dir'] === 'true') {
return 'dir';
} else {
return 'file';
@@ -203,7 +203,7 @@ class Dropbox extends \OC\Files\Storage\Common {
}
public function file_exists($path) {
- if ($path == '' || $path == '/') {
+ if ($path === '' || $path === '/') {
return true;
}
if ($this->getDropBoxMetaData($path)) {
diff --git a/apps/files_external/lib/Lib/Storage/FTP.php b/apps/files_external/lib/Lib/Storage/FTP.php
index 22fe2090f30..1bbdfaba468 100644
--- a/apps/files_external/lib/Lib/Storage/FTP.php
+++ b/apps/files_external/lib/Lib/Storage/FTP.php
@@ -56,7 +56,7 @@ class FTP extends StreamWrapper{
$this->secure = false;
}
$this->root=isset($params['root'])?$params['root']:'/';
- if ( ! $this->root || $this->root[0]!='/') {
+ if ( ! $this->root || $this->root[0]!=='/') {
$this->root='/'.$this->root;
}
if (substr($this->root, -1) !== '/') {
diff --git a/apps/files_external/lib/Lib/Storage/OwnCloud.php b/apps/files_external/lib/Lib/Storage/OwnCloud.php
index 9669b5f3dad..34838b9891e 100644
--- a/apps/files_external/lib/Lib/Storage/OwnCloud.php
+++ b/apps/files_external/lib/Lib/Storage/OwnCloud.php
@@ -40,10 +40,10 @@ class OwnCloud extends \OC\Files\Storage\DAV{
// (owncloud install path on host)
$host = $params['host'];
// strip protocol
- if (substr($host, 0, 8) == "https://") {
+ if (substr($host, 0, 8) === "https://") {
$host = substr($host, 8);
$params['secure'] = true;
- } else if (substr($host, 0, 7) == "http://") {
+ } else if (substr($host, 0, 7) === "http://") {
$host = substr($host, 7);
$params['secure'] = false;
}
diff --git a/apps/files_external/lib/Lib/Storage/SFTP.php b/apps/files_external/lib/Lib/Storage/SFTP.php
index a4dfea94bf7..8d489551264 100644
--- a/apps/files_external/lib/Lib/Storage/SFTP.php
+++ b/apps/files_external/lib/Lib/Storage/SFTP.php
@@ -102,11 +102,11 @@ class SFTP extends \OC\Files\Storage\Common {
$this->root
= isset($params['root']) ? $this->cleanPath($params['root']) : '/';
- if ($this->root[0] != '/') {
+ if ($this->root[0] !== '/') {
$this->root = '/' . $this->root;
}
- if (substr($this->root, -1, 1) != '/') {
+ if (substr($this->root, -1, 1) !== '/') {
$this->root .= '/';
}
}
@@ -128,7 +128,7 @@ class SFTP extends \OC\Files\Storage\Common {
// The SSH Host Key MUST be verified before login().
$currentHostKey = $this->client->getServerPublicHostKey();
if (array_key_exists($this->host, $hostKeys)) {
- if ($hostKeys[$this->host] != $currentHostKey) {
+ if ($hostKeys[$this->host] !== $currentHostKey) {
throw new \Exception('Host public key does not match known key');
}
} else {
@@ -248,7 +248,7 @@ class SFTP extends \OC\Files\Storage\Common {
if ($lines) {
foreach ($lines as $line) {
$hostKeyArray = explode("::", $line, 2);
- if (count($hostKeyArray) == 2) {
+ if (count($hostKeyArray) === 2) {
$hosts[] = $hostKeyArray[0];
$keys[] = $hostKeyArray[1];
}
@@ -300,7 +300,7 @@ class SFTP extends \OC\Files\Storage\Common {
$id = md5('sftp:' . $path);
$dirStream = array();
foreach($list as $file) {
- if ($file != '.' && $file != '..') {
+ if ($file !== '.' && $file !== '..') {
$dirStream[] = $file;
}
}
@@ -316,11 +316,11 @@ class SFTP extends \OC\Files\Storage\Common {
public function filetype($path) {
try {
$stat = $this->getConnection()->stat($this->absPath($path));
- if ($stat['type'] == NET_SFTP_TYPE_REGULAR) {
+ if ((int) $stat['type'] === NET_SFTP_TYPE_REGULAR) {
return 'file';
}
- if ($stat['type'] == NET_SFTP_TYPE_DIRECTORY) {
+ if ((int) $stat['type'] === NET_SFTP_TYPE_DIRECTORY) {
return 'dir';
}
} catch (\Exception $e) {
diff --git a/apps/files_external/lib/Lib/Storage/SMB.php b/apps/files_external/lib/Lib/Storage/SMB.php
index 7afdb746a98..4af6df5d84a 100644
--- a/apps/files_external/lib/Lib/Storage/SMB.php
+++ b/apps/files_external/lib/Lib/Storage/SMB.php
@@ -81,10 +81,10 @@ class SMB extends Common implements INotifyStorage {
$this->share = $this->server->getShare(trim($params['share'], '/'));
$this->root = isset($params['root']) ? $params['root'] : '/';
- if (!$this->root || $this->root[0] != '/') {
+ if (!$this->root || $this->root[0] !== '/') {
$this->root = '/' . $this->root;
}
- if (substr($this->root, -1, 1) != '/') {
+ if (substr($this->root, -1, 1) !== '/') {
$this->root .= '/';
}
} else {
diff --git a/apps/files_external/lib/Lib/Storage/Swift.php b/apps/files_external/lib/Lib/Storage/Swift.php
index db5b5bf6d95..57df4aa01a5 100644
--- a/apps/files_external/lib/Lib/Storage/Swift.php
+++ b/apps/files_external/lib/Lib/Storage/Swift.php
@@ -435,7 +435,7 @@ class Swift extends \OC\Files\Storage\Common {
}
$metadata = array('timestamp' => $mtime);
if ($this->file_exists($path)) {
- if ($this->is_dir($path) && $path != '.') {
+ if ($this->is_dir($path) && $path !== '.') {
$path .= '/';
}
@@ -640,7 +640,7 @@ class Swift extends \OC\Files\Storage\Common {
}, $cachedContent);
sort($cachedNames);
sort($content);
- return $cachedNames != $content;
+ return $cachedNames !== $content;
}
/**