Actions
バグ #1046
open改行文字のみでも投稿可能状態になる
Description
改行のみではTwitterに投稿できないが、投稿ボタンが投稿可能状態になってしまっている。
Updated by あひる 家鴨 over 7 years ago
実装について考えていたんですが、Twitterの仕様?に則って、先頭の空白のみおよび改行のみを削除する方向で進めたいと思います。
Updated by toshi_a 初音 over 7 years ago
賛成です。先頭の空白文字(半角スペースとTABのみ?)をトリムしてから入力判定するので良いと思います
Updated by あひる 家鴨 over 7 years ago
- Status changed from 新規 to パッチ適用待ち
- Assignee changed from あひる 家鴨 to toshi_a 初音
ブランチ topix/1046-fix-postable
にプッシュしました。
確認をお願いします。
Updated by toshi_a 初音 over 7 years ago
- Assignee changed from toshi_a 初音 to あひる 家鴨
これって、String#stripを呼んでる意味ってありますか?
Updated by あひる 家鴨 over 7 years ago
意味ないですね…
眺めてて思ったのが、こちらの方がシンプルでしょうか。
def postable?
text = widget_post.buffer.text
text.delete("\n").strip.empty?
end
Updated by toshi_a 初音 over 7 years ago
- Status changed from パッチ適用待ち to 新規
そちらで問題なければレビューするので、改めてcommitお願いします。
Updated by あひる 家鴨 over 7 years ago
他の依存関係の問題などで面倒になったので、最新の hotfix/3.5 からブランチ切って作り直してコミットします。
Updated by あひる 家鴨 over 7 years ago
プッシュしました。
topic/1046-fix-postable ブランチです。
よろしくお願いします。
Updated by toshi_a 初音 almost 7 years ago
- Status changed from パッチ適用待ち to まだダメ
- Target version changed from 3.5 to 3.6
そのあたりを3.6ではいじってしまっていて衝突してしまいました。
見てもらったらわかりやすいと思うんですが、spellのconditionを使って、各Worldに条件を移動してるので、これはTwitterプラグインに方に書く感じですかね
def postable?
<<<<<<< HEAD
not(widget_post.buffer.text.empty?) and (/[^\p{blank}]/ === widget_post.buffer.text) and Plugin[:gtk].compose?(current_world, to_display_only? ? nil : @to.first, visibility: @visibility)
=======
text = widget_post.buffer.text || ''
!text.delete("\n").strip.empty?
>>>>>>> 改行文字のみで投稿可能状態になってしまわないように修正 refs #1046
end
Updated by あひる 家鴨 almost 7 years ago
Twitterプラグインに移行していたのを忘れてました。
Twitterプラグイン見て修正して上げ直します。
Actions