Implement saving queries.

Bryan Cuccioli · epriestley · commit 7ad2eae47f35 · 2013-04-30T10:48:16.000-07:00
Summary: Enable saved query objects to actually be saved to the database. Test Plan: Insert a call to save() and check that the query is written correctly. Reviewers: epriestley Reviewed By: epriestley CC: aran, Korvin, AnhNhan Maniphest Tasks: T2625 Differential Revision: https://secure.phabricator.com/D5775 Conflicts: src/infrastructure/storage/patch/PhabricatorBuiltinPatchList.php
diff --git a/resources/sql/patches/20130426.search_savedquery.sql b/resources/sql/patches/20130426.search_savedquery.sql
@@ -0,0 +1,11 @@
+CREATE TABLE {$NAMESPACE}_search.search_savedquery (
+  id INT(10) UNSIGNED NOT NULL AUTO_INCREMENT,
+  engineClassName VARCHAR(255) NOT NULL COLLATE utf8_bin,
+  parameters LONGTEXT NOT NULL COLLATE utf8_bin,
+  dateCreated INT(10) UNSIGNED NOT NULL,
+  dateModified INT(10) UNSIGNED NOT NULL,
+  queryKey VARCHAR(12) NOT NULL COLLATE utf8_bin,
+  PRIMARY KEY(id),
+  UNIQUE KEY key_queryKey (queryKey)
+)
+ENGINE=InnoDB, COLLATE utf8_general_ci
diff --git a/src/applications/search/storage/PhabricatorSavedQuery.php b/src/applications/search/storage/PhabricatorSavedQuery.php
@@ -6,6 +6,8 @@
 final class PhabricatorSavedQuery extends PhabricatorSearchDAO {
 
   protected $parameters = array();
+  protected $queryKey = "";
+  protected $engineClassName = "PhabricatorPasteSearchEngine";
 
   public function getConfiguration() {
     return array(
@@ -22,4 +24,15 @@ public function setParameter($key, $value) {
   public function getParameter($key, $default = null) {
     return idx($this->parameters, $key, $default);
   }
+
+  public function save() {
+    if ($this->getEngineClass() === null) {
+      throw new Exception(pht("Engine class is null."));
+    }
+
+    $serial = $this->getEngineClass().serialize($this->parameters);
+    $this->queryKey = PhabricatorHash::digestForIndex($serial);
+
+    return parent::save();
+  }
 }
diff --git a/src/infrastructure/storage/patch/PhabricatorBuiltinPatchList.php b/src/infrastructure/storage/patch/PhabricatorBuiltinPatchList.php
@@ -1254,6 +1254,10 @@ public function getPatches() {
         'type' => 'sql',
         'name' => $this->getPatchPath('20130423.conpherenceindices.sql'),
       ),
+      '20130426.search_savedquery.sql' => array(
+        'type' => 'sql',
+        'name' => $this->getPatchPath('20130426.search_savedquery.sql'),
+      ),
     );
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -1254,6 +1254,10 @@ public function getPatches() {`
`1254`	`1254`	`'type' => 'sql',`
`1255`	`1255`	`'name' => $this->getPatchPath('20130423.conpherenceindices.sql'),`
`1256`	`1256`	`),`
	`1257`	`+ '20130426.search_savedquery.sql' => array(`
	`1258`	`+ 'type' => 'sql',`
	`1259`	`+ 'name' => $this->getPatchPath('20130426.search_savedquery.sql'),`
	`1260`	`+ ),`
`1257`	`1261`	`);`
`1258`	`1262`	`}`
`1259`	`1263`	`}`