[fix] Cannot update schedules after an upgrade

[anonymous]

After an upgrade from an earlier version of MythTV to 0.20.2, any attempt to update the recording schedules fails, whether it is attempted from within mythfrontend or through mythweb. The problem is that in the database upgrade statements, record.recordid is created WITHOUT the AUTO_INCREMENT attribute, whereas if the table is created by libmythtv/dbcheck.cpp, it is created WITH the AUTO_INCREMENT attribute.

Fix is fairly simply, just increment the database schema version and add a MySQL statement for that version upgrade to ADD the AUTO_INCREMENT attribute to record.recordid.

[tilghman@…]

Patch file

[danielk]

tilghman, I can't find the DB update you speak of.

The patch is incorrect as it would break the upgrade path to 0.21, but if I find the source of the problem I'm sure I can fix it.

[tilghman@…]

In the branch 0.20-fixes (svn), in libs/libmythtv/dbcheck.cpp (revision 14557), see line number 2935. You'll see that the recordid is specified as auto_increment. Now look at the upgrade path from previous dbschemas. You'll find that the recordid field never gets changed to have that attribute. That attribute is absolutely critical, as the recordid field is inserted in multiple places with the value 0, which triggers mysql to use the next auto-incremented value in its place.

That is why you cannot change the schedule on an upgraded mythtv box and why this needs to be fixed.

[danielk]

I see where recordid is added with the attribute, but where is recordid added without the auto increment attribute?

I don't want to apply a general fix if only your database is affected.

[tilghman@…]

Okay, I don't see anywhere where it didn't have it, so chalk this up to mysql (maybe a recovery after a crash somewhere) silently dropping the auto_increment attribute.

I have to wonder now how many of my other tables are missing a necessary auto_increment attribute.

Close this out.