REVERT: turn auth role choice back on #248

media/js/sign.js

@@ -23,13 +23,12 @@ $(function () {
     const tryToAuthorizeByOpenID = function () {
         const login = $loginOpenID.val();
         // userRole для http://gitlab.mmcs.sfedu.ru/it-lab/grade/issues/248
-        // const userRole = $('#user_role').val();
+        const userRole = $('#user_role').val();
         if (! login.trim()) {
             alert('Необходимо ввести логин ЮФУ')
             return;
         }
-        // window.location = g_URLdir + "handler/sign/openidlogin?loginopenid=" + login + "&user_role=" + userRole;
-        window.location = g_URLdir + "handler/sign/openidlogin?loginopenid=" + login;
+        window.location = g_URLdir + "handler/sign/openidlogin?loginopenid=" + login + "&user_role=" + userRole;
     };
     $signInOpenIDButton.click(tryToAuthorizeByOpenID);
 

~dev_rating/application/classes/Controller/Handler/Sign.php

@@ -85,12 +85,12 @@ class Controller_Handler_Sign extends Controller_Handler
 
         $loginopenid = explode('@', $_GET["loginopenid"])[0];
         // user_role было добавлено для http://gitlab.mmcs.sfedu.ru/it-lab/grade/issues/248
-        // $user_role = $_GET["user_role"];
+        $user_role = $_GET["user_role"];
         $openid->SetIdentity("https://openid.sfedu.ru/server.php/idpage?user=" . $loginopenid);
         $openid->SetTrustRoot('http://' . $_SERVER["HTTP_HOST"]);
         $openid->SetOptionalFields(['email', 'nickname', 'r61globalkey', 'staff', 'student', 'r61studentid']);
         if ($openid->GetOpenIDServer()) {
-            //$openid->SetApprovedURL('http://' . $_SERVER["HTTP_HOST"] . Kohana::$base_url . 'handler/sign/openidfinish?user_role='.$user_role);
+            $openid->SetApprovedURL('http://' . $_SERVER["HTTP_HOST"] . Kohana::$base_url . 'handler/sign/openidfinish?user_role='.$user_role);
             $openid->SetApprovedURL('http://' . $_SERVER["HTTP_HOST"] . Kohana::$base_url . 'handler/sign/openidfinish');
             $openid->Redirect();
         } else {
@@ -130,14 +130,14 @@ class Controller_Handler_Sign extends Controller_Handler
         $studentId = $_GET["openid_sreg_r61studentid"];
         $email = $_GET["openid_sreg_email"];
         // user_role для http://gitlab.mmcs.sfedu.ru/it-lab/grade/issues/248
-        // $chosen_role = $_GET["user_role"];
+        $chosen_role = $_GET["user_role"];
 
         $id = 0;
         $error = null;
         try {
             if ($isStudent && $isStaff) {
                 // http://gitlab.mmcs.sfedu.ru/it-lab/grade/issues/248
-                // if ($chosen_role === 'student') {
+                if ($chosen_role === 'student') {
                 // find student with same ext id
                 $inila = str_replace('st-ss-', '', $globalKey);
                 $extId = Model_Account::getAccountExtIdByInila($inila);
@@ -164,13 +164,13 @@ class Controller_Handler_Sign extends Controller_Handler
                         }
                     }
                 }
-//                } else {
-//                    $globalKey = str_replace('st-ss-', 'ss-', $globalKey);
-//                    $id = User::instance()->signInByOpenID($globalKey);
-//                    if ($id === 0) {
-//                        $error = "не найден аккаунт преподавателя!";
-//                    }
-//                }
+                } else {
+                    $globalKey = str_replace('st-ss-', 'ss-', $globalKey);
+                    $id = User::instance()->signInByOpenID($globalKey);
+                    if ($id === 0) {
+                        $error = "не найден аккаунт преподавателя!";
+                    }
+                }
             } elseif ($isStudent) {
                 $globalKey = 'st-' . str_pad(str_replace('st-', '', $globalKey), 9, '0', STR_PAD_LEFT);
                 $id = User::instance()->signInByOpenID($globalKey);

~dev_rating/application/views/sign/in.twig

@@ -14,18 +14,17 @@
                 | {{ HTML.anchor('sign/up', 'Активировать аккаунт')|raw }}
                 | {{ HTML.anchor('remind', 'Забыли пароль?')|raw }}
                 {# http://gitlab.mmcs.sfedu.ru/it-lab/grade/issues/248 #}
-                {# решили подождать с изменениями авторизации до решения об учетных записях #}
-                {#|#}
-                {#<button class = "collapsible">выбор роли</button>#}
-                {#<div class="roleToggler">#}
-                    {# <label title="При наличии аккаунтов студента и преподавателя следует выбрать какой из них использовать"> #}
-                        {#<select id="user_role">#}
-                            {#<option value="student"> студент </option>#}
-                            {#<option value="staff"> преподаватель </option>#}
-                        {#</select>#}
-                        {#Выберите роль студента или преподавателя для авторизации#}
-                    {# </label> #}
-                {#</div>#}
+                |
+                <button class = "collapsible">выбор роли</button>
+                <div class="roleToggler">
+                     <label title="При наличии аккаунтов студента и преподавателя следует выбрать какой из них использовать">
+                        <select id="user_role">
+                            <option value="student"> студент </option>
+                            <option value="staff"> преподаватель </option>
+                        </select>
+                        Выберите роль студента или преподавателя для авторизации
+                     </label>
+                </div>
             {% endif %}
         </div>
     </div>