Migration_Favourites_Rework_Revert

Complexity

Total Complexity

4

Size/Duplication

Total Lines	79
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	up()	0	68	2
A	down()	0	3	1

<?php defined('BASEPATH') OR exit('No direct script access allowed');

class Migration_Favourites_Rework_Revert extends CI_Migration {
	public function __construct() {
		parent::__construct();

It seems like you code against a specific sub-type and not the parent class CI_Migration as the method __construct() does only exist in the following sub-classes of CI_Migration: Migration_Auth_add_apikey, Migration_Favourites_Rework, Migration_Favourites_Rework_Revert, Migration_Install_ion_auth, Migration_Setup_Favourites, Migration_Setup_Rate_Limit, Migration_Setup_Sessions, Migration_Setup_Signup_Verification, Migration_Setup_Title_History, Migration_Setup_Tracker, Migration_Setup_User_History, Migration_Setup_User_Options, Migration_Tracker_Add_Active, Migration_Tracker_Add_Site_Status, Migration_Tracker_Add_Title_Status, Migration_Tracker_add_category, Migration_Tracker_add_complete, Migration_Tracker_add_last_checked, Migration_Tracker_add_tags. Maybe you want to instanceof check for one of these explicitly?

		$this->load->dbforge();
	}

	public function up() {
		$this->dbforge->add_field(array(
			'id' => array(
				'type'           => 'MEDIUMINT',
				'constraint'     => '8',
				'unsigned'       => TRUE,
				'auto_increment' => TRUE
			),
			'chapter_id' => array(
				'type'           => 'MEDIUMINT',
				'constraint'     => '8',
				'unsigned'       => TRUE
				//FOREIGN KEY
			),

			'chapter' => array(
				'type'       => 'VARCHAR',
				'constraint' => '255',
				'null'       => FALSE
			),

			'updated_at' => array(
				//Despite not actually creating the field here (it's instead done below), we still need this here so a key can be created properly.
			//	'type'    => 'TIMESTAMP',
			//	'null'    => FALSE,
			//	'on_update' => FALSE
			)
		));
		$this->dbforge->add_field('updated_at TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP'); //CI is annoying and auto-appends ON UPDATE which we don't want.

'updated_at TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP' is of type string, but the function expects a array.

		$this->dbforge->add_key('id', TRUE);
		$this->dbforge->add_key('chapter_id');
		$this->dbforge->add_key('updated_at');
		$this->dbforge->create_table('tracker_favourites_new');

		//We need to add data from the old favourites table
		$query = $this->db->select('
		tf.id AS id,
		tc.id AS chapter_id,
		tf.chapter AS chapter,
		tf.updated_at AS updated_at')
		                  ->from('tracker_favourites AS tf')
		                  //NOTE: This may break, but we'll be only running this once on a live site so it can sort itself out.
		                  ->join('tracker_chapters AS tc', 'tf.user_id = tc.user_id AND tf.title_id = tc.title_id', 'left')
		                  ->order_by('tf.id ASC')
		                  ->get();
		if($data = $query->result_array()) {
			$this->db->insert_batch('tracker_favourites_new', $data);
		}

		//Rename the old DB
		$this->dbforge->rename_table('tracker_favourites', 'tracker_favourites_old');

		//Rename the new one to old name
		$this->dbforge->rename_table('tracker_favourites_new', 'tracker_favourites');


		/*** Unique/Foreign Keys ***/
		//For whatever reason, dbforge lacks a unique/foreign key function.
		$this->db->query('ALTER TABLE `tracker_favourites` ADD UNIQUE(`chapter_id`, `chapter`);');

		$this->db->query('
			ALTER TABLE `tracker_favourites`
				ADD CONSTRAINT `FK_tracker_favourites_tracker_chapters` FOREIGN KEY (`chapter_id`) REFERENCES `tracker_chapters` (`id`) ON UPDATE NO ACTION ON DELETE NO ACTION;'
		);

		//Remove old DB
		$this->dbforge->drop_table('tracker_favourites_old');
	}

	public function down() {
		//There's no going back.
	}
}