Use short for of substr to not need strlen()
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
This commit is contained in:
parent
b9bbb894f8
commit
e6efa755c9
4 changed files with 4 additions and 4 deletions
|
@ -167,7 +167,7 @@ class Storage {
|
|||
// to get the right target
|
||||
$ext = pathinfo($filename, PATHINFO_EXTENSION);
|
||||
if ($ext === 'part') {
|
||||
$filename = substr($filename, 0, strlen($filename) - 5);
|
||||
$filename = substr($filename, 0, -5);
|
||||
}
|
||||
|
||||
// we only handle existing files
|
||||
|
|
|
@ -108,7 +108,7 @@ class Adapter {
|
|||
$query .= ' = ? AND ';
|
||||
}
|
||||
}
|
||||
$query = substr($query, 0, strlen($query) - 5);
|
||||
$query = substr($query, 0, -5);
|
||||
$query .= ' HAVING COUNT(*) = 0';
|
||||
|
||||
return $this->conn->executeUpdate($query, $inserts);
|
||||
|
|
|
@ -79,7 +79,7 @@ class AdapterSqlite extends Adapter {
|
|||
$query .= ' = ? AND ';
|
||||
}
|
||||
}
|
||||
$query = substr($query, 0, strlen($query) - 5);
|
||||
$query = substr($query, 0, -5);
|
||||
$query .= ')';
|
||||
|
||||
return $this->conn->executeUpdate($query, $inserts);
|
||||
|
|
|
@ -132,7 +132,7 @@ class OC_Template extends \OC\Template\Base {
|
|||
foreach(array_reverse($coreDependencies['vendor']) as $vendorLibrary) {
|
||||
//remove trailing ".js" as addVendorScript will append it
|
||||
OC_Util::addVendorScript(
|
||||
substr($vendorLibrary, 0, strlen($vendorLibrary) - 3),null,true);
|
||||
substr($vendorLibrary, 0, -3),null,true);
|
||||
}
|
||||
} else {
|
||||
throw new \Exception('Cannot read core/js/core.json');
|
||||
|
|
Loading…
Reference in a new issue