silverstripe / silverstripe-subsites

tests/php/FileSubsitesTest.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use SilverStripe\Assets\File;
 use SilverStripe\Assets\Folder;
 use SilverStripe\Forms\FieldList;
-use SilverStripe\Security\Member;
 use SilverStripe\Subsites\Extensions\FileSubsites;
 use SilverStripe\Subsites\Model\Subsite;
 

tests/php/SubsitesVirtualPageTest.php

Doc Comments +3 added lines

@@ -303,6 +303,9 @@
         );
     }
 
+    /**
+     * @param SubsitesVirtualPage $page
+     */
     protected function fixVersionNumberCache($page)
     {
         $pages = func_get_args();

tests/php/SubsiteAdminFunctionalTest.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use SilverStripe\Core\Config\Config;
 use SilverStripe\Dev\FunctionalTest;
 use SilverStripe\Subsites\Model\Subsite;
-use SilverStripe\Subsites\State\SubsiteState;
 
 class SubsiteAdminFunctionalTest extends FunctionalTest
 {

src/Extension/SiteTreeSubsites.php

Doc Comments +6 added lines, -6 removed lines

@@ -261,8 +261,8 @@ 
      * - Is in a group which has access to the subsite this page belongs to
      * - Is in a group with edit permissions on the "main site"
      *
-     * @param null $member
-     * @return bool
+     * @param null|\SilverStripe\Security\Member $member
+     * @return false|null
      */
     public function canEdit($member = null)
     {
@@ -292,7 +292,7 @@ 
 
     /**
      * @param null $member
-     * @return bool
+     * @return false|null
      */
     public function canDelete($member = null)
     {
@@ -305,7 +305,7 @@ 
 
     /**
      * @param null $member
-     * @return bool
+     * @return false|null
      */
     public function canAddChildren($member = null)
     {
@@ -318,7 +318,7 @@ 
 
     /**
      * @param null $member
-     * @return bool
+     * @return false|null
      */
     public function canPublish($member = null)
     {
@@ -460,7 +460,7 @@ 
 
     /**
      * @param Member
-     * @return boolean|null
+     * @return false|null
      */
     public function canCreate($member = null)
     {

src/Forms/SubsitesTreeDropdownField.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use SilverStripe\Control\Controller;
 use SilverStripe\Control\HTTPRequest;
-use SilverStripe\Control\Session;
 use SilverStripe\Core\Manifest\ModuleLoader;
 use SilverStripe\Forms\TreeDropdownField;
 use SilverStripe\View\Requirements;

src/Model/Subsite.php

Doc Comments +6 added lines, -2 removed lines

@@ -204,7 +204,7 @@ 
      * for example matching all subdomains on *.example.com with one subsite,
      * and all subdomains on *.example.org on another.
      *
-     * @param $host string The host to find the subsite for.  If not specified, $_SERVER['HTTP_HOST'] is used.
+     * @param string $host string The host to find the subsite for.  If not specified, $_SERVER['HTTP_HOST'] is used.
      * @param bool $checkPermissions
      * @return int Subsite ID
      */
@@ -329,6 +329,10 @@ 
      *
      * @return ArrayList of {@link Subsite} instances.
      */
+
+    /**
+     * @param DataObject $member
+     */
     public static function all_accessible_sites($includeMainSite = true, $mainSiteTitle = 'Main site', $member = null)
     {
         // Rationalise member arguments
@@ -371,7 +375,7 @@ 
      * @param $permCode array|string Either a single permission code or an array of permission codes.
      * @param $includeMainSite bool If true, the main site will be included if appropriate.
      * @param $mainSiteTitle string The label to give to the main site
-     * @param $member int|Member The member attempting to access the sites
+     * @param DataObject $member int|Member The member attempting to access the sites
      * @return DataList|ArrayList of {@link Subsite} instances
      */
     public static function accessible_sites(

src/Tasks/SubsiteCopyPagesTask.php

Doc Comments +3 added lines

@@ -70,6 +70,9 @@
         }
     }
 
+    /**
+     * @param string $msg
+     */
     public function log($msg)
     {
         echo $msg . "\n";