newsにて「トピック画像を表示」の選択肢が両方選択

• 回報者: okuhiki
• 負責人: (無)
• 類型: 問題回報
• 狀態: 關閉
• 元件: (無)
• 里程碑: (無)
• 優先權: 5 - 中
• 嚴重程度: 5 - 中
• 處理結果: 修正
• 檔案: 無

細節 回覆

newsを管理メニューより新規作成時にトピック画像の表示を
制御するラジオボタンの、「はい」「いいえ」双方に
checked='checked'が書き出されています。

news\admin\storyform.inc.php Line 59~67

echo "<br /><b>"._AM_TOPICDISPLAY."</b>&nbsp;&nbsp;<inp
ut type='radio' name='topicdisplay' value='1'";
if ( !isset($topicdisplay) || $topicdisplay==1 ) {
echo " checked='checked'";
}
ECHO " />"._AM_YES."&nbsp;<input type='radio'
name='topicdisplay' value='0'";
if (empty($topicdisplay)) {
echo " checked='checked'";
}
echo " />"._AM_NO."&nbsp;&nbsp;&nbsp;";

これでは、$topicdisplayが設定されていない場合双方に書き
出されてしまうのでは。

if ( isset($topicdisplay) && $topicdisplay==1 ) {
$tdispy=" checked='checked'";
$tdispn="";
} else {
$tdispy="";
$tdispn=" checked='checked'";
}
echo "<br /><b>"._AM_TOPICDISPLAY."</b>&nbsp;&nbsp;<inp
ut type='radio' name='topicdisplay'
value='1'$tdispy />"._AM_YES;
ECHO "&nbsp;<input type='radio' name='topicdisplay'
value='0'$tdispn" />"._AM_NO."&nbsp;&nbsp;&nbsp;"

勝手な名前の変数使ってますから注意が必用ですが、ifを1回
で済ませると、重複を防げませんか？
私が理解出来ないので、echoの区切りを怪変してます。
比較が素早くできなかったらすみません。

Ticket History (3/3 Histories)

2005-07-02 18:29 Updated by: okuhiki

評語 回覆

Logged In: YES
user_id=13441

あら　変更例のソース最下行に、不要な文字が。

value='0'$tdispn />"._AM_NO."&nbsp;&nbsp;&nbsp;"

2005-07-18 17:24 Updated by: onokazu

• Ticket Close date is changed to 2005-07-18 17:24
• 處理結果 Update from 無 to 修正
• 狀態 Update from 開啟 to 關閉

評語 回覆

Logged In: YES
user_id=754

ありがとうございます。CVSにて修正いたしました。

2005-07-19 06:24 Updated by: okuhiki

• Ticket Close date is changed to 2005-07-19 06:24

評語 回覆

Logged In: YES
user_id=13441

あ、そうか 「$topicdisplayが設定されていない場合」と書いて置
きならが、設定してから判断する事を思いつかない私。
さっそくの対応、ありがとうございます。