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

• Reporter: okuhiki
• 소유자: (None)
• Type: Bugs
• Status: Closed
• Component: (None)
• MileStone: (None)
• Priority: 5 - Medium
• Severity: 5 - Medium
• Resolution: Fixed
• File: None

Details Reply

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

댓글 올리기 Reply

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
• Resolution Update from None to Fixed
• Status Update from Open to Closed

댓글 올리기 Reply

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

댓글 올리기 Reply

Logged In: YES
user_id=13441

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