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;

Indentation +8 added lines, -8 removed lines

@@ -76,17 +76,17 @@
             // include this event if filtering is off...
             $this->isEnabled() == false ||
             (
-               (
-                   ( // if filtering is on, and there's an include pattern test that pattern...
-                       $this->getIncludeExpression() &&
+                (
+                    ( // if filtering is on, and there's an include pattern test that pattern...
+                        $this->getIncludeExpression() &&
                        preg_match('%' . $this->getIncludeExpression() . '%', $event->getProperty('SUMMARY'))
-                   )
-               ) &&
+                    )
+                ) &&
                !( // if there is an exclude pattern, make sure that this event is NOT excluded
-                   $this->getExcludeExpression() &&
+                    $this->getExcludeExpression() &&
                    preg_match('%' . $this->getExcludeExpression() . '%', $event->getProperty('SUMMARY'))
-               )
-           )
+                )
+            )
         );
     }
 }

src/SyncIntoCanvas/Syncable.php

Doc Comments -1 removed lines

@@ -53,7 +53,6 @@
     /**
      * Update the MySQL connection
      *
-     * @param PDO $db
      * @throws Exception If `$db` is null
      */
     public static function setDatabase(PDO $database)

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 +3 added lines, -3 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);
         }
 
@@ -285,10 +285,10 @@ 
          *     today's date, probably a better system? issue:12
          */
         foreach ($ics->selectComponents(
-            date('Y')-1, // startYear
+            date('Y') - 1, // startYear
             date('m'), // startMonth
             date('d'), // startDay
-            date('Y')+1, // endYEar
+            date('Y') + 1, // endYEar
             date('m'), // endMonth
             date('d'), // endDay
             'vevent', // cType

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__)))

import.php

Indentation +3 added lines, -3 removed lines

@@ -73,9 +73,9 @@
 function filterEvent($event, $calendarCache)
 {
 
-     return (
-         // include this event if filtering is off...
-         $calendarCache['enable_regexp_filter'] == false ||
+        return (
+            // include this event if filtering is off...
+            $calendarCache['enable_regexp_filter'] == false ||
          (
             (
                 ( // if filtering is on, and there's an include pattern test that pattern...

Spacing +2 added lines, -2 removed lines

@@ -186,10 +186,10 @@
                 // TODO:0 the best window for syncing would be the term of the course in question, right? issue:12
                 // TODO:0 Arbitrarily selecting events in for a year on either side of today's date, probably a better system? issue:12
                 foreach ($ics->selectComponents(
-                    date('Y')-1, // startYear
+                    date('Y') - 1, // startYear
                     date('m'), // startMonth
                     date('d'), // startDay
-                    date('Y')+1, // endYEar
+                    date('Y') + 1, // endYEar
                     date('m'), // endMonth
                     date('d'), // endDay
                     'vevent', // cType

Braces +3 added lines, -1 removed lines

@@ -424,7 +424,9 @@
             'The Canvas URL you submitted could not be parsed.<pre>' . $_REQUEST['canvas_url'] . '</pre>',
             NotificationMessage::ERROR
         );
-        if (php_sapi_name() != 'cli') $smarty->display('page.tpl');
+        if (php_sapi_name() != 'cli') {
+            $smarty->display('page.tpl');
+        }
         exit;
     }
 }

visualize.php

Spacing +1 added lines, -1 removed lines

@@ -33,5 +33,5 @@
 }
 
 $smarty->assign('ics', $ics);
-$smarty->assign('veventProperties', array('unique' => array('CLASS','CREATED','SUMMARY','DESCRIPTION','DTSTART','X-CURRENT-DTSTART','DTEND','X-CURRENT-DTEND','DURATION','GEO','LAST-MOD','LOCATION','ORGANIZER','PRIORITY','DTSTAMP','SEQ','STATUS','TRANSP','UID','URL','RECURID'),'multiple'=>array('ATTACH','ATTENDEE','CATEGORIES','COMMENT','CONTACT','EXDATE','EXRULE','RSTATUS','RELATED','RESOURCES','RDATE','RRULE','X-PROP')));
+$smarty->assign('veventProperties', array('unique' => array('CLASS', 'CREATED', 'SUMMARY', 'DESCRIPTION', 'DTSTART', 'X-CURRENT-DTSTART', 'DTEND', 'X-CURRENT-DTEND', 'DURATION', 'GEO', 'LAST-MOD', 'LOCATION', 'ORGANIZER', 'PRIORITY', 'DTSTAMP', 'SEQ', 'STATUS', 'TRANSP', 'UID', 'URL', 'RECURID'), 'multiple'=>array('ATTACH', 'ATTENDEE', 'CATEGORIES', 'COMMENT', 'CONTACT', 'EXDATE', 'EXRULE', 'RSTATUS', 'RELATED', 'RESOURCES', 'RDATE', 'RRULE', 'X-PROP')));
 $smarty->display('visualize.tpl');