commit 2e09c835efc07a9b5e93d94b6698e193b04dd48d
parent 72c3c191b51117035367048d151273a1468b627b
Author: Jan Dankert <devnull@localhost>
Date: Thu, 7 Dec 2017 23:18:17 +0100
Refactoring: Statement::getAll() benötigt keinen Parameter mehr.
Diffstat:
9 files changed, 60 insertions(+), 60 deletions(-)
diff --git a/action/LoginAction.class.php b/action/LoginAction.class.php
@@ -798,13 +798,14 @@ class LoginAction extends Action
return;
}
}
-
-
- /**
- * Synchronisiert die bisherigen Gruppen des Benutzers mit den Gruppen, die sich aus der Authentifzierung ergeben haben.
- *
- * @param unknown_type $groups Einfaches Array von Gruppennamen.
- */
+
+
+ /**
+ * Synchronisiert die bisherigen Gruppen des Benutzers mit den Gruppen, die sich aus der Authentifzierung ergeben haben.
+ *
+ * @param $user User Benutzerobjekt
+ * @param $groups array $groups Einfaches Array von Gruppennamen.
+ */
private function checkGroups($user, $groups)
{
if ( $groups == null )
@@ -830,7 +831,7 @@ class LoginAction extends Action
catch (ObjectNotFoundException $e)
{
// Gruppe fehlt. Anlegen?
- if ( $conf['ldap']['authorize']['auto_add'] )
+ if ( config('ldap','authorize','auto_add' )
{
// Die Gruppe in der OpenRat-Datenbank hinzufuegen.
$g = new Group();
@@ -902,7 +903,7 @@ class LoginAction extends Action
{
$auth = new InternalAuth();
- if ( $auth->login($loginName, $loginPassword) || $auth->mustChangePassword )
+ if ( $auth->login($loginName, $loginPassword,$token) || $auth->mustChangePassword )
{
if ( $newPassword1 != $newPassword2 )
{
diff --git a/model/Folder.class.php b/model/Folder.class.php
@@ -195,7 +195,7 @@ class Folder extends Object
$sql->setInt('objectid' ,$this->objectid );
$liste = array();
- $res = $sql->getAll( $sql );
+ $res = $sql->getAll();
foreach( $res as $row )
{
$o = new Object( $row['id'] );
@@ -889,7 +889,7 @@ SQL
$language = \Session::getProjectLanguage();
$sql->setInt( 'languageid', $language->languageid );
- return $sql->getAll( $sql );
+ return $sql->getAll();
}
}
diff --git a/model/Group.class.php b/model/Group.class.php
@@ -268,7 +268,7 @@ class Group
$sql->setInt('projectid',$projectid );
$sql->setInt('groupid' ,$this->groupid );
- $acls = $sql->getAll( $sql );
+ $acls = $sql->getAll();
foreach( $acls as $acl )
{
@@ -348,7 +348,7 @@ class Group
$aclList = array();
- foreach( $sql->getAll( $sql ) as $row )
+ foreach($sql->getAll() as $row )
{
$acl = new Acl();
$acl->setDatabaseRow( $row );
diff --git a/model/Object.class.php b/model/Object.class.php
@@ -274,7 +274,7 @@ SQL
$sql->setInt ( 'objectid' ,$this->objectid );
$sql->setInt ( 'userid' ,$user->userid );
- foreach( $sql->getAll( $sql ) as $row )
+ foreach($sql->getAll() as $row )
{
$acl = new Acl();
$acl->setDatabaseRow( $row );
@@ -446,7 +446,7 @@ SQL
/**
* Stellt fest, ob das Objekt mit der angegebenen Id existiert.
*/
- function available( $objectid )
+ public static function available( $objectid )
{
$db = db_connection();
diff --git a/model/Project.class.php b/model/Project.class.php
@@ -818,7 +818,7 @@ SQL
$user = \Session::getUser();
$sql->setInt( 'userid', $user->userid );
- return $sql->getAll( $sql );
+ return $sql->getAll();
}
@@ -851,7 +851,7 @@ SQL
SQL
);
- return $sql->getAll( $sql );
+ return $sql->getAll();
}
@@ -892,7 +892,7 @@ SQL
$language = \Session::getProjectLanguage();
$sql->setInt( 'languageid', $language->languageid );
- return $sql->getAll( $sql );
+ return $sql->getAll();
}
}
diff --git a/model/Template.class.php b/model/Template.class.php
@@ -1,4 +1,4 @@
-<?php
+<?php
namespace cms\model;
// OpenRat Content Management System
// Copyright (C) 2002-2012 Jan Dankert, cms@jandankert.de
@@ -248,7 +248,7 @@ class Template
' WHERE templateid={templateid}'.
' ORDER BY name ASC' );
$sql->setInt( 'templateid',$this->templateid );
- foreach( $sql->getAll( $sql ) as $row )
+ foreach($sql->getAll() as $row )
{
$e = new Element( $row['id'] );
$e->setDatabaseRow( $row );
@@ -273,18 +273,18 @@ class Template
$readonlyList = "'".implode("','",$e->readonlyElementNames)."'";
$db = db_connection();
-
+
$sql = $db->sql( <<<SQL
SELECT * FROM {{element}}
WHERE templateid={templateid}
AND writable=1
- AND type NOT IN ($readonlyList)
- ORDER BY name ASC
+ AND type NOT IN ($readonlyList)
+ ORDER BY name ASC
SQL
-);
+);
$sql->setInt ( 'templateid' ,$this->templateid );
- foreach( $sql->getAll( $sql ) as $row )
- {
+ foreach($sql->getAll() as $row )
+ {
$e = new Element( $row['id'] );
$e->setDatabaseRow( $row );
@@ -405,35 +405,35 @@ SQL
' WHERE id={templateid}' );
$sql->setInt( 'templateid',$this->templateid );
$sql->query( $sql );
- }
-
-
- /**
+ }
+
+
+ /**
* Ermittelt den Mime-Type zu diesem Template.
*
* Es wird die Extension des Templates betrachtet und dann mit Hilfe der
- * Konfigurationsdatei 'mime-types.ini' der Mime-Type bestimmt.
- *
- * @return String Mime-Type
- */
- function mimeType()
- {
- global $conf;
- $mime_types = $conf['mime-types'];
+ * Konfigurationsdatei 'mime-types.ini' der Mime-Type bestimmt.
+ *
+ * @return String Mime-Type
+ */
+ function mimeType()
+ {
+ global $conf;
+ $mime_types = $conf['mime-types'];
// Nur den letzten Teil der Extension auswerten:
// Aus 'mobile.html' wird nur 'html' verwendet.
- $parts = explode('.',$this->extension);
- $extension = strtolower(array_pop($parts));
-
- if ( !empty($mime_types[$extension]) )
- $this->mime_type = $mime_types[$extension];
- else
- // Wenn kein Mime-Type gefunden, dann Standardwert setzen
- $this->mime_type = 'application/octet-stream';
-
- return( $this->mime_type );
- }
+ $parts = explode('.',$this->extension);
+ $extension = strtolower(array_pop($parts));
+
+ if ( !empty($mime_types[$extension]) )
+ $this->mime_type = $mime_types[$extension];
+ else
+ // Wenn kein Mime-Type gefunden, dann Standardwert setzen
+ $this->mime_type = 'application/octet-stream';
+
+ return( $this->mime_type );
+ }
}
diff --git a/model/User.class.php b/model/User.class.php
@@ -92,7 +92,7 @@ class User extends ModelBase
' FROM {{user}}'.
' ORDER BY name' );
- foreach( $sql->getAll( $sql ) as $row )
+ foreach($sql->getAll() as $row )
{
$user = new User();
$user->setDatabaseRow( $row );
@@ -590,7 +590,7 @@ SQL
/**
* Gruppen ermitteln, in denen der Benutzer Mitglied ist.
*
- * @return Array mit Id:Name
+ * @return array mit Id:Name
*/
function getGroups()
{
@@ -722,7 +722,7 @@ SQL
$aclList = array();
- foreach( $sql->getAll( $sql ) as $row )
+ foreach($sql->getAll() as $row )
{
$acl = new Acl();
$acl->setDatabaseRow( $row );
@@ -966,7 +966,7 @@ SQL
$sql->setInt( 'userid', $this->userid );
- return $sql->getAll( $sql );
+ return $sql->getAll();
}
diff --git a/model/Value.class.php b/model/Value.class.php
@@ -257,7 +257,7 @@ class Value
$sql->setInt( 'languageid',$this->languageid);
$list = array();
- foreach( $sql->getAll( $sql ) as $row )
+ foreach($sql->getAll() as $row )
{
$val = new Value();
$val->valueid = $row['id'];
diff --git a/modules/database/Statement.class.php b/modules/database/Statement.class.php
@@ -225,12 +225,11 @@ class Statement
}
- /**
- * Ermittelt alle Datenbankergebniszeilen.
- *
- * @param String $this->query
- * @return Array
- */
+ /**
+ * Ermittelt alle Datenbankergebniszeilen.
+ *
+ * @return array
+ */
public function &getAll()
{
$result = $this->query();
