板名変更時に、専ブラで板登録時に自動取得される板名が反映されるように変更 #40

Merged
stat2 merged 1 commits from SaturdayTheMsgr/delightly-v2fork:fix-bbstitle-no-apply into main 2023-11-04 03:35:12 +09:00
Contributor

現状では板名を変更してもindex.htmlには反映されず板作成時のディレクトリ名で固定される仕様なので、この変更ではそれが反映されるようにします。

主な変更点

index.htmlをindex.phpに置換

delightlyの汎用ブラウザ版は文字エンコードがUTF-8となっているので、板名も当然UTF-8で記述されます。
しかし、板登録時に板名が自動取得される専ブラのうち、UTF-8非対応なもの(JaneStyle等)だと文字化けしてしまうのでどうにかして専ブラを識別して出力する文字エンコードを変える必要があります。

特段いいやり方が思いつかなかったんですがここではテンプレートとなっている/test/board/index.htmlindex.phpに変更し、その中でUAを識別し場合分けで文字エンコードを変更しつつhtml出力するようにしています。
/(板ディレクトリ)/index.phpにはUTF-8の時とShift-JISの時の板名のみ保持し、/test/board/index.phpを呼び出すという処理を行います。

板名変更時に/(板ディレクトリ)/index.phpに反映されるように

/test/operate/view.phpおよび/test/kanri2.phpにて、index.phpへUTF-8/Shift_JIS両者の板名を持つ変数を出力するようにした。

副次的な変更

絵文字を含みうる掲示板名をSJISへ変換するのにmb_substitute_character('entity')を指定する必要があり、その結果管理画面の掲示板設定>表示設定で変更できる名無し名および削除テキストに絵文字がある場合に文字化けするのが解消される。

懸念点

UTF-8非対応の専ブラに対応する必要があるか

専ブラの総数で考えるとそのような仕様の専ブラはシェアがたかが知れているので、わざわざ処理が重くなる方式にせずUTF-8一本に絞ってindex.htmlへ反映させるという手も考えられます。

UAによる識別が曖昧

JaneXeno, JaneStyle, ChMate, Firefox, Chrome, Edgeで動作確認済みだが、この変更におけるUAによる識別7abe831839/test/board/index.php (L3-L14)が適切なのかどうか自信がないので問題ありそうなら助言頂きたい所です。

それ以前にPHPで処理すべきか

あまり詳しくわからないので良い方法があるならそっちにしたい

その他

動作検証およびレビュー頂ければ幸いです

現状では板名を変更してもindex.htmlには反映されず板作成時のディレクトリ名で固定される仕様なので、この変更ではそれが反映されるようにします。 ## 主な変更点 ### index.htmlをindex.phpに置換 delightlyの汎用ブラウザ版は文字エンコードがUTF-8となっているので、板名も当然UTF-8で記述されます。 しかし、板登録時に板名が自動取得される専ブラのうち、UTF-8非対応なもの(JaneStyle等)だと文字化けしてしまうのでどうにかして専ブラを識別して出力する文字エンコードを変える必要があります。 特段いいやり方が思いつかなかったんですがここではテンプレートとなっている`/test/board/index.html`を`index.php`に変更し、その中でUAを識別し場合分けで文字エンコードを変更しつつhtml出力するようにしています。 `/(板ディレクトリ)/index.php`にはUTF-8の時とShift-JISの時の板名のみ保持し、`/test/board/index.php`を呼び出すという処理を行います。 ### 板名変更時に`/(板ディレクトリ)/index.php`に反映されるように `/test/operate/view.php`および`/test/kanri2.php`にて、`index.php`へUTF-8/Shift_JIS両者の板名を持つ変数を出力するようにした。 ## 副次的な変更 絵文字を含みうる掲示板名をSJISへ変換するのに`mb_substitute_character('entity')`を指定する必要があり、その結果管理画面の掲示板設定>表示設定で変更できる名無し名および削除テキストに絵文字がある場合に文字化けするのが解消される。 ## 懸念点 ### UTF-8非対応の専ブラに対応する必要があるか 専ブラの総数で考えるとそのような仕様の専ブラはシェアがたかが知れているので、わざわざ処理が重くなる方式にせずUTF-8一本に絞ってindex.htmlへ反映させるという手も考えられます。 ### UAによる識別が曖昧 JaneXeno, JaneStyle, ChMate, Firefox, Chrome, Edgeで動作確認済みだが、この変更におけるUAによる識別https://git.3chan.cc/SaturdayTheMsgr/delightly-v2fork/src/commit/7abe83183940dc82d4387c35b2aa1937a2aec133/test/board/index.php#L3-L14が適切なのかどうか自信がないので問題ありそうなら助言頂きたい所です。 ### それ以前にPHPで処理すべきか あまり詳しくわからないので良い方法があるならそっちにしたい ## その他 動作検証およびレビュー頂ければ幸いです
SaturdayTheMsgr added 1 commit 2023-10-29 22:47:32 +09:00
Contributor

Chrome、Siki、JaneXenoで動作確認しましたが、特に動作に問題はありませんでした。コードもおかしな点は無いように思います。
JaneXenoでは板名取得でディレクトリ名しか取れなかったのが掲示板名を取得可能に変わりました。
恩恵を受ける専ブラは少ないかもしれませんが有意義な修正かと思います。

Chrome、Siki、JaneXenoで動作確認しましたが、特に動作に問題はありませんでした。コードもおかしな点は無いように思います。 JaneXenoでは板名取得でディレクトリ名しか取れなかったのが掲示板名を取得可能に変わりました。 恩恵を受ける専ブラは少ないかもしれませんが有意義な修正かと思います。
Author
Contributor

@konkon-fox 検証していただきありがとうございました。以前に妙な仕様だというレスを見たのと個人的にも気になっていたんですよね

@konkon-fox 検証していただきありがとうございました。以前に妙な仕様だというレスを見たのと個人的にも気になっていたんですよね
Author
Contributor

WIPは外しますがマージするかは統計部員さんの方針に任せます。

WIPは外しますがマージするかは統計部員さんの方針に任せます。
SaturdayTheMsgr changed title from WIP: 板名変更時に、専ブラで板登録時に自動取得される板名が反映されるように変更 to 板名変更時に、専ブラで板登録時に自動取得される板名が反映されるように変更 2023-11-03 13:30:26 +09:00
Owner

ありがとうございます。
私自身確認できておりませんが、良い変更で問題なさそうなのでマージしました。
UA部分については今後もしかすると変更が必要なこともあるかもしれません。

ありがとうございます。 私自身確認できておりませんが、良い変更で問題なさそうなのでマージしました。 UA部分については今後もしかすると変更が必要なこともあるかもしれません。
stat2 merged commit 988e862712 into main 2023-11-04 03:35:12 +09:00
Sign in to join this conversation.
No description provided.