[BUGFIX] Methods of tx_em_Database should be static
authorXavier Perseguers <xavier@typo3.org>
Mon, 19 Sep 2011 21:40:32 +0000 (23:40 +0200)
committerXavier Perseguers <xavier@typo3.org>
Mon, 19 Sep 2011 21:45:42 +0000 (23:45 +0200)
Methods of class tx_em_Database miss the static keyword and, sometimes,
the visibility as well.

Change-Id: I477ca961ae9dc6fc456c3e90fd8f0871be2cc145
Fixes: #30059
Releases: 4.6
Reviewed-on: http://review.typo3.org/5086
Reviewed-by: Xavier Perseguers
Tested-by: Xavier Perseguers
typo3/sysext/em/classes/database/class.tx_em_database.php

index f586637..6a3370d 100644 (file)
@@ -62,7 +62,7 @@ final class tx_em_Database {
         * @param   integer  $repository  (optional) repository uid of extensions to count
         * @return  integer  sum of extensions in database
         */
-       public function getExtensionCountFromRepository($repository = NULL) {
+       public static function getExtensionCountFromRepository($repository = NULL) {
                if (is_null($repository)) {
                        return $GLOBALS['TYPO3_DB']->exec_SELECTcountRows(
                                'DISTINCT extkey',
@@ -87,7 +87,7 @@ final class tx_em_Database {
         * @param string $limit
         * @return array
         */
-       public function getExtensionListFromRepository($repository, $addFields = '', $andWhere = '', $order = '', $limit = '') {
+       public static function getExtensionListFromRepository($repository, $addFields = '', $andWhere = '', $order = '', $limit = '') {
                $ret = array();
                $temp = $GLOBALS['TYPO3_DB']->exec_SELECTgetRows(
                        'count(*) AS count',
@@ -117,7 +117,7 @@ final class tx_em_Database {
         * @param string $extKey
         * @return array $versions
         */
-       public function getExtensionVersionsFromRepository($repository, $extKey) {
+       public static function getExtensionVersionsFromRepository($repository, $extKey) {
                $versions = array();
                //TODO: implement
                return $versions;
@@ -130,7 +130,7 @@ final class tx_em_Database {
         * @param   tx_em_Repository $repository  repository object
         * @return  void
         */
-       public function updateRepository(tx_em_Repository $repository) {
+       public static function updateRepository(tx_em_Repository $repository) {
                $repositoryData = array(
                        'title' => $repository->getTitle(),
                        'description' => $repository->getDescription(),
@@ -155,7 +155,7 @@ final class tx_em_Database {
         * @param   tx_em_Repository $repository  repository object
         * @return  integer  UID of the newly inserted repository object
         */
-       public function insertRepository(tx_em_Repository $repository) {
+       public static function insertRepository(tx_em_Repository $repository) {
                $repositoryData = array(
                        'title' => $repository->getTitle(),
                        'description' => $repository->getDescription(),
@@ -177,7 +177,7 @@ final class tx_em_Database {
         * @param  tx_em_Repository $repository  repository object
         * @return void
         */
-       public function deleteRepository(tx_em_Repository $repository) {
+       public static function deleteRepository(tx_em_Repository $repository) {
                $GLOBALS['TYPO3_DB']->exec_DELETEquery(
                         self::TABLE_REPOSITORY,
                        'uid=' . $repository->getId()
@@ -190,7 +190,7 @@ final class tx_em_Database {
         * @param int $uid
         * @return void
         */
-       public function updateRepositoryCount($extCount, $uid = 1) {
+       public static function updateRepositoryCount($extCount, $uid = 1) {
                $GLOBALS['TYPO3_DB']->exec_UPDATEquery(
                         self::TABLE_REPOSITORY,
                         'uid=' . intval($uid),
@@ -206,7 +206,7 @@ final class tx_em_Database {
         * @param  $arrFields
         * @return void
         */
-       public function insertVersion(array $arrFields) {
+       public static function insertVersion(array $arrFields) {
                $GLOBALS['TYPO3_DB']->exec_INSERTquery(self::TABLE_EXTENSION, $arrFields);
        }
 
@@ -216,7 +216,7 @@ final class tx_em_Database {
         * @param int $repositoryUid
         * @return integer
         */
-       public function insertLastVersion($repositoryUid = 1) {
+       public static function insertLastVersion($repositoryUid = 1) {
                $groupedRows = $GLOBALS['TYPO3_DB']->exec_SELECTgetRows(
                        'extkey, version, max(intversion) maxintversion',
                        'cache_extensions',
@@ -246,7 +246,6 @@ final class tx_em_Database {
                return $extensions;
        }
 
-
        /**
         * Method finds and returns repository fields identified by its UID.
         *
@@ -254,7 +253,7 @@ final class tx_em_Database {
         * @param   int  $uid  repository UID
         * @return array
         */
-       public function getRepositoryByUID($uid) {
+       public static function getRepositoryByUID($uid) {
                $row = $GLOBALS['TYPO3_DB']->exec_SELECTgetSingleRow('*', self::TABLE_REPOSITORY, 'uid=' . intval($uid));
 
                return $row;
@@ -266,7 +265,7 @@ final class tx_em_Database {
         * @param  $title
         * @return array
         */
-       public function getRepositoryByTitle($title) {
+       public static function getRepositoryByTitle($title) {
                return $GLOBALS['TYPO3_DB']->exec_SELECTgetRows(
                        '*',
                        self::TABLE_REPOSITORY,
@@ -281,7 +280,7 @@ final class tx_em_Database {
         * @param string $where
         * @return array
         */
-       public function getRepositories($where = NULL) {
+       public static function getRepositories($where = NULL) {
                return $GLOBALS['TYPO3_DB']->exec_SELECTgetRows(
                        '*',
                        self::TABLE_REPOSITORY,
@@ -297,7 +296,7 @@ final class tx_em_Database {
         * @param       array           Field structure
         * @return      string          SQL Content of dump (INSERT statements)
         */
-       function dumpTableContent($table, $fieldStructure) {
+       protected static function dumpTableContent($table, array $fieldStructure) {
 
                // Substitution of certain characters (borrowed from phpMySQL):
                $search = array('\\', '\'', "\x00", "\x0a", "\x0d", "\x1a");
@@ -329,9 +328,9 @@ final class tx_em_Database {
         * Which fields and which tables are determined from the ext_tables.sql file
         *
         * @param       string          Array with table.field values
-        * @return      array           Array of tables and fields splitted.
+        * @return      array           Array of tables and fields split.
         */
-       function getTableAndFieldStructure($parts) {
+       public static function getTableAndFieldStructure($parts) {
                        /** @var $instObj t3lib_install_Sql */
                $instObj = t3lib_div::makeInstance('t3lib_install_Sql');
                $dbFields = $instObj->getFieldDefinitions_database(TYPO3_db);
@@ -360,7 +359,6 @@ final class tx_em_Database {
                return $outTables;
        }
 
-
        /**
         * Makes a dump of the tables/fields definitions for an extension
         *
@@ -368,7 +366,7 @@ final class tx_em_Database {
         * @return      string          SQL for the table definitions
         * @see dumpStaticTables()
         */
-       function dumpTableAndFieldStructure($arr) {
+       public static function dumpTableAndFieldStructure(array $arr) {
                $tables = array();
 
                if (count($arr)) {
@@ -394,7 +392,7 @@ final class tx_em_Database {
         * @param       array  $additionalLinkParameter
         * @return      string          HTML
         */
-       function dumpDataTablesLine($tablesArray, $extKey, $additionalLinkParameter = array()) {
+       public static function dumpDataTablesLine(array $tablesArray, $extKey, array $additionalLinkParameter = array()) {
                $tables = array();
                $tablesNA = array();
                $allTables = array_keys($GLOBALS['TYPO3_DB']->admin_get_tables());
@@ -448,7 +446,7 @@ final class tx_em_Database {
         * @return      string          Returns the content
         * @see dumpTableAndFieldStructure()
         */
-       function dumpStaticTables($tableList) {
+       public static function dumpStaticTables($tableList) {
                        /** @var $instObj t3lib_install_Sql */
                $instObj = t3lib_div::makeInstance('t3lib_install_Sql');
                $dbFields = $instObj->getFieldDefinitions_database(TYPO3_db);
@@ -481,7 +479,7 @@ final class tx_em_Database {
         *
         * @return      string          Table header
         */
-       function dumpHeader() {
+       protected static function dumpHeader() {
                return trim('
 # TYPO3 Extension Manager dump 1.1
 #
@@ -498,7 +496,7 @@ final class tx_em_Database {
         * @param       boolean         If TRUE, add "DROP TABLE IF EXISTS"
         * @return      string          Table definition SQL
         */
-       function dumpTableHeader($table, $fieldKeyInfo, $dropTableIfExists = 0) {
+       protected static function dumpTableHeader($table, array $fieldKeyInfo, $dropTableIfExists = FALSE) {
                $lines = array();
                $dump = '';