「nee」



レス送信モード


お絵かきレス


name
URL
sage
sub
com *
UpFile []
Pass 記事の削除に必要
  • 画像を添付する場合は、'No File'のチェックを外して下さい。
  • 添付可能ファイル:GIF, JPG, PNG ブラウザによっては正常に添付できないことがあります。
  • 画像は横 300ピクセル、縦 300ピクセルを超えると縮小表示されます。
  • 最大投稿データ量は 2000 KB までです。sage機能付き。

No Pass


106 無題


106 無題 : funige : 2018/07/08(Sun) 13:38

お世話になってます。
potiboard.phpなのですが、USE_COMが0のとき(本文必須でないとき)英語のスパムが通ってしまうので、以下のように修正してNEOのsamplebbsに導入しました。

//本文に日本語がなければ拒絶
// if(USE_COM&&strlen($com) == mb_strlen($com,'utf8')) error(MSG035,$dest);
if(strlen($com) == mb_strlen($com,'utf8')) error(MSG035,$dest); // samplebbs

IP制限はキリがないので日本語必須はかなり有効なフィルタだと思うのですが、このままではあまり使えないので
USE_COMを外すか、またはUSE_JAPANESEみたいな適当な名前のフラグをconfig.phpに追加してあればいいなあと思いました。


107 無題 : さとぴあ : 2018/07/09(Mon) 23:14 URL

funigeさんの記述で最初は書いていたのですが、
本文無しの投稿をすると、
日本語がないといって拒絶されるから、使えないという話になり、
USE_COMが&&に入りました…。
ただ、逆に日本語以外で使いたいという方もいるでしょうから、
日本語がなければ拒絶するしないを別途設定できるようにしたほうがいいのかもしれないですね…。


108 無題 : funige : 2018/07/10(Tue) 09:21

ほんと仕事が雑で申し訳ありません。
if(strlen($com) > 0 && strlen($com) == mb_strlen($com,'utf8')) error(MSG035,$dest);
でどうでしょうか……


109 無題 : さとぴあ : 2018/07/10(Tue) 15:00 URL

ありがとうございます。
ifを入れ子にすればいいのかなぁ…と思っていましたが、スマートに書けるんですね…。
GITの更新はサ骨さんに頼りっきりでいまだに頭に用語が入っていかないので、
他の更新ネタとあわせて、
動作確認してまとまってから、
ここにZIPであげる…感じでしょうか。
その前にサ骨さんが作業されるかもしれませんが…。


110 無題 : サ骨 : 2018/07/11(Wed) 20:33

恥ずかしながら只今入院中でございまして
作業は来週以降になると思いますゆえにまとめていただければ幸いです。
ありがとうございます…


111 無題 : funige : 2018/07/12(Thu) 08:30

変更点をプルリクエストにまとめておきますね。
恥ずかしい話ですが自分でプルリク作ったことがないので(マージは何度もやっているのですが)
今作り方勉強しています。お待ちを……。


112 無題 : さとぴあ : 2018/07/13(Fri) 02:34 URL

ほぼ書き直しが終わって、最終チェックしているところです。Gitのインストールもしてなくてお手数をおかけします…。
もうちょっとチェックしたらZIPファイルをここに貼り付けます。


113 無題 : サ骨 : 2018/07/13(Fri) 16:48

退院しましたマン


114 ZIPファイル : さとぴあ : 2018/07/13(Fri) 20:16 URL *

http://pbbs.sakura.ne.jp/archives/temp/potiboard180713.zip

お手数をおかけしますが、2つの
ファイルをZIPにまとめました。

変更点。

POTI-boardのエラーを減らすために $mode編: STP^3
http://stp.sblo.jp/article/183848483.html
コンティニュー認証時にWarningが発生する箇所が何もしていなかったのでコメントアウト。
if文の中身をechoでチェックしても、どこにも関わっていない。

switch($mode)の、$modeがレス送信画面で大量のNoticeを発生させているので、
(isset($mode))で、未定義になっている時にはswitchの処理をしないように変更。

画面を切り替えます

の文字が、スマートフォンでは小さく表示されるので、viewportを設定。


115 無題 : さとぴあ : 2018/07/13(Fri) 20:14 URL

つづき

本文を必須にしない設定でも、本文に日本語がなければ拒絶できるように修正。
funige氏のsamplebbsと同じもの。

POTI-boardのNEOを使うをチェックした時の動作を変更: STP^3
http://stp.sblo.jp/article/183805394.html

しぃペインターを選択した状態でNEOを使うにチェックを入れると
<APPLET-DUMMY
が有効にならなくなっていたのを修正。

picpost.php

samplebbsでfunige氏が修正したものと同じ。
動画の拡張子設定が有効にならなくなっていたのを修正。


116 無題 : サ骨 : 2018/07/13(Fri) 21:59

1.43として公開しました


117 無題 : funige : 2018/07/14(Sat) 09:39

あわわ……
何もできないでいるうちにどんどん話が進んで行く……


118 無題 : funige : 2018/07/14(Sat) 12:06

プルリクエスト送ってみました。
初プルリクなんで、何か間違ってたらごめんなさい!


119 無題 : サ骨 : 2018/07/14(Sat) 15:28

見てみます!


120 無題 : サ骨 : 2018/07/14(Sat) 16:00

1.44としてマージしてみました


121 無題 : さとぴあ : 2018/07/14(Sat) 16:32 URL

1.44を落とそうと思ったのですが
見つけられませんでした…。


122 無題 : funige : 2018/07/14(Sat) 18:55

ええとですね
https://github.com/sakots/poti-kai
の一番上のPull Requestsというところに入っているのですが
サ骨さんの方で何かしないと
masterブランチにマージされないと思うのですが
僕もよくわかってないのではっきりしたことはちょっと……


124 無題 : funige : 2018/07/14(Sat) 19:16

たぶんサ骨さんのGithubではMerge pull requestというボタンが表示されてるはずなので
それをポチッと押していただければ……


125 無題 : サ骨 : 2018/07/15(Sun) 02:37

pushしてなかったという…