Skip to content

Commit

Permalink
Last minute fix for people who already installed the ext
Browse files Browse the repository at this point in the history
Maybe they run into errors because I had to rename the migration class
I added the effectively_installed() function then to second migration
because it made me feel more safe.
  • Loading branch information
Elsensee committed Jan 8, 2015
1 parent 8f2537e commit 2bacbdb
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 0 deletions.
7 changes: 7 additions & 0 deletions migrations/release_1_0_0_rc1.php
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,13 @@

class release_1_0_0_rc1 extends \phpbb\db\migration\migration
{
public function effectively_installed()
{
// This should be implemented because I had to rename the class (from uppercase to lowercase)
// And checking for config should be enough because schema won't exist if config doesn't exist.
return isset($this->config['ppp_maximum_ppp']);
}

public function update_schema()
{
return array(
Expand Down
5 changes: 5 additions & 0 deletions migrations/release_1_0_0_rc2.php
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,11 @@ static public function depends_on()
return array('\elsensee\postsperpage\migrations\release_1_0_0_rc1');
}

public function effectively_installed()
{
return isset($this->config['ppp_maximum_topic_ppp']);
}

public function update_schema()
{
return array(
Expand Down

0 comments on commit 2bacbdb

Please sign in to comment.