silverstripe / silverstripe-subsites

code/tasks/SubsiteCopyPagesTask.php

Doc Comments +3 added lines

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

code/extensions/SiteTreeSubsites.php

Doc Comments +6 added lines, -6 removed lines

@@ -251,8 +251,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 Member|null $member
+     * @return false|null
      */
     public function canEdit($member = null)
     {
@@ -282,7 +282,7 @@ 
 
     /**
      * @param null $member
-     * @return bool
+     * @return false|null
      */
     public function canDelete($member = null)
     {
@@ -295,7 +295,7 @@ 
 
     /**
      * @param null $member
-     * @return bool
+     * @return false|null
      */
     public function canAddChildren($member = null)
     {
@@ -308,7 +308,7 @@ 
 
     /**
      * @param null $member
-     * @return bool
+     * @return false|null
      */
     public function canPublish($member = null)
     {
@@ -448,7 +448,7 @@ 
 
     /**
      * @param Member
-     * @return boolean|null
+     * @return false|null
      */
     public function canCreate($member = null)
     {

code/forms/SubsitesTreeDropdownField.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 
 use SilverStripe\Control\Controller;
 use SilverStripe\Control\HTTPRequest;
-use SilverStripe\Control\Session;
 use SilverStripe\Forms\TreeDropdownField;
 use SilverStripe\View\Requirements;
 

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

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();