Merge pull request #1992 from opensource-workshop/2.17.4

Merge v1.17.4 into 2

Author: akagane99

app/Http/Controllers/Auth/LoginController.php

@@ -16,6 +16,8 @@
 // ログインエラーをCatch するために追加。
 use Illuminate\Validation\ValidationException;
 
+use App\Plugins\Manage\UserManage\UsersTool;
+
 use App\User;
 use App\Models\Core\Configs;
 use App\Models\Core\UsersRoles;
@@ -306,6 +308,7 @@ public function authMethod($request)
                         $user->userid = $request['userid'];
                         $user->password = Hash::make($request['password']);
                         $user->created_event = AuthMethodType::netcommons2;
+                        $user->columns_set_id = UsersTool::COLUMNS_SET_ID_DEFAULT;
                         $user->save();
 
                         // 追加権限設定があれば作成
@@ -411,6 +414,7 @@ public function authMethod($request)
                         $user->userid = $request->userid;
                         $user->password = $password;
                         $user->created_event = AuthMethodType::ldap;
+                        $user->columns_set_id = UsersTool::COLUMNS_SET_ID_DEFAULT;
                         $user->save();
 
                     } else {
@@ -495,6 +499,7 @@ private function authMethodShibboleth($request)
                 $user->email = $email;
                 $user->password = $password;
                 $user->created_event = AuthMethodType::shibboleth;
+                $user->columns_set_id = UsersTool::COLUMNS_SET_ID_DEFAULT;
                 $user->save();
 
                 // [TODO] 区分 (unscoped-affiliation),    faculty (教員)，staff (職員), student (学生)

app/Plugins/Api/Nc2Sso/Nc2Sso.php

@@ -2,25 +2,21 @@
 
 namespace App\Plugins\Api\Nc2Sso;
 
-use Illuminate\Http\Request;
-use Illuminate\Support\Facades\Auth;
-use Illuminate\Support\Facades\Log;
-use Illuminate\Support\Facades\Validator;
-
-use DB;
-
 use App\Models\Core\UsersRoles;
-use App\User;
 use App\Plugins\Api\ApiPluginBase;
+use App\Plugins\Manage\UserManage\UsersTool;
 use App\Traits\ConnectCommonTrait;
+use App\User;
+use Illuminate\Support\Facades\Auth;
 
 /**
  * NC2からのSSO管理クラス
  *
  * @author 永原　篤 <nagahara@opensource-workshop.jp>
+ * @author 牟田口 満 <mutaguchi@opensource-workshop.jp>
  * @copyright OpenSource-WorkShop Co.,Ltd. All Rights Reserved
  * @category ページ管理
- * @package Contoroller
+ * @package Controller
  */
 class Nc2Sso extends ApiPluginBase
 {
@@ -78,7 +74,7 @@ public function index($request, $site_key, $login_id, $token)
         if (!empty($user)) {
             // ユーザ権限データ取得
             //$roles = UsersRoles::getUsersRoles($user->id);
-            $users_roles = new UsersRoles();
+            // $users_roles = new UsersRoles();
 
             // 権限が一般 or ゲストの場合は、自動ログイン
             // if ($user->role == config('cc_role.ROLE_PAGE_MANAGER') || $user->role == config('cc_role.ROLE_GUEST')) {
@@ -121,6 +117,7 @@ public function index($request, $site_key, $login_id, $token)
             $user->userid   = $login_id;
             $user->password = 'sso-invalid-password';   // プレーンテキストのパスワードは設定しても、入力パスワードと一致する事はないため、無効になる
             //$user->role     = 0;
+            $user->columns_set_id = UsersTool::COLUMNS_SET_ID_DEFAULT;
             $user->save();
 
             // ユーザ権限の登録

app/Plugins/Manage/UserManage/UserManage.php

@@ -577,7 +577,7 @@ private function getColumnsSetIdManageDefault(): int
             $columns_set_id = 0;
         } else {
             // 1:ユーザ一覧（基本）
-            $columns_set_id = 1;
+            $columns_set_id = UsersTool::COLUMNS_SET_ID_DEFAULT;
         }
 
         return $columns_set_id;

app/Plugins/Manage/UserManage/UsersTool.php

@@ -26,6 +26,9 @@ class UsersTool
 {
     const CHECKBOX_SEPARATOR = '|';
 
+    /** columns_set_id 初期値 1:基本 */
+    const COLUMNS_SET_ID_DEFAULT = 1;
+
     /**
      * ユーザーのカラム取得
      */

app/Plugins/User/Forms/FormsPlugin.php

@@ -97,6 +97,7 @@ public function getPublicFunctions()
             'publicConfirmToken',
             'listInputs',
             'editInput',
+            'thanks',
         ];
         $functions['post'] = [
             'index',
@@ -898,24 +899,20 @@ public function publicStore($request, $page_id, $frame_id, $id = null)
 
         // 表示期間外か
         if ($this->isOutOfTermDisplay($form)) {
-            // 表示しない
-            return false;
+            // 初期表示にリダイレクトして、初期表示処理にまかせる（表示しない）
+            return collect(['redirect_path' => url($this->page->permanent_link)]);
         }
 
         // 登録期間外か
         if ($this->isOutOfTermRegist($form)) {
-            // エラー画面へ
-            return $this->commonView('error_messages', [
-                'error_messages' => ['登録期間外のため、登録出来ません。'],
-            ]);
+            // 初期表示にリダイレクトして、初期表示処理にまかせる（エラー表示）
+            return collect(['redirect_path' => url($this->page->permanent_link)]);
         }
 
         // 登録制限数オーバーか
         if ($this->isOverEntryLimit($form->id, $form->entry_limit)) {
-            // エラー画面へ
-            return $this->commonView('error_messages', [
-                'error_messages' => [$form->entry_limit_over_message],
-            ]);
+            // 初期表示にリダイレクトして、初期表示処理にまかせる（エラー表示）
+            return collect(['redirect_path' => url($this->page->permanent_link)]);
         }
 
         // forms_inputs 登録
@@ -1027,10 +1024,11 @@ public function publicStore($request, $page_id, $frame_id, $id = null)
         $contents_text = trim($contents_text);
 
         if ($form->use_temporary_regist_mail_flag) {
-            // 仮登録
+            // *** 仮登録
             // ユーザ側のみメール送信する
 
             $after_message = $form->temporary_regist_after_message;
+            session()->flash("after_message{$frame_id}", $after_message);
 
             // メール送信
             // メール件名の組み立て
@@ -1066,16 +1064,15 @@ public function publicStore($request, $page_id, $frame_id, $id = null)
             // メール送信（ユーザー側）
             foreach ($user_mailaddresses as $user_mailaddress) {
                 if (!empty($user_mailaddress)) {
-                    // Mail::to(trim($user_mailaddress))->send(new ConnectMail($mail_options, ['content' => $mail_text]));
                     // メール送信はログ出力の追加に伴いTrait のメソッドに移行
                     $this->sendMail($user_mailaddress, $mail_options, ['content' => $mail_text], $this->getPluginName());
                 }
             }
         } else {
-            // 本登録
-
+            // *** 本登録
             // 登録後メッセージ内の採番文字列を置換
             $after_message = str_replace('[[number]]', $number, $form->after_message);
+            session()->flash("after_message{$frame_id}", $after_message);
 
             // メール送信
             if ($form->mail_send_flag || $form->user_mail_send_flag) {
@@ -1114,7 +1111,6 @@ public function publicStore($request, $page_id, $frame_id, $id = null)
                 if ($form->mail_send_flag) {
                     $mail_addresses = explode(',', $form->mail_send_address);
                     foreach ($mail_addresses as $mail_address) {
-                        // Mail::to(trim($mail_address))->send(new ConnectMail($mail_options, ['content' => $mail_text]));
                         // メール送信はログ出力の追加に伴いTrait のメソッドに移行
                         $this->sendMail($mail_address, $mail_options, ['content' => $mail_text], $this->getPluginName());
                     }
@@ -1124,7 +1120,6 @@ public function publicStore($request, $page_id, $frame_id, $id = null)
                 if ($form->user_mail_send_flag) {
                     foreach ($user_mailaddresses as $user_mailaddress) {
                         if (!empty($user_mailaddress)) {
-                            // Mail::to(trim($user_mailaddress))->send(new ConnectMail($mail_options, ['content' => $mail_text]));
                             // メール送信はログ出力の追加に伴いTrait のメソッドに移行
                             $this->sendMail($user_mailaddress, $mail_options, ['content' => $mail_text], $this->getPluginName());
                         }
@@ -1151,9 +1146,17 @@ public function publicStore($request, $page_id, $frame_id, $id = null)
             }
         }
 
-        // 表示テンプレートを呼び出す。
+        // 登録後のリダイレクト表示
+        return collect(['redirect_path' => url('/') . "/plugin/forms/thanks/{$page_id}/{$frame_id}#frame-{$frame_id}"]);
+    }
+
+    /**
+     * 登録後画面
+     */
+    public function thanks($request, $page_id, $frame_id, $id = null)
+    {
         return $this->view('forms_thanks', [
-            'after_message' => $after_message
+            'after_message' => session("after_message{$frame_id}")
         ]);
     }
 
@@ -1257,33 +1260,30 @@ public function publicStoreToken($request, $page_id, $frame_id, $id)
 
         // 表示期間外か
         if ($this->isOutOfTermDisplay($form)) {
-            // 表示しない
-            return false;
+            // 初期表示にリダイレクトして、初期表示処理にまかせる（表示しない）
+            return collect(['redirect_path' => url($this->page->permanent_link)]);
         }
 
         // 登録期間外か
         if ($this->isOutOfTermRegist($form)) {
-            // エラー画面へ
-            return $this->commonView('error_messages', [
-                'error_messages' => ['登録期間外のため、登録出来ません。'],
-            ]);
+            // 初期表示にリダイレクトして、初期表示処理にまかせる（エラー表示）
+            return collect(['redirect_path' => url($this->page->permanent_link)]);
         }
 
         // 登録制限数オーバーか
         if ($this->isOverEntryLimit($form->id, $form->entry_limit)) {
-            // エラー画面へ
-            return $this->commonView('error_messages', [
-                'error_messages' => [$form->entry_limit_over_message],
-            ]);
+            // 初期表示にリダイレクトして、初期表示処理にまかせる（エラー表示）
+            return collect(['redirect_path' => url($this->page->permanent_link)]);
         }
 
         // $id がなかったら、エラー画面へ
         // $forms_inputs がなかったら、エラー画面へ
         $forms_inputs = FormsInputs::find($id);
         if (empty($forms_inputs)) {
-            return $this->commonView('error_messages', [
-                'error_messages' => ['有効期限切れのため、そのURLはご利用できません。'],
-            ]);
+            session()->flash("error_messages{$frame_id}", '有効期限切れのため、そのURLはご利用できません。');
+
+            // 初期表示にリダイレクトして、初期表示処理にまかせる（エラー表示）
+            return collect(['redirect_path' => url($this->page->permanent_link)]);
         }
 
         // 項目のエラーチェック
@@ -1293,9 +1293,11 @@ public function publicStoreToken($request, $page_id, $frame_id, $id)
 
         // getで日付形式エラーは表示しない（通常URLをコピペミス等でいじらなければエラーにならない想定）
         if ($validator->fails()) {
-            return $this->commonView('error_messages', [
-                'error_messages' => $validator->errors()->all(),
-            ]);
+            $error_messages = $validator->errors()->all();
+            session()->flash("error_messages{$frame_id}", $error_messages[0]);
+
+            // 初期表示にリダイレクトして、初期表示処理にまかせる（エラー表示）
+            return collect(['redirect_path' => url($this->page->permanent_link)]);
         }
 
         // forms_inputs 更新
@@ -1314,22 +1316,17 @@ public function publicStoreToken($request, $page_id, $frame_id, $id)
 
         // フォームの登録データ
         $forms_input_cols = FormsInputCols::where('forms_inputs_id', $id)
-                                            ->get()
-                                            // keyをforms_columns_idにした結果をセット
-                                            ->mapWithKeys(function ($item) {
-                                                return [$item['forms_columns_id'] => $item];
-                                            });
+            ->get()
+            // keyをforms_columns_idにした結果をセット
+            ->mapWithKeys(function ($item) {
+                return [$item['forms_columns_id'] => $item];
+            });
 
         // メールの送信文字列
         $contents_text = '';
 
         // 登録者のメールアドレス
         $user_mailaddresses = array();
-        // dd($id, $forms_input_cols->first()->forms_columns_id);
-        // foreach ($forms_input_cols as $forms_input_col) {
-        //     var_dump($forms_input_col->forms_columns_id);
-        // }
-        // dd($forms_input_cols->count());
 
         // 添付ファイルID
         $attach_uploads_ids = [];
@@ -1367,12 +1364,11 @@ public function publicStoreToken($request, $page_id, $frame_id, $id)
         }
         // 最後の改行を除去
         $contents_text = trim($contents_text);
-        // dd($user_mailaddresses);
-
-        // 本登録
 
+        // *** 本登録
         // 登録後メッセージ内の採番文字列を置換
         $after_message = str_replace('[[number]]', $number, $form->after_message);
+        session()->flash("after_message{$frame_id}", $after_message);
 
         // メール送信
         if ($form->mail_send_flag || $form->user_mail_send_flag) {
@@ -1411,7 +1407,6 @@ public function publicStoreToken($request, $page_id, $frame_id, $id)
             if ($form->mail_send_flag) {
                 $mail_addresses = explode(',', $form->mail_send_address);
                 foreach ($mail_addresses as $mail_address) {
-                    // Mail::to(trim($mail_address))->send(new ConnectMail($mail_options, ['content' => $mail_text]));
                     // メール送信はログ出力の追加に伴いTrait のメソッドに移行
                     $this->sendMail($mail_address, $mail_options, ['content' => $mail_text], $this->getPluginName());
                 }
@@ -1421,18 +1416,15 @@ public function publicStoreToken($request, $page_id, $frame_id, $id)
             if ($form->user_mail_send_flag) {
                 foreach ($user_mailaddresses as $user_mailaddress) {
                     if (!empty($user_mailaddress)) {
-                        // Mail::to(trim($user_mailaddress))->send(new ConnectMail($mail_options, ['content' => $mail_text]));
                         // メール送信はログ出力の追加に伴いTrait のメソッドに移行
                         $this->sendMail($user_mailaddress, $mail_options, ['content' => $mail_text], $this->getPluginName());
                     }
                 }
             }
         }
 
-        // 表示テンプレートを呼び出す。
-        return $this->view('forms_thanks', [
-            'after_message' => $after_message
-        ]);
+        // 登録後のリダイレクト表示
+        return collect(['redirect_path' => url('/') . "/plugin/forms/thanks/{$page_id}/{$frame_id}#frame-{$frame_id}"]);
     }
 
     /**

app/Traits/Migration/MigrationTrait.php

@@ -84,7 +84,6 @@
 use App\Models\User\Rsses\Rsses;
 use App\Models\User\Rsses\RssUrls;
 
-
 use App\User;
 
 use App\Models\Migration\MigrationMapping;
@@ -162,6 +161,7 @@
 use App\Models\Migration\Nc2\Nc2Simplemovie;
 use App\Models\Migration\Nc2\Nc2RssBlock;
 
+use App\Plugins\Manage\UserManage\UsersTool;
 use App\Traits\ConnectCommonTrait;
 use App\Utilities\Migration\MigrationUtils;
 
@@ -1778,7 +1778,7 @@ private function importUsers($redo)
                     $user->userid     = $user_item['userid'];
                     $user->password   = $user_item['password'];
                     $user->status     = $user_item['status'];
-                    $user->columns_set_id = 1;
+                    $user->columns_set_id = UsersTool::COLUMNS_SET_ID_DEFAULT;
                     $user->created_at = $user_item['created_at'];
                     $user->updated_at = $user_item['updated_at'];
                     $user->save();

app/Utilities/Request/RequestUtils.php

@@ -59,6 +59,7 @@ public static function isBot($request): bool
             'python-requests',
             'meg',
             'Scrapy',
+            'GoogleOther',
         ];
 
         foreach ($bots as $bot) {

config/version.php

@@ -12,7 +12,7 @@
     |
     */
 
-    'cc_version' => '2.17.3',
+    'cc_version' => '2.17.4',
 
     'show_cc_version' => true,
 ];

resources/views/plugins/user/forms/default/forms.blade.php

@@ -12,6 +12,17 @@
 
 @include('plugins.common.errors_form_line')
 
+{{-- エラーメッセージ：publicStoreToken()リダイレクト時 --}}
+@if (session('error_messages' . $frame_id))
+    <div class="card border-danger form-group">
+        <div class="card-body">
+            <span class="text-danger">
+                <i class="fas fa-exclamation-triangle"></i> {{ session('error_messages' . $frame_id) }}
+            </span>
+        </div>
+    </div>
+@endif
+
 @if ($form->can_view_inputs_moderator)
     @can('role_article')
         <div class="row">