From 9f36bb81aebf71015f9cd32b4fc50954930ab09e Mon Sep 17 00:00:00 2001
From: Touhidur Rahman <abir.ruet@gmail.com>
Date: Mon, 25 Nov 2024 00:07:51 +0600
Subject: [PATCH] pkp/pkp-lib#10292 make context id required on controlled
 vocabs operations

---
 .../install/ControlledVocabMigration.php      | 34 +++++++++++++++++++
 dbscripts/xml/install.xml                     |  2 +-
 tools/cleanReviewerInterests.php              |  5 ++-
 3 files changed, 39 insertions(+), 2 deletions(-)
 create mode 100644 classes/migration/install/ControlledVocabMigration.php

diff --git a/classes/migration/install/ControlledVocabMigration.php b/classes/migration/install/ControlledVocabMigration.php
new file mode 100644
index 00000000000..689d71a63cf
--- /dev/null
+++ b/classes/migration/install/ControlledVocabMigration.php
@@ -0,0 +1,34 @@
+<?php
+
+/**
+ * @file classes/migration/install/ControlledVocabMigration.php
+ *
+ * Copyright (c) 2024 Simon Fraser University
+ * Copyright (c) 2024 John Willinsky
+ * Distributed under the GNU GPL v3. For full terms see the file docs/COPYING.
+ *
+ * @class ControlledVocabMigration
+ *
+ * @brief Describe database table structures.
+ */
+
+namespace APP\migration\install;
+
+class ControlledVocabMigration extends \PKP\migration\install\ControlledVocabMigration
+{
+    /**
+     * @copydoc \PKP\migration\install\ControlledVocabMigration::getContextTable()
+     */
+    protected function getContextTable(): string
+    {
+        return 'journals';
+    }
+
+    /**
+     * @copydoc \PKP\migration\install\ControlledVocabMigration::getContextPrimaryKey()
+     */
+    protected function getContextPrimaryKey(): string
+    {
+        return 'journal_id';
+    }
+}
diff --git a/dbscripts/xml/install.xml b/dbscripts/xml/install.xml
index ef296cee6dc..bd0a103c072 100644
--- a/dbscripts/xml/install.xml
+++ b/dbscripts/xml/install.xml
@@ -21,7 +21,7 @@
 	<migration class="PKP\migration\install\CommonMigration" />
 	<migration class="PKP\migration\install\SessionsMigration" />
 	<migration class="PKP\migration\install\GenresMigration" />
-	<migration class="PKP\migration\install\ControlledVocabMigration" />
+	<migration class="APP\migration\install\ControlledVocabMigration" />
 	<migration class="PKP\migration\install\FailedJobsMigration" />
 	<migration class="PKP\migration\install\FilesMigration" />
 	<migration class="PKP\migration\install\JobsMigration" />
diff --git a/tools/cleanReviewerInterests.php b/tools/cleanReviewerInterests.php
index ca161309b58..d127033cf19 100755
--- a/tools/cleanReviewerInterests.php
+++ b/tools/cleanReviewerInterests.php
@@ -90,7 +90,10 @@ public function execute(): void
     protected function _getOrphanVocabInterests(): Collection
     {
         $controlledVocab = Repo::controlledVocab()->build(
-            UserInterest::CONTROLLED_VOCAB_INTEREST
+            UserInterest::CONTROLLED_VOCAB_INTEREST,
+            UserInterest::CONTROLLED_VOCAB_INTEREST_ASSOC_TYPE,
+            UserInterest::CONTROLLED_VOCAB_INTEREST_ASSOC_ID,
+            UserInterest::CONTROLLED_VOCAB_INTEREST_CONTEXT_ID
         );
 
         return ControlledVocabEntry::query()