Error while moving columns #12841

Closed
ibennetch opened this Issue Dec 20, 2016 · 0 comments

Projects

None yet

2 participants

@ibennetch
Contributor

Steps to reproduce

  1. From a database Structure tab, use the "Move columns" link
  2. Drag to reorder some columns
  3. Press Go.

Expected behaviour

No error

Actual behaviour

The message appears "A fatal JavaScript error has occurred. Would you like to send an error report?"

{
    "pma_version": "4.7.0-dev",
    "browser_name": "FIREFOX",
    "browser_version": "50.0",
    "user_os": "Mac",
    "server_software": "Apache/2.4.10 (Debian)",
    "user_agent_string": "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:50.0) Gecko/20100101 Firefox/50.0",
    "locale": "en",
    "configuration_storage": "enabled",
    "php_version": "5.6.29-0+deb8u1",
    "exception_type": "js",
    "exception": {
        "mode": "stack",
        "name": "Error",
        "message": "Syntax error, unrecognized expression: input:checkbox[value=COL 1]",
        "stack": [
            {
                "func": "t</ga.error",
                "line": 2,
                "column": "12481",
                "context": [
                    "/*! jQuery v2.1.4 | (c) 2005, 2015 jQuery Foundation, Inc. | jquery.org/lic//...",
                    "!function(a,b){\"object\"==typeof module&&\"object\"==typeof module.exports?mod//...",
                    "return M.access(a,b,c)},removeData:function(a,b){M.remove(a,b)},_data:funct//...",
                    "void 0===c?d&&\"get\"in d&&null!==(e=d.get(a,b))?e:(e=n.find.attr(a,b),null==//...",
                    ";",
                    "",
                    "function sprintf() {"
                ],
                "filename": "jquery/jquery-2.1.4.min.js"
            },
            {
                "func": "t</ga.tokenize",
                "line": 2,
                "column": "18405",
                "context": [
                    "/*! jQuery v2.1.4 | (c) 2005, 2015 jQuery Foundation, Inc. | jquery.org/lic//...",
                    "!function(a,b){\"object\"==typeof module&&\"object\"==typeof module.exports?mod//...",
                    "return M.access(a,b,c)},removeData:function(a,b){M.remove(a,b)},_data:funct//...",
                    "void 0===c?d&&\"get\"in d&&null!==(e=d.get(a,b))?e:(e=n.find.attr(a,b),null==//...",
                    ";",
                    "",
                    "function sprintf() {"
                ],
                "filename": "jquery/jquery-2.1.4.min.js"
            },
            {
                "func": "ga",
                "line": 2,
                "column": "6830",
                "context": [
                    "/*! jQuery v2.1.4 | (c) 2005, 2015 jQuery Foundation, Inc. | jquery.org/lic//...",
                    "!function(a,b){\"object\"==typeof module&&\"object\"==typeof module.exports?mod//...",
                    "return M.access(a,b,c)},removeData:function(a,b){M.remove(a,b)},_data:funct//...",
                    "void 0===c?d&&\"get\"in d&&null!==(e=d.get(a,b))?e:(e=n.find.attr(a,b),null==//...",
                    ";",
                    "",
                    "function sprintf() {"
                ],
                "filename": "jquery/jquery-2.1.4.min.js"
            },
            {
                "func": ".find",
                "line": 2,
                "column": "23408",
                "context": [
                    "/*! jQuery v2.1.4 | (c) 2005, 2015 jQuery Foundation, Inc. | jquery.org/lic//...",
                    "!function(a,b){\"object\"==typeof module&&\"object\"==typeof module.exports?mod//...",
                    "return M.access(a,b,c)},removeData:function(a,b){M.remove(a,b)},_data:funct//...",
                    "void 0===c?d&&\"get\"in d&&null!==(e=d.get(a,b))?e:(e=n.find.attr(a,b),null==//...",
                    ";",
                    "",
                    "function sprintf() {"
                ],
                "filename": "jquery/jquery-2.1.4.min.js"
            },
            {
                "func": "button_options[PMA_messages.strGo]/<",
                "line": 366,
                "column": "40",
                "context": [
                    "                    // remove all existing rows and remember them",
                    "                    var $rows = $fields_table.find(\"tr\").remove();",
                    "                    // loop through the correct order",
                    "                    for (var i in data.columns) {",
                    "                        var the_column = data.columns[i];",
                    "                        var $the_row = $rows",
                    "                            .find(\"input:checkbox[value=\" + the_column + \"]\")",
                    "                            .closest(\"tr\");",
                    "                        // append the row for this column to the table",
                    "                        $fields_table.append($the_row);",
                    "                    }"
                ],
                "filename": "tbl_structure.js"
            },
            {
                "func": "n.Callbacks/j",
                "line": 2,
                "column": "26920",
                "context": [
                    "/*! jQuery v2.1.4 | (c) 2005, 2015 jQuery Foundation, Inc. | jquery.org/lic//...",
                    "!function(a,b){\"object\"==typeof module&&\"object\"==typeof module.exports?mod//...",
                    "return M.access(a,b,c)},removeData:function(a,b){M.remove(a,b)},_data:funct//...",
                    "void 0===c?d&&\"get\"in d&&null!==(e=d.get(a,b))?e:(e=n.find.attr(a,b),null==//...",
                    ";",
                    "",
                    "function sprintf() {"
                ],
                "filename": "jquery/jquery-2.1.4.min.js"
            },
            {
                "func": "n.Callbacks/k.fireWith",
                "line": 2,
                "column": "27738",
                "context": [
                    "/*! jQuery v2.1.4 | (c) 2005, 2015 jQuery Foundation, Inc. | jquery.org/lic//...",
                    "!function(a,b){\"object\"==typeof module&&\"object\"==typeof module.exports?mod//...",
                    "return M.access(a,b,c)},removeData:function(a,b){M.remove(a,b)},_data:funct//...",
                    "void 0===c?d&&\"get\"in d&&null!==(e=d.get(a,b))?e:(e=n.find.attr(a,b),null==//...",
                    ";",
                    "",
                    "function sprintf() {"
                ],
                "filename": "jquery/jquery-2.1.4.min.js"
            },
            {
                "func": "x",
                "line": 4,
                "column": "11251",
                "context": [
                    "/*! jQuery v2.1.4 | (c) 2005, 2015 jQuery Foundation, Inc. | jquery.org/lic//...",
                    "!function(a,b){\"object\"==typeof module&&\"object\"==typeof module.exports?mod//...",
                    "return M.access(a,b,c)},removeData:function(a,b){M.remove(a,b)},_data:funct//...",
                    "void 0===c?d&&\"get\"in d&&null!==(e=d.get(a,b))?e:(e=n.find.attr(a,b),null==//...",
                    ";",
                    "",
                    "function sprintf() {",
                    "/*",
                    " * Copyright (c) 2013 Kevin van Zonneveld (http://kvz.io)"
                ],
                "filename": "jquery/jquery-2.1.4.min.js"
            },
            {
                "func": ".send/b/<",
                "line": 4,
                "column": "14765",
                "context": [
                    "/*! jQuery v2.1.4 | (c) 2005, 2015 jQuery Foundation, Inc. | jquery.org/lic//...",
                    "!function(a,b){\"object\"==typeof module&&\"object\"==typeof module.exports?mod//...",
                    "return M.access(a,b,c)},removeData:function(a,b){M.remove(a,b)},_data:funct//...",
                    "void 0===c?d&&\"get\"in d&&null!==(e=d.get(a,b))?e:(e=n.find.attr(a,b),null==//...",
                    ";",
                    "",
                    "function sprintf() {",
                    "/*",
                    " * Copyright (c) 2013 Kevin van Zonneveld (http://kvz.io)"
                ],
                "filename": "jquery/jquery-2.1.4.min.js"
            }
        ],
        "uri": "index.php?"
    },
    "script_name": "index.php",
    "microhistory": null
}

Server configuration

Operating system:

Web server:

Database:

PHP version:

phpMyAdmin version:
QA_4_6 (4.6.6-dev) and master (4.7.0-dev)

Client configuration

Browser:
Firefox/Safari
Operating system:

@nijel nijel self-assigned this Dec 21, 2016
@nijel nijel added the bug label Dec 21, 2016
@nijel nijel added this to the 4.6.6 milestone Dec 21, 2016
@nijel nijel added a commit that closed this issue Dec 21, 2016
@nijel nijel Fixed moving of columns with whitespace in name
Fixes #12841

Signed-off-by: Michal Čihař <michal@cihar.com>
6f01864
@nijel nijel closed this in 6f01864 Dec 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment