Completed
Push — authenticator-refactor ( 638690...7dc887 )
by Simon
06:10
created
tests/php/Forms/GridField/GridFieldEditButtonTest.php 1 patch
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -8,10 +8,8 @@
 block discarded – undo
8 8
 use SilverStripe\Forms\Tests\GridField\GridFieldTest\Team;
9 9
 use SilverStripe\ORM\ArrayList;
10 10
 use SilverStripe\ORM\DataList;
11
-use SilverStripe\Security\Member;
12 11
 use SilverStripe\Dev\CSSContentParser;
13 12
 use SilverStripe\Dev\SapphireTest;
14
-use SilverStripe\Control\Controller;
15 13
 use SilverStripe\Forms\FieldList;
16 14
 use SilverStripe\Forms\Form;
17 15
 use SilverStripe\Forms\GridField\GridFieldConfig;
Please login to merge, or discard this patch.
src/Core/Convert.php 1 patch
Doc Comments   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -304,6 +304,7 @@  discard block
 block discarded – undo
304 304
      * call this function directly, Please use {@link Convert::xml2array()}
305 305
      *
306 306
      * @param SimpleXMLElement
307
+     * @param SimpleXMLElement $xml
307 308
      *
308 309
      * @return mixed
309 310
      */
@@ -517,7 +518,7 @@  discard block
 block discarded – undo
517 518
      * - IDField => idField
518 519
      * - iDField => iDField
519 520
      *
520
-     * @param $str
521
+     * @param string $str
521 522
      * @return string
522 523
      */
523 524
     public static function upperCamelToLowerCamel($str)
Please login to merge, or discard this patch.
src/Core/Injector/Injector.php 1 patch
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -372,7 +372,7 @@  discard block
 block discarded – undo
372 372
      *
373 373
      * @param string $property
374 374
      *                the name of the property
375
-     * @param object $object
375
+     * @param string $object
376 376
      *                the object to be set
377 377
      * @return $this
378 378
      */
@@ -467,7 +467,7 @@  discard block
 block discarded – undo
467 467
      *              The name of the service to update the definition for
468 468
      * @param string $property
469 469
      *              The name of the property to update.
470
-     * @param mixed $value
470
+     * @param string $value
471 471
      *              The value to set
472 472
      * @param boolean $append
473 473
      *              Whether to append (the default) when the property is an array
@@ -789,7 +789,7 @@  discard block
 block discarded – undo
789 789
     /**
790 790
      * @deprecated 4.0.0:5.0.0 Use Injector::has() instead
791 791
      * @param $name
792
-     * @return string
792
+     * @return boolean
793 793
      */
794 794
     public function hasService($name)
795 795
     {
Please login to merge, or discard this patch.
src/ORM/Connect/DBSchemaManager.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -331,7 +331,7 @@  discard block
 block discarded – undo
331 331
      *   - array('fields' => array('A','B','C'), 'type' => 'index/unique/fulltext'): This gives you full
332 332
      *     control over the index.
333 333
      * @param boolean $hasAutoIncPK A flag indicating that the primary key on this table is an autoincrement type
334
-     * @param array $options Create table options (ENGINE, etc.)
334
+     * @param string|null $options Create table options (ENGINE, etc.)
335 335
      * @param array|bool $extensions List of extensions
336 336
      */
337 337
     public function requireTable(
@@ -595,7 +595,7 @@  discard block
 block discarded – undo
595 595
      *
596 596
      * @param string $table The table name.
597 597
      * @param string $field The field name.
598
-     * @param array|string $spec The field specification. If passed in array syntax, the specific database
598
+     * @param string $spec The field specification. If passed in array syntax, the specific database
599 599
      *  driver takes care of the ALTER TABLE syntax. If passed as a string, its assumed to
600 600
      *  be prepared as a direct SQL framgment ready for insertion into ALTER TABLE. In this case you'll
601 601
      *  need to take care of database abstraction in your DBField subclass.
Please login to merge, or discard this patch.
src/ORM/DataObjectSchema.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -488,7 +488,7 @@
 block discarded – undo
488 488
      * Cache all indexes for the given class.
489 489
      * Will do nothing if already cached
490 490
      *
491
-     * @param $class
491
+     * @param string $class
492 492
      */
493 493
     protected function cacheDatabaseIndexes($class)
494 494
     {
Please login to merge, or discard this patch.
src/ORM/Search/FulltextSearchable.php 1 patch
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -4,9 +4,7 @@
 block discarded – undo
4 4
 
5 5
 use SilverStripe\Assets\File;
6 6
 use SilverStripe\CMS\Model\SiteTree;
7
-use SilverStripe\Core\Config\Config;
8 7
 use SilverStripe\CMS\Controllers\ContentController;
9
-use SilverStripe\ORM\Connect\MySQLSchemaManager;
10 8
 use SilverStripe\ORM\DataExtension;
11 9
 use Exception;
12 10
 
Please login to merge, or discard this patch.
src/Security/MemberAuthenticator/LogoutHandler.php 1 patch
Unused Use Statements   -3 removed lines patch added patch discarded remove patch
@@ -2,13 +2,10 @@
 block discarded – undo
2 2
 
3 3
 namespace SilverStripe\Security\MemberAuthenticator;
4 4
 
5
-use SilverStripe\Control\Cookie;
6 5
 use SilverStripe\Control\RequestHandler;
7
-use SilverStripe\Control\Session;
8 6
 use SilverStripe\Core\Injector\Injector;
9 7
 use SilverStripe\Security\IdentityStore;
10 8
 use SilverStripe\Security\Member;
11
-use SilverStripe\Security\RememberLoginHash;
12 9
 use SilverStripe\Security\Security;
13 10
 
14 11
 /**
Please login to merge, or discard this patch.
src/Security/MemberAuthenticator/SessionAuthenticationHandler.php 1 patch
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -3,10 +3,8 @@
 block discarded – undo
3 3
 namespace SilverStripe\Security\MemberAuthenticator;
4 4
 
5 5
 use SilverStripe\Control\Cookie;
6
-use SilverStripe\ORM\DataObject;
7 6
 use SilverStripe\Security\Member;
8 7
 use SilverStripe\Control\HTTPRequest;
9
-use SilverStripe\Control\HTTPResponse;
10 8
 use SilverStripe\Control\Session;
11 9
 use SilverStripe\Control\Director;
12 10
 use SilverStripe\Security\AuthenticationHandler as AuthenticationHandlerInterface;
Please login to merge, or discard this patch.