Upgrade doesn't work if there are more than 1 lines in the script

[QuickBrownFoxy]

For example
in

alter table RECORDING_FILE add ORIGINAL_SAMPLERATE TEXT;
alter table RECORDING_FILE add ORIGINAL_BITRATE TEXT;

only the first line executed. Sugar seems to proccess it file but on db.execSQL(text.toString()); in SugarDB.java only first column added

[whoshuu]

Have you tried putting both columns into one line? I'll test this out myself some time tomorrow and get back to you.

[doublerebel]

Spent a good hour today hunting this one down. I could not get multiple commands to execute whether on two lines or one. However, adding both 2.sql and 3.sql with one line each, and bumping the version to 3 in the manifest, executed the upgrade.

[whoshuu]

Migration troubles are a frequent area of concern. I'm glad you were able to get your migrations to work. In the future I hope we can get some work done on that part of the codebase to make procedures like this much easier.

[doublerebel]

Hello,

As this is a major issue with migrations, I don't believe the issue should be closed. At the minimum the documentation (http://satyan.github.io/sugar/migration.html) should be updated to reflect the actual behavior of the current code. Unless this has been fixed?

[whoshuu]

Hey @doublerebel, thanks for bringing this up again. In my haste I closed this issue because 1.5.0 is the release we're planning on overhauling the migration system. You make a good point, it's important to keep this issue around to make sure we don't repeat the mistake of only allowing for one sql statement per migration file. Reopening this and tagging it appropriately.

[doublerebel]

thanks 😃