silverstripe / silverstripe-fulltextsearch

code/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;
 

code/search/updaters/SearchUpdater.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use SilverStripe\Core\Config\Configurable;
 use SilverStripe\Core\Injector\Injector;
-use SilverStripe\Dev\SapphireTest;
 use SilverStripe\ORM\DataObject;
 use SilverStripe\ORM\DB;
 use SilverStripe\FullTextSearch\Search\FullTextSearch;

code/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, &$a1 = null, &$a2 = null, &$a3 = null, &$a4 = null, &$a5 = null, &$a6 = null, &$a7 = null)
     {
@@ -160,6 +161,7 @@ 
     /**
      * Get the current state of every variant
      * @static
+     * @param string $class
      * @return array
      */
     public static function current_state($class = null, $includeSubclasses = true)
@@ -193,7 +195,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)
     {

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace SilverStripe\FullTextSearch\Search\Variants;
 
-use SilverStripe\ORM\DataObject;
 use SilverStripe\Core\ClassInfo;
 use SilverStripe\FullTextSearch\Utils\CombinationsArrayIterator;
 use ReflectionClass;

code/search/variants/SearchVariantSubsites.php

Doc Comments +4 added lines, -1 removed lines

@@ -19,6 +19,9 @@ 
         return class_exists('Subsite');
     }
 
+    /**
+     * @param boolean $includeSubclasses
+     */
     public function appliesTo($class, $includeSubclasses)
     {
         // Include all DataExtensions that contain a SubsiteID.
@@ -135,7 +138,7 @@ 
     /**
      * Determine if a field with a certain name is filtered by the search query or on the index
      * This is the equivalent of saying "show me the results that do ONLY contain this value"
-     * @param $field string name of the field being filtered
+     * @param string $field string name of the field being filtered
      * @param $query SearchQuery currently being executed
      * @param $index SearchIndex which specifies a filter field
      * @return bool true if $field is being filtered, false if it is not being filtered

code/search/variants/SearchVariantVersioned.php

Doc Comments +3 added lines

@@ -10,6 +10,9 @@
 
 class SearchVariantVersioned extends SearchVariant
 {
+    /**
+     * @param boolean $includeSubclasses
+     */
     public function appliesTo($class, $includeSubclasses)
     {
         return SearchIntrospection::has_extension($class, Versioned::class, $includeSubclasses);

code/solr/services/Solr4Service_Core.php

Doc Comments +1 added lines

@@ -38,6 +38,7 @@
     /**
      * Solr 4.0 compat http://wiki.apache.org/solr/UpdateXmlMessages#Optional_attributes_for_.22add.22
      * Remove allowDups, overwritePending and overwriteComitted
+     * @param \Apache_Solr_Document[] $documents
      */
     public function addDocuments(
         $documents,

code/solr/SolrIndex.php

Doc Comments +7 added lines, -1 removed lines

@@ -290,7 +290,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)
     {
@@ -299,6 +299,9 @@ 
     }
 
 
+    /**
+     * @param string $field
+     */
     public function fieldData($field, $forceType = null, $extraOptions = array())
     {
         // Ensure that 'boost' is recorded here without being captured by solr
@@ -558,6 +561,9 @@ 
         }
     }
 
+    /**
+     * @param string $base
+     */
     protected function _addAs($object, $base, $options)
     {
         $includeSubs = $options['include_children'];

code/solr/tasks/Solr_Reindex.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 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;

code/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);