From 3e02da6b7ca4cafeb9e97c0f330371bba30ecac8 Mon Sep 17 00:00:00 2001
From: Andrew Rudenets <andrey.rudenets@gmail.com>
Date: Mon, 31 Aug 2015 14:32:23 +0300
Subject: [PATCH] Fix semester switcher

---
 .../application/classes/Controller/Handler/Settings.php   | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/~dev_rating/application/classes/Controller/Handler/Settings.php b/~dev_rating/application/classes/Controller/Handler/Settings.php
index cc9405501..e6181d7fe 100644
--- a/~dev_rating/application/classes/Controller/Handler/Settings.php
+++ b/~dev_rating/application/classes/Controller/Handler/Settings.php
@@ -81,9 +81,11 @@ class Controller_Handler_Settings extends Controller_Handler
     }
 
     public function action_setSemesterID() {
-        $semester = $this->post['semesterID'];
-        if (is_int($semester) && $semester > 0) {
-            $this->user->SemesterID = $semester;
+        $this->post
+            ->rule('semesterID', 'not_empty')
+            ->rule('semesterID', 'digit');
+        if ($this->post->check()) {
+            $this->user->SemesterID = (int) $this->post['semesterID'];
         }
     }
 }
-- 
GitLab