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

• Auswertung: okuhiki
• Verantwortlicher: (Keine)
• Typ: Fehler
• Status: Geschlossen
• Komponente: (Keine)
• Meilenstein: (Keine)
• Priorität: 5 - Mittel
• Schweregrad: 5 - Mittel
• Lösung: Gefixt
• Datei: Keine

Details Antworten

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-Verlauf (3/3 Historien)

2005-07-02 18:29 Aktualisiert von: okuhiki

Kommentar Antworten

Logged In: YES
user_id=13441

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

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

2005-07-18 17:24 Aktualisiert von: onokazu

• Ticket Close date is changed to 2005-07-18 17:24
• Lösung Update from Keine to Gefixt
• Status Update from Offen to Geschlossen

Kommentar Antworten

Logged In: YES
user_id=754

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

2005-07-19 06:24 Aktualisiert von: okuhiki

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

Kommentar Antworten

Logged In: YES
user_id=13441

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