Skip to content

Commit

Permalink
Fix wrong column name
Browse files Browse the repository at this point in the history
  • Loading branch information
Luke Carbis committed Apr 16, 2014
1 parent bec915e commit ed643b3
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions includes/install.php
Original file line number Diff line number Diff line change
Expand Up @@ -213,9 +213,9 @@ public static function update( $db_version, $current ) {
}

// If version is lower than 1.3.2, do the update routine
// Update Stream table to include author_id and author_role
// Update Stream table to include author_role
if ( version_compare( $db_version, '1.3.2', '<' ) ) {
$wpdb->query( "ALTER TABLE {$prefix}stream ADD author_role varchar(20) NOT NULL AFTER author_id" );
$wpdb->query( "ALTER TABLE {$prefix}stream ADD author_role varchar(20) NOT NULL AFTER author" );
}
}

Expand Down

3 comments on commit ed643b3

@frankiejarrett
Copy link
Contributor

Choose a reason for hiding this comment

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

@lukecarbis This can be removed since you are doing the upgrade routine in #379, right?

@lukecarbis
Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah - it can. Figured we'd remove it when we merge, so the branch can still be tested on it's own, and so I didn't confuse whoever was merging. Happy to bin it now, though.

@frankiejarrett
Copy link
Contributor

Choose a reason for hiding this comment

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

@lukecarbis Yeah please go ahead and remove it. I'll remember.

Please sign in to comment.