Re: 『?preview=a』を指定すると管理画面にリダイレクトされてしまう件について
- momo_r > 『?preview=a』を指定すると管理画面にリダイレクトされてしまう件について @ 2018/5/11 16:51
- momo_r > Re: 『?preview=a』を指定すると管理画面にリダイレクトされてしまう件について @ 2018/5/11 22:51
- ryuring > Re: 『?preview=a』を指定すると管理画面にリダイレクトされてしまう件について @ 2018/5/13 13:51
- ryuring > Re: 『?preview=a』を指定すると管理画面にリダイレクトされてしまう件について @ 2018/5/13 14:04
- » Doguu > Re: 『?preview=a』を指定すると管理画面にリダイレクトされてしまう件について @ 2018/5/14 20:05
Doguu > Re: 『?preview=a』を指定すると管理画面にリダイレクトされてしまう件について @ 2018/5/14 20:05 |
---|
この件について修正を実施し、githubのほうにpullリクエストを依頼をさせていただきました。
https://github.com/baserproject/basercms/pull/914 ご確認よろしくお願いいたします。 追記: 修正内容について全く記載しておりませんでしたので追記します。 > 1. Aさんが記事を作成し、公開前にBさんに確認してもらう為にプレビュー用のURLを共有 > (例)http://localhost/service?preview=default > 2. BさんがURLをリクエストしたが、ログインしていない為、ログイン画面に遷移 > 3. ログインをすると、ダッシュボードでなく、リクエストしたURLに遷移 > 今回の修正内容だと、上記のような場合、以下のように手順が増えてしまいます。 > 1. Aさんが記事を作成し、公開前にBさんに確認してもらう為にプレビュー用のURLを共有 > (例)http://localhost/service?preview=default > 2. BさんがURLをリクエストしたが、ログインしていない為、Not Fountページを表示 > 3. ログイン画面のURLを確認して、ログイン画面をリクエスト > 4. ログインするとダッシュボードを表示 > 5. プレビュー用のURLを再度リクエスト > ログイン画面を知らせたくないという事も理解できますので、上の動作とするのか、下の動作とするのかを選択して設定できるようにして頂けるとコアにも取り込むことができそうです。 コメントでいただいた上記のように『?preview=xxx』や『?force=xxx』を指定された場合の挙動を設定ファイルにて選択できるようにしました。 デフォルト設定は現在の仕様どおり、指定されかつ、未ログインだった場合はログイン画面へリダイレクトする処理となっております。 lib/Baser/Config/setting.php に『BcAuthPrefix.admin.previewRedirect』を追加しました。 boolean値にて設定する仕様で、 trueの場合はリダイレクトして、 falseの場合はリダイレクトしないという感じになっております。 各々設置した環境で修正する場合は app/Baser/Config/setting.php にて 『BcAuthPrefix.admin.previewRedirect』の値を操作することによって変更します。 ご確認よろしくお願いいたします。 |