Updated methods to correctly check to see if a file exists on admin actions

application/modules/bulletin/controllers/AdminController.php

@@ -79,7 +79,7 @@ class Bulletin_AdminController extends Creq_Controller_Action
 
         // get file path and name for major
         $filePath = 'data' .DIRECTORY_SEPARATOR. Bulletin_RepositoryModel::getInstance()->getFilePathForCollegeMajorYear($in['college'], $major, $currentYear);
-        if (!$filePath) {
+        if (!is_file($filePath)) {
             $this->_helper->getHelper('FlashMessenger')->addMessage('Unable to find an existing file for that college/major.');
             $this->redirect('/bulletin/admin/change-major-name');
         }
@@ -147,7 +147,7 @@ class Bulletin_AdminController extends Creq_Controller_Action
 
         // get file path and name for major
         $filePath = 'data' .DIRECTORY_SEPARATOR. Bulletin_RepositoryModel::getInstance()->getFilePathForCollegeMajorYear($in['college'], $major, $currentYear);
-        if (!$filePath) {
+        if (!is_file($filePath)) {
             $this->_helper->getHelper('FlashMessenger')->addMessage('Unable to find an existing file for that college/major.');
             $this->redirect('/bulletin/admin/change-major-name');
         }
@@ -468,7 +468,7 @@ class Bulletin_AdminController extends Creq_Controller_Action
 
         // get file path and name for major
         $filePath = 'data' .DIRECTORY_SEPARATOR. Bulletin_RepositoryModel::getInstance()->getFilePathForCollegeMajorYear($in['college'], $major, $currentYear);
-        if (!$filePath) {
+        if (!is_file($filePath)) {
             $this->_helper->getHelper('FlashMessenger')->addMessage('Unable to find an existing file for that college/major.');
             $this->redirect('/bulletin/admin/remove-major');
         }