プロジェクト

全般

プロフィール

バグ #1046

改行文字のみでも投稿可能状態になる

あひる 家鴨7ヶ月前に追加. 4ヶ月前に更新.

ステータス:
新規
優先度:
通常
対象バージョン:
プラグイン名:

説明

改行のみではTwitterに投稿できないが、投稿ボタンが投稿可能状態になってしまっている。

関係しているリビジョン

リビジョン edcc2825 (差分)
あひる 家鴨6ヶ月前に追加

改行文字のみで投稿可能状態になってしまわないように修正 refs #1046

改行のみの場合は postable? が false を返すようにした
postbox のテキストが empty かもしくは改行文字を削除した状態で空白文字のみでないかをチェックするようにしました

リビジョン e61be885 (差分)
あひる 家鴨4ヶ月前に追加

改行文字のみで投稿可能状態になってしまわないように修正 refs #1046

改行のみの場合は postable? が false を返すようにしました。
postbox のテキストから改行文字を削除し、空白文字を削除した際に empty? かどうかをチェックするようにしました。

履歴

#1 あひる 家鴨6ヶ月前に更新

実装について考えていたんですが、Twitterの仕様?に則って、先頭の空白のみおよび改行のみを削除する方向で進めたいと思います。

#2 toshi_a 初音6ヶ月前に更新

賛成です。先頭の空白文字(半角スペースとTABのみ?)をトリムしてから入力判定するので良いと思います

#3 あひる 家鴨6ヶ月前に更新

  • ステータス新規 から パッチ適用待 に変更
  • 担当者あひる 家鴨 から toshi_a 初音 に変更

ブランチ topix/1046-fix-postable にプッシュしました。
確認をお願いします。

#4 toshi_a 初音6ヶ月前に更新

  • 担当者toshi_a 初音 から あひる 家鴨 に変更

これって、String#stripを呼んでる意味ってありますか?

#5 あひる 家鴨5ヶ月前に更新

意味ないですね…
眺めてて思ったのが、こちらの方がシンプルでしょうか。

def postable?
      text = widget_post.buffer.text
      text.delete("\n").strip.empty?
end

#6 toshi_a 初音4ヶ月前に更新

  • ステータスパッチ適用待 から 新規 に変更

そちらで問題なければレビューするので、改めてcommitお願いします。

#7 あひる 家鴨4ヶ月前に更新

他の依存関係の問題などで面倒になったので、最新の hotfix/3.5 からブランチ切って作り直してコミットします。

#8 あひる 家鴨4ヶ月前に更新

プッシュしました。
topic/1046-fix-postable ブランチです。
よろしくお願いします。

他の形式にエクスポート: Atom PDF