Update FileMigration.php

[xepozz]

Using exec is a very very bad idea!

[schmunk42]

Why is exec in general a bad idea?

The issue with file_get_contents and using Yii and PHP for this instead of the native CLI command is that you might run into problems with large files (several 100s of MBs) and multiple commands.

[xepozz]

Because a mysql server may installed on remote server.
You may write drivers for it

[schmunk42]

The only way I could think about adding this feature is via a global flag and an if/else statement.

CC: @handcode @eluhr WDYT?

[handcode]

If mysql server is installed on remote i can see no problem. This is our default setup ;-)
But of course exec 'mysql' from PHP can be a problem if the mysql client is not installed locally.

Apart from that i also think file_get_contents with large dumps is not a good idea.

So I agree with @schmunk42 : it can only be implemented as a configurable alternative to execCommand.

[schmunk42]

Changes should be moved to a separate file, so the developer can choose which one to use.

[xepozz]

I'm ready to complete this PR 😄
#27