https://dev.mikutter.hachune.net/
https://dev.mikutter.hachune.net/favicon.ico?1619448608
2018-05-21T10:28:17Z
やること
mikutter - 機能 #1250: 残り文字数カウントの処理を remain_charcount Spell に分離する
https://dev.mikutter.hachune.net/issues/1250?journal_id=3371
2018-05-21T10:28:17Z
toshi_a 初音
toshi.alternative@gmail.com
<ul><li><strong>ステータス</strong> を <i>新規</i> から <i>実装待ち</i> に変更</li><li><strong>担当者</strong> を <i>Shibafu Midorino</i> にセット</li><li><strong>プラグイン名</strong> を <i>topic/1250-remain-charcount</i> にセット</li></ul><p>土曜日隣で直接レビューしていたのでcommitしました。</p>
<a name="DMなど本文以外の投稿オプションで字数制限が変化するケース"></a>
<h1 >DMなど、本文以外の投稿オプションで字数制限が変化するケース<a href="#DMなど本文以外の投稿オプションで字数制限が変化するケース" class="wiki-anchor">¶</a></h1>
<p>土曜日、口頭で</p>
<blockquote>
<p>PostBoxがDiva::Modelなら、remain_charcount[world, postbox] のような構文で残り文字数を取得できると良さそう</p>
</blockquote>
<p>と言いましたが、よく考えると投稿する手段はPostBoxだけじゃないし、World系プラグインがguiに依存することになるのでよくありませんね。</p>
<p>compose spellと全く同じ引数を渡すことにしたら、どのような場合でも常に判断材料が揃うのではないかなと思いました。<br />残り文字数を知りたい場所では、compose spellを呼ぶはずですから、そのようなルールならわかりやすいと思います。</p>
mikutter - 機能 #1250: 残り文字数カウントの処理を remain_charcount Spell に分離する
https://dev.mikutter.hachune.net/issues/1250?journal_id=3373
2018-05-21T14:32:46Z
Shibafu Midorino
<ul><li><strong>ファイル</strong> <a href="/attachments/452">0002-fix_stack_overflow.patch</a> <a class="icon-only icon-download" title="ダウンロード" href="/attachments/download/452/0002-fix_stack_overflow.patch">0002-fix_stack_overflow.patch</a> を追加</li><li><strong>ファイル</strong> <a href="/attachments/453">0003-change_remain_charcount_options.patch</a> <a class="icon-only icon-download" title="ダウンロード" href="/attachments/download/453/0003-change_remain_charcount_options.patch">0003-change_remain_charcount_options.patch</a> を追加</li><li><strong>ステータス</strong> を <i>実装待ち</i> から <i>パッチ適用待ち</i> に変更</li><li><strong>担当者</strong> を <i>Shibafu Midorino</i> から <i>toshi_a 初音</i> に変更</li></ul><p>本題の前に、patch 0001にてURLを含む文字列を入力した際にスタックオーバーフローを起こしてしまう問題がありましたので、patch 0002にて対応しました。<br />メソッドの移動を行った際に、名前がかぶっていることに気づいていませんでした...</p>
<blockquote>
<p>compose spellと全く同じ引数を渡すことにしたら、どのような場合でも常に判断材料が揃うのではないかなと思いました。</p>
</blockquote>
<p>とても良いと思います。そのような形で対応しましたので、patch 0003をご確認ください。</p>
<p>対応にあたってPostBox内のcompose呼び出しを探していたところ、remain_charcountの呼び出し側でフッタを差し込むかのフラグの考慮が漏れていることが分かりました。<br />そのため、双方のオプションを統一できるような形にしています。</p>
mikutter - 機能 #1250: 残り文字数カウントの処理を remain_charcount Spell に分離する
https://dev.mikutter.hachune.net/issues/1250?journal_id=3399
2018-06-04T15:06:50Z
toshi_a 初音
toshi.alternative@gmail.com
<ul><li><strong>ステータス</strong> を <i>パッチ適用待ち</i> から <i>レビュー待ち</i> に変更</li><li><strong>担当者</strong> を <i>toshi_a 初音</i> から <i>Shibafu Midorino</i> に変更</li></ul><p>topic/1250-remain-charcount ブランチにcommitしました。こっちで試してるぶんには問題ないかなと思います。そのまま当てただけですが、もう一度確認お願いします。</p>
mikutter - 機能 #1250: 残り文字数カウントの処理を remain_charcount Spell に分離する
https://dev.mikutter.hachune.net/issues/1250?journal_id=3421
2018-06-08T23:58:09Z
Shibafu Midorino
<ul><li><strong>担当者</strong> を <i>Shibafu Midorino</i> から <i>toshi_a 初音</i> に変更</li></ul><p>確認しました。問題ありません。</p>
<p>機能トラッカーには レビュー待ち → マージ待ち の状態遷移がないようですので、ステータスはそのままにしておきます。</p>
mikutter - 機能 #1250: 残り文字数カウントの処理を remain_charcount Spell に分離する
https://dev.mikutter.hachune.net/issues/1250?journal_id=3424
2018-06-09T02:45:09Z
toshi_a 初音
toshi.alternative@gmail.com
<ul><li><strong>ステータス</strong> を <i>レビュー待ち</i> から <i>終了</i> に変更</li></ul><p>mergeしました</p>
mikutter - 機能 #1250: 残り文字数カウントの処理を remain_charcount Spell に分離する
https://dev.mikutter.hachune.net/issues/1250?journal_id=3496
2018-07-01T03:48:58Z
Izumi Tsutsui
tsutsui@ceres.dti.ne.jp
<ul><li><strong>関連している</strong> <i><a class="issue tracker-12 status-5 priority-4 priority-default closed" href="/issues/1153">提案 #1153</a>: Twitter postbox 残り文字数表示 280字対応</i> を追加</li></ul>