search

tacitknowledge / autopatch

An automated database patching framework for Java.
41 stars 19 forks source link

Create systemname patch level row if it doesn't exist #32

Closed bradrust closed 12 years ago

bradrust commented 12 years ago

In PatchTable.createPatchStoreIfNeeded, the try-catch is testing for the existence of the patches table itself. However, if the patches tables exists, it does not call the createSystemPatchRecord() method to create the patchlevel=0 for systemname.

The pull request just tests if there are any rows in the current patch row (by systemname) and if there are no rows, it calls createSystemPatchRecord().

scottfromsf commented 12 years ago

Unfortunately this commit fails the unit tests. I'll look into exactly why a bit later but for now I've rolled it back.

bradrust commented 12 years ago

I'll try and fix them On Sep 27, 2012 12:25 PM, "Scott Askew" notifications@github.com wrote:

Unfortunately this commit fails the unit tests. I'll look into exactly why a bit later but for now I've rolled it back.

— Reply to this email directly or view it on GitHubhttps://github.com/tacitknowledge/autopatch/pull/32#issuecomment-8948047.