Project

General

Profile

バグ #1046

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

Added by あひる 家鴨 over 2 years ago. Updated almost 2 years ago.

Status:
まだダメ
Priority:
通常
Target version:
プラグイン名:
ブランチ:
クラッシュする:

Description

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

Associated revisions

Revision edcc2825 (diff)
Added by あひる 家鴨 over 2 years ago

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

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

Revision e61be885 (diff)
Added by あひる 家鴨 over 2 years ago

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

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

History

#1

Updated by あひる 家鴨 over 2 years ago

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

#2

Updated by toshi_a 初音 over 2 years ago

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

#3

Updated by あひる 家鴨 over 2 years ago

  • Status changed from 新規 to パッチ適用待ち
  • Assignee changed from あひる 家鴨 to toshi_a 初音

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

#4

Updated by toshi_a 初音 over 2 years ago

  • Assignee changed from toshi_a 初音 to あひる 家鴨

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

#5

Updated by あひる 家鴨 over 2 years ago

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

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

#6

Updated by toshi_a 初音 over 2 years ago

  • Status changed from パッチ適用待ち to 新規

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

#7

Updated by あひる 家鴨 over 2 years ago

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

#8

Updated by あひる 家鴨 over 2 years ago

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

#9

Updated by あひる 家鴨 almost 2 years ago

  • Status changed from 新規 to パッチ適用待ち
#10

Updated by toshi_a 初音 almost 2 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

#11

Updated by あひる 家鴨 almost 2 years ago

Twitterプラグインに移行していたのを忘れてました。
Twitterプラグイン見て修正して上げ直します。

#12

Updated by toshi_a 初音 almost 2 years ago

ヽ('ω')ノ三ヽ('ω')ノ

Also available in: Atom PDF