Incidencia #3095

アバターマネージャーのカスタムアバターの削除にバグ
Abrir Fecha: 2003-10-02 11:29 Última actualización: 2003-10-04 17:31

Informador:
Propietario:
(Ninguno)
Tipo:
Estado:
Cerrado
Componente:
(Ninguno)
Hito:
(Ninguno)
Prioridad:
5 - Medium
Gravedad:
5 - Medium
Resolución:
Fixed
Fichero:
Ninguno

Details

Xoops2.0.3 or 2.0,4 or 2.0.5 共通です。

 あまりする事が無いのと、深刻な症状にならないので気がつか
なかったですが、
アバターマネージャーのカスタムアバターの削除にバグが2つあ
るのに気がつきました。報告が上がっていたのかもしれませんが?
 見つけられなかったので書いておきます。

<バグ1>
アバターマネジャーのカスタムアバター[一覧]をしたときに、
(アバター・マネジャー »» カスタムアバター)
「画像名:欄のユーザー名」と「削除」の文字のリンクに、getす
るユーザーIDがセットされてません。
 マウスをあててリンクをみると、user_id= が空っぽです。
http://XOOPSのurl/modules/system/admin.php?
fct=avatars&op=delfile&avatar_id=39&user_id=

このため、
<バグ1の症状>
「画像名:欄のユーザー名」は、ジャンプできないだけですが、
「削除」のときの症状として、ユーザーuser_id= は存在しない
ので
テーブルusersの user_avatar='blank.gif'への update が実
際には出来てません。
$xoopsDB->query("UPDATE ".$xoopsDB->prefix('users')."
SET user_avatar='blank.gif' WHERE uid=".intval
($user_id));
(ただし、このSQL文は、エラーとはならないはず)
 
<バグ1の修正案>
記述の誤記みたいです
modules/system/admin/avatars/main.php line 95,96 
[code]変更前
line 95 echo '<table ・・//この行、このforumに書くと変に
改行されてみずらいので略
line 96 $userids =& $avt_handler->getUser($avt);
[/code]
[code]変更後
line 95 $userids =& $avt_handler->getUser($avatars
[$i]);
line 96 echo '<table ・・//この行、このforumに書くと変に
改行されてみずらいので略
[/code]

<バグ2>
上記、カスタムアバターの削除のとき、avatar_user_link の削
除してません。(ゴミとして残るがあまり深刻な症状は?でないは
ず)
記述の漏れみたいです、queryしてません。
kernel/avatar.php  
[code]変更前
line 144 $sql = sprintf("DELETE FROM %s WHERE
avatar_id = %u", $this->db->prefix
('avatar_user_link'), $id);
[/code]
これ、削除を失敗してもあまり影響がないから、queryするだけ
でいいかな?この下に
[code]変更後
line 144 $sql = sprintf("DELETE FROM %s WHERE
avatar_id = %u", $this->db->prefix
('avatar_user_link'), $id);
line 145 $result = $this->db->query($sql);
[/code]

Ticket History (1/1 Histories)

2003-10-04 17:31 Updated by: onokazu
  • Ticket Close date is changed to 2003-10-04 17:31
  • Resolución Update from Ninguno to Fixed
  • Estado Update from Open to Cerrado
Comentario
Logged In: YES
user_id=754

CVSへと反映しておきました。

Attachment File List

No attachments

Editar

You are not logged in. I you are not logged in, your comment will be treated as an anonymous post. » Entrar