Differential D10035 Diff 30222 src/applications/macro/storage/PhabricatorFileImageMacroUsage.php

Changeset View

Standalone View

				<?php
				final class PhabricatorFileImageMacroUsage extends PhabricatorFileDAO {

				/********************************/
				/* Update our macro usage table */
				nipunnAuthorUnsubmitted Not Done Inline Actions This is nasty because it actually backfills/populates the table synchronous to the request. We cap the amount of backfill done per-request so as not to timeout / overload a mysql query. But there's gotta be a better way. Is there a way to do a backfill as part of a migration? Perhaps as an async background task? nipunn: This is nasty because it actually backfills/populates the table synchronous to the request. We…
				/********************************/

				private $macroPHID;
				private $authorPHID;
				private $count;

				protected function getConfiguration() {
				return array(
				self::CONFIG_COLUMN_SCHEMA => array(
				'macroPHID' => 'phid?',
				'authorPHID' => 'phid?',
				'count' => 'uint32',
				),
				self::CONFIG_KEY_SCHEMA => array(
				),
				) + parent::getConfiguration();
				}

				public function updateUsage($author_phid, $macros) {
				if (empty($macros)) {
				return;
				}

				$conn = $this->establishConnection('w');

				// Append to our wonderful macrousage table
				$values = array();
				foreach ($macros as $macro_phid => $count) {
				$values[] = qsprintf(
				$conn,
				'(%s, %s, %d)',
				$macro_phid,
				$author_phid,
				$count);
				}
				$values_clause = implode(',', $values);

				queryfx(
				$conn,
				'INSERT INTO %T
				(macroPHID, authorPHID, count)
				VALUES %Q',
				$this->getTableName(),
				$values_clause);
				}
				}