GCTC-NTGC / TalentCloud

public_html/inc/applicant/page-view-job-poster.php

Braces +1 added lines, -1 removed lines

@@ -240,7 +240,7 @@
                         $loginLink .= "?response_type=" . URL_RESPONSE_TYPES;
                         if (strpos($url, 'admin') !== false) {
                             $loginLink .= "&redirect_uri=" . urlencode(REDIRECT_URI_ADMIN);
-                        }else {
+                        } else {
                             $loginLink .= "&redirect_uri=" . urlencode(REDIRECT_URI);
                         }
                         $loginLink .= "&nonce=" . $nonce;

public_html/inc/common/landing-hero.php

Braces +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
                 $url = 'http://' . $_SERVER['SERVER_NAME'] . $_SERVER['REQUEST_URI'];
                 if (strpos($url, 'admin') !== false) {
                     echo '<p id="adminTagline" class="centered">Changing government through it\'s people</p>';
-                }else {
+                } else {
 
                 }
             ?>

public_html/inc/common/header-nav.php

Braces +25 added lines, -10 removed lines

@@ -19,23 +19,29 @@ 
             <div class="page-hero__navigation-item box med-1of5 lg-1of6" id="navigationHomeLinkWrapper" aria-hidden="false">
                 <?php if (strpos($url, 'admin') !== false) : ?>
                     <a href="/admin/" id="navigationHomeLink">Home</a>
-                <?php else : ?>
+                <?php else {
+    : ?>
                     <a href="/" id="navigationHomeLink">Home</a>
-                <?php endif; ?>
+                <?php endif;
+}
+?>
             </div>
 
             <?php /* Checks to see if the page is the admin portal and then displays the admin home link. */ ?>
             <?php $url = 'http://' . $_SERVER['SERVER_NAME'] . $_SERVER['REQUEST_URI']; ?>
             <?php if (strpos($url, 'admin') !== false) : ?>
                 <?php /* No link for you! */ ?>
-            <?php else : ?>
+            <?php else {
+    : ?>
                 <?php /* Browse Jobs */ ?>
                 <div class="page-hero__navigation-item box med-1of5 lg-1of6" id="navigationBrowseLinkWrapper" aria-hidden="false">
                     <a href="#BrowseJobs" id="navigationBrowseLink" onclick="JobPostAPI.showBrowseJobs();TalentCloudAPI.setNav('navigationBrowseLinkWrapper');">
                         Browse Jobs
                     </a>
                 </div>
-            <?php endif; ?>
+            <?php endif;
+}
+?>
 
             <?php /* Dashboard */ ?>
             <div class="page-hero__navigation-item box med-1of5 lg-1of6 hidden" id="navigationDashboardLinkWrapper" aria-hidden="true">
@@ -78,18 +84,24 @@ 
                     <a href="javascript:void(0)" id="navigationPosterLink" onclick="CreateJobPosterAPI.showCreateJobPosterForm(this);TalentCloudAPI.setNav('navigationPosterLinkWrapper');">Job Poster
                     </a>
                 </div>
-            <?php else : ?>
+            <?php else {
+    : ?>
                 <?php /* No link for you! */ ?>
-            <?php endif; ?>
+            <?php endif;
+}
+?>
 
             <?php /* Register */
                 if (strpos($url, 'admin') !== false) : ?>
                     <div class="page-hero__navigation-item box med-1of5 lg-1of6 hidden" id="navigationRegisterLinkWrapper" aria-hidden="true">
                         <?php echo("<a href=\"\" id=\"navigationRegisterLink\">Register</a>"); ?>
                     </div>
-                <?php else : ?>
+                <?php else {
+    : ?>
                     <div class="page-hero__navigation-item box med-1of5 lg-1of6" id="navigationRegisterLinkWrapper" aria-hidden="false">
-                        <?php echo("<a href=\"https://account.gccollab.ca/register?redirect_uri=https%3A%2F%2Ftc.gccollab.ca%2F\" id=\"navigationRegisterLink\">Register</a>"); ?>
+                        <?php echo("<a href=\"https://account.gccollab.ca/register?redirect_uri=https%3A%2F%2Ftc.gccollab.ca%2F\" id=\"navigationRegisterLink\">Register</a>");
+}
+?>
                     </div>
                 <?php endif; ?>
 
@@ -125,13 +137,16 @@ 
                         ?>
                     </div>
 
-            <?php else : ?>
+            <?php else {
+    : ?>
 
                 <?php if ($hasUser != null) : ?>
                     <div class="page-hero__navigation-item box med-1of5 lg-1of6 hidden" id="navigationLoginLinkWrapper">
                 <?php else :?>
                     <div class="page-hero__navigation-item box med-1of5 lg-1of6" id="navigationLoginLinkWrapper">
-                <?php endif; ?>
+                <?php endif;
+}
+?>
                         <?php
                             $loginLink = "";
                             $loginLink .= OPENID_URI . AUTH_URI;

public_html/index.php

Braces +2 added lines, -2 removed lines

@@ -62,7 +62,7 @@ 
 
         header("Refresh:0; url=\"" . REDIRECT_URI . "");
     }
-}else {
+} else {
     $_SESSION['openid_connect_state'] = $state;
     $_SESSION['openid_connect_nonce'] = $nonce;
 }
@@ -108,7 +108,7 @@ 
                 echo("UserAPI.storeSessionUser(" . json_encode($userInfo) . ");");
                 echo("UserAPI.login();");
             }
-        }else {
+        } else {
             echo("UserAPI.login();");
         }
         //var isExistingUser = UserAPI.authenticate(UserAPI.getSessionUserAsJSON());

public_html/tc/dao/JobPosterDAO.php

Braces +12 added lines, -8 removed lines

@@ -503,14 +503,18 @@
             $sqlEssentialType->execute() or die("ERROR: " . implode(":", $link->errorInfo()));

             $sqlAssetType->execute() or die("ERROR: " . implode(":", $link->errorInfo()));

             

-            if (sizeof($key_task_data) > 0)

-                    $sql4->execute($key_task_data) or die("ERROR: " . implode(":", $link->errorInfo()));

-            if (sizeof($core_competency_data) > 0)

-                $sql5->execute($core_competency_data) or die("ERROR: " . implode(":", $link->errorInfo()));

-            if (sizeof($dev_competency_data) > 0)

-                $sql6->execute($dev_competency_data) or die("ERROR: " . implode(":", $link->errorInfo()));

-            if (sizeof($question_data) > 0)

-                $sqlQuestions->execute($question_data) or die("ERROR: " . implode(":", $link->errorInfo()));

+            if (sizeof($key_task_data) > 0) {
+                                $sql4->execute($key_task_data) or die("ERROR: " . implode(":", $link->errorInfo()));
+            }
+            if (sizeof($core_competency_data) > 0) {
+                            $sql5->execute($core_competency_data) or die("ERROR: " . implode(":", $link->errorInfo()));
+            }
+            if (sizeof($dev_competency_data) > 0) {
+                            $sql6->execute($dev_competency_data) or die("ERROR: " . implode(":", $link->errorInfo()));
+            }
+            if (sizeof($question_data) > 0) {
+                            $sqlQuestions->execute($question_data) or die("ERROR: " . implode(":", $link->errorInfo()));
+            }
 

             $link->commit();

         } catch (PDOException $e) {

public_html/tc/index.php

Braces +3 added lines, -3 removed lines

@@ -8,7 +8,7 @@ 
         header('HTTP/1.0 401 Unauthorized');
         echo 'Text to send if user hits Cancel button';
         exit;
-    }else {
+    } else {
         if ($_SERVER['PHP_AUTH_USER'] == 'test' && $_SERVER['PHP_AUTH_PW'] == 'test') {
             header("Access-Control-Allow-Origin: *");
             header("Content-Type: application/json");
@@ -25,7 +25,7 @@ 
                         $result = array('user' => 'test', 'pass' => 'test');
                         $json = json_encode($result);
                         echo($json);
-                    }else {
+                    } else {
                         $result1 = array('user' => 'test', 'pass' => 'test');
                         $result2 = array('user' => 'test1', 'pass' => 'test1');
                         $result3 = array('user' => 'test2', 'pass' => 'test2');
@@ -55,7 +55,7 @@ 
                     echo("");
                     break;
             }
-        }else {
+        } else {
             header('WWW-Authenticate: Basic realm="My Realm"');
             header('HTTP/1.0 401 Unauthorized');
             echo 'Authorization declined';

public_html/tc/controller/ManagerProfileController.php

Braces +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
             }
             
             return ["manager_profile_id"=>$profileId];
-        }else {
+        } else {
             $profileId = ManagerProfileDAO::createManagerProfile($managerProfile, $managerProfileDetails);
             return ["manager_profile_id"=>$profileId];
         }

public_html/tc/controller/WorkEnvironmentController.php

Braces +5 added lines, -5 removed lines

@@ -18,7 +18,7 @@ 
         if ($workEnvId === 0) {
             $workEnvironment = self::createWorkEnvironment($workEnvironment);
             WorkEnvironmentDAO::setManagerProfileWorkEnvironment($managerProfileId, $workEnvironment->getBasic_work_environment()->getId());
-        }else {
+        } else {
             $workEnvironment->getBasic_work_environment()->setId($workEnvId);
             self::updateWorkEnvironment($workEnvironment);
         }
@@ -44,7 +44,7 @@ 
                 //already exists, so just update description
                 $oldCaption->setDescription($newCaption->getDescription());
                 WorkEnvironmentDAO::updateWorkplacePhotoCaption($oldCaption);
-            }else { 
+            } else { 
                 //doesn't exist, so make sure it has correct work env id and blank photo id, and insert it
                 $newCaption->setWork_environment_id($workEnvId);
                 $newCaption->setWorkplace_photo_id(null);
@@ -71,7 +71,7 @@ 
                 //already exists, so just update description
                 $oldCaption->setDescription($newCaption->getDescription());
                 WorkEnvironmentDAO::updateWorkplacePhotoCaption($oldCaption);
-            }else { 
+            } else { 
                 //doesn't exist, so make sure it has correct work env id and blank photo id, and insert it
                 $newCaption->setWork_environment_id($workEnvId);
                 $newCaption->setWorkplace_photo_id(null);
@@ -97,7 +97,7 @@ 
         $exists = WorkEnvironmentDAO::workplacePhotoExistsForManagerAndName($managerProfileId, $photoName);
         if ($exists) {
             WorkEnvironmentDAO::updateWorkplacePhoto($workplacePhoto, $managerProfileId, $photoName);      
-        }else {
+        } else {
             
             $photoId = WorkEnvironmentDAO::insertWorkplacePhoto($workplacePhoto);
 
@@ -113,7 +113,7 @@ 
             if ($caption) {
                 $caption->setWorkplace_photo_id($photoId);
                 WorkEnvironmentDAO::updateWorkplacePhotoCaption($caption);
-            }else {                
+            } else {                
                 $caption = new WorkplacePhotoCaption($workEnvironmentId, $photoName, $photoId, '');      
                 WorkEnvironmentDAO::insertWorkplacePhotoCaption($caption);
             }

public_html/tc/controller/UserController.php

Braces +3 added lines, -3 removed lines

@@ -61,7 +61,7 @@ 
         //if user_id is not null, then log the user in automatically
         if ($existingUser) {
             return $existingUser;
-        }else {
+        } else {
             //if user_id is null, then the user is not registered and we should register them automatically
             //register new user
             $newUser = UserController::registerUser($user);
@@ -87,7 +87,7 @@ 
                 $userId = $registeredUser->getUser_id();
                 $jobSeekerProfile = new JobSeekerProfile();                
                 $result = JobSeekerController::addJobSeekerProfile($jobSeekerProfile, $userId);
-            }else if ($registeredUser->getUser_role() === 'administrator') {
+            } else if ($registeredUser->getUser_role() === 'administrator') {
                 
                 $userId = $registeredUser->getUser_id();
                 $managerProfile = new ManagerProfile();
@@ -129,7 +129,7 @@ 
             if ($oldUser->getEmail() != $updatedUser->getEmail()) {
                 $confEmailSent = UserController::confirmEmail($updatedUser);
                 $updatedUser->setIs_confirmed(false);
-            }else {
+            } else {
                 $updatedUser->setIs_confirmed($oldUser->getIs_confirmed());
             }
             $updateSuccessful = UserDAO::updateUser($updatedUser); //do updates