プロジェクト

全般

プロフィール

操作
リンクをコピー

バグ #1046

未完了

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

あひる 家鴨 さんが7年以上前に追加. ほぼ7年前に更新.

ステータス:
まだダメ
優先度:
通常
担当者:
あひる 家鴨
対象バージョン:
3.6
プラグイン名:
ブランチ:
クラッシュする:

説明

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

操作
リンクをコピー
 #1

あひる 家鴨 さんが7年以上前に更新

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

操作
リンクをコピー
 #2

toshi_a 初音 さんが7年以上前に更新

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

操作
リンクをコピー
 #3

あひる 家鴨 さんが7年以上前に更新

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

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

操作
リンクをコピー
 #4

toshi_a 初音 さんが7年以上前に更新

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

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

操作
リンクをコピー
 #5

あひる 家鴨 さんが7年以上前に更新

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

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

操作
リンクをコピー
 #6

toshi_a 初音 さんが7年以上前に更新

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

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

操作
リンクをコピー
 #7

あひる 家鴨 さんが7年以上前に更新

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

操作
リンクをコピー
 #8

あひる 家鴨 さんが7年以上前に更新

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

操作
リンクをコピー
 #9

あひる 家鴨 さんがほぼ7年前に更新

  • ステータス を 新規 から パッチ適用待ち に変更
操作
リンクをコピー
 #10

toshi_a 初音 さんがほぼ7年前に更新

  • ステータス を パッチ適用待ち から まだダメ に変更
  • 対象バージョン を 3.5 から 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

あひる 家鴨 さんがほぼ7年前に更新

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

操作
リンクをコピー
 #12

toshi_a 初音 さんがほぼ7年前に更新

ヽ('ω')ﾉ三ヽ('ω')ﾉ

操作
リンクをコピー

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