Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix #1829 親フォルダが公開状態の非公開の固定ページが閲覧できる #1832

Merged
merged 1 commit into from
Jun 20, 2022

Conversation

seto1
Copy link
Collaborator

@seto1 seto1 commented Jun 17, 2022

issue: #1829

@ryuring mtgでお時間をとっていただいて申し訳ないのですが、原因としてはシンプルでしたので軽い修正ですみました。

原因としては親要素に公開期間が設定されている場合、親要素の公開状態まで上書きされてしまうということでした。
でしたので、以下のように調整しています。

  • 親要素が非公開の場合は子要素も非公開
  • 親要素に公開期間が設定されている場合は子要素の公開期間を上書き

今までこのあたりの処理が混ざっていたので分割しています。

また、親要素と子要素、両方に公開期間が設定されていた場合はどうするか、という問題なのですが、
現在の仕様でも親要素の公開期間を優先する仕様でしたので特に対応は不要でした。

ご確認お願いします。

@ryuring ryuring self-requested a review June 17, 2022 05:16
@ryuring ryuring added the Bug バグ label Jun 17, 2022
@ryuring ryuring self-assigned this Jun 17, 2022
@ryuring
Copy link
Collaborator

ryuring commented Jun 17, 2022

@seto1 これで、親が2階層以上に渡りそれぞれで公開期間を定められている問題も解決しますか??

@seto1
Copy link
Collaborator Author

seto1 commented Jun 17, 2022

@ryuring はい。公開期間が設定されている中での最上位のフォルダの公開期間を参照するようです。

@ryuring ryuring merged commit 4d68c98 into baserproject:dev-4 Jun 20, 2022
@ryuring
Copy link
Collaborator

ryuring commented Jun 20, 2022

@seto1 ありがとうございました!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug バグ
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants