Remove copy field feature.

artefactual · Jun 4, 2024 · 000048b · 000048b
1 parent 9bf3cee
commit 000048b
Showing 1 changed file with 7 additions and 19 deletions.
diff --git a/plugins/arSolrPlugin/lib/arSolrPlugin.class.php b/plugins/arSolrPlugin/lib/arSolrPlugin.class.php
@@ -339,12 +339,10 @@ protected function initialize()
             arSolrPlugin::makeHttpRequest($url);
 
             $addFieldQuery = '';
-            $addCopyFieldQuery = '';
 
             // Add fields to 'all' field
             $q = $this->getFieldQuery('all', 'text_general', true, false, false);
             $addFieldQuery .= $q[0];
-            $addCopyFieldQuery .= $q[1];
 
             $url = $this->solrBaseUrl.'/api/collections/'.$this->solrClientOptions['collection'].'/config/';
             $updateDefaultHandler = '{"update-requesthandler": {"name": "/select", "class": "solr.SearchHandler", "defaults": {"df": "all", "echoParams": "explicit"}}}';
@@ -358,17 +356,14 @@ protected function initialize()
                 $typeName = 'Qubit'.sfInflector::camelize($typeName);
 
                 foreach ($typeProperties['properties'] as $key => $value) {
-                    $includeInCopy = $value['include_in_all'] ? $value['include_in_all'] : true;
                     if (null != $value['type'] && 'nested' !== $value['type'] && 'object' !== $value['type']) {
-                        $q = $this->getFieldQuery($key, $this->setType($value['type']), false, $includeInCopy);
+                        $q = $this->getFieldQuery($key, $this->setType($value['type']), false);
                         $addFieldQuery .= $q[0];
-                        $addCopyFieldQuery .= $q[1];
                     } else {
                         if (null === $value['type']) {
                             // array fields
                             $q = $this->getFieldQuery($key, '_nest_path_', true);
                             $addFieldQuery .= $q[0];
-                            $addCopyFieldQuery .= $q[1];
                             $this->addNestedFields($key, $value['properties']);
                         } else {
                             // object and nested fields
@@ -377,15 +372,13 @@ protected function initialize()
                                 if ('object' === $value['type']) {
                                     foreach ($value['properties'] as $k => $v) {
                                         $fields .= '"'.$k.':/'.$key.'/'.$k.'",';
-                                        $q = $this->getFieldQuery($k, $this->setType($v['type']), false, $includeInCopy);
+                                        $q = $this->getFieldQuery($k, $this->setType($v['type']), false);
                                         $addFieldQuery .= $q[0];
-                                        $addCopyFieldQuery .= $q[1];
                                     }
                                 } elseif (null != $value['type']) {
                                     $fields .= '"'.$key.':/'.$key.'",';
-                                    $q = $this->getFieldQuery($key, $this->setType($value['type']), false, $includeInCopy);
+                                    $q = $this->getFieldQuery($key, $this->setType($value['type']), false);
                                     $addFieldQuery .= $q[0];
-                                    $addCopyFieldQuery .= $q[1];
                                 }
                             }
                             $fields = rtrim($fields, ',').']';
@@ -396,8 +389,7 @@ protected function initialize()
             }
 
             $addFieldQuery = rtrim($addFieldQuery, ',');
-            $addCopyFieldQuery = rtrim($addCopyFieldQuery, ',');
-            $addQuery = '{"add-field":['.$addFieldQuery.'],"add-copy-field":['.$addCopyFieldQuery.']}';
+            $addQuery = '{"add-field":['.$addFieldQuery.']}';
 
             $this->addFieldsToType($addQuery);
         }
@@ -409,8 +401,7 @@ private function addNestedFields($key, $properties)
             if (null === $v['type']) {
                 $this->addNestedFields($k, $v['properties']);
             } else {
-                $includeInCopy = $v['include_in_all'] ? $v['include_in_all'] : true;
-                $this->getFieldQuery($key.'.'.$k, $this->setType($v['type']), false, $includeInCopy);
+                $this->getFieldQuery($key.'.'.$k, $this->setType($v['type']), false);
             }
         }
     }
@@ -446,17 +437,14 @@ private function setType($type)
         return $type;
     }
 
-    private function getFieldQuery($field, $type, $multiValue, $stored = true, $includeInCopy = true)
+    private function getFieldQuery($field, $type, $multiValue, $stored = true)
     {
         $stored = $stored ? 'true' : 'false';
         $multiValue = $multiValue ? 'true' : 'false';
         $addFieldQuery = '{"name": "'.$field.'","stored": "'.$stored.'","type": "'.$type.'","indexed": "true","multiValued": "'.$multiValue.'"},';
-        if ($includeInCopy) {
-            $addCopyFieldQuery = '{"source": "'.$field.'", "dest": "all"},';
-        }
         $this->log(sprintf('Defining mapping %s...', $field));
 
-        return [$addFieldQuery.$addCopyFieldQuery];
+        return $addFieldQuery;
     }
 
     private function addFieldsToType($query)