From 620600f3b9f1d9d5211f4ba91ff59b078b468666 Mon Sep 17 00:00:00 2001 From: Nisarg Jhaveri Date: Fri, 12 Jun 2015 13:14:41 +0530 Subject: [PATCH 1/8] Foreign key check checkbox for empty/drop in db_structure Signed-off-by: Nisarg Jhaveri --- js/db_structure.js | 22 ++++++++++++++++++---- js/functions.js | 25 ++++++++++++++++++------- libraries/Header.class.php | 3 ++- libraries/Util.class.php | 22 ++++++++++------------ 4 files changed, 48 insertions(+), 24 deletions(-) diff --git a/js/db_structure.js b/js/db_structure.js index b48094266d..656b6a70a2 100644 --- a/js/db_structure.js +++ b/js/db_structure.js @@ -243,14 +243,21 @@ AJAX.registerOnload('db_structure.js', function () { /** * @var question String containing the question to be asked for confirmation */ - var question = PMA_messages.strTruncateTableStrongWarning + ' ' + - PMA_sprintf(PMA_messages.strDoYouReally, 'TRUNCATE ' + escapeHtml(curr_table_name)); + var question = PMA_messages.strTruncateTableStrongWarning + ' ' + + PMA_sprintf(PMA_messages.strDoYouReally, 'TRUNCATE ' + escapeHtml(curr_table_name)) + + getForeignKeyCheckbox(); $this_anchor.PMA_confirm(question, $this_anchor.attr('href'), function (url) { PMA_ajaxShowMessage(PMA_messages.strProcessingRequest); - $.get(url, {'is_js_confirmed' : 1, 'ajax_request' : true}, function (data) { + var params = { + 'is_js_confirmed' : 1, + 'ajax_request' : true, + 'fk_checks': $(this).find('#fk_checks').is(':checked') ? 1 : 0 + }; + + $.get(url, params, function (data) { if (typeof data !== 'undefined' && data.success === true) { PMA_ajaxShowMessage(data.message); // Adjust table statistics @@ -305,12 +312,19 @@ AJAX.registerOnload('db_structure.js', function () { question = PMA_sprintf(PMA_messages.strDoYouReally, 'DROP VIEW ' + escapeHtml(curr_table_name)); } + question += getForeignKeyCheckbox(); $this_anchor.PMA_confirm(question, $this_anchor.attr('href'), function (url) { var $msg = PMA_ajaxShowMessage(PMA_messages.strProcessingRequest); - $.get(url, {'is_js_confirmed' : 1, 'ajax_request' : true}, function (data) { + var params = { + 'is_js_confirmed' : 1, + 'ajax_request' : true, + 'fk_checks': $(this).find('#fk_checks').is(':checked') ? 1 : 0 + }; + + $.get(url, params, function (data) { if (typeof data !== 'undefined' && data.success === true) { PMA_ajaxShowMessage(data.message); toggleRowColors($curr_row.next()); diff --git a/js/functions.js b/js/functions.js index baa164f59d..0d34dbae40 100644 --- a/js/functions.js +++ b/js/functions.js @@ -1675,6 +1675,23 @@ function pdfPaperSize(format, axis) return 0; } +/** + * Get checkbox for foreign key checks + * + * @return string + */ +function getForeignKeyCheckbox() { + default_fk_check_value = PMA_commonParams.get('default_fk_check_value'); + var html = ""; + html += "
"; + html += ""; + html += ""; + html += ""; + html += "
"; + return html; +} + /** * Unbind all event handlers before tearing down a page */ @@ -1723,15 +1740,9 @@ AJAX.registerOnload('functions.js', function () { var sql_query = $form.find("input[name='sql_query']").val().trim(); var $inner_sql = $(this).parent().prev().find('code.sql'); var old_text = $inner_sql.html(); - var default_fk_check_value = $form.find("input[name='default_fk_check_value']").val() == 'true'; var new_content = "\n"; - new_content += "
"; - new_content += ""; - new_content += ""; - new_content += ""; - new_content += "
"; + new_content += getForeignKeyCheckbox(); new_content += "\n"; new_content += "\n"; var $editor_area = $('div#inline_editor'); diff --git a/libraries/Header.class.php b/libraries/Header.class.php index 0a3ac7dc75..8c6cb7f039 100644 --- a/libraries/Header.class.php +++ b/libraries/Header.class.php @@ -254,7 +254,8 @@ class PMA_Header 'pftext' => $pftext, 'confirm' => $GLOBALS['cfg']['Confirm'], 'LoginCookieValidity' => $GLOBALS['cfg']['LoginCookieValidity'], - 'logged_in' => isset($GLOBALS['userlink']) ? true : false + 'logged_in' => isset($GLOBALS['userlink']) ? true : false, + 'default_fk_check_value' => PMA_Util::getDefaultFKCheckValue() ? 1 : 0 ); if (isset($GLOBALS['cfg']['Server']) && isset($GLOBALS['cfg']['Server']['auth_type']) diff --git a/libraries/Util.class.php b/libraries/Util.class.php index fced591974..d23e8c0785 100644 --- a/libraries/Util.class.php +++ b/libraries/Util.class.php @@ -1291,11 +1291,6 @@ class PMA_Util 'profiling', __('Profiling'), isset($_SESSION['profiling']), true ); } - - // Pass default foreign_key_checks - $default_fk_check_value = $GLOBALS['dbi']->getVariable('FOREIGN_KEY_CHECKS') == 'ON'; - $retval .= ''; $retval .= ''; /** @@ -3206,6 +3201,15 @@ class PMA_Util } } + public static function getDefaultFKCheckValue() { + if ($GLOBALS['cfg']['DefaultForeignKeyChecks'] === 'enable') { + return true; + } else if ($GLOBALS['cfg']['DefaultForeignKeyChecks'] === 'disable') { + return false; + } + return ($GLOBALS['dbi']->getVariable('FOREIGN_KEY_CHECKS') == 'ON'); + } + /** * Get HTML for Foreign key check checkbox * @@ -3213,13 +3217,7 @@ class PMA_Util */ public static function getFKCheckbox() { - if ($GLOBALS['cfg']['DefaultForeignKeyChecks'] === 'enable') { - $checked = true; - } else if ($GLOBALS['cfg']['DefaultForeignKeyChecks'] === 'disable') { - $checked = false; - } else { - $checked = $GLOBALS['dbi']->getVariable('FOREIGN_KEY_CHECKS') == 'ON'; - } + $checked = self::getDefaultFKCheckValue(); $html = ''; $html .= ' Date: Fri, 12 Jun 2015 15:10:49 +0530 Subject: [PATCH 2/8] Fix tests Signed-off-by: Nisarg Jhaveri --- libraries/dbi/DBIDummy.class.php | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/libraries/dbi/DBIDummy.class.php b/libraries/dbi/DBIDummy.class.php index 5876979806..f179dee016 100644 --- a/libraries/dbi/DBIDummy.class.php +++ b/libraries/dbi/DBIDummy.class.php @@ -125,6 +125,10 @@ $GLOBALS['dummy_queries'] = array( 'query' => 'SHOW VARIABLES LIKE \'language\';', 'result' => array(), ), + array( + 'query' => 'SHOW SESSION VARIABLES LIKE \'FOREIGN_KEY_CHECKS\';', + 'result' => array('foreign_key_checks', 'ON'), + ), array( 'query' => 'SHOW TABLES FROM `pma_test`;', 'result' => array( From 0491bb8e410d5a4f26c0cb83be48fbdac98f2e5c Mon Sep 17 00:00:00 2001 From: Kurt Eilertsen Date: Fri, 12 Jun 2015 17:02:35 +0200 Subject: [PATCH 3/8] =?UTF-8?q?Translated=20using=20Weblate=20(Norwegian?= =?UTF-8?q?=20Bokm=C3=A5l)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Currently translated at 60.7% (1884 of 3102 strings) [CI skip] --- po/nb.po | 79 +++++++++++++++++++++----------------------------------- 1 file changed, 29 insertions(+), 50 deletions(-) diff --git a/po/nb.po b/po/nb.po index a237ed8484..8bb0c6a9cd 100644 --- a/po/nb.po +++ b/po/nb.po @@ -4,10 +4,10 @@ msgstr "" "Project-Id-Version: phpMyAdmin 4.5.0-dev\n" "Report-Msgid-Bugs-To: phpmyadmin-devel@lists.sourceforge.net\n" "POT-Creation-Date: 2015-06-11 09:31-0400\n" -"PO-Revision-Date: 2015-06-09 13:19+0200\n" -"Last-Translator: Børge Holm-Wennberg \n" -"Language-Team: Norwegian Bokmål \n" +"PO-Revision-Date: 2015-06-12 17:02+0200\n" +"Last-Translator: Kurt Eilertsen \n" +"Language-Team: Norwegian Bokmål " +"\n" "Language: nb\n" "MIME-Version: 1.0\n" "Content-Type: text/plain; charset=UTF-8\n" @@ -2986,10 +2986,9 @@ msgid "Expand" msgstr "" #: libraries/Console.class.php:87 libraries/Console.class.php:213 -#, fuzzy #| msgid "in query" msgid "Requery" -msgstr "i spørring" +msgstr "Spør igjen" #: libraries/Console.class.php:89 libraries/DbSearch.class.php:365 #: libraries/DisplayResults.class.php:3521 @@ -3021,12 +3020,12 @@ msgid "Database" msgstr "Database" #: libraries/Console.class.php:100 -#, fuzzy, php-format +#, php-format #| msgid "Delete relation" msgid "Total %d bookmark" msgid_plural "Total %d bookmarks" -msgstr[0] "Slett relasjon" -msgstr[1] "Slett relasjon" +msgstr[0] "Totalt %d bokmerke" +msgstr[1] "Totalt %d bokmerker" #: libraries/Console.class.php:108 msgid "private" @@ -3038,28 +3037,25 @@ msgstr "delt" #. l10n: First parameter will be replaced with the translation for Total and the number of bookmarks, second one with the translation for private and the third one, with the translation for shared #: libraries/Console.class.php:116 -#, fuzzy, php-format +#, php-format #| msgid "The bookmark has been deleted." msgid "%1$s, %2$s and %3$s bookmarks included" -msgstr "Bokmerket er slettet." +msgstr "%1$s, %2$s og %3$s bokmerker er inkludert" #: libraries/Console.class.php:122 -#, fuzzy #| msgid "Delete relation" msgid "No bookmarks" -msgstr "Slett relasjon" +msgstr "Ingen bokmerker" #: libraries/Console.class.php:183 -#, fuzzy #| msgid "Skip current error" msgid "During current session" -msgstr "Hopp over nåværende feil" +msgstr "Gjennom gjeldende sesjon" #: libraries/Console.class.php:215 -#, fuzzy #| msgid "Explain SQL" msgid "Explain" -msgstr "Forklar SQL" +msgstr "Forklar" #: libraries/Console.class.php:216 libraries/Util.class.php:1291 #: libraries/sql.lib.php:278 @@ -3067,28 +3063,24 @@ msgid "Profiling" msgstr "Profilering" #: libraries/Console.class.php:218 -#, fuzzy #| msgid "Bookmark table" msgid "Bookmark" -msgstr "Bokmerketabell" +msgstr "Bokmerke" #: libraries/Console.class.php:219 -#, fuzzy #| msgid "Query cache" msgid "Query failed" -msgstr "Spørringsmellomlager" +msgstr "Spørring feilet" #: libraries/Console.class.php:223 -#, fuzzy #| msgid "Query execution time" msgid "Queried time" -msgstr "Spørringens utførelsestid" +msgstr "Spørringens tidsforbruk" #: libraries/Console.class.php:230 -#, fuzzy #| msgid "SQL Query box" msgid "SQL Query Console" -msgstr "SQL spørringsboks" +msgstr "SQL spørrings konsoll" #: libraries/Console.class.php:231 msgid "Console" @@ -3100,10 +3092,9 @@ msgid "Clear" msgstr "Fjern" #: libraries/Console.class.php:237 -#, fuzzy #| msgid "SQL history" msgid "History" -msgstr "SQL-historie" +msgstr "Historikk" #: libraries/Console.class.php:240 libraries/Console.class.php:313 #: libraries/DisplayResults.class.php:1726 libraries/import.lib.php:1262 @@ -3135,15 +3126,13 @@ msgid "Options" msgstr "Innstillinger" #: libraries/Console.class.php:244 libraries/Console.class.php:275 -#, fuzzy #| msgid "Bookmark table" msgid "Bookmarks" -msgstr "Bokmerketabell" +msgstr "Bokmerker" #: libraries/Console.class.php:254 -#, fuzzy msgid "Press Ctrl+Enter to execute query" -msgstr "SQL spørringer" +msgstr "Tast Ctrl+Enter for å utføre spørring" #: libraries/Console.class.php:256 #, fuzzy @@ -3158,48 +3147,42 @@ msgid "Refresh" msgstr "Oppdater" #: libraries/Console.class.php:282 -#, fuzzy #| msgid "And" msgid "Add" -msgstr "og" +msgstr "Legg til" #: libraries/Console.class.php:291 -#, fuzzy #| msgid "Delete relation" msgid "Add bookmark" -msgstr "Slett relasjon" +msgstr "Legg til bokmerke" #: libraries/Console.class.php:295 -#, fuzzy #| msgid "Label" msgid "Label" -msgstr "Navn" +msgstr "Etikett" #: libraries/Console.class.php:297 msgid "Target database" msgstr "Måldatabase" #: libraries/Console.class.php:300 -#, fuzzy #| msgid "Delete relation" msgid "Share this bookmark" -msgstr "Slett relasjon" +msgstr "Del dette bokmerke" #: libraries/Console.class.php:317 -#, fuzzy #| msgid "Restore default value" msgid "Set default" -msgstr "Gjennopprett standard verdi" +msgstr "Angi standard" #: libraries/Console.class.php:321 msgid "Always expand query messages" msgstr "" #: libraries/Console.class.php:323 -#, fuzzy #| msgid "SQL query history table" msgid "Show query history at start" -msgstr "SQL spørringshistorietabell" +msgstr "Vis spørringshistorikk ved oppstart" #: libraries/Console.class.php:325 msgid "Show current browsing query" @@ -3510,7 +3493,6 @@ msgid "Show BLOB contents" msgstr "Vis BLOB innhold" #: libraries/DisplayResults.class.php:1781 -#, fuzzy #| msgid "Browser transformation" msgid "Hide browser transformation" msgstr "Skjul nettlesertransformasjon" @@ -3616,16 +3598,14 @@ msgid "Too many error messages, some are not displayed." msgstr "For mange feilmeldinger, noen vises ikke." #: libraries/Error_Handler.class.php:333 -#, fuzzy #| msgid "Import" msgid "Report" -msgstr "Importer" +msgstr "Rapporter" #: libraries/Error_Handler.class.php:338 libraries/error_report.lib.php:373 -#, fuzzy #| msgid "Automatically create versions" msgid "Automatically send report next time" -msgstr "Automatisk opprette versjoner" +msgstr "Automatisk send rapport neste gang" #: libraries/File.class.php:225 msgid "File was not an uploaded file." @@ -3901,10 +3881,9 @@ msgstr "Designer" #: libraries/Menu.class.php:500 libraries/Util.class.php:4312 #: libraries/structure.lib.php:362 -#, fuzzy #| msgid "CHAR textarea columns" msgid "Central columns" -msgstr "CHAR textarea kolonner" +msgstr "Senter kolonner" #: libraries/Menu.class.php:537 libraries/Util.class.php:4284 #: libraries/config/messages.inc.php:243 libraries/config/messages.inc.php:316 From edad7fcb3ccf351b8a8aa4d6987f1cf8232aa5c9 Mon Sep 17 00:00:00 2001 From: Deven Bansod Date: Tue, 9 Jun 2015 23:08:11 +0530 Subject: [PATCH 4/8] Replace script names as config options with plain english words Signed-off-by: Deven Bansod --- doc/config.rst | 55 +++++----- index.php | 8 +- libraries/Header.class.php | 4 +- libraries/Menu.class.php | 12 ++- libraries/Util.class.php | 102 +++++++++++++++++- libraries/config.default.php | 56 +++++----- libraries/config.values.php | 48 ++++----- libraries/db_common.inc.php | 5 +- .../navigation/Nodes/Node_Database.class.php | 6 +- .../navigation/Nodes/Node_Table.class.php | 47 ++++---- .../auth/AuthenticationConfig.class.php | 4 +- libraries/select_server.lib.php | 9 +- libraries/server_privileges.lib.php | 16 ++- libraries/special_schema_links.lib.php | 60 ++++++++--- libraries/sql_query_form.lib.php | 8 +- libraries/tbl_common.inc.php | 10 +- sql.php | 8 +- tbl_chart.php | 12 ++- tbl_find_replace.php | 4 +- tbl_gis_visualization.php | 4 +- tbl_select.php | 4 +- tbl_zoom_select.php | 4 +- test/classes/PMA_Footer_test.php | 2 +- test/classes/PMA_Menu_test.php | 6 +- test/classes/dbi/DBIDrizzle_test.php | 2 +- .../PMA_Node_DatabaseChild_test.php | 2 +- .../navigation/PMA_Node_Database_test.php | 2 +- .../navigation/PMA_Node_Table_test.php | 13 +-- .../auth/PMA_AuthenticationConfig_test.php | 1 + test/libraries/PMA_build_html_for_db_test.php | 2 +- test/libraries/PMA_select_server_test.php | 6 +- test/libraries/PMA_server_databases_test.php | 2 +- test/libraries/PMA_server_privileges_test.php | 14 ++- test/libraries/PMA_sql_query_form_test.php | 2 +- test/libraries/common/PMA_getDbLink_test.php | 17 ++- test/theme.php | 12 ++- 36 files changed, 393 insertions(+), 176 deletions(-) diff --git a/doc/config.rst b/doc/config.rst index 2537981489..b65799bb01 100644 --- a/doc/config.rst +++ b/doc/config.rst @@ -1598,16 +1598,16 @@ Navigation panel setup .. config:option:: $cfg['NavigationTreeDefaultTabTable'] :type: string - :default: ``'tbl_structure.php'`` + :default: ``'structure'`` Defines the tab displayed by default when clicking the small icon next to each table name in the navigation panel. Possible values: - * ``tbl_structure.php`` - * ``tbl_sql.php`` - * ``tbl_select.php`` - * ``tbl_change.php`` - * ``sql.php`` + * ``structure`` + * ``sql`` + * ``search`` + * ``insert`` + * ``browse`` .. config:option:: $cfg['NavigationTreeDefaultTabTable2'] @@ -1618,11 +1618,11 @@ Navigation panel setup to each table name in the navigation panel. Possible values: * ``(empty)`` - * ``tbl_structure.php`` - * ``tbl_sql.php`` - * ``tbl_select.php`` - * ``tbl_change.php`` - * ``sql.php`` + * ``structure`` + * ``sql`` + * ``search`` + * ``insert`` + * ``browse`` .. config:option:: $cfg['NavigationTreeEnableExpansion'] @@ -1957,40 +1957,41 @@ Tabs display settings .. config:option:: $cfg['DefaultTabServer'] :type: string - :default: ``'index.php'`` + :default: ``'welcome'`` Defines the tab displayed by default on server view. Possible values: - * ``main.php`` (recommended for multi-user setups) - * ``server_databases.php``, - * ``server_status.php`` - * ``server_variables.php`` - * ``server_privileges.php`` + * ``welcome`` (recommended for multi-user setups) + * ``databases``, + * ``status`` + * ``variables`` + * ``privileges`` .. config:option:: $cfg['DefaultTabDatabase'] :type: string - :default: ``'db_structure.php'`` + :default: ``'structure'`` Defines the tab displayed by default on database view. Possible values: - * ``db_structure.php`` - * ``db_sql.php`` - * ``db_search.php``. + * ``structure`` + * ``sql`` + * ``search`` + * ``operations`` .. config:option:: $cfg['DefaultTabTable'] :type: string - :default: ``'sql.php'`` + :default: ``'browse'`` Defines the tab displayed by default on table view. Possible values: - * ``tbl_structure.php`` - * ``tbl_sql.php`` - * ``tbl_select.php`` - * ``tbl_change.php`` - * ``sql.php`` + * ``structure`` + * ``sql`` + * ``search`` + * ``insert`` + * ``browse`` PDF Options ----------- diff --git a/index.php b/index.php index bb4e12742a..5d27ef28c6 100644 --- a/index.php +++ b/index.php @@ -61,9 +61,13 @@ if (isset($_REQUEST['ajax_request']) && ! empty($_REQUEST['access_time'])) { if (! empty($_REQUEST['db'])) { $page = null; if (! empty($_REQUEST['table'])) { - $page = $GLOBALS['cfg']['DefaultTabTable']; + $page = PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabTable'], 'table' + ); } else { - $page = $GLOBALS['cfg']['DefaultTabDatabase']; + $page = PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabDatabase'], 'database' + ); } include $page; exit; diff --git a/libraries/Header.class.php b/libraries/Header.class.php index 0a3ac7dc75..5d3b29d576 100644 --- a/libraries/Header.class.php +++ b/libraries/Header.class.php @@ -230,7 +230,9 @@ class PMA_Header $params = array( 'common_query' => PMA_URL_getCommon(array(), 'text'), - 'opendb_url' => $GLOBALS['cfg']['DefaultTabDatabase'], + 'opendb_url' => PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabDatabase'], 'database' + ), 'safari_browser' => PMA_USR_BROWSER_AGENT == 'SAFARI' ? 1 : 0, 'collation_connection' => $GLOBALS['collation_connection'], 'lang' => $GLOBALS['lang'], diff --git a/libraries/Menu.class.php b/libraries/Menu.class.php index 6a0f978713..e33f0ee348 100644 --- a/libraries/Menu.class.php +++ b/libraries/Menu.class.php @@ -191,7 +191,9 @@ class PMA_Menu } $retval .= sprintf( $item, - $GLOBALS['cfg']['DefaultTabServer'], + PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabServer'], 'server' + ), PMA_URL_getCommon(), htmlspecialchars($server_info), __('Server') @@ -208,7 +210,9 @@ class PMA_Menu } $retval .= sprintf( $item, - $GLOBALS['cfg']['DefaultTabDatabase'], + PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabDatabase'], 'database' + ), PMA_URL_getCommon(array('db' => $this->_db)), htmlspecialchars($this->_db), __('Database') @@ -231,7 +235,9 @@ class PMA_Menu } $retval .= sprintf( $item, - $GLOBALS['cfg']['DefaultTabTable'], + PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabTable'], 'table' + ), PMA_URL_getCommon( array( 'db' => $this->_db, 'table' => $this->_table diff --git a/libraries/Util.class.php b/libraries/Util.class.php index 4e94fc5e78..1bd3814842 100644 --- a/libraries/Util.class.php +++ b/libraries/Util.class.php @@ -2657,7 +2657,10 @@ class PMA_Util $database = self::unescapeMysqlWildcards($database); } - return ' $database)) . '" title="' . htmlspecialchars( sprintf( @@ -3333,18 +3336,26 @@ class PMA_Util * $cfg['NavigationTreeDefaultTabTable2'], * $cfg['DefaultTabTable'] or $cfg['DefaultTabDatabase'] * - * @return array + * @return string Title for the $cfg value */ public static function getTitleForTarget($target) { $mapping = array( + 'structure' => __('Structure'), + 'sql' => __('SQL'), + 'search' =>__('Search'), + 'insert' =>__('Insert'), + 'browse' => __('Browse'), + 'operations' => __('Operations'), + + // For backward compatiblity + // Values for $cfg['DefaultTabTable'] 'tbl_structure.php' => __('Structure'), 'tbl_sql.php' => __('SQL'), 'tbl_select.php' =>__('Search'), 'tbl_change.php' =>__('Insert'), 'sql.php' => __('Browse'), - // Values for $cfg['DefaultTabDatabase'] 'db_structure.php' => __('Structure'), 'db_sql.php' => __('SQL'), @@ -3354,6 +3365,91 @@ class PMA_Util return isset($mapping[$target]) ? $mapping[$target] : false; } + /** + * Get the script name corresponding to a plain English config word + * in order to append in links on navigation and main panel + * + * @param string $target a valid value for $cfg['NavigationTreeDefaultTabTable'], + * $cfg['NavigationTreeDefaultTabTable2'], + * $cfg['DefaultTabTable'], $cfg['DefaultTabDatabase'] or + * $cfg['DefaultTabServer'] + * @param string $location one out of 'server', 'table', 'database' + * + * @return string script name corresponding to the config word + */ + public static function getScriptNameForOption($target, $location) + { + $retval = ''; + + if ($location == 'server') { + // Values for $cfg['DefaultTabServer'] + switch ($target) { + case 'welcome': + $retval = 'index.php'; + break; + case 'databases': + $retval = 'server_databases.php'; + break; + case 'status': + $retval = 'server_status.php'; + break; + case 'variables': + $retval = 'server_variables.php'; + break; + case 'privileges': + $retval = 'server_privileges.php'; + break; + default: + $retval = $target; + } + } elseif ($location == 'database') { + // Values for $cfg['DefaultTabDatabase'] + switch ($target) { + case 'structure': + $retval = 'db_structure.php'; + break; + case 'sql': + $retval = 'db_sql.php'; + break; + case 'search': + $retval = 'db_search.php'; + break; + case 'operations': + $retval = 'db_operations.php'; + break; + default: + $retval = $target; + } + } elseif ($location == 'table') { + // Values for $cfg['DefaultTabTable'], + // $cfg['NavigationTreeDefaultTabTable'] and + // $cfg['NavigationTreeDefaultTabTable2'] + switch ($target) { + case 'structure': + $retval = 'tbl_structure.php'; + break; + case 'sql': + $retval = 'tbl_sql.php'; + break; + case 'search': + $retval = 'tbl_select.php'; + break; + case 'insert': + $retval = 'tbl_change.php'; + break; + case 'browse': + $retval = 'sql.php'; + break; + default: + $retval = $target; + } + } else { + $retval = $target; + } + + return $retval; + } + /** * Formats user string, expanding @VARIABLES@, accepting strftime format * string. diff --git a/libraries/config.default.php b/libraries/config.default.php index 648a4b7132..06cb63ebad 100644 --- a/libraries/config.default.php +++ b/libraries/config.default.php @@ -958,25 +958,25 @@ $cfg['NavigationTreeDisplayDbFilterMinimum'] = 30; * target of the navigation panel quick access icon * * Possible values: - * 'tbl_structure.php' = fields list - * 'tbl_sql.php' = SQL form - * 'tbl_select.php' = search page - * 'tbl_change.php' = insert row page - * 'sql.php' = browse page + * 'structure.php' = fields list + * 'sql' = SQL form + * 'search' = search page + * 'insert' = insert row page + * 'browse' = browse page * * @global string $cfg['NavigationTreeDefaultTabTable'] */ -$cfg['NavigationTreeDefaultTabTable'] = 'tbl_structure.php'; +$cfg['NavigationTreeDefaultTabTable'] = 'structure'; /** * target of the navigation panel quick second access icon * * Possible values: - * 'tbl_structure.php' = fields list - * 'tbl_sql.php' = SQL form - * 'tbl_select.php' = search page - * 'tbl_change.php' = insert row page - * 'sql.php' = browse page + * 'structure' = fields list + * 'sql' = SQL form + * 'search' = search page + * 'insert' = insert row page + * 'browse' = browse page * '' = no link * * @global string $cfg['NavigationTreeDefaultTabTable2'] @@ -1285,39 +1285,39 @@ $cfg['PropertiesNumColumns'] = 1; /** * Possible values: - * 'index.php' = the welcome page + * 'welcome' = the welcome page * (recommended for multiuser setups) - * 'server_databases.php' = list of databases - * 'server_status.php' = runtime information - * 'server_variables.php' = MySQL server variables - * 'server_privileges.php' = user management + * 'databases' = list of databases + * 'status' = runtime information + * 'variables' = MySQL server variables + * 'privileges' = user management * * @global string $cfg['DefaultTabServer'] */ -$cfg['DefaultTabServer'] = 'index.php'; +$cfg['DefaultTabServer'] = 'welcome'; /** * Possible values: - * 'db_structure.php' = tables list - * 'db_sql.php' = SQL form - * 'db_search.php' = search query - * 'db_operations.php' = operations on database + * 'structure' = tables list + * 'sql' = SQL form + * 'search' = search query + * 'operations' = operations on database * * @global string $cfg['DefaultTabDatabase'] */ -$cfg['DefaultTabDatabase'] = 'db_structure.php'; +$cfg['DefaultTabDatabase'] = 'structure'; /** * Possible values: - * 'tbl_structure.php' = fields list - * 'tbl_sql.php' = SQL form - * 'tbl_select.php' = search page - * 'tbl_change.php' = insert row page - * 'sql.php' = browse page + * 'structure' = fields list + * 'sql' = SQL form + * 'search' = search page + * 'insert' = insert row page + * 'browse' = browse page * * @global string $cfg['DefaultTabTable'] */ -$cfg['DefaultTabTable'] = 'sql.php'; +$cfg['DefaultTabTable'] = 'browse'; /** * Whether to display image or text or both image and text in table row diff --git a/libraries/config.values.php b/libraries/config.values.php index ca811814b0..78e3880386 100644 --- a/libraries/config.values.php +++ b/libraries/config.values.php @@ -36,19 +36,19 @@ $cfg_db['OBGzip'] = array('auto', true, false); $cfg_db['MemoryLimit'] = 'short_string'; $cfg_db['NavigationLogoLinkWindow'] = array('main', 'new'); $cfg_db['NavigationTreeDefaultTabTable'] = array( - 'tbl_structure.php', // fields list - 'tbl_sql.php', // SQL form - 'tbl_select.php', // search page - 'tbl_change.php', // insert row page - 'sql.php' // browse page + 'structure' => __('Structure'), // fields list + 'sql' => __('SQL'), // SQL form + 'search' => __('Search'), // search page + 'insert' => __('Insert'), // insert row page + 'browse' => __('Browse') // browse page ); $cfg_db['NavigationTreeDefaultTabTable2'] = array( '', //don't display - 'tbl_structure.php', // fields list - 'tbl_sql.php', // SQL form - 'tbl_select.php', // search page - 'tbl_change.php', // insert row page - 'sql.php' // browse page + 'structure' => __('Structure'), // fields list + 'sql' => __('SQL'), // SQL form + 'search' => __('Search'), // search page + 'insert' => __('Insert'), // insert row page + 'browse' => __('Browse') // browse page ); $cfg_db['NavigationTreeDbSeparator'] = 'short_string'; $cfg_db['NavigationTreeTableSeparator'] = 'short_string'; @@ -99,24 +99,24 @@ $cfg_db['RelationalDisplay'] = array( 'D' => __('display column') ); $cfg_db['DefaultTabServer'] = array( - 'index.php', // the welcome page (recommended for multiuser setups) - 'server_databases.php', // list of databases - 'server_status.php', // runtime information - 'server_variables.php', // MySQL server variables - 'server_privileges.php' // user management + 'welcome' => __('Welcome'), // the welcome page (recommended for multiuser setups) + 'databases' => __('Databases'), // list of databases + 'status' => __('Status'), // runtime information + 'variables' => __('Variables'), // MySQL server variables + 'privileges' => __('Privileges') // user management ); $cfg_db['DefaultTabDatabase'] = array( - 'db_structure.php', // tables list - 'db_sql.php', // SQL form - 'db_search.php', // search query - 'db_operations.php' // operations on database + 'structure' => __('Structure'), // tables list + 'sql' => __('SQL'), // SQL form + 'search' => __('Search'), // search query + 'operations' => __('Operations') // operations on database ); $cfg_db['DefaultTabTable'] = array( - 'tbl_structure.php', // fields list - 'tbl_sql.php', // SQL form - 'tbl_select.php', // search page - 'tbl_change.php', // insert row page - 'sql.php' // browse page + 'structure' => __('Structure'), // fields list + 'sql' => __('SQL'), // SQL form + 'search' => __('Search'), // search page + 'insert' => __('Insert'), // insert row page + 'browse' => __('Browse') // browse page ); $cfg_db['InitialSlidersState'] = array( 'open' => __('Open'), diff --git a/libraries/db_common.inc.php b/libraries/db_common.inc.php index ac9c793466..e555784165 100644 --- a/libraries/db_common.inc.php +++ b/libraries/db_common.inc.php @@ -27,7 +27,10 @@ if ($db_is_system_schema) { * Defines the urls to return to in case of error in a sql statement */ $err_url_0 = 'index.php' . PMA_URL_getCommon(); -$err_url = $cfg['DefaultTabDatabase'] + +$err_url = PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabDatabase'], 'database' +) . PMA_URL_getCommon(array('db' => $db)); /** @var PMA_String $pmaString */ diff --git a/libraries/navigation/Nodes/Node_Database.class.php b/libraries/navigation/Nodes/Node_Database.class.php index 2db8896ada..261ae3d0a6 100644 --- a/libraries/navigation/Nodes/Node_Database.class.php +++ b/libraries/navigation/Nodes/Node_Database.class.php @@ -40,8 +40,12 @@ class Node_Database extends Node 's_db.png', __('Database operations') ); + + $script_name = PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabDatabase'], 'database' + ); $this->links = array( - 'text' => $GLOBALS['cfg']['DefaultTabDatabase'] + 'text' => $script_name . '?server=' . $GLOBALS['server'] . '&db=%1$s&token=' . $_SESSION[' PMA_token '], 'icon' => 'db_operations.php?server=' . $GLOBALS['server'] diff --git a/libraries/navigation/Nodes/Node_Table.class.php b/libraries/navigation/Nodes/Node_Table.class.php index 10a521a52c..2d01059843 100644 --- a/libraries/navigation/Nodes/Node_Table.class.php +++ b/libraries/navigation/Nodes/Node_Table.class.php @@ -32,36 +32,39 @@ class Node_Table extends Node_DatabaseChild { parent::__construct($name, $type, $is_group); $this->icon = array(); - $this->_addIcon($GLOBALS['cfg']['NavigationTreeDefaultTabTable']); - $this->_addIcon($GLOBALS['cfg']['NavigationTreeDefaultTabTable2']); - switch($GLOBALS['cfg']['DefaultTabTable']) { - case 'tbl_structure.php': - $this->title = __('Structure'); - break; - case 'tbl_select.php': - $this->title = __('Search'); - break; - case 'tbl_change.php': - $this->title = __('Insert'); - break; - case 'tbl_sql.php': - $this->title = __('SQL'); - break; - case 'sql.php': - $this->title = __('Browse'); - break; - } + $this->_addIcon( + PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['NavigationTreeDefaultTabTable'], 'table' + ) + ); + $this->_addIcon( + PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['NavigationTreeDefaultTabTable2'], 'table' + ) + ); + $title = PMA_Util::getTitleForTarget( + $GLOBALS['cfg']['DefaultTabTable'] + ); + $this->title = $title; + + $script_name = PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabTable'], 'table' + ); $this->links = array( - 'text' => $GLOBALS['cfg']['DefaultTabTable'] + 'text' => $script_name . '?server=' . $GLOBALS['server'] . '&db=%2$s&table=%1$s' . '&pos=0&token=' . $_SESSION[' PMA_token '], 'icon' => array( - $GLOBALS['cfg']['NavigationTreeDefaultTabTable'] + PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['NavigationTreeDefaultTabTable'], 'table' + ) . '?server=' . $GLOBALS['server'] . '&db=%2$s&table=%1$s&token=' . $_SESSION[' PMA_token '], - $GLOBALS['cfg']['NavigationTreeDefaultTabTable2'] + PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['NavigationTreeDefaultTabTable2'], 'table' + ) . '?server=' . $GLOBALS['server'] . '&db=%2$s&table=%1$s&token=' . $_SESSION[' PMA_token '] diff --git a/libraries/plugins/auth/AuthenticationConfig.class.php b/libraries/plugins/auth/AuthenticationConfig.class.php index 19a8ed2a16..f4ffd6e3d8 100644 --- a/libraries/plugins/auth/AuthenticationConfig.class.php +++ b/libraries/plugins/auth/AuthenticationConfig.class.php @@ -159,7 +159,9 @@ class AuthenticationConfig extends AuthenticationPlugin ' . "\n"; echo '' . __('Retry to connect') . '' . "\n"; diff --git a/libraries/select_server.lib.php b/libraries/select_server.lib.php index db06332f36..1fe4d691f5 100644 --- a/libraries/select_server.lib.php +++ b/libraries/select_server.lib.php @@ -31,7 +31,10 @@ function PMA_selectServer($not_only_options, $omit_fieldset) if ($not_only_options) { $retval .= '
'; + . PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabServer'], 'server' + ) + . '" class="disableAjax">'; $retval .= PMA_getHiddenFields(array('token' => $_SESSION[' PMA_token '])); if (! $omit_fieldset) { @@ -84,7 +87,9 @@ function PMA_selectServer($not_only_options, $omit_fieldset) } else { $retval .= '' . htmlspecialchars($label) . ''; } diff --git a/libraries/server_privileges.lib.php b/libraries/server_privileges.lib.php index c6616df946..3019e9309b 100644 --- a/libraries/server_privileges.lib.php +++ b/libraries/server_privileges.lib.php @@ -2060,7 +2060,9 @@ function PMA_getHtmlForSpecificDbPrivileges($db) . ' ' . sprintf( __('Users having access to "%s"'), - ' $db)) . '">' . htmlspecialchars($db) . '' @@ -2124,7 +2126,9 @@ function PMA_getHtmlForSpecificTablePrivileges($db, $table) . PMA_Util::getIcon('b_usrcheck.png') . sprintf( __('Users having access to "%s"'), - ' $db, @@ -2757,7 +2761,9 @@ function PMA_getChangeLoginInformationHtmlForm($username, $hostname) function PMA_getLinkToDbAndTable($url_dbname, $dbname, $tablename) { $html_output = '[ ' . __('Database') - . ' $url_dbname, @@ -2773,7 +2779,9 @@ function PMA_getLinkToDbAndTable($url_dbname, $dbname, $tablename) if (/*overload*/mb_strlen($tablename)) { $html_output .= ' [ ' . __('Table') . ' $db)) . '"'; $tmp_db_link .= '>' . htmlspecialchars($db) . ''; @@ -182,7 +184,9 @@ function PMA_getHtmlForSqlQueryFormInsert( $db, $GLOBALS['table'], null, true ); - $tmp_db_link = ' $db)) . '"'; $tmp_db_link .= '>' . htmlspecialchars($db) . ''; diff --git a/libraries/tbl_common.inc.php b/libraries/tbl_common.inc.php index a2e7b260b2..850feacedd 100644 --- a/libraries/tbl_common.inc.php +++ b/libraries/tbl_common.inc.php @@ -35,9 +35,15 @@ $url_params['table'] = $table; /** * Defines the urls to return to in case of error in a sql statement */ -$err_url_0 = $cfg['DefaultTabDatabase'] +$err_url_0 = PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabDatabase'], 'database' +) . PMA_URL_getCommon(array('db' => $db)); -$err_url = $cfg['DefaultTabTable'] . PMA_URL_getCommon($url_params); + +$err_url = PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabTable'], 'table' +) + . PMA_URL_getCommon($url_params); /** diff --git a/sql.php b/sql.php index 6b972d4124..9c2e790552 100644 --- a/sql.php +++ b/sql.php @@ -54,9 +54,13 @@ if (! empty($goto)) { } } else { if (empty($table)) { - $goto = $cfg['DefaultTabDatabase']; + $goto = PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabDatabase'], 'database' + ); } else { - $goto = $cfg['DefaultTabTable']; + $goto = PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabTable'], 'table' + ); } $is_gotofile = true; } // end if diff --git a/tbl_chart.php b/tbl_chart.php index aedeea0b82..5fff8f5236 100644 --- a/tbl_chart.php +++ b/tbl_chart.php @@ -80,17 +80,23 @@ $scripts->addFile('jqplot/plugins/jqplot.highlighter.js'); * Runs common work */ if (/*overload*/mb_strlen($GLOBALS['table'])) { - $url_params['goto'] = $cfg['DefaultTabTable']; + $url_params['goto'] = PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabTable'], 'table' + ); $url_params['back'] = 'tbl_sql.php'; include 'libraries/tbl_common.inc.php'; include 'libraries/tbl_info.inc.php'; } elseif (/*overload*/mb_strlen($GLOBALS['db'])) { - $url_params['goto'] = $cfg['DefaultTabDatabase']; + $url_params['goto'] = PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabDatabase'], 'database' + ); $url_params['back'] = 'sql.php'; include 'libraries/db_common.inc.php'; include 'libraries/db_info.inc.php'; } else { - $url_params['goto'] = $cfg['DefaultTabServer']; + $url_params['goto'] = PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabServer'], 'server' + ); $url_params['back'] = 'sql.php'; include 'libraries/server_common.inc.php'; } diff --git a/tbl_find_replace.php b/tbl_find_replace.php index 39eefa48bb..ddc0cea937 100644 --- a/tbl_find_replace.php +++ b/tbl_find_replace.php @@ -54,7 +54,9 @@ if (isset($_POST['replace'])) { } if (! isset($goto)) { - $goto = $GLOBALS['cfg']['DefaultTabTable']; + $goto = PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabTable'], 'table' + ); } // Defines the url to return to in case of error in the next sql statement $params = array('db' => $db, 'table' => $table); diff --git a/tbl_gis_visualization.php b/tbl_gis_visualization.php index 4be47b8391..cecabe35d2 100644 --- a/tbl_gis_visualization.php +++ b/tbl_gis_visualization.php @@ -10,7 +10,9 @@ require_once 'libraries/common.inc.php'; // Runs common work require_once 'libraries/db_common.inc.php'; -$url_params['goto'] = $cfg['DefaultTabDatabase']; +$url_params['goto'] = PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabDatabase'], 'database' +); $url_params['back'] = 'sql.php'; // Import visualization functions diff --git a/tbl_select.php b/tbl_select.php index 5366250331..914b547157 100644 --- a/tbl_select.php +++ b/tbl_select.php @@ -53,7 +53,9 @@ if (! isset($_POST['columnsToDisplay']) && ! isset($_POST['displayAllColumns'])) include_once 'libraries/tbl_info.inc.php'; if (! isset($goto)) { - $goto = $GLOBALS['cfg']['DefaultTabTable']; + $goto = PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabTable'], 'table' + ); } // Defines the url to return to in case of error in the next sql statement $err_url = $goto . PMA_URL_getCommon(array('db' => $db, 'table' => $table)); diff --git a/tbl_zoom_select.php b/tbl_zoom_select.php index 2f0e738abd..3efcfb574e 100644 --- a/tbl_zoom_select.php +++ b/tbl_zoom_select.php @@ -97,7 +97,9 @@ $url_query .= '&goto=tbl_select.php&back=tbl_select.php'; require_once './libraries/tbl_info.inc.php'; if (! isset($goto)) { - $goto = $GLOBALS['cfg']['DefaultTabTable']; + $goto = PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabTable'], 'table' + ); } // Defines the url to return to in case of error in the next sql statement $err_url = $goto . PMA_URL_getCommon(array('db' => $db, 'table' => $table)); diff --git a/test/classes/PMA_Footer_test.php b/test/classes/PMA_Footer_test.php index 85aece7d00..47968d9d1b 100644 --- a/test/classes/PMA_Footer_test.php +++ b/test/classes/PMA_Footer_test.php @@ -60,7 +60,7 @@ class PMA_Footer_Test extends PHPUnit_Framework_TestCase $GLOBALS['PMA_Config']->enableBc(); $GLOBALS['collation_connection'] = 'utf8_general_ci'; $GLOBALS['cfg']['Server']['verbose'] = 'verbose host'; - $GLOBALS['cfg']['DefaultTabDatabase'] = 'db_structure.php'; + $GLOBALS['cfg']['DefaultTabDatabase'] = 'structure'; $GLOBALS['server'] = '1'; $_GET['reload_left_frame'] = '1'; $GLOBALS['focus_querywindow'] = 'main_pane_left'; diff --git a/test/classes/PMA_Menu_test.php b/test/classes/PMA_Menu_test.php index 0f5033f1c8..6682751e57 100644 --- a/test/classes/PMA_Menu_test.php +++ b/test/classes/PMA_Menu_test.php @@ -47,9 +47,9 @@ class PMA_Menu_Test extends PHPUnit_Framework_TestCase $_SESSION['PMA_Theme'] = new PMA_Theme(); $GLOBALS['pmaThemePath'] = $_SESSION['PMA_Theme']->getPath(); $GLOBALS['pmaThemeImage'] = 'theme/'; - $GLOBALS['cfg']['DefaultTabServer'] = 'main.php'; - $GLOBALS['cfg']['DefaultTabDatabase'] = 'db_structure.php'; - $GLOBALS['cfg']['DefaultTabTable'] = 'sql.php'; + $GLOBALS['cfg']['DefaultTabServer'] = 'welcome'; + $GLOBALS['cfg']['DefaultTabDatabase'] = 'structure'; + $GLOBALS['cfg']['DefaultTabTable'] = 'browse'; $GLOBALS['cfg']['OBGzip'] = false; $GLOBALS['cfg']['NaturalOrder'] = true; $GLOBALS['cfg']['TabsMode'] = 'both'; diff --git a/test/classes/dbi/DBIDrizzle_test.php b/test/classes/dbi/DBIDrizzle_test.php index 3236667320..83d6c28fe6 100644 --- a/test/classes/dbi/DBIDrizzle_test.php +++ b/test/classes/dbi/DBIDrizzle_test.php @@ -69,7 +69,7 @@ class PMA_DBI_Drizzle_Test extends PHPUnit_Framework_TestCase $GLOBALS['cfg']['Server']['ssl'] = false; $GLOBALS['cfg']['MaxCharactersInDisplayedSQL'] = 1000; $GLOBALS['cfg']['ActionLinksMode'] = "both"; - $GLOBALS['cfg']['DefaultTabDatabase'] = 'db_structure.php'; + $GLOBALS['cfg']['DefaultTabDatabase'] = 'structure'; $GLOBALS['pmaThemeImage'] = 'image'; //$_SESSION diff --git a/test/classes/navigation/PMA_Node_DatabaseChild_test.php b/test/classes/navigation/PMA_Node_DatabaseChild_test.php index 0b643d1e33..b394ba36a4 100644 --- a/test/classes/navigation/PMA_Node_DatabaseChild_test.php +++ b/test/classes/navigation/PMA_Node_DatabaseChild_test.php @@ -38,7 +38,7 @@ class Node_DatabaseChildTest extends PHPUnit_Framework_TestCase $_SESSION['PMA_Theme'] = new PMA_Theme(); $GLOBALS['pmaThemePath'] = $_SESSION['PMA_Theme']->getPath(); $GLOBALS['pmaThemeImage'] = 'theme/'; - $GLOBALS['cfg']['DefaultTabDatabase'] = 'db_structure.php'; + $GLOBALS['cfg']['DefaultTabDatabase'] = 'structure'; $GLOBALS['server'] = 1; $GLOBALS['cfg']['ServerDefault'] = 1; $_SESSION['relation'][1]['navwork'] = true; diff --git a/test/classes/navigation/PMA_Node_Database_test.php b/test/classes/navigation/PMA_Node_Database_test.php index 26852e298d..834fb1faf0 100644 --- a/test/classes/navigation/PMA_Node_Database_test.php +++ b/test/classes/navigation/PMA_Node_Database_test.php @@ -26,7 +26,7 @@ class Node_Database_Test extends PHPUnit_Framework_TestCase public function setup() { $GLOBALS['server'] = 0; - $GLOBALS['cfg']['DefaultTabDatabase'] = 'db_structure.php'; + $GLOBALS['cfg']['DefaultTabDatabase'] = 'structure'; $GLOBALS['cfg']['MaxNavigationItems'] = 250; $_SESSION['PMA_Theme'] = PMA_Theme::load('./themes/pmahomme'); } diff --git a/test/classes/navigation/PMA_Node_Table_test.php b/test/classes/navigation/PMA_Node_Table_test.php index a92f099d13..2d15cf14f3 100644 --- a/test/classes/navigation/PMA_Node_Table_test.php +++ b/test/classes/navigation/PMA_Node_Table_test.php @@ -27,7 +27,8 @@ class Node_Table_Test extends PHPUnit_Framework_TestCase { $GLOBALS['server'] = 0; $GLOBALS['cfg']['NavigationTreeDefaultTabTable'] = 'b_browse'; - $GLOBALS['cfg']['DefaultTabTable'] = 'sql.php'; + $GLOBALS['cfg']['NavigationTreeDefaultTabTable2'] = ''; + $GLOBALS['cfg']['DefaultTabTable'] = 'browse'; $GLOBALS['cfg']['MaxNavigationItems'] = 250; $GLOBALS['cfg']['NavigationTreeEnableGrouping'] = true; $GLOBALS['cfg']['NavigationTreeDbSeparator'] = '_'; @@ -80,11 +81,11 @@ class Node_Table_Test extends PHPUnit_Framework_TestCase public function providerForTestIcon() { return array( - array('tbl_structure.php', 'b_props'), - array('tbl_select.php', 'b_search'), - array('tbl_change.php', 'b_insrow'), - array('tbl_sql.php', 'b_sql'), - array('sql.php', 'b_browse'), + array('structure', 'b_props'), + array('search', 'b_search'), + array('insert', 'b_insrow'), + array('sql', 'b_sql'), + array('browse', 'b_browse'), ); } } diff --git a/test/classes/plugin/auth/PMA_AuthenticationConfig_test.php b/test/classes/plugin/auth/PMA_AuthenticationConfig_test.php index 4dd337d390..627568351f 100644 --- a/test/classes/plugin/auth/PMA_AuthenticationConfig_test.php +++ b/test/classes/plugin/auth/PMA_AuthenticationConfig_test.php @@ -35,6 +35,7 @@ class PMA_AuthenticationConfig_Test extends PHPUnit_Framework_TestCase $GLOBALS['PMA_Config'] = new PMA_Config(); $GLOBALS['PMA_Config']->enableBc(); $GLOBALS['server'] = 0; + $GLOBALS['cfg']['DefaultTabServer'] = 'welcome'; $GLOBALS['token_provided'] = true; $GLOBALS['token_mismatch'] = false; $this->object = new AuthenticationConfig(); diff --git a/test/libraries/PMA_build_html_for_db_test.php b/test/libraries/PMA_build_html_for_db_test.php index dd5b539e37..36424e3fcc 100644 --- a/test/libraries/PMA_build_html_for_db_test.php +++ b/test/libraries/PMA_build_html_for_db_test.php @@ -48,7 +48,7 @@ class PMA_BuildHtmlForDb_Test extends PHPUnit_Framework_TestCase $GLOBALS['pmaThemePath'] = $_SESSION['PMA_Theme']->getPath(); $GLOBALS['pmaThemeImage'] = 'theme/'; - $GLOBALS['cfg']['DefaultTabDatabase'] = 'db_structure.php'; + $GLOBALS['cfg']['DefaultTabDatabase'] = 'structure'; } /** diff --git a/test/libraries/PMA_select_server_test.php b/test/libraries/PMA_select_server_test.php index 1651f66d1f..564084d342 100644 --- a/test/libraries/PMA_select_server_test.php +++ b/test/libraries/PMA_select_server_test.php @@ -69,7 +69,7 @@ class PMA_SelectServer_Test extends PHPUnit_Framework_TestCase $not_only_options = false; $omit_fieldset = false; - $GLOBALS['cfg']['DefaultTabServer'] = "DefaultTabServer"; + $GLOBALS['cfg']['DefaultTabServer'] = "welcome"; $GLOBALS['cfg']['Servers'] = array( '0' => array( @@ -119,7 +119,9 @@ class PMA_SelectServer_Test extends PHPUnit_Framework_TestCase //$GLOBALS['cfg']['DefaultTabServer'] $this->assertContains( - $GLOBALS['cfg']['DefaultTabServer'], + PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabServer'], 'server' + ), $html ); diff --git a/test/libraries/PMA_server_databases_test.php b/test/libraries/PMA_server_databases_test.php index 37d8a5f9f0..e0842a6ad6 100644 --- a/test/libraries/PMA_server_databases_test.php +++ b/test/libraries/PMA_server_databases_test.php @@ -59,7 +59,7 @@ class PMA_ServerDatabases_Test extends PHPUnit_Framework_TestCase $GLOBALS['cfg']['LimitChars'] = 100; $GLOBALS['cfg']['DBG']['sql'] = false; $GLOBALS['cfg']['ActionLinksMode'] = "both"; - $GLOBALS['cfg']['DefaultTabDatabase'] = 'db_structure.php'; + $GLOBALS['cfg']['DefaultTabDatabase'] = 'structure'; $GLOBALS['table'] = "table"; $GLOBALS['replication_info']['master']['status'] = false; diff --git a/test/libraries/PMA_server_privileges_test.php b/test/libraries/PMA_server_privileges_test.php index 17657feb57..f8b88b498a 100644 --- a/test/libraries/PMA_server_privileges_test.php +++ b/test/libraries/PMA_server_privileges_test.php @@ -64,10 +64,10 @@ class PMA_ServerPrivileges_Test extends PHPUnit_Framework_TestCase $GLOBALS['cfg']['LimitChars'] = 100; $GLOBALS['cfg']['AllowThirdPartyFraming'] = false; $GLOBALS['cfg']['ActionLinksMode'] = "both"; - $GLOBALS['cfg']['DefaultTabDatabase'] = 'db_structure.php'; + $GLOBALS['cfg']['DefaultTabDatabase'] = 'structure'; $GLOBALS['cfg']['PmaAbsoluteUri'] = "PmaAbsoluteUri"; - $GLOBALS['cfg']['DefaultTabTable'] = "db_structure.php"; - $GLOBALS['cfg']['NavigationTreeDefaultTabTable'] = "db_structure.php"; + $GLOBALS['cfg']['DefaultTabTable'] = "structure"; + $GLOBALS['cfg']['NavigationTreeDefaultTabTable'] = "structure"; $GLOBALS['cfg']['Confirm'] = "Confirm"; $GLOBALS['cfg']['ShowHint'] = true; @@ -1672,7 +1672,9 @@ class PMA_ServerPrivileges_Test extends PHPUnit_Framework_TestCase $html ); $this->assertContains( - $GLOBALS['cfg']['DefaultTabDatabase'], + PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabDatabase'], 'database' + ), $html ); $item = PMA_URL_getCommon( @@ -1696,7 +1698,9 @@ class PMA_ServerPrivileges_Test extends PHPUnit_Framework_TestCase $html ); $this->assertContains( - $GLOBALS['cfg']['DefaultTabTable'], + PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabTable'], 'table' + ), $html ); $item = PMA_URL_getCommon( diff --git a/test/libraries/PMA_sql_query_form_test.php b/test/libraries/PMA_sql_query_form_test.php index a77e898828..3f97e9a883 100644 --- a/test/libraries/PMA_sql_query_form_test.php +++ b/test/libraries/PMA_sql_query_form_test.php @@ -57,7 +57,7 @@ class PMA_SqlQueryForm_Test extends PHPUnit_Framework_TestCase $GLOBALS['cfg']['TextareaAutoSelect'] = true; $GLOBALS['cfg']['TextareaRows'] = 100; $GLOBALS['cfg']['TextareaCols'] = 11; - $GLOBALS['cfg']['DefaultTabDatabase'] = "default_database"; + $GLOBALS['cfg']['DefaultTabDatabase'] = "structure"; $GLOBALS['cfg']['RetainQueryBox'] = true; $GLOBALS['cfg']['ActionLinksMode'] = 'both'; diff --git a/test/libraries/common/PMA_getDbLink_test.php b/test/libraries/common/PMA_getDbLink_test.php index d108f1ab94..22f0d41830 100644 --- a/test/libraries/common/PMA_getDbLink_test.php +++ b/test/libraries/common/PMA_getDbLink_test.php @@ -62,7 +62,11 @@ class PMA_GetDbLink_Test extends PHPUnit_Framework_TestCase $GLOBALS['db'] = 'test_db'; $database = $GLOBALS['db']; $this->assertEquals( - '' @@ -81,7 +85,10 @@ class PMA_GetDbLink_Test extends PHPUnit_Framework_TestCase global $cfg; $database = 'test_database'; $this->assertEquals( - '' . htmlspecialchars($database) . '', @@ -99,7 +106,11 @@ class PMA_GetDbLink_Test extends PHPUnit_Framework_TestCase global $cfg; $database = 'test&data\'base'; $this->assertEquals( - '' diff --git a/test/theme.php b/test/theme.php index 02d0436640..24ff49ebbb 100644 --- a/test/theme.php +++ b/test/theme.php @@ -52,7 +52,9 @@ $item = '' echo '
' . "\n"; printf( $item, - $GLOBALS['cfg']['DefaultTabServer'], + PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabserver'], 'server' + ), PMA_URL_getCommon(), 'Server', __('Server'), @@ -62,7 +64,9 @@ printf( echo $separator; printf( $item, - $GLOBALS['cfg']['DefaultTabDatabase'], + PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabDatabase'], 'database' + ), '', 'Database', __('Database'), @@ -72,7 +76,9 @@ printf( echo $separator; printf( $item, - $GLOBALS['cfg']['DefaultTabTable'], + PMA_Util::getScriptNameForOption( + $GLOBALS['cfg']['DefaultTabTable'], 'table' + ), '', 'Table', (isset($GLOBALS['tbl_is_view']) && $GLOBALS['tbl_is_view'] From 26f271cfa63ad9a036e621409fe0f3a1ce6b8dbf Mon Sep 17 00:00:00 2001 From: Kurt Eilertsen Date: Fri, 12 Jun 2015 17:12:40 +0200 Subject: [PATCH 5/8] =?UTF-8?q?Translated=20using=20Weblate=20(Norwegian?= =?UTF-8?q?=20Bokm=C3=A5l)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Currently translated at 61.1% (1897 of 3102 strings) [CI skip] --- po/nb.po | 39 ++++++++++++++------------------------- 1 file changed, 14 insertions(+), 25 deletions(-) diff --git a/po/nb.po b/po/nb.po index 8bb0c6a9cd..bea47aae6c 100644 --- a/po/nb.po +++ b/po/nb.po @@ -4,7 +4,7 @@ msgstr "" "Project-Id-Version: phpMyAdmin 4.5.0-dev\n" "Report-Msgid-Bugs-To: phpmyadmin-devel@lists.sourceforge.net\n" "POT-Creation-Date: 2015-06-11 09:31-0400\n" -"PO-Revision-Date: 2015-06-12 17:02+0200\n" +"PO-Revision-Date: 2015-06-12 17:12+0200\n" "Last-Translator: Kurt Eilertsen \n" "Language-Team: Norwegian Bokmål " "\n" @@ -4027,7 +4027,6 @@ msgid "SQL query" msgstr "SQL-spørring" #: libraries/ServerStatusData.class.php:118 -#, fuzzy msgid "Handler" msgstr "Behandler" @@ -4125,20 +4124,20 @@ msgid "Advisor" msgstr "Rådgiver" #: libraries/ServerStatusData.class.php:494 -#, fuzzy, php-format +#, php-format #| msgid "Second" msgid "%d second" msgid_plural "%d seconds" -msgstr[0] "Sekund" -msgstr[1] "Sekund" +msgstr[0] "%d sekund" +msgstr[1] "%d sekunder" #: libraries/ServerStatusData.class.php:499 -#, fuzzy, php-format +#, php-format #| msgid "Minute" msgid "%d minute" msgid_plural "%d minutes" -msgstr[0] "Minutt" -msgstr[1] "Minutt" +msgstr[0] "%d minutt" +msgstr[1] "%d minutter" #: libraries/StorageEngine.class.php:272 msgid "" @@ -4235,19 +4234,16 @@ msgid "No index parts defined!" msgstr "Ingen indeksdeler definert!" #: libraries/TableSearch.class.php:184 -#, fuzzy #| msgid "Table Search" msgid "Table search" msgstr "Tabellsøk" #: libraries/TableSearch.class.php:191 -#, fuzzy #| msgid "Zoom Search" msgid "Zoom search" msgstr "Utvidet søk" #: libraries/TableSearch.class.php:196 templates/table/selection_form.phtml:59 -#, fuzzy #| msgid "Find and Replace" msgid "Find and replace" msgstr "Finn og erstatt" @@ -4710,10 +4706,9 @@ msgid "The directory you set for upload work cannot be reached." msgstr "Katalogen du anga for opplasting kan ikke nåes." #: libraries/Util.class.php:3546 -#, fuzzy #| msgid "There are no files to upload" msgid "There are no files to upload!" -msgstr "Det er ingen filer å laste opp" +msgstr "Det er ingen filer å laste opp!" #: libraries/Util.class.php:3571 libraries/Util.class.php:3572 #: libraries/structure.lib.php:329 @@ -4821,10 +4816,9 @@ msgid "Could not add %1$s as they already exist in central list!" msgstr "" #: libraries/central_columns.lib.php:298 -#, fuzzy #| msgid "Could not save configuration" msgid "Could not add columns!" -msgstr "Kunne ikke lagre konfigurasjonen" +msgstr "Kunne ikke legge til kolonner!" #: libraries/central_columns.lib.php:373 #, php-format @@ -4833,10 +4827,9 @@ msgid "" msgstr "" #: libraries/central_columns.lib.php:385 -#, fuzzy #| msgid "Could not save recent table" msgid "Could not remove columns!" -msgstr "Kunne ikke lagre tabell" +msgstr "Kunne ikke fjerne kolonner!" #: libraries/central_columns.lib.php:530 msgid "YES" @@ -4869,10 +4862,9 @@ msgid "Length/Values" msgstr "Lengde/Sett*" #: libraries/central_columns.lib.php:706 -#, fuzzy #| msgid "Attributes" msgid "Attribute" -msgstr "Attributter" +msgstr "Attributt" #: libraries/central_columns.lib.php:710 #: libraries/central_columns.lib.php:1208 @@ -4880,20 +4872,18 @@ msgid "A_I" msgstr "" #: libraries/central_columns.lib.php:750 -#, fuzzy #| msgid "Select Tables" msgid "Select a table" -msgstr "Velg tabeller" +msgstr "Velg en tabell" #: libraries/central_columns.lib.php:804 libraries/structure.lib.php:1746 msgid "Add column" msgstr "Legg til kolonne(r)" #: libraries/central_columns.lib.php:816 -#, fuzzy #| msgid "Select two columns" msgid "Select a column." -msgstr "Velg to kolonner" +msgstr "Velg en kolonne." #: libraries/central_columns.lib.php:1148 #, fuzzy @@ -15890,10 +15880,9 @@ msgid "Replace with:" msgstr "Erstatt med:" #: templates/table/search_and_replace.phtml:22 -#, fuzzy #| msgid "as regular expression" msgid "Use regular expression" -msgstr "som \"regular expression\"" +msgstr "Bruk \"regulert uttrykk\"" #: templates/table/selection_form.phtml:12 msgid "Do a \"query by example\" (wildcard: \"%\") for two different columns" From fa268f9b6cbd008733ee7d58127dba0107e594e5 Mon Sep 17 00:00:00 2001 From: Burak Yavuz Date: Fri, 12 Jun 2015 18:12:18 +0200 Subject: [PATCH 6/8] Translated using Weblate (Turkish) Currently translated at 99.9% (3101 of 3102 strings) [CI skip] --- po/tr.po | 33 +++++++++++++-------------------- 1 file changed, 13 insertions(+), 20 deletions(-) diff --git a/po/tr.po b/po/tr.po index 08ed4a142e..8c6730d3e8 100644 --- a/po/tr.po +++ b/po/tr.po @@ -4,16 +4,16 @@ msgstr "" "Project-Id-Version: phpMyAdmin 4.5.0-dev\n" "Report-Msgid-Bugs-To: phpmyadmin-devel@lists.sourceforge.net\n" "POT-Creation-Date: 2015-06-11 09:31-0400\n" -"PO-Revision-Date: 2015-05-31 16:18+0200\n" +"PO-Revision-Date: 2015-06-12 18:12+0200\n" "Last-Translator: Burak Yavuz \n" -"Language-Team: Turkish \n" +"Language-Team: Turkish " +"\n" "Language: tr\n" "MIME-Version: 1.0\n" "Content-Type: text/plain; charset=UTF-8\n" "Content-Transfer-Encoding: 8bit\n" "Plural-Forms: nplurals=2; plural=n > 1;\n" -"X-Generator: Weblate 2.3-dev\n" +"X-Generator: Weblate 2.4-dev\n" #: changelog.php:37 license.php:28 #, php-format @@ -1034,14 +1034,13 @@ msgid "" msgstr "" #: js/messages.php:85 -#, fuzzy #| msgid "Are you sure you wish to change the collation and convert the data?" msgid "" "Are you sure you wish to change all the column collations and convert the " "data?" msgstr "" -"Karşılaştırmayı değiştirmek ve verileri dönüştürmek istediğinizden emin " -"misiniz?" +"Tüm sütun karşılaştırmalarını değiştirmek ve verileri dönüştürmek " +"istediğinizden emin misiniz?" #: js/messages.php:88 msgid "Save & Close" @@ -5019,22 +5018,19 @@ msgid "Never send error reports" msgstr "Hata raporlarını asla gönderme" #: libraries/config.values.php:132 -#, fuzzy #| msgid "Set default" msgid "Server default" -msgstr "Varsayılana ayarla" +msgstr "Sunucu varsayılanı" #: libraries/config.values.php:133 -#, fuzzy #| msgid "Enabled" msgid "Enable" -msgstr "Etkinleştirildi" +msgstr "Etkinleştir" #: libraries/config.values.php:134 -#, fuzzy #| msgid "Disabled" msgid "Disable" -msgstr "Etkisizleştirildi" +msgstr "Etkisizleştir" #: libraries/config.values.php:163 #: libraries/plugins/export/ExportHtmlword.class.php:69 @@ -5902,10 +5898,9 @@ msgstr "" "kontrol eder." #: libraries/config/messages.inc.php:218 -#, fuzzy #| msgid "Foreign key check:" msgid "Foreign key checks" -msgstr "Dış anahtar kontrolü:" +msgstr "Dış anahtar kontrolleri" #: libraries/config/messages.inc.php:219 msgid "Browse mode" @@ -6546,7 +6541,6 @@ msgstr "" "Veritabanı süzgeci kutusunu görüntülemek için en az veritabanı sayısıdır" #: libraries/config/messages.inc.php:467 -#, fuzzy #| msgid "" #| "Group items in the navigation tree (determined by the separator defined " #| "below)." @@ -6554,8 +6548,8 @@ msgid "" "Group items in the navigation tree (determined by the separator defined in " "the Databases and Tables tabs above)." msgstr "" -"Gezinti ağacındaki öğeleri gruplar (aşağıda tanımlanan ayıraç tarafından " -"belirlenir)." +"Gezinti ağacındaki öğeleri gruplar (Veritabanları ve Tablolar sekmeleri " +"üzerini tanımlanan ayıraç tarafından belirlenir)." #: libraries/config/messages.inc.php:469 msgid "Group items in the tree" @@ -9588,10 +9582,9 @@ msgid "Storage Engine" msgstr "Depolama Motoru" #: libraries/operations.lib.php:995 -#, fuzzy #| msgid "Make all columns atomic" msgid "Change all column collations" -msgstr "Tüm sütunları çok küçük yap" +msgstr "Tüm sütun karşılaştırmalarını değiştir" #: libraries/operations.lib.php:1164 msgid "Copy table to (database.table)" From 10f5d6e5867d673e0bea2294274b3763b4df8f5e Mon Sep 17 00:00:00 2001 From: Isaac Bennetch Date: Fri, 12 Jun 2015 15:18:39 -0400 Subject: [PATCH 7/8] Changelog for rfe 1594 Signed-off-by: Isaac Bennetch --- ChangeLog | 1 + 1 file changed, 1 insertion(+) diff --git a/ChangeLog b/ChangeLog index eabd055b9a..8dedbc575d 100644 --- a/ChangeLog +++ b/ChangeLog @@ -54,6 +54,7 @@ phpMyAdmin - ChangeLog + rfe #1634 Don't group tables in tree if the result has only one group - bug #4946 When hide table structure actions is false, action should be in a row + rfe #726 Batch changing the collation of each column in a table ++ rfe #1594 Use plain English descriptors instead of script names for icon link destinations 4.4.10.0 (not yet released) - bug #4950 Issues in database selection for replication From 3827e3e9886fc18b7533f4aad5caedfd68a92faa Mon Sep 17 00:00:00 2001 From: Hugues Peccatte Date: Fri, 12 Jun 2015 22:24:32 +0200 Subject: [PATCH 8/8] Make PHPcs happy. Signed-off-by: Hugues Peccatte --- libraries/DatabaseInterface.class.php | 4 ++-- libraries/Table.class.php | 26 +++++++++++++++---------- tbl_chart.php | 25 +++++++++++++----------- tbl_indexes.php | 11 +++++++---- test/libraries/PMA_tbl_indexes_test.php | 4 +++- 5 files changed, 42 insertions(+), 28 deletions(-) diff --git a/libraries/DatabaseInterface.class.php b/libraries/DatabaseInterface.class.php index 56900e48f5..78265a0ff8 100644 --- a/libraries/DatabaseInterface.class.php +++ b/libraries/DatabaseInterface.class.php @@ -3115,8 +3115,8 @@ class PMA_DatabaseInterface /** * Get a table with database name and table name * - * @param $db_name - * @param $table_name + * @param string $db_name DB name + * @param string $table_name Table name * * @return PMA_Table */ diff --git a/libraries/Table.class.php b/libraries/Table.class.php index 999cf50f91..344ba355ce 100644 --- a/libraries/Table.class.php +++ b/libraries/Table.class.php @@ -74,9 +74,9 @@ class PMA_Table /** * Constructor * - * @param string $table_name table name - * @param string $db_name database name - * @param PMA_DatabaseInterface $dbi database interface for the table + * @param string $table_name table name + * @param string $db_name database name + * @param PMA_DatabaseInterface $dbi database interface for the table */ function __construct($table_name, $db_name, PMA_DatabaseInterface $dbi = null) { @@ -1625,7 +1625,7 @@ class PMA_Table $server_id = $GLOBALS['server']; // set session variable if it's still undefined - if (! isset($_SESSION['tmpval']['table_uiprefs'][$server_id][$this->_db_name][$this->_name])) { + if (!isset($_SESSION['tmpval']['table_uiprefs'][$server_id][$this->_db_name][$this->_name])) { // check whether we can get from pmadb $_SESSION['tmpval']['table_uiprefs'][$server_id][$this->_db_name] [$this->_name] = $cfgRelation['uiprefswork'] @@ -1819,11 +1819,14 @@ class PMA_Table public function getNameAndTypeOfTheColumns() { $columns = array(); - foreach ($this->_dbi->getColumnsFull($this->_db_name, $this->_name) as $row) { + foreach ($this->_dbi->getColumnsFull( + $this->_db_name, $this->_name + ) as $row) { if (preg_match('@^(set|enum)\((.+)\)$@i', $row['Type'], $tmp)) { - $tmp[2] = /*overload*/mb_substr( - preg_replace('@([^,])\'\'@', '\\1\\\'', ',' . $tmp[2]), 1 - ); + $tmp[2] = /*overload*/ + mb_substr( + preg_replace('@([^,])\'\'@', '\\1\\\'', ',' . $tmp[2]), 1 + ); $columns[$row['Field']] = $tmp[1] . '(' . str_replace(',', ', ', $tmp[2]) . ')'; } else { @@ -1836,7 +1839,8 @@ class PMA_Table /** * Get index with index name * - * @param $index + * @param string $index Index name + * * @return PMA_Index */ public function getIndex($index) @@ -1890,7 +1894,9 @@ class PMA_Table case 'INDEX': case 'SPATIAL': if ($index->getName() == 'PRIMARY') { - $error = PMA_Message::error(__('Can\'t rename index to PRIMARY!')); + $error = PMA_Message::error( + __('Can\'t rename index to PRIMARY!') + ); } $sql_query .= sprintf( ' ADD %s ', diff --git a/tbl_chart.php b/tbl_chart.php index 0b384ce44f..f6cd4c0644 100644 --- a/tbl_chart.php +++ b/tbl_chart.php @@ -140,14 +140,17 @@ $url_params['reload'] = 1; /** * Displays the page */ -$response->addHTML(Template::get('tbl_chart')->render( - array( - 'url_query' => $url_query, - 'url_params' => $url_params, - 'keys' => $keys, - 'fields_meta' => $fields_meta, - 'numeric_types' => $numeric_types, - 'numeric_column_count' => $numeric_column_count, - 'sql_query' => $sql_query - ) -)); +$response->addHTML( + Template::get('tbl_chart') + ->render( + array( + 'url_query' => $url_query, + 'url_params' => $url_params, + 'keys' => $keys, + 'fields_meta' => $fields_meta, + 'numeric_types' => $numeric_types, + 'numeric_column_count' => $numeric_column_count, + 'sql_query' => $sql_query + ) + ) +); diff --git a/tbl_indexes.php b/tbl_indexes.php index 69801f44d3..1a6c3e78d5 100644 --- a/tbl_indexes.php +++ b/tbl_indexes.php @@ -47,9 +47,11 @@ if (isset($_REQUEST['do_save_data'])) { PMA_Response::getInstance()->addJSON( 'sql_data', PMA\Template::get('preview_sql') - ->render(array( - 'query_data' => $sql_query - )) + ->render( + array( + 'query_data' => $sql_query + ) + ) ); } elseif (!$error) { @@ -121,7 +123,8 @@ if (isset($_REQUEST['create_index'])) { } $response = PMA_Response::getInstance(); -$response->addHTML(PMA\Template::get('index_form') +$response->addHTML( + PMA\Template::get('index_form') ->render( array( 'fields' => $fields, diff --git a/test/libraries/PMA_tbl_indexes_test.php b/test/libraries/PMA_tbl_indexes_test.php index 9ff4240007..35a2c73513 100644 --- a/test/libraries/PMA_tbl_indexes_test.php +++ b/test/libraries/PMA_tbl_indexes_test.php @@ -122,7 +122,9 @@ class PMA_TblIndexTest extends PHPUnit_Framework_TestCase * ); * $add_fields = 3; * - * $html = PMA_getHtmlForIndexForm($fields, $index, $form_params, $add_fields); + * $html = PMA_getHtmlForIndexForm( + * $fields, $index, $form_params, $add_fields + * ); * * //PMA_URL_getHiddenInputs * $this->assertContains(