Skip to content

Commit

Permalink
add idShow to the episode table and drop the unneeded tvshowlinkepisode
Browse files Browse the repository at this point in the history
  • Loading branch information
Jonathan Marshall committed May 8, 2012
1 parent ed6a28a commit 2fc3a8f
Show file tree
Hide file tree
Showing 2 changed files with 56 additions and 53 deletions.
Loading

4 comments on commit 2fc3a8f

@ScudLee
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When exactly does the idShow value get set when scanning in new episodes? Because none of my newly scraped episodes are showing up, and looking in the database all their idShow fields are empty and if I manually add the right value, the episode shows up. All the old episodes are fine from the update, just nothing new.

@drae
Copy link

@drae drae commented on 2fc3a8f May 10, 2012

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same thing on a fresh install. Not looked through all the SQL statements and not an expert by any means but shouldn't "CREATE TABLE episode" gain an ", idShow integer" on L195? Also a little later an index is created on season rather than seasons - L312. Plus whatever is causing the idShow integer to not be set in the episode table. I very quickly and very very blindly tried modifying L1138 to set it but that didn't seem to fix it.

@jmarshallnz
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The season/seasons thing is already fixed.

Looks like I missed the update for show - will fix.

@jmarshallnz
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rest should be fixed now as well.

Please sign in to comment.