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 script_escape #410

Merged
merged 1 commit into from
Aug 5, 2020
Merged

Fix script_escape #410

merged 1 commit into from
Aug 5, 2020

Conversation

nanasess
Copy link
Contributor

@nanasess nanasess commented Jul 11, 2020

開発コミュニティより
https://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=24267&forum=9&post_id=98863#forumpost98863

以下、テンプレート編集ページで script_escape がかかって、#script tag escaped# となっていたのを修正

  • ページ詳細設定
  • ヘッダーフッター設定

ブロック編集ページの実装と合わせた

@nobuhiko
Copy link
Contributor

これ結構なバグだし、リリースしたいっすね・・

@nanasess
Copy link
Contributor Author

@nobuhiko 修正自体は軽微なので、リリースタグだけ打ってもらいますか。。。

@okazy okazy added this to the 2.17.2 milestone Aug 5, 2020
@okazy okazy merged commit d70c588 into EC-CUBE:master Aug 5, 2020
@okazy
Copy link
Contributor

okazy commented Aug 5, 2020

ありがとうございます!取り込みました。

@okazy
Copy link
Contributor

okazy commented Aug 5, 2020

@nobuhiko @nanasess
bug:Middle のタグをつけさせていただきました。(システムエラーになるわけではないため)
コメントいただいたのは重要なバグなのですでに利用されておられる方に早くお届けしたいとの意図だと存じます。
どうリリースしていくべきか、私だけでは判断しかねていますのでご意見をいただければと思います。

2.17.2 としてリリースですとリリースまでにお時間をいただいたり、現在ご利用中の方がアップデートされたりということが考えられます。
本件の修正は量としては軽微な修正なので、SNS等で不具合の事実と修正方法のお知らせをする形でいかがでしょうか?

@nobuhiko
Copy link
Contributor

nobuhiko commented Aug 5, 2020

@okazy
軽微ですが明確なバグが有るバージョンを配布し続けると、それによって無駄なフォーラムへの書き込みなどもあると思いますし、使う人も不便ではないでしょうか?
利用中の方への不具合修正方法は案内したほうが親切ですが、配布バージョンを2.17.2 としてしまうほうが結果早い気がします。

あと、次バージョンがいつでるかもわかりませんし、やれることはやれる時にやってしまうほうがいい主義です。

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

Successfully merging this pull request may close these issues.

3 participants