smtech / canvas-ics-sync

src/SyncIntoCanvas/Filter.php

Doc Comments +9 added lines

@@ -34,6 +34,9 @@ 
         $this->setExcludeExpression($exclude);
     }
 
+    /**
+     * @param boolean $enabled
+     */
     public function setEnabled($enabled)
     {
         $this->enabled = (boolean) $enabled;
@@ -44,6 +47,9 @@ 
         return $enabled;
     }
 
+    /**
+     * @param string|null $regex
+     */
     public function setIncludeExpression($regex)
     {
         $this->include = (string) $regex;
@@ -57,6 +63,9 @@ 
         return $this->include;
     }
 
+    /**
+     * @param string|null $regex
+     */
     public function setExcludeExpression($regex)
     {
         $this->exclude = (string) $regex;

src/SyncIntoCanvas/Calendar.php

Doc Comments +3 added lines

@@ -323,6 +323,9 @@
         $this->log(static::getTimestamp() . ' sync finished', $log);
     }
 
+    /**
+     * @param string $message
+     */
     private function log($message, Log $log, $flag = PEAR_LOG_INFO)
     {
         if ($log) {

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace smtech\CanvasICSSync\SyncIntoCanvas;
 
-use DateTime;
 use vcalendar;
 use Battis\DataUtilities;
 

Spacing +1 added lines, -1 removed lines

@@ -257,7 +257,7 @@
             $this->logThrow(new Exception("Cannot sync calendars without a valid Canvas context"), $log);
         }
 
-        if (!DataUtilities::URLexists($this>getFeedUrl())) {
+        if (!DataUtilities::URLexists($this > getFeedUrl())) {
             $this->logThrow(new Exception("Cannot sync calendars with a valid calendar feed"), $log);
         }
 

src/SyncIntoCanvas/Event.php

Doc Comments +3 added lines

@@ -204,6 +204,9 @@
         }
     }
 
+    /**
+     * @param Calendar $calendar
+     */
     public static function purgeUnmatched($timestamp, $calendar)
     {
         $db = static::getDatabase();

index.php

Spacing +3 added lines, -5 removed lines

@@ -13,8 +13,7 @@ 
 
 /* store any requested actions for future handling */
 $action = (empty($_REQUEST['action']) ?
-    ACTION_UNSPECIFIED :
-    strtolower($_REQUEST['action'])
+    ACTION_UNSPECIFIED : strtolower($_REQUEST['action'])
 );
 
 /* action requests only come from outside the LTI! */
@@ -49,8 +48,7 @@ 
 /* if not authenticated, default to showing credentials */
 } else {
     $action = (empty($action) ?
-        ACTION_CONFIG :
-        $action
+        ACTION_CONFIG : $action
     );
 }
 
@@ -59,7 +57,7 @@ 
     /* reset cached install data from config file */
     case ACTION_INSTALL:
         $_SESSION['toolbox'] = Toolbox::fromConfiguration(CONFIG_FILE, true);
-        $toolbox =& $_SESSION['toolbox'];
+        $toolbox = & $_SESSION['toolbox'];
 
         /* test to see if we can connect to the API */
         try {

common.inc.php

Spacing +2 added lines, -2 removed lines

@@ -13,9 +13,9 @@
 
 /* prepare the toolbox */
 if (empty($_SESSION[Toolbox::class])) {
-    $_SESSION[Toolbox::class] =& Toolbox::fromConfiguration(CONFIG_FILE);
+    $_SESSION[Toolbox::class] = & Toolbox::fromConfiguration(CONFIG_FILE);
 }
-$toolbox =& $_SESSION[Toolbox::class];
+$toolbox = & $_SESSION[Toolbox::class];
 $toolbox->smarty_prependTemplateDir(__DIR__ . '/templates', basename(__DIR__));
 $toolbox->smarty_assign([
     'category' => DataUtilities::titleCase(preg_replace('/[\-_]+/', ' ', basename(__DIR__)))