+global\Tracker_Admin_Model::updateLatestChapters()

Complexity

Conditions	8
Paths	7

Size

Total Lines	113
Code Lines	61

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	72

Importance

Changes

0

<?php declare(strict_types=1); defined('BASEPATH') OR exit('No direct script access allowed');
	public function updateLatestChapters() {
		// @formatter:off
		$query = $this->db
			->select('
				tracker_titles.id as title_id,
				tracker_titles.title,
				tracker_titles.title_url,
				tracker_titles.status,
				tracker_sites.site,
				tracker_sites.site_class,
				tracker_sites.status,
				tracker_titles.latest_chapter,
				tracker_titles.last_updated,
				from_unixtime(MAX(auth_users.last_login)) AS timestamp
			')
			->from('tracker_titles')
			->join('tracker_sites', 'tracker_sites.id = tracker_titles.site_id', 'left')
			->join('tracker_chapters', 'tracker_titles.id = tracker_chapters.title_id', 'left')
			->join('auth_users', 'tracker_chapters.user_id = auth_users.id', 'left')
			->where('tracker_sites.status', 'enabled')
			->group_start()
				//Check if title is marked as on-going...
				->where('tracker_titles.status', 0)
				//AND matches one of where queries below
				->group_start()
					//Then check if it's NULL (only occurs for new series)
					->where('latest_chapter', NULL)
					//OR if it hasn't updated within the past 12 hours AND isn't a custom update site
					->or_group_start()
						->where('tracker_sites.use_custom', 'N')
						->where('last_checked < DATE_SUB(NOW(), INTERVAL 12 HOUR)')
					->group_end()
					//OR it is a custom update site, has more than one follower and hasn't updated within the past 72 hours.
					->or_group_start()
						->where('tracker_titles.id IN (
							SELECT title_id
							FROM tracker_chapters
							GROUP BY title_id
							HAVING COUNT(title_id) > 1
						)', NULL, FALSE)

						->where('last_checked < DATE_SUB(NOW(), INTERVAL 72 HOUR)')
					->group_end()
					//OR it is a custom update site and hasn't updated within the past 120 hours (5 days)
					->or_where('last_checked < DATE_SUB(NOW(), INTERVAL 120 HOUR)')
				->group_end()
			->group_end()
			->or_group_start()
				//Check if title is marked as complete...
				->where('tracker_titles.status', 1)
				//Then check if it hasn't updated within the past week
				->where('last_checked < DATE_SUB(NOW(), INTERVAL 1 WEEK)')
			->group_end()
			//Status 2 (One-shot) & 255 (Ignore) are both not updated intentionally.
			->group_by('tracker_titles.id, tracker_chapters.active')
			//Check if the series is actually being tracked by someone
			->having('timestamp IS NOT NULL')
			//AND if it's currently marked as active by the user
			->having('tracker_chapters.active', 'Y')
			//AND if they have been active in the last 120 hours (5 days)
			->having('timestamp > DATE_SUB(NOW(), INTERVAL 120 HOUR)')
			->order_by('tracker_titles.title', 'ASC')
			->get();
		// @formatter:on

		if($query->num_rows() > 0) {
			foreach ($query->result() as $row) {
				print "> {$row->title} <{$row->site_class} - {$row->title_url}> | <{$row->title_id}>"; //Print this prior to doing anything so we can more easily find out if something went wrong
				$titleData = $this->sites->{$row->site_class}->getTitleData($row->title_url);
				if(is_array($titleData) && !is_null($titleData['latest_chapter'])) {
					if(count($titleData) === 3) {
						// Normal update.

						//FIXME: "At the moment" we don't seem to be doing anything with TitleData['last_updated'].
						//       Should we even use this? Y/N
						if($this->Tracker->title->updateByID((int) $row->title_id, $titleData['latest_chapter'])) {
							//Make sure last_checked is always updated on successful run.
							//CHECK: Is there a reason we aren't just doing this in updateByID?
							$this->db->set('last_checked', 'CURRENT_TIMESTAMP', FALSE)
							         ->where('id', $row->title_id)
							         ->update('tracker_titles');

							print " - ({$titleData['latest_chapter']})\n";
						} else {
							log_message('error', "{$row->title} failed to update successfully");

							print " - Something went wrong?\n";
						}
					} else {
						// No chapters were returned, but site allows this.
						if($this->Tracker->title->updateByID((int) $row->title_id, NULL)) {
							//Make sure last_checked is always updated on successful run.
							//CHECK: Is there a reason we aren't just doing this in updateByID?
							$this->db->set('last_checked', 'CURRENT_TIMESTAMP', FALSE)
							         ->where('id', $row->title_id)
							         ->update('tracker_titles');

							print " - (No chapters found?)\n";
						} else {
							log_message('error', "{$row->title} failed to update successfully");

							print " - Something went wrong?\n";
						}
					}
				} else {
					log_message('error', "{$row->title} failed to update successfully");
					$this->Tracker->title->updateFailedChecksByID((int) $row->title_id);

					print " - FAILED TO PARSE\n";
				}
			}
		}
	}