[go: nahoru, domu]

Skip to content

Commit

Permalink
Removes default new database name
Browse files Browse the repository at this point in the history
Issue #13099

Signed-off-by: Raghuram Vadapalli<raghuram.vadapalli@research.iiit.ac.in>
Signed-off-by: Michal Čihař <michal@cihar.com>
  • Loading branch information
Achilles-96 authored and nijel committed Mar 21, 2017
1 parent b98fc39 commit 553d1ff
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions ChangeLog
Original file line number Diff line number Diff line change
Expand Up @@ -107,6 +107,7 @@ phpMyAdmin - ChangeLog
- issue #13105 Fixed changing attribute for virtual field
- issue #12757 Fixed setting password on recent MariaDB with non working plugins
- issue #12349 Fixed undefined variable on import from some formats
- issue #13103 Do not offer default names for copying/renaming databases

4.6.6 (2017-01-23)
- issue #12759 Fix Notice regarding 'Undefined index: old_usergroup'
Expand Down
4 changes: 2 additions & 2 deletions libraries/operations.lib.php
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ function PMA_getHtmlForRenameDatabase($db)

$html_output .= '<input id="new_db_name" type="text" name="newname" '
. 'maxlength="64" size="30" class="textfield" required="required" '
. 'value="' . htmlspecialchars($db) . '"/>';
. 'value=""/>';

if ($GLOBALS['db_priv'] && $GLOBALS['table_priv']
&& $GLOBALS['col_priv'] && $GLOBALS['proc_priv']
Expand Down Expand Up @@ -201,7 +201,7 @@ function PMA_getHtmlForCopyDatabase($db)
$html_output .= __('Copy database to')
. '</legend>'
. '<input type="text" maxlength="64" name="newname" size="30" '
. 'class="textfield" value="' . htmlspecialchars($db) . '" '
. 'class="textfield" value="" '
. 'required="required" /><br />'
. Util::getRadioFields(
'what', $choices, 'data', true
Expand Down

2 comments on commit 553d1ff

@OlafvdSpek
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not remove the value attribute entirely?

@nijel
Copy link
Contributor
@nijel nijel commented on 553d1ff Apr 12, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed it's not needed at all, I've done little cleanup in 780bf96.

Please sign in to comment.