• Thanks for BackWpUp!

    I imported a Mysql-Dump and run into a SQL-Syntax error. I figured out that it was a missing
    setting of DELIMITER in backwpup/inc/class-mysqldump.php.

    For triggers it says:
    $create .= “DELIMITER //\n”;
    $create .= $create_trigger[ ‘SQL Original Statement’ ] . “;\n”;
    $create .= “//\nDELIMITER ;\n”;

    But for functions and procedures it says:

    $create .= “/*!40101 SET character_set_client = ‘” . $this->mysqli->character_set_name() . “‘ */;\n”;
    $create .= $create_function[ ‘Create Function’ ] . “;\n”;
    $create .= “/*!40101 SET character_set_client = @saved_cs_client */;\n”;

    Here the two lines are missing. If I included them manually in the .sql file, the .sql file could be imported.

    I think it’s a bug, but I may be wrong. If you see it the same way, I would appreciate it, if you would fix the bug.

    Best wishes
    Tobias

Viewing 2 replies - 1 through 2 (of 2 total)
  • Plugin Support happyAnt

    (@duongcuong96)

    Hi @polzin
    hmm sorry, we have not received any report about the unable import database file created by BackWPUp also never got your issue before.
    Could you give me a more detailed steps on how to reproduce the issue?

    Thread Starter polzin

    (@polzin)

    Hi @duongcuong96,
    thanks for commenting!
    In my point of view the bug is obvious from the code: Procedures and function, like triggers, may contain code that contains ; and requires a redefinition of the DELIMITER.

    To reproduce the bug, dump a database with procedures or functions that include code with “;” and try in import it again.

    Best wishes
    Tobias

Viewing 2 replies - 1 through 2 (of 2 total)
  • The topic ‘BUG in backwpup/inc/class-mysql: Dump of Database Functions Misses DELIMITER’ is closed to new replies.