silverstripe / silverstripe-fulltextsearch

tests/SearchVariantVersionedTest.php

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use SilverStripe\CMS\Model\SiteTree;
 use SilverStripe\Dev\SapphireTest;
 use SilverStripe\FullTextSearch\Search\FullTextSearch;
-use SilverStripe\FullTextSearch\Search\Indexes\SearchIndex_Recording;
 use SilverStripe\FullTextSearch\Search\Variants\SearchVariantVersioned;
 use SilverStripe\FullTextSearch\Tests\SearchVariantVersionedTest\SearchVariantVersionedTest_Index;
 use SilverStripe\FullTextSearch\Tests\SearchVariantVersionedTest\SearchVariantVersionedTest_Item;

tests/SolrReindexTest/SolrReindexTest_Item.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace SilverStripe\FullTextSearch\Tests\SolrReindexTest;
 
-use SilverStripe\Dev\TestOnly;
 use SilverStripe\ORM\DataObject;
 use SilverStripe\FullTextSearch\Tests\SolrReindexTest\SolrReindexTest_ItemExtension;
 

src/Search/FullTextSearch.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 
 use SilverStripe\Core\ClassInfo;
 use SilverStripe\Core\Config\Config;
-use SilverStripe\ORM\DataObject;
 use SilverStripe\FullTextSearch\Search\Indexes\SearchIndex;
 use ReflectionClass;
 

src/Solr/Solr.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace SilverStripe\FullTextSearch\Solr;
 
-use SilverStripe\Control\Director;
 use SilverStripe\Core\Injector\Injector;
 use SilverStripe\Core\Manifest\Module;
 use SilverStripe\Core\Manifest\ModuleLoader;

src/Utils/WebDAV.php

Doc Comments +9 added lines

@@ -3,6 +3,9 @@ 
 
 class WebDAV
 {
+    /**
+     * @param string $method
+     */
     public static function curl_init($url, $method)
     {
         $ch = curl_init($url);
@@ -46,6 +49,9 @@ 
         return $code == 201;
     }
 
+    /**
+     * @param resource $handle
+     */
     public static function put($handle, $url)
     {
         $ch = curl_init($url);
@@ -70,6 +76,9 @@ 
         return self::put($fh, $url);
     }
 
+    /**
+     * @param string $url
+     */
     public static function upload_from_file($string, $url)
     {
         return self::put(fopen($string, 'rb'), $url);

src/Solr/SolrIndex.php

Doc Comments +10 added lines, -1 removed lines

@@ -289,7 +289,7 @@ 
      * @param string $forceType The type to force this field as (required in some cases, when not
      * detectable from metadata)
      * @param array $extraOptions Dependent on search implementation
-     * @param float $boost Numeric boosting value (defaults to 2)
+     * @param integer $boost Numeric boosting value (defaults to 2)
      */
     public function addBoostedField($field, $forceType = null, $extraOptions = array(), $boost = 2)
     {
@@ -298,6 +298,9 @@ 
     }
 
 
+    /**
+     * @param string $field
+     */
     public function fieldData($field, $forceType = null, $extraOptions = array())
     {
         // Ensure that 'boost' is recorded here without being captured by solr
@@ -508,6 +511,9 @@ 
         return $class === $base || is_subclass_of($class, $base);
     }
 
+    /**
+     * @param \Apache_Solr_Document $doc
+     */
     protected function _addField($doc, $object, $field)
     {
         $class = get_class($object);
@@ -557,6 +563,9 @@ 
         }
     }
 
+    /**
+     * @param string $base
+     */
     protected function _addAs($object, $base, $options)
     {
         $includeSubs = $options['include_children'];

src/Search/Variants/SearchVariant.php

Doc Comments +3 added lines, -1 removed lines

@@ -151,6 +151,7 @@ 
      * A shortcut to with when calling without passing in a class,
      *
      * SearchVariant::call(...) ==== SearchVariant::with()->call(...);
+     * @param string $method
      */
 
     public static function call($method, &...$args)
@@ -161,6 +162,7 @@ 
     /**
      * Get the current state of every variant
      * @static
+     * @param string $class
      * @return array
      */
     public static function current_state($class = null, $includeSubclasses = true)
@@ -194,7 +196,7 @@ 
      * @static
      * @param string $class - The class name to get variants for
      * @param bool $includeSubclasses - True if variants should be included if they apply to at least one subclass of $class
-     * @return SearchVariant_ReindexStateIteratorRet - The iterator to foreach loop over
+     * @return \Traversable - The iterator to foreach loop over
      */
     public static function reindex_states($class = null, $includeSubclasses = true)
     {

src/Solr/Tasks/Solr_Reindex.php

Unused Use Statements -2 removed lines

@@ -4,9 +4,7 @@
 use ReflectionClass;
 use SilverStripe\Core\ClassInfo;
 use SilverStripe\Core\Injector\Injector;
-use SilverStripe\Dev\Debug;
 use SilverStripe\FullTextSearch\Search\Variants\SearchVariant;
-use SilverStripe\ORM\DataList;
 use SilverStripe\FullTextSearch\Solr\Reindex\Handlers\SolrReindexHandler;
 use SilverStripe\FullTextSearch\Solr\SolrIndex;
 

src/Search/Updaters/SearchUpdater.php

Unused Use Statements -3 removed lines

@@ -4,15 +4,12 @@
 
 use SilverStripe\Core\Config\Configurable;
 use SilverStripe\Core\Injector\Injector;
-use SilverStripe\Dev\SapphireTest;
 use SilverStripe\ORM\Connect\Database;
 use SilverStripe\ORM\DataObject;
-use SilverStripe\ORM\DB;
 use SilverStripe\FullTextSearch\Search\FullTextSearch;
 use SilverStripe\FullTextSearch\Search\SearchIntrospection;
 use SilverStripe\FullTextSearch\Search\Variants\SearchVariant;
 use SilverStripe\FullTextSearch\Search\Processors\SearchUpdateImmediateProcessor;
-use ReflectionClass;
 
 /**
  * This class is responsible for capturing changes to DataObjects and triggering index updates of the resulting dirty